-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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
[WIP] Allow update instead for recreate for aws_security_group_rule. #10193
Conversation
I have not tested this at all yet. I'm also not entirely sure what tests if any I should add. |
Pull request #21306 has significantly refactored the AWS Provider codebase. As a result, most PRs opened prior to the refactor now have merge conflicts that must be resolved before proceeding. Specifically, PR #21306 relocated the code for all AWS resources and data sources from a single We recognize that many pull requests have been open for some time without yet being addressed by our maintainers. Therefore, we want to make it clear that resolving these conflicts in no way affects the prioritization of a particular pull request. Once a pull request has been prioritized for review, the necessary changes will be made by a maintainer -- either directly or in collaboration with the pull request author. For a more complete description of this refactor, including examples of how old filepaths and function names correspond to their new counterparts: please refer to issue #20000. For a quick guide on how to amend your pull request to resolve the merge conflicts resulting from this refactor and bring it in line with our new code patterns: please refer to our Service Package Refactor Pull Request Guide. |
20ddc50
to
84b16a9
Compare
With this change, changes it cidr_blocks and ipv6_cidr_blocks will only remove/add the cidr ranges that were removed/added in config, rather than destroying the entire resource and recreating it. It also changes the type of those attributes to sets to make the diffs more readable.
84b16a9
to
a5f2c9d
Compare
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. |
With this change, changes it cidr_blocks and ipv6_cidr_blocks will only
remove/add the cidr ranges that were removed/added in config, rather
than destroying the entire resource and recreating it.
It also changes the type of those attributes to sets to make the diffs
more readable.
Currently,
aws_security_group_rule
is destroyed and recreated whenever there is a change tocidr_blocks
. Which is unacceptable, since adding a single cidr block to the resource could result in a service interruption.Community Note
Relates #10097
This is also related to #4726 which does something similar for the
aws_security_group
resource.Release note for CHANGELOG:
Output from acceptance testing: