-
Notifications
You must be signed in to change notification settings - Fork 1.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Job Summary - Part 2 #1455
Merged
Merged
Job Summary - Part 2 #1455
Changes from all commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
bac783a
Setting the number of queued allocations per task group
diptanu 38b1ba7
Added the job summary related endpoints
diptanu 6e9b955
Updating the job summary table only if an evaluation has any Queued A…
diptanu 7497403
Fixed the logic for decrementing the count of queued based on plan re…
diptanu 5b2f2ee
Not setting the desired and client status of an allocation during in-…
diptanu 36abb97
Applying changes to job updates via FSM
diptanu 0347f60
Updating the job summary while mutating jobs and allocation objects
diptanu 7628283
Moved the job endpoint around
diptanu 0a7489d
Adding the summary to the Job Stub
diptanu ba18de7
Added support for retreiving job summary in api
diptanu 4095e33
Displaying the job summary in nomad status command
diptanu 8f45222
Fixed some error messages and conditions
diptanu 03c6692
Fixed some bugs
diptanu 5e86e9a
Review comments
diptanu c79784d
Added code to create missing job summaries
diptanu e9d8e10
Added some more tests
diptanu 46907a1
Added a test to ensure system scheduler records the correct number of…
diptanu fb3b109
Added a test to ensure failed batch allocations are being added to th…
diptanu 7928f29
Added test for blocking query of job summary endpoint
diptanu 230a59c
Fixed some more tests
diptanu 73c6fbb
Added a test to ensure we record the queued allocations correctly whe…
diptanu ddf10e1
Initializing the queued allocations late
diptanu c18e431
Fixed more tests
diptanu c91f477
Renamed Job.GetJobSummary to Job.Summary
diptanu a9c995b
Added a test for adjustQueuedAllocations
diptanu 3d4c185
Reconciling the queued allocations during restore
diptanu 53a2cc9
Setting the right indexes while creating Job Summary
diptanu 64a944c
Making the queued allocations bind late
diptanu c8870af
Fixed a test
diptanu d1da78a
Running the tests in verbose mode
diptanu 2892224
Updated some tests
diptanu File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Output this as well for the
nomad status
case