Repository navigation
Initial Blueprint G4 - #4685
Merged
Merged
Initial Blueprint G4#4685
Conversation
cboneti
previously requested changes
Nov 6, 2025
Member
|
/gemini review |
Contributor
There was a problem hiding this comment.
Code Review
This pull request introduces a new ml-slurm-g4 blueprint, which is a valuable addition for users looking to deploy GPU-based machine learning workloads on Slurm. The configuration is mostly well-structured. My review includes a few suggestions to improve it. The most important ones are to ensure that the Slurm controller and login nodes use the same custom OS image as the compute nodes for a consistent environment. I've also pointed out a couple of opportunities to clean up the configuration by removing an extraneous key and some verbose network settings. Overall, these are good changes that will provide a useful example to users.
LAVEEN
force-pushed
the
g4new
branch
7 times, most recently
from
November 11, 2025 20:29
2c44dca to
d9f8e90
Compare
LAVEEN
enabled auto-merge
November 11, 2025 20:38
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add
ml-slurm-g4blueprint for GPU-based ML workloads.This change introduces a new example blueprint,
ml-slurm-g4, to deploy a Slurm cluster optimized for machine learning workloads on Google Cloud.The blueprint provisions the following resources:
g4_partition) withg4-standard-48machine types.slurm-gcp-6-11-ubuntu-2204-lts-nvidia-570image family.This provides users with a reference architecture for setting up a performant, GPU-enabled HPC environment for machine learning applications.