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-6156: Add POST option to get power states to avoid size limitations with GET parameters #30

Merged
merged 1 commit into from
Mar 27, 2024

Conversation

mharding-hpe
Copy link
Contributor

@mharding-hpe mharding-hpe commented Mar 27, 2024

Summary and Scope

See source PR for full explanation of the PCS changes associated with this, and the testing that was done:
Cray-HPE/hms-power-control#41

Adopted app version 2.3.0 for CSM 1.5 (helm chart 2.0.8)
Adopted app version 2.3.0 for CSM 1.6 (helm chart 2.1.5)

Issues and Related PRs

Testing

See source PR for full explanation of the PCS changes associated with this, and the testing that was done:
Cray-HPE/hms-power-control#41

For this PR specifically, I also made sure I could upgrade and deploy the new charts on mug.

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

Copy link

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

Chart Download Link
cray-power-control-2.1.5-20240327213533+ef32f67.tgz https://artifactory.algol60.net/artifactory/csm-helm-charts/unstable/cray-hms-power-control/cray-power-control-2.1.5-20240327213533+ef32f67.tgz
cray-power-control-2.0.8-20240327213533+ef32f67.tgz https://artifactory.algol60.net/artifactory/csm-helm-charts/unstable/cray-hms-power-control/cray-power-control-2.0.8-20240327213533+ef32f67.tgz

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

Build summary

Build metadata

Key Value
Artifactory Component cray-hms-power-control
Stable Build false
Unstable Build Prefix -20240327213533+ef32f67

Publish Helm charts results

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

@mharding-hpe mharding-hpe requested review from shunr-hpe and jwlv March 27, 2024 21:36
@mharding-hpe
Copy link
Contributor Author

The Snyk failures all appear to be pre-existing in this branch and unrelated to this PR

@mharding-hpe mharding-hpe merged commit 97c900a into main Mar 27, 2024
17 of 23 checks passed
@mharding-hpe mharding-hpe deleted the CASMHMS-6156 branch March 27, 2024 22:03
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