Skip to content

Minimal provider version for aws_s3_directory_bucket #263

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

salemgolemugoo
Copy link

@salemgolemugoo salemgolemugoo commented Dec 11, 2024

what

Updated AWS provider version for the new resource used in the module aws_s3_directory_bucket

why

Old provider doesn't support this resource The provider hashicorp/aws does not support resource type

references

https://github.com/hashicorp/terraform-provider-aws/blob/main/CHANGELOG.md#5280-november-29-2023

@salemgolemugoo salemgolemugoo requested review from a team as code owners December 11, 2024 12:48
@mergify mergify bot added the triage Needs triage label Dec 11, 2024
@nitrocode
Copy link
Member

/terratest

@nitrocode
Copy link
Member

@Gowiem this would require a major version, right? If so, then we'd probably need to add all the non major changes first before this goes in

@Gowiem
Copy link
Member

Gowiem commented Jun 2, 2025

@nitrocode I think we are treating major provider version bumps as major module version bumps, so that sounds right 👍

@Gowiem Gowiem added enhancement New feature or request major Breaking changes (or first stable release) labels Jun 2, 2025
@Gowiem
Copy link
Member

Gowiem commented Jun 2, 2025

Oh but @salemgolemugoo is saying that we're already using aws_s3_directory_bucket (which we are) and that resources REQUIRES v5.x of the AWS Provider... so we should've already did a major rev when we introduced that resource... damn.

@nitrocode I feel like it probably makes sense to raise this as a major rev and update all of the release docs from when that resource was added to mention that it should've included this provider rev to major. Mind taking that on?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request major Breaking changes (or first stable release) triage Needs triage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants