Skip to content

Auto-delete s3://nextstrain-ncov-private/*#73

Merged
victorlin merged 2 commits into
mainfrom
victorlin/update-lifecycle-rules
Jun 22, 2026
Merged

Auto-delete s3://nextstrain-ncov-private/*#73
victorlin merged 2 commits into
mainfrom
victorlin/update-lifecycle-rules

Conversation

@victorlin

@victorlin victorlin commented Jun 12, 2026

Copy link
Copy Markdown
Member

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

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

Checklist

  • Checks pass
  • Post-merge: apply plan

@victorlin victorlin self-assigned this Jun 12, 2026

filter {
prefix = "trial/"
prefix = "branch/"

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.

Are all files under the branch/ prefix gone and not to come back?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

They're gone now, but new files could be added if ncov-ingest is run with config/debug_sample_gisaid.yaml. Same with .github/workflows/fetch-and-ingest-gisaid-master.yml for trial/.

Comment thread env/production/aws-s3-bucket-nextstrain-ncov-private.tf
Use consistent naming and remove the now-redundant
abort_incomplete_multipart_upload config from prefix-specific rules.
@victorlin victorlin force-pushed the victorlin/update-lifecycle-rules branch from 8f21bf7 to f1c350e Compare June 18, 2026 21:21
@victorlin victorlin merged commit c20ecbd into main Jun 22, 2026
1 check passed
@victorlin victorlin deleted the victorlin/update-lifecycle-rules branch June 22, 2026 21:01
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