Skip to content

Auto-delete s3://nextstrain-ncov-private/trial*#71

Merged
victorlin merged 1 commit into
mainfrom
victorlin/s3-lifecycle-rules
Jun 3, 2026
Merged

Auto-delete s3://nextstrain-ncov-private/trial*#71
victorlin merged 1 commit into
mainfrom
victorlin/s3-lifecycle-rules

Conversation

@victorlin

@victorlin victorlin commented Jun 2, 2026

Copy link
Copy Markdown
Member

Description of proposed changes

These are not meant for long-term storage.

Plan: 0 to add, 1 to change, 0 to destroy.

Related issue(s)

https://github.com/nextstrain/private/issues/180

Checklist

  • Checks pass
  • Post-merge: apply plan

@victorlin victorlin self-assigned this Jun 2, 2026

@jameshadfield jameshadfield left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can't comment on the terraform aspects, but 👍 for the direction

These are not meant for long-term storage.

Plan: 0 to add, 1 to change, 0 to destroy.
@victorlin victorlin force-pushed the victorlin/s3-lifecycle-rules branch from 2799b4d to 5821730 Compare June 3, 2026 21:28
Comment thread env/production/aws-s3-bucket-nextstrain-ncov-private.tf
@victorlin victorlin merged commit 59ef8b5 into main Jun 3, 2026
1 check passed
@victorlin victorlin deleted the victorlin/s3-lifecycle-rules branch June 3, 2026 21:38
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.

3 participants