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

CASMHMS-6310: Fix FAS resource leaks and scaling issues #38

Merged
merged 6 commits into from
Dec 11, 2024
Merged

Conversation

jwlv
Copy link
Contributor

@jwlv jwlv commented Dec 3, 2024

Summary and Scope

Various changes including:

  • Resolved various scaling/resource issues
  • Update to latest TRS
  • Update to Go 1.23 as result of updating to latest TRS
  • Updated python to alpine:3.18 in all Dockerfiles to avoid pipenv install issue

Adopted app version 1.35.0 for CSM 1.5.3 (helm chart 3.0.10)
Adopted app version 1.35.0 for CSM 1.6.1 (helm chart 3.1.10)

Issues and Related PRs

Testing

Tested Description:

All integration and unit tests pass in the build pipeline.

Deployed on mug (CSM 1.6) and rocket (CSM 1.5) and verified all /opt/cray/csm/scripts/hms_verification/run_hms_ct_tests.sh -t fas tests succeeded.

Mike performed manual testing on mug.

Testing Check List:

  • Were the install/upgrade-based validation checks/tests run (goss tests/install-validation doc)? Y
  • Were continuous integration tests run? If not, why? Y
  • Was upgrade tested? If not, why? Y
  • Was downgrade tested? If not, why? Y

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

@jwlv jwlv requested review from a team as code owners December 3, 2024 21:51
@jwlv jwlv requested review from jsollom-hpe, mbuchmann-hpe and shunr-hpe and removed request for jsollom-hpe December 3, 2024 21:51
Copy link

github-actions bot commented Dec 3, 2024

👋 Hey! Here are the chart(s) we built for you:

Chart Download Link
cray-hms-firmware-action-3.0.10-20241211160544+4d281f6.tgz https://artifactory.algol60.net/artifactory/csm-helm-charts/unstable/cray-hms-firmware-action/cray-hms-firmware-action-3.0.10-20241211160544+4d281f6.tgz
cray-hms-firmware-action-3.1.10-20241211160544+4d281f6.tgz https://artifactory.algol60.net/artifactory/csm-helm-charts/unstable/cray-hms-firmware-action/cray-hms-firmware-action-3.1.10-20241211160544+4d281f6.tgz

Note: this SHA is the merge of 8ea0796 and the PR base branch. Good luck and make rocket go now! 🌮 🚀

Build summary

Build metadata

Key Value
Artifactory Component cray-hms-firmware-action
Stable Build false
Unstable Build Prefix -20241211160544+4d281f6

Publish Helm charts results

Key Value
Status success
Successfully uploaded charts count 2
Failed uploaded charts count 0

@jwlv jwlv merged commit 5b33dfd into main Dec 11, 2024
21 of 25 checks passed
@jwlv jwlv deleted the CASMHMS-6310 branch December 11, 2024 16:47
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.

3 participants