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

Adopt Slurm 6.5.6 to workaround long hostname issues in Debian 12 #2644

Conversation

tpdownes
Copy link
Member

@tpdownes tpdownes commented May 29, 2024

Adopt Slurm 6.5.6 which builds upon 6.5.5 to extend long hostname workaround to dynamic nodes, including login nodes. See GoogleCloudPlatform/slurm-gcp#142 for fuller description of changes.

Submission Checklist

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 Cloud HPC Toolkit Contribution guidelines #

@tpdownes tpdownes self-assigned this May 29, 2024
@tpdownes tpdownes force-pushed the long_hostname_fix_slurm branch from 9fe91d7 to ca83a92 Compare May 29, 2024 18:23
@tpdownes tpdownes requested a review from mr0re1 May 29, 2024 18:23
@tpdownes tpdownes assigned mr0re1 and unassigned mr0re1 and tpdownes May 29, 2024
@tpdownes tpdownes marked this pull request as ready for review May 29, 2024 18:24
@tpdownes
Copy link
Member Author

All integration tests passed on the draft PR. 1 automated retry was necessary on the Lustre example.

@mr0re1 mr0re1 assigned tpdownes and unassigned mr0re1 May 29, 2024
@tpdownes tpdownes enabled auto-merge May 29, 2024 18:29
@tpdownes tpdownes merged commit 6f8a20b into GoogleCloudPlatform:release-candidate May 29, 2024
6 of 47 checks passed
@tpdownes tpdownes deleted the long_hostname_fix_slurm branch May 29, 2024 18:36
@tpdownes tpdownes restored the long_hostname_fix_slurm branch May 29, 2024 18:37
@tpdownes tpdownes deleted the long_hostname_fix_slurm branch May 29, 2024 18:37
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.

2 participants