-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Make workload_identity_config dynamic #237
Conversation
@sylvioneto can you please update provider version for ./examples in this PR as well? |
Closed temporarily because it requires more tests, which I'm doing right now. I'll reopen once it is done. |
Reopening the PR because tests worked successfully. Solution was creating a local |
@kopachevsky, @morgante, all changes done. |
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.
Thank you for the contribution!
Make workload_identity_config dynamic
This PR makes workload_identity_config dynamic in the cluster configuration in order to solve the problem reported by @richardmcsong in this PR #234 (review), when a cluster was created without that config and should not prompt any change in the terraform plan.