Skip to content

split terraform bundles for gpu operator#3911

Merged
ighosh98 merged 5 commits into
GoogleCloudPlatform:developfrom
ighosh98:a4x
Apr 10, 2025
Merged

split terraform bundles for gpu operator#3911
ighosh98 merged 5 commits into
GoogleCloudPlatform:developfrom
ighosh98:a4x

Conversation

@ighosh98

Copy link
Copy Markdown
Contributor

Submission Checklist

  • Splitting the tf module to avoid race conditions around installing dependencies.

NOTE: Community submissions can take up to 2 weeks to be reviewed.

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

@ighosh98 ighosh98 requested review from a team and samskillman as code owners April 10, 2025 07:45
@ighosh98 ighosh98 requested a review from parulbajaj01 April 10, 2025 07:45
@ighosh98 ighosh98 added release-module-improvements Added to release notes under the "Module Improvements" heading. release-bugfix Added to release notes under the "Bug fixes" heading. labels Apr 10, 2025

@parulbajaj01 parulbajaj01 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ighosh98 ighosh98 enabled auto-merge April 10, 2025 07:54
@ighosh98 ighosh98 merged commit e080364 into GoogleCloudPlatform:develop Apr 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-bugfix Added to release notes under the "Bug fixes" heading. release-module-improvements Added to release notes under the "Module Improvements" heading.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants