Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat: add transition_default_minimum_object_size#274

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
jjurzyn wants to merge2 commits intocloudposse:main
base:main
Choose a base branch
Loading
fromjjurzyn:main

Conversation

@jjurzyn
Copy link

what

why

  • In September 2024 Amazon S3 updated the default transition behavior for small objects that prevents objects smaller than 128 KB from being transitioned to any storage class.
  • All lifecycle rules created before September 2024 have default transition behavior set to varies_by_storage_class.
  • Currently, the module doesn't support changing it to the previous default value.

terraform plan wants to modify lifecycle configuration as follows:

# module.s3.aws_s3_bucket_lifecycle_configuration.default[0] will be updated in-place! resource "aws_s3_bucket_lifecycle_configuration" "default" {        id                                     = <name>!       transition_default_minimum_object_size = "varies_by_storage_class" -> "all_storage_classes_128K"        # (1 unchanged attribute hidden)        # (1 unchanged block hidden)    }

references

sht reacted with thumbs up emoji
@jjurzynjjurzyn requested review froma team ascode ownersMay 12, 2025 16:29
@mergifymergifybot added the triageNeeds triage labelMay 12, 2025
@jjurzyn
Copy link
Author

Fixes#267

@oycyc
Copy link

/terratest

github-actions[bot] reacted with thumbs up emoji

@mergify
Copy link

mergifybot commentedJun 11, 2025

💥 This pull request now has conflicts. Could you fix it @jjuinfare? 🙏

@mergifymergifybot added the conflictThis PR has conflicts labelJun 11, 2025
@sht
Copy link

sht commentedJul 4, 2025

Can somebody please resolve the conflicts and merge this? I need this feature 😄

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@johncblandiijohncblandiiAwaiting requested review from johncblandiijohncblandii is a code owner automatically assigned from cloudposse/contributors

@joe-nilandjoe-nilandAwaiting requested review from joe-nilandjoe-niland is a code owner automatically assigned from cloudposse/contributors

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

conflictThis PR has conflictstriageNeeds triage

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jjurzyn@oycyc@sht

[8]ページ先頭

©2009-2025 Movatter.jp