-
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
[ready] [breaking] Added monitoring_config and logging_config variables to beta clusters #1028
Conversation
@rosmo I assume this is to allow enabling new |
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 contributing.
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.
Since you increased the minimum required version, versions in examples will need to be updated.
Thanks for the PR! 🚀 |
…riables to beta clusters and updated provider version.
I updated all the examples to the newer provider version. |
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 contributing!
This should be merged in our next breaking release. |
@rosmo Can you review the conflicts? |
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.
Test failure is unrelated.
Did anyone get this change to work? I'm getting issue of not being able to use it at all, getting errors as in this issue: hashicorp/terraform-provider-google#10361 |
…nts variables to beta clusters (#1028)
…nts variables to beta clusters (terraform-google-modules#1028)
Related changes: hashicorp/terraform-provider-google#10125