Skip to content
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

Terraform should indicate which changes are impossible #1870

Closed
inferiorhumanorgans opened this issue May 8, 2015 · 2 comments · Fixed by #1871
Closed

Terraform should indicate which changes are impossible #1870

inferiorhumanorgans opened this issue May 8, 2015 · 2 comments · Fixed by #1871

Comments

@inferiorhumanorgans
Copy link

Playing around with security groups a bit last night, and I guess there's no API for changing the name or description of a SG. However if you change the description of an SG and run the plan and apply actions everything goes through without a hitch implying that it's possible to change the description.

phinze added a commit that referenced this issue May 8, 2015
Description cannot be handled in Update (there is no ModifySecurityGroup
API call), so we have to recreate to change the description.

Closes #1870
@phinze
Copy link
Contributor

phinze commented May 8, 2015

Thanks for the report. We do have a mechanism for handling attributes that cannot be updated - Terraform sees changes to these attributes as forcing a recreate of the resource. So it was just an oversight not to turn that flag on for SG descriptions. Attached PR will fix that. 👍

@ghost
Copy link

ghost commented May 2, 2020

I'm going to lock this issue 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 similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators May 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants