-
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
feat!: Add support for enable_l4_ilb_subsetting
flag
#896
feat!: Add support for enable_l4_ilb_subsetting
flag
#896
Conversation
Thanks for the PR! 🚀 |
cc6732a
to
7bcf966
Compare
@bharathkkb @morgante @terraform-google-modules/cft-admins this is ready for review :) |
enable_l4_ilb_subsetting
flag
enable_l4_ilb_subsetting
flagenable_l4_ilb_subsetting
flag
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 @Jberlinsky
Since we are bumping min provider versions this will be a breaking change.
enable_l4_ilb_subsetting
flagenable_l4_ilb_subsetting
flag
Remove unnecessary mention of `enabled` RE: `enable_l4_ilb_subsetting` variable Co-authored-by: Bharath KKB <[email protected]>
After this commit - 7531f90,
Is the issue because of this Change or global issue with GKE Appreciate any help on this. |
Hi @gauravkr19 |
Thanks @bharathkkb for the quick response. Can get the cluster up now, must be due to the outage at that point of time(tried 3 times). By the way, I am implementing the jenkins-pipeline from below article and its working with terraform 0.12.24. |
…gle-modules#896) * Add support for enable_l4_ilb_subsetting flag * Fix typo * Bump minimum google-beta provider version * Bump google-provider-beta version in test definitions * Update autogen/main/variables.tf.tmpl Remove unnecessary mention of `enabled` RE: `enable_l4_ilb_subsetting` variable Co-authored-by: Bharath KKB <[email protected]> * Remove unnecessary `enabled` identifier in description Co-authored-by: Bharath KKB <[email protected]>
Adds support for the
google_container_cluster
enable_l4_ilb_subsetting flag, which is particularly useful for clusters that require more than 250 total Nodes and build ILBs.Note that
enable_l4_ilb_subsetting
is only supported ingoogle-beta >= 3.63.0
. Minimum module versions are bumped in accordance with this version change.