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

fix: remove datapath provider from Autopilot modules #1556

Merged
merged 2 commits into from
Feb 3, 2023

Conversation

mastersingh24
Copy link
Contributor

You cannot set datapath for Autopilot clusters.

Fixes #1446

@mastersingh24 mastersingh24 requested review from a team, Jberlinsky and ericyz as code owners February 3, 2023 14:03
@mastersingh24 mastersingh24 changed the title Remove datapath provider from Autopilot modules fix: remove datapath provider from Autopilot modules Feb 3, 2023
You cannot set datapath for Autopilot clusters.

Fixes terraform-google-modules#1446

Signed-off-by: Gari Singh <[email protected]>
@mastersingh24
Copy link
Contributor Author

No idea why this is failing test 27. Don't see how this change would have any affect there.

@bharathkkb
Copy link
Member

@mastersingh24 it may have been a flake. I will retry

@comment-bot-dev
Copy link

@mastersingh24
Thanks for the PR! 🚀
✅ Lint checks have passed.

@bharathkkb bharathkkb merged commit ea012f5 into terraform-google-modules:master Feb 3, 2023
CPL-markus pushed a commit to WALTER-GROUP/terraform-google-kubernetes-engine that referenced this pull request Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Terraform re-apply causes cluster deletion and recreation due to datapath_provider parameter
3 participants