Skip to content

Power down non-responding node if there is not instance attached#4627

Merged
ankitkumar-quad merged 1 commit into
GoogleCloudPlatform:release-candidatefrom
abbas1902:no-res
Sep 9, 2025
Merged

Power down non-responding node if there is not instance attached#4627
ankitkumar-quad merged 1 commit into
GoogleCloudPlatform:release-candidatefrom
abbas1902:no-res

Conversation

@abbas1902

@abbas1902 abbas1902 commented Sep 8, 2025

Copy link
Copy Markdown
Collaborator

This is to fix any stale NOT_RESPONDING flags given in the nodestate in Slurm and resetting the nodes to power down. Considered the instance is non-existant in this scenarios this shouldn't harbor any side effects for already running VMs which may correctly carry a NOT_RESPONDING flag for other reasons.

Submission Checklist

NOTE: Community submissions can take up to 2 weeks to be reviewed.

Please take the following actions before submitting this pull request.

  • Fork your PR branch from the Toolkit "develop" branch (not main)
  • Test all changes with pre-commit in a local branch #
  • Confirm that "make tests" passes all tests
  • Add or modify unit tests to cover code changes
  • Ensure that unit test coverage remains above 80%
  • Update all applicable documentation
  • Follow Cluster Toolkit Contribution guidelines #

@abbas1902 abbas1902 added the release-bugfix Added to release notes under the "Bug fixes" heading. label Sep 8, 2025
@abbas1902 abbas1902 marked this pull request as ready for review September 8, 2025 16:59
@abbas1902 abbas1902 requested review from a team and samskillman as code owners September 8, 2025 16:59
@abbas1902 abbas1902 requested a review from mr0re1 September 8, 2025 17:04
@ankitkumar-quad ankitkumar-quad merged commit 22f1665 into GoogleCloudPlatform:release-candidate Sep 9, 2025
37 of 69 checks passed
@abbas1902 abbas1902 deleted the no-res branch September 11, 2025 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-bugfix Added to release notes under the "Bug fixes" heading.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants