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

FIX: Add missing lifecycle rule for S3 Inventory #26996

Closed
wants to merge 1 commit into from

Conversation

tgxworld
Copy link
Contributor

When enable_s3_inventory site setting is enabled, inventory files will
be generated on a daily basis. However, we were not cleaning up the
inventory files and leading to large AWS S3 storage costs as the files
pile up overtime.

This commit adds a S3Inventory.purge_inventory class method which is
called on a daily basis by the Jobs::EnsureS3UploadsExistence job. The
S3Inventory.purge_inventory class method adds a lifecycle policy to
purge files in the prefix which the inventory files are generated after
2 days.

@github-actions github-actions bot added the i18n PRs which update English locale files or i18n related code label May 13, 2024
When `enable_s3_inventory` site setting is enabled, inventory files will
be generated on a daily basis. However, we were not cleaning up the
inventory files and leading to large AWS S3 storage costs as the files
pile up overtime.

This commit adds a `S3Inventory.purge_inventory` class method which is
called on a daily basis by the `Jobs::EnsureS3UploadsExistence` job. The
`S3Inventory.purge_inventory` class method adds a lifecycle policy to
purge files in the prefix which the inventory files are generated after
2 days.
@tgxworld tgxworld force-pushed the add_lifecycle_rule_for_inventories branch from 1d52e09 to 9bdaff6 Compare May 13, 2024 07:03
@tgxworld
Copy link
Contributor Author

tgxworld commented Jun 5, 2024

Superseded by #27304

@tgxworld tgxworld closed this Jun 5, 2024
@tgxworld tgxworld deleted the add_lifecycle_rule_for_inventories branch June 5, 2024 02:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i18n PRs which update English locale files or i18n related code
1 participant