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

CRAYSAT-1787: Adding ceph health check bypass in sat bootsys ncn-power #239

Conversation

shivaprasad-metimath
Copy link
Contributor

@shivaprasad-metimath shivaprasad-metimath commented Jul 15, 2024

IM:CRAYSAT-1787
Reviewer:Ryan

Summary and Scope

Adding ceph health check bypass prompt for the user to decide whether to wait or proceed with skipping the health check after unfreezing of ceph is done. As it may take some time and the next steps may not explicitly require, by the time it comes back it would be good to use.

Issues and Related PRs

List and characterize relationship to Jira/Github issues and other pull requests. Be sure to list dependencies.

Testing

List the environments in which these changes were tested.

Tested on:

Tested on Rocket system

Test description:

Will have to power off and power on the ncn's to verify how the unfreeze ceph and ceph health check bypass works during ncn-power stage of sat bootsys

Risks and Mitigations

minimal

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
  • HPC Product Announcement prepared, if applicable

@shivaprasad-metimath shivaprasad-metimath force-pushed the CRAYSAT-1787-ceph-healthcheck-bypass-in-bootsys branch 2 times, most recently from 6539c8e to e80a5d5 Compare July 15, 2024 13:06
@shivaprasad-metimath shivaprasad-metimath force-pushed the CRAYSAT-1787-ceph-healthcheck-bypass-in-bootsys branch 3 times, most recently from bcd11b6 to 1eea073 Compare July 24, 2024 05:24
@shivaprasad-metimath
Copy link
Contributor Author

sat/cli/bootsys/mgmt_power.py Outdated Show resolved Hide resolved
sat/cli/bootsys/mgmt_power.py Outdated Show resolved Hide resolved
sat/cli/bootsys/mgmt_power.py Outdated Show resolved Hide resolved
@shivaprasad-metimath shivaprasad-metimath force-pushed the CRAYSAT-1787-ceph-healthcheck-bypass-in-bootsys branch 3 times, most recently from c70e2d8 to d72dfa3 Compare July 25, 2024 06:03
IM:CRAYSAT-1787
Reviewer:Ryan
Adding ceph health check bypass prompt for the user to decide whether to wait
or proceed with skipping the health check after unfreezing of ceph is done.
As it may take some time and the next steps may not explicitly require, by the
time it comes back it would be good to use.
@shivaprasad-metimath shivaprasad-metimath force-pushed the CRAYSAT-1787-ceph-healthcheck-bypass-in-bootsys branch from d72dfa3 to 53b4981 Compare July 25, 2024 06:16
Copy link
Contributor

@annapoorna-s-alt annapoorna-s-alt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@shivaprasad-metimath shivaprasad-metimath merged commit 3f9a619 into feature/CRAYSAT-1740 Jul 25, 2024
3 checks passed
@shivaprasad-metimath shivaprasad-metimath deleted the CRAYSAT-1787-ceph-healthcheck-bypass-in-bootsys branch July 25, 2024 06:28
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.

None yet

3 participants