-
-
Notifications
You must be signed in to change notification settings - Fork 559
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: Add security_group_use_name_prefix
variable to enable/disable name prefix usage
#289
feat: Add security_group_use_name_prefix
variable to enable/disable name prefix usage
#289
Conversation
b582f93
to
34b1c02
Compare
Thanks for the PR @naseemkullah ! can you please run |
df698b3
to
77e8c0b
Compare
my pleasure @bryantbiggs , done! would it be possible to cut a release immediately upon merging? |
@naseemkullah releases are automatic after each merge |
security_group_use_name_prefix
variable to enable/disable name prefix usage
77e8c0b
to
0f386ba
Compare
Alright, perfect! Looking forward to it. |
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.
looks good to me - @antonbabenko your thoughts?
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.
As long as it is backward-compatible, it is good.
## [7.1.0](v7.0.0...v7.1.0) (2022-05-02) ### Features * Add `security_group_use_name_prefix` variable to enable/disable name prefix usage ([#289](#289)) ([75ffb30](75ffb30))
This PR is included in version 7.1.0 🎉 |
I'm going to lock this pull request 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 related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Description
Add
security_group_use_name_prefix
that defaults to true, which toggles wether to use name_prefix or name within the security groupMotivation and Context
name/name-prefix changes in security groups are destructive, in order to import resources to this module I need to be able to use just name and not name_prefix.
Breaking Changes
keeping name-prefix as the default does not introduce breaking changes
How Has This Been Tested?
examples/*
to demonstrate and validate my change(s)examples/*
projectspre-commit run -a
on my pull request