-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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: Ignore changes to bootstrap_cluster_creator_admin_permissions
which is disabled by default
#3042
fix: Ignore changes to bootstrap_cluster_creator_admin_permissions
which is disabled by default
#3042
Conversation
…w upgrading module and importing existing clusters
This fix is way simpler than I was anticipating! Thanks @wevanscfi! |
Co-authored-by: Igor Brites <igor@brites.dev>
❤️ |
bootstrap_cluster_creator_admin_permissions
which is disabled by default
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!
## [20.11.1](v20.11.0...v20.11.1) (2024-05-21) ### Bug Fixes * Ignore changes to `bootstrap_cluster_creator_admin_permissions` which is disabled by default ([#3042](#3042)) ([c65d308](c65d308))
This PR is included in version 20.11.1 🎉 |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Description
Ignore changes to bootstrap_cluster_creator_admin_permissions to allow upgrading module and importing existing clusters
Motivation and Context
Fixes: #3025
Breaking Changes
None
How Has This Been Tested?
examples/*
to demonstrate and validate my change(s)examples/*
projectspre-commit run -a
on my pull request