-
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
r/aws_efs_replication_configuration: support an existing FS as destination #34955
r/aws_efs_replication_configuration: support an existing FS as destination #34955
Conversation
Community NoteVoting for Prioritization
For Submitters
|
719e81e
to
d84a349
Compare
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.
Welcome @yaronya 👋
It looks like this is your first Pull Request submission to the Terraform AWS Provider! If you haven’t already done so please make sure you have checked out our CONTRIBUTOR guide and FAQ to make sure your contribution is adhering to best practice and has all the necessary elements in place for a successful approval.
Also take a look at our FAQ which details how we prioritize Pull Requests for inclusion.
Thanks again, and welcome to the community! 😃
destination
block of resource aws_efs_replication_configuration
resourcedestination
block of resource aws_efs_replication_configuration
destination
block of resource aws_efs_replication_configuration
2ce8fc2
to
2fea129
Compare
Waiting on #35029 to complete tests |
…ion_configuration resource Signed-off-by: Yaron Yarimi <yaron.yarimi@env0.com>
Signed-off-by: Yaron Yarimi <yaron.yarimi@env0.com>
Signed-off-by: Yaron Yarimi <yaron.yarimi@env0.com>
Signed-off-by: Yaron Yarimi <yaron.yarimi@env0.com>
Signed-off-by: Yaron Yarimi <yaron.yarimi@env0.com>
Signed-off-by: Yaron Yarimi <yaron.yarimi@env0.com>
Signed-off-by: Yaron Yarimi <yaron.yarimi@env0.com>
c67748b
to
8b05dc2
Compare
…tion test Signed-off-by: Yaron Yarimi <yaron.yarimi@env0.com>
Hey @justinretzolk @ewbankkit See logs:
|
Signed-off-by: Yaron Yarimi <yaron.yarimi@env0.com>
@yaronya If there is a better default timeout value then you can change the value in |
Signed-off-by: Yaron Yarimi <yaron.yarimi@env0.com>
Signed-off-by: Yaron Yarimi <yaron.yarimi@env0.com>
Signed-off-by: Yaron Yarimi <yaron.yarimi@env0.com>
…ternate region Signed-off-by: Yaron Yarimi <yaron.yarimi@env0.com>
Signed-off-by: Yaron Yarimi <yaron.yarimi@env0.com>
fae2a6d
to
13b9fa2
Compare
Signed-off-by: Yaron Yarimi <yaron.yarimi@env0.com>
…fault Create timeout value.
…tionConfiguration_disappears' (prevent panic).
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.
LGTM 🚀.
% make testacc TESTARGS='-run=TestAccEFSMountTarget_disappears' PKG=efs ACCTEST_PARALLELISM=2
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/efs/... -v -count 1 -parallel 2 -run=TestAccEFSMountTarget_disappears -timeout 360m
=== RUN TestAccEFSMountTarget_disappears
=== PAUSE TestAccEFSMountTarget_disappears
=== CONT TestAccEFSMountTarget_disappears
--- PASS: TestAccEFSMountTarget_disappears (123.10s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/efs 132.496s
% make testacc TESTARGS='-run=TestAccEFSReplicationConfiguration_basic\|TestAccEFSReplicationConfiguration_disappears' PKG=efs
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/efs/... -v -count 1 -parallel 20 -run=TestAccEFSReplicationConfiguration_basic\|TestAccEFSReplicationConfiguration_disappears -timeout 360m
=== RUN TestAccEFSReplicationConfiguration_basic
=== PAUSE TestAccEFSReplicationConfiguration_basic
=== RUN TestAccEFSReplicationConfiguration_disappears
=== PAUSE TestAccEFSReplicationConfiguration_disappears
=== CONT TestAccEFSReplicationConfiguration_basic
=== CONT TestAccEFSReplicationConfiguration_disappears
--- PASS: TestAccEFSReplicationConfiguration_disappears (462.95s)
--- PASS: TestAccEFSReplicationConfiguration_basic (547.15s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/efs 553.120s
% make testacc TESTARGS='-run=TestAccEFSReplicationConfiguration_allAttributes' PKG=efs
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/efs/... -v -count 1 -parallel 20 -run=TestAccEFSReplicationConfiguration_allAttributes -timeout 360m
=== RUN TestAccEFSReplicationConfiguration_allAttributes
=== PAUSE TestAccEFSReplicationConfiguration_allAttributes
=== CONT TestAccEFSReplicationConfiguration_allAttributes
--- PASS: TestAccEFSReplicationConfiguration_allAttributes (544.90s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/efs 551.105s
% make testacc TESTARGS='-run=TestAccEFSReplicationConfiguration_existingDestination' PKG=efs
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/efs/... -v -count 1 -parallel 20 -run=TestAccEFSReplicationConfiguration_existingDestination -timeout 360m
=== RUN TestAccEFSReplicationConfiguration_existingDestination
=== PAUSE TestAccEFSReplicationConfiguration_existingDestination
=== CONT TestAccEFSReplicationConfiguration_existingDestination
--- PASS: TestAccEFSReplicationConfiguration_existingDestination (1437.97s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/efs 1443.721s
@yaronya Thanks for the contribution 🎉 👏. |
This functionality has been released in v5.32.0 of the Terraform AWS Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you! |
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. |
Description
AWS now supports EFS replication to an existing file system, and won't force you to replicate your source FS into a new one.
This PR takes the existing computed
file_system_id
attribute ofaws_efs_replication_configuration
resource and sets it to be also optional and sends it in the CreateReplicationConfiguration API callRelations
Closes #34863
References
Output from Acceptance Testing