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-8899 - support for Paradise nodes. #53

Merged
merged 1 commit into from
May 3, 2024
Merged

Conversation

dlaine-hpe-2
Copy link
Contributor

Summary and Scope

Paradise nodes (Cray xd224) needs to connect to the consoles through ssh but with a username and password, which is a different way of connecting than to regular river, mountain, or hill nodes. The changes in this service mostly have to do with identifying and labeling nodes that are paradise hardware.

Issues and Related PRs

Testing

Tested on:

  • Tyr

Test description:

The new services were installed via helm and I manually tested that the existing mountain, hill, river, and paradise nodes all are identified correctly and the console services can connect with them.

  • 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? Y
  • Was downgrade tested? If not, why? Y
  • Were new tests (or test issues/Jiras) created for this change? N

Risks and Mitigations

This is a low risk, but there are changes that impact the other node types as well as the new paradise nodes.

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-2 dlaine-hpe-2 requested a review from a team as a code owner May 2, 2024 19:59
@dlaine-hpe-2 dlaine-hpe-2 merged commit 476cda6 into develop May 3, 2024
4 checks passed
@dlaine-hpe-2 dlaine-hpe-2 deleted the CASMCMS-8899 branch May 3, 2024 19:27
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.

4 participants