-
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!: align keepers with ForceNew: true fields #1698
feat!: align keepers with ForceNew: true fields #1698
Conversation
This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days |
Not stale, still an open issue |
/gcbrun |
2 similar comments
/gcbrun |
/gcbrun |
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 contribution @lauraseidler!
Could you add the referenced tf state migration to (new) docs/upgrading_to_v29.0.md
Will cause node pools to be re-created on update, unless remote state of the `random_id` resources is manually modified to reflect the new keepers. Fixes terraform-google-modules#1695
4fda7c1
to
1defeaf
Compare
@apeabody sorry, it took me a while to find time for this again - done now :) |
/gcbrun |
/gcbrun |
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 contribution @lauraseidler!
Will cause node pools to be re-created on update, unless remote state
of the
random_id
resources is manually modified to reflect the new keepers.Fixes #1695