Skip to content
This repository was archived by the owner on Jan 25, 2023. It is now read-only.

Add option to use_existing_s3_bucket instead of creating one.#240

Open
queglay wants to merge 1 commit intohashicorp:masterfrom
queglay:pull-request-236
Open

Add option to use_existing_s3_bucket instead of creating one.#240
queglay wants to merge 1 commit intohashicorp:masterfrom
queglay:pull-request-236

Conversation

@queglay
Copy link
Contributor

@queglay queglay commented Apr 2, 2021

If var.use_existing_s3_bucket is true, will acquire the bucket id and arn by the provided var.s3_bucket_name instead of creating a new bucket in this module. This allows a user to avoid creating and destroying the S3 bucket repeatedly, and they can persist configuration from the s3 backend even after destroying this module.

@queglay queglay requested a review from robmorgan as a code owner April 2, 2021 05:23
@queglay
Copy link
Contributor Author

queglay commented May 8, 2021

I'm just checking, did I create this pull request correctly? It relates to the enhancement suggestion - #236

Copy link
Collaborator

@brikis98 brikis98 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!

@queglay
Copy link
Contributor Author

queglay commented Sep 24, 2021

I'm wondering what other needs might need to be met for this to be approved?

@queglay
Copy link
Contributor Author

queglay commented Mar 1, 2022

Is anything preventing this from being merged?

@hashicorp-cla
Copy link

hashicorp-cla commented Mar 12, 2022

CLA assistant check
All committers have signed the CLA.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments