Skip to content

split kubectl installation into two parts to reduce race conditions#4081

Merged
ighosh98 merged 1 commit into
GoogleCloudPlatform:developfrom
ighosh98:develop
May 7, 2025
Merged

split kubectl installation into two parts to reduce race conditions#4081
ighosh98 merged 1 commit into
GoogleCloudPlatform:developfrom
ighosh98:develop

Conversation

@ighosh98

@ighosh98 ighosh98 commented May 7, 2025

Copy link
Copy Markdown
Contributor

Submission Checklist

kubectl installation will start only after cluster creation is complete to reduce the race conditions we have observed.

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 May 7, 2025 17:48
@ighosh98 ighosh98 added the release-bugfix Added to release notes under the "Bug fixes" heading. label May 7, 2025
Comment thread modules/management/kubectl-apply/main.tf
Comment thread modules/management/kubectl-apply/main.tf
@ighosh98 ighosh98 merged commit 708cec8 into GoogleCloudPlatform:develop May 7, 2025
15 of 65 checks passed
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants