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

provider/aws: Trap Instance error from mismatched SG IDs and Names #4240

Merged
merged 1 commit into from
Jan 13, 2016

Conversation

catsby
Copy link
Contributor

@catsby catsby commented Dec 9, 2015

Closes #3798 – there isn't much we can do to prevent this (that I see at this time), but this at least proves a clearer error message if someone hits this.

@catsby
Copy link
Contributor Author

catsby commented Dec 9, 2015

ping @pawelsawicz – let me know if you feel this is enough to close out #3798

@pawelsawicz
Copy link

@catsby
Tested. For me seems ok, message is clear enough to understand.

Thanks!

@catsby
Copy link
Contributor Author

catsby commented Dec 10, 2015

Thanks for the follow up @pawelsawicz – it's not perfect but it's better 😄

@pawelsawicz
Copy link

Yeah true :) 👍 @catsby good work!

@phinze
Copy link
Contributor

phinze commented Jan 6, 2016

LGTM

catsby added a commit that referenced this pull request Jan 13, 2016
provider/aws: Trap Instance error from mismatched SG IDs and Names
@catsby catsby merged commit c9231a7 into master Jan 13, 2016
@mitchellh mitchellh deleted the b-aws-catch-sg-name-id-error branch March 22, 2016 19:21
@ghost
Copy link

ghost commented Apr 27, 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 Apr 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

aws_instance security_groups when provide id throwing error
4 participants