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

OFE - various updates and fixes #2921

Merged
merged 51 commits into from
Aug 14, 2024

Conversation

scott-nag
Copy link
Collaborator

OFE fixes and updates:

  • Upgrade to use Slurm-GCP v6 modules instead of v5
  • Migrated to Rocky8
  • Use Terraform 1.4 (Using a newer version of Terraform can lead to controller replacement on reconfigure for Slurm GCP v6 #2774)
  • Added validation for clusters that use reservations. It validates whether:
    • reservation exists in the project, region and zone
    • reservation supports the selected node type
    • reservation has capacity for the requested instances
    • reservation is 'specific'
  • Added support to display disk type options which are valid for the selected machine type
  • Changed mentions of ‘hpc-toolkit’ to ‘cluster-toolkit’

ek-nag and others added 30 commits June 17, 2024 13:54
Typo in cluster additional disk javascript.
Fixing the logic for selecting custom images for controller and login nodes.
Fixing disk selection for login and controller nodes logic
Migrating to V6
Migrating to V6 and Rocky8
Migrating to V6 and Rocky8
Migrating to V6 and Rocky8
Migrating to v6 and Rocky8
Migrating to v6 and Rocky8
Migrating to v6 and Rocky8
Migrating to v6 and Rocky8
Migrating to v6 and Rocky8
Migrating to V6 and Rocky8
Lustre client install fix no longer needed
@nick-stroud nick-stroud added the release-improvements Added to release notes under the "Improvements" heading. label Aug 13, 2024
@nick-stroud
Copy link
Collaborator

/gcbrun

@nick-stroud nick-stroud merged commit 12697cb into GoogleCloudPlatform:develop Aug 14, 2024
11 of 53 checks passed
@rohitramu rohitramu mentioned this pull request Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-improvements Added to release notes under the "Improvements" heading.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants