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

feat: add support for logging_variant in Autopilot clusters #1962

Conversation

abdel
Copy link
Contributor

@abdel abdel commented May 31, 2024

Leverage the existing logging_variant [ref] option at a cluster-level in google_container_cluster module to (optionally) allow enabling MAX_THROUGHPUT logging in Autopilot clusters.

@abdel abdel marked this pull request as ready for review May 31, 2024 03:27
@abdel abdel requested review from ericyz, gtsorbo and a team as code owners May 31, 2024 03:27
Copy link
Contributor

@apeabody apeabody left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution @abdel!

Can you please make your change in the source template as described here: https://github.com/terraform-google-modules/terraform-google-kubernetes-engine/blob/master/CONTRIBUTING.md#templating. Thanks!

@abdel
Copy link
Contributor Author

abdel commented Jul 10, 2024

@apeabody Sorry for the late response, didn't get a notification for some reason! I made the changes to the autogen templates, and did the re-build for the modules and the docs.

Let me know if that looks OK!

@abdel abdel requested a review from apeabody July 10, 2024 04:15
Copy link
Contributor

@apeabody apeabody left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution @abdel!

autogen/main/variables.tf.tmpl Outdated Show resolved Hide resolved
@apeabody apeabody self-assigned this Jul 10, 2024
@apeabody
Copy link
Contributor

/gcbrun

@apeabody
Copy link
Contributor

/gcbrun

Copy link
Contributor

@apeabody apeabody left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution @abdel!

@apeabody apeabody merged commit 08028a8 into terraform-google-modules:master Jul 11, 2024
4 checks passed
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
2 participants