-
Notifications
You must be signed in to change notification settings - Fork 56
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
Vaerh/issue268 #287
Vaerh/issue268 #287
Conversation
@@ -9,7 +9,7 @@ import ( | |||
func ResourceInterfaceEoip() *schema.Resource { | |||
resSchema := map[string]*schema.Schema{ | |||
MetaResourcePath: PropResourcePath("/interface/eoip"), | |||
MetaId: PropId(Name), | |||
MetaId: PropId(Id), |
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.
@vaerh I tested this particular change. Functionally, that works as expected, but it breaks the existing state. The id
property is outdated in the state, and hence terraform loses the link. That can be easily fixed by reimporting the resource, which will update the id
property only.
That's kind of a breaking change, but we just released this feature, so I am not sure if that's critical.
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.
Ok, let's make it pretty. I have added state migration, please test it. Maybe there are some other requests.
For resources with the "Name" identifier
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.
Tested schema version bump of all the changed resources as I had all of them in my configuration. State migrated seamlessly -- I've got the correct ID's in the state instead of the resource names.
Great job! LGTM 👍
🎉 This PR is included in version 1.21.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
No description provided.