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

providers/aws: Update Security Group docs #4139

Merged
merged 1 commit into from
Dec 2, 2015
Merged

Conversation

catsby
Copy link
Contributor

@catsby catsby commented Dec 2, 2015

Small doc update: Security Group names are optional; Terraform will assign a unique id if omitted.

helping

@jen20
Copy link
Contributor

jen20 commented Dec 2, 2015

LGTM.

catsby added a commit that referenced this pull request Dec 2, 2015
providers/aws: Update Security Group docs
@catsby catsby merged commit 1d71ffa into master Dec 2, 2015
@catsby catsby deleted the aws-sg-doc-update branch December 2, 2015 16:39
@scalp42
Copy link
Contributor

scalp42 commented Dec 2, 2015

pigging backing on this, any chance we can see a name_prefix attribute for this as well @jen20 or @catsby ? 🙏

@catsby
Copy link
Contributor Author

catsby commented Dec 3, 2015

@scalp42 I opened #4152 to track that request, though I don't know that I specifically will get to it anytime soon

@catsby
Copy link
Contributor Author

catsby commented Dec 4, 2015

#4167 has been opened to add the name_prefix, should land today I imagine

@scalp42
Copy link
Contributor

scalp42 commented Dec 4, 2015

Thanks a lot! :love:

@ghost
Copy link

ghost commented Apr 29, 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 29, 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.

3 participants