Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add shielded instance config to default pool #3507

Conversation

annuay-google
Copy link
Contributor

Why?

If a project has constraints/compute.requireShieldedVm enabled, then the default pool cannot be created and fails with

Current errors: [CONDITION_NOT_MET]: Instance 'gke-xyz-cluster-default-pool-...' creation failed: Constraint constraints/compute.requireShieldedVm violated for project projects/project-name.

Submission Checklist

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 #

@annuay-google annuay-google changed the base branch from main to develop January 7, 2025 20:44
@annuay-google annuay-google added the release-module-improvements Added to release notes under the "Module Improvements" heading. label Jan 7, 2025
@annuay-google annuay-google force-pushed the annuay/add-shielded-instance-config-to-default-pool branch from d73a58e to 551a6c4 Compare January 7, 2025 21:04
@annuay-google annuay-google merged commit 3213e22 into GoogleCloudPlatform:develop Jan 7, 2025
8 of 52 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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