Skip to content
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

[ui, deployments] Denominator based on completed allocations for batch jobs #17147

Conversation

philrenaud
Copy link
Contributor

@philrenaud philrenaud commented May 10, 2023

We shouldn't count "complete" allocations against the denominator in the "7/10 Allocations Running" heading, when the 3 non-running are of "complete" status. Instead, in this case, we would say "7/7 Remaining Allocations Running".

Resolves #17139

@github-actions
Copy link

github-actions bot commented May 10, 2023

Ember Asset Size action

As of 18364f4

Files that got Bigger 🚨:

File raw gzip
nomad-ui.js +635 B +120 B

Files that stayed the same size 🤷‍:

File raw gzip
vendor.js 0 B 0 B
nomad-ui.css 0 B 0 B
vendor.css 0 B 0 B

@philrenaud philrenaud marked this pull request as ready for review May 10, 2023 20:16
Base automatically changed from 17114-ui-deployments-redesign-batch to main May 10, 2023 20:59
@philrenaud philrenaud force-pushed the 17139-running-allocations-denominator-for-batch-jobs-should-combine-all-non-complete-allocs branch from 18364f4 to 53fc47f Compare May 10, 2023 21:01
@philrenaud philrenaud merged commit 106aef9 into main May 11, 2023
3 of 6 checks passed
@philrenaud philrenaud deleted the 17139-running-allocations-denominator-for-batch-jobs-should-combine-all-non-complete-allocs branch May 11, 2023 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Running Allocations denominator for batch jobs should combine all non-complete allocs
2 participants