Adding kueue support for GKE A4X-Max#5389
Adding kueue support for GKE A4X-Max#5389vikramvs-gg wants to merge 1 commit intoGoogleCloudPlatform:developfrom
Conversation
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 enhances the GKE A4X-Max bare metal example by integrating Kueue for advanced job scheduling and resource management, ensuring optimal utilization of GPU resources. It also addresses a critical compatibility issue by updating the 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 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 counter productive. 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 introduces Kueue support for the GKE A4X-Max example, which is a valuable addition. The changes include adding the necessary Kueue configuration, updating the example blueprint to install and configure Kueue, and modifying the NCCL jobset example to use it. The upgrade of the asapd-lite image and the adjustment to the additional_networks configuration are also well-justified.
My review has identified one critical issue where the specified Kueue version is not yet supported by the kubectl-apply module, which will cause deployment failures. Please see the specific comment for details on how to resolve this.
1ada989 to
80790ea
Compare
Feat: Adding kueue support for GKE A4X-Max
This PR adds kueue support for GKE A4X-Max. A kueue configuration manifest is added along with defined topologies of block, subblock and host levels. Kueue version is
0.14.4. Corresponding annotation of subblock topology is used in the NCCL jobset.This PR also upgrades the asapd-lite container image version from
0.0.3to0.0.7because the older version did not recognize the newer firmware installed on A4X-Max bare metal nodes.We are also removing the additional_networks from gke-cluster in blueprint, keeping it on the node-pool. Machine type a4x-maxgpu-4g-metal is not supported when multi-networking is enabled on the cluster.
NOTE: Community submissions can take up to 2 weeks to be reviewed.
Please take the following actions before submitting this pull request.