Update A4high blueprints to use pre-built ACI image#5932
Conversation
…DCGM to 1:4.6.0-1
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request updates the A4high blueprints to leverage pre-built Advanced Compute Images (ACI), significantly reducing build complexity. It also introduces Filestore for shared home directories, optimizes GCSFuse mount configurations, and improves the robustness of startup scripts for Enroot and GPU health checks. These changes collectively streamline the deployment process and improve environment stability for A4high clusters. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize the Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counterproductive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request refactors the Slurm blueprints to use pre-built advanced compute images instead of building custom images with Packer, simplifies the startup scripts, and updates GCSfuse mount options to use standard profiles. Feedback on these changes highlights that downloading and installing the CUDA toolkit during node startup is redundant since the selected image already has CUDA 13.0 pre-installed. Additionally, it is recommended to configure the AppArmor unprivileged user namespace restriction permanently using sysctl via /etc/sysctl.d/ rather than writing directly to /proc/sys.
|
/gcbrun |
|
/gcbrun |
|
@ksaishree please wait for the integration test "PR-test-ml-a4-highgpu-onspot-slurm" to pass before merging. |
c2030c4
into
GoogleCloudPlatform:develop
Replacing the Ubuntu accelerator image with ACI image in A4high blueprint.
Integration tests and NCCL tests passed successfully.
Submission Checklist
NOTE: Community submissions can take up to 2 weeks to be reviewed.
Please take the following actions before submitting this pull request.