Skip to content

Use setsid resume.py to reduce reconfigure time#4436

Merged
samskillman merged 1 commit into
GoogleCloudPlatform:developfrom
samskillman:fix/setsid-resume
Jul 28, 2025
Merged

Use setsid resume.py to reduce reconfigure time#4436
samskillman merged 1 commit into
GoogleCloudPlatform:developfrom
samskillman:fix/setsid-resume

Conversation

@samskillman

Copy link
Copy Markdown
Collaborator

Using setsid instead of disown unlinks the resume.py process from the parent (slurmctld) process, allowing scontrol reconfigure to continue prior to the resume.py process exiting.

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 #

@samskillman samskillman requested review from alyssa-sm and mr0re1 July 22, 2025 23:03
@samskillman samskillman requested a review from a team as a code owner July 22, 2025 23:03
@samskillman samskillman added the release-bugfix Added to release notes under the "Bug fixes" heading. label Jul 22, 2025
@samskillman

Copy link
Copy Markdown
Collaborator Author

Failure looks unrelated

Using setsid instead of disown unlinks the resume.py process
from the parent (slurmctld) process, allowing scontrol reconfigure
to continue prior to the resume.py process exiting.
@samskillman samskillman merged commit 4889560 into GoogleCloudPlatform:develop Jul 28, 2025
34 of 62 checks passed
@samskillman samskillman deleted the fix/setsid-resume branch July 28, 2025 18:35
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.

2 participants