Skip to content

ETT-316 issues with glacier expiration job#188

Merged
aelkiss merged 1 commit into
mainfrom
ETT-316_glacier_s3api
Jul 2, 2026
Merged

ETT-316 issues with glacier expiration job#188
aelkiss merged 1 commit into
mainfrom
ETT-316_glacier_s3api

Conversation

@moseshll

Copy link
Copy Markdown
Contributor
  • HTFeed::Storage::ObjectStore calls s3api delete-objects with zip and mets instead of separate rm calls.
  • Should halve the number of aws calls needed for Glacier expiration.
  • If successful should provide a model for deleting up to 1k items at a time.

- `HTFeed::Storage::ObjectStore` calls `s3api delete-objects` with zip and mets instead of separate `rm` calls.
- Should halve the number of aws calls needed for Glacier expiration.
- If successful should provide a model for deleting up to 1k items at a time.
@moseshll moseshll marked this pull request as ready for review June 29, 2026 19:25
@moseshll moseshll requested a review from aelkiss June 29, 2026 19:25

@aelkiss aelkiss 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.

Let's go for it.

One option to try it without needing to deploy (probably worthwhile, especially since we will likely want to increase the number of objects per call):

  • Start a pod with the necessary secrets etc with the command sleep infinity (e.g. by changing the command for the cron job and manually triggering the job)
  • Copy the updated script into /tmp on the pod (with kubectl cp)
  • Exec into the pod
  • Run the script using nohup

@aelkiss

aelkiss commented Jul 2, 2026

Copy link
Copy Markdown
Member

I'm going to go ahead and merge and deploy, and we can try this, although I suspect we'll probably want to still increase the number of objects per API call.

@aelkiss aelkiss merged commit 2b01546 into main Jul 2, 2026
1 check passed
@aelkiss aelkiss deleted the ETT-316_glacier_s3api branch July 2, 2026 11:28
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.

2 participants