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

CASMCMS-8979 - update status unknown jobs. #137

Merged
merged 1 commit into from
Aug 26, 2024
Merged

Conversation

dlaine-hpe
Copy link
Contributor

Summary and Scope

Clean up the default value of number of jobs for the remote build node status object. Using 10000 was weird - made it -1 for unknown.

Issues and Related PRs

Testing

Tested on:

  • Mug

Test description:

Updated to the new image and tested with valid and invalid remote build nodes. Ran multiple jobs, make sure the number of current jobs was correct.

  • Were the install/upgrade-based validation checks/tests run (goss tests/install-validation doc)? N
  • Were continuous integration tests run? If not, why? N
  • Was upgrade tested? If not, why? N
  • Was downgrade tested? If not, why? N
  • Were new tests (or test issues/Jiras) created for this change? N

Risks and Mitigations

Very low risk.

Pull Request Checklist

  • Version number(s) incremented, if applicable
  • Copyrights updated
  • License file intact
  • Target branch correct
  • CHANGELOG.md updated
  • Testing is appropriate and complete, if applicable

@dlaine-hpe dlaine-hpe requested a review from a team as a code owner August 26, 2024 18:04
src/server/models/jobs.py Show resolved Hide resolved
@dlaine-hpe dlaine-hpe merged commit 6ac0dfd into develop Aug 26, 2024
4 checks passed
@dlaine-hpe dlaine-hpe deleted the CASMCMS-8979-v2 branch August 26, 2024 18:52
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.

2 participants