-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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: make master_ipv4_cidr_block optional for autopilot #1902
fix: make master_ipv4_cidr_block optional for autopilot #1902
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR @cloud-pharaoh
Could you make the changes to autogen and run make build
to generate the modules.
Thanks @bharathkkb! Looks like I haven't contributed in so long. Things have changed a lot :) |
/gcbrun |
@gtsorbo looks like an unrelated error and just needs to be rerun?
|
Fixing a bug with Autopilot Private cluster submodule
The default value provided in the module makes it not possible to create multiple clusters in the same VPC. This was likely added to cover for a provider bug that required
master_ipv4_cidr_block
to be set even for autopilot clusters but it was fixed days after the module additions for autopilot private were merged.