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

S3 resource supports AWS provider v4 #77

Closed
wants to merge 12 commits into from

Conversation

sreetejap
Copy link

what

  • Modified aws_s3_bucket resource to support AWS provider v4.
  • Upgraded cloudposse/s3-log-storage/aws module to 0.28.0 to support AWS provider v4.

why

  • Current aws_s3_bucket resource does not support AWS provider v4.

@sreetejap sreetejap requested review from a team as code owners March 24, 2022 10:20
@sreetejap sreetejap requested review from florian0410 and SweetOps and removed request for a team March 24, 2022 10:20
Copy link

@bridgecrew bridgecrew bot left a comment

Choose a reason for hiding this comment

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

Bridgecrew has found infrastructure configuration errors in this PR ⬇️

main.tf Show resolved Hide resolved
main.tf Show resolved Hide resolved
main.tf Show resolved Hide resolved
main.tf Show resolved Hide resolved
main.tf Show resolved Hide resolved
main.tf Show resolved Hide resolved
main.tf Show resolved Hide resolved
main.tf Show resolved Hide resolved
main.tf Show resolved Hide resolved
Copy link

@Nuru Nuru left a comment

Choose a reason for hiding this comment

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

See below

Nuru
Nuru previously requested changes Apr 3, 2022
@Nuru Nuru dismissed their stale review April 3, 2022 06:15

Duplicate

@Nuru Nuru self-requested a review April 3, 2022 06:17
Copy link

@Nuru Nuru left a comment

Choose a reason for hiding this comment

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

Our road map calls for S3 bucket resources to be provisioned with our s3-bucket module, with its inputs passed through where relevant, rather than to piecemeal update the S3 bucket resources everywhere. Therefore these changes are not acceptable. You can view cloudposse/terraform-aws-s3-log-storage#71 as an example.

The migration to s3-log-storage v0.28.0 has a high risk of data loss. Any module this is updating to this version needs, at a minimum:

  • Explicit warnings about potential data loss
  • A clear migration document detailing how to safely perform the migration
  • A breaking change that will prevent automatic application of the upgrade in a situation that might trigger data loss

You can view the s3-log-storage migration documents as an example, and please note you do not need to copy or rewrite them, you can just reference them in the module-specific migration documents.

Once the module has been updated to the current s3-bucket and s3-log-storage modules and fully supports AWS provider v4, we will need to make an internal decision about version number changes. We may want to release this as v1.0.0 or v2.0.0.

@Nuru Nuru closed this in #93 Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants