Skip to content

delete S3 Watcher (since we and others use queue based ingestion now)#4645

Merged
twrichards merged 1 commit intomainfrom
remove-s3watcher
Mar 26, 2026
Merged

delete S3 Watcher (since we and others use queue based ingestion now)#4645
twrichards merged 1 commit intomainfrom
remove-s3watcher

Conversation

@twrichards
Copy link
Copy Markdown
Contributor

@twrichards twrichards commented Feb 27, 2026 •
edited
Loading

As per #4026 and #4201 (plus lots of follow-up PRs) we have been using queue based ingestion for some time, rather than S3 Watcher - now is finally the team to clean-up.

mrs-doubtfire-cleaning

@twrichards twrichards added the maintenance Departmental tracking: maintenance work, not a fix or a feature label Feb 27, 2026
@twrichards
Copy link
Copy Markdown
Contributor Author

twrichards commented Feb 27, 2026

Hey @AndyKilmory please could I get a review of this given I imagine you're off S3 Watcher now

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 27, 2026 •
edited
Loading

Deploy build 14122 of media-service::grid::all to TEST

All deployment options

From guardian/actions-riff-raff.

@AndyKilmory
Copy link
Copy Markdown
Collaborator

AndyKilmory commented Mar 2, 2026

Hi @twrichards, I think we're happy for you to remove the S3 watcher code - just checking with Richard LE for confirmation and will let you know once he's got back to me. Thanks

@AndyKilmory
Copy link
Copy Markdown
Collaborator

AndyKilmory commented Mar 4, 2026

Have spoken with Ricahrd LE and he is happy for S3 Watcher to be removed given bespoke BBC sync loader doesn't use it and it will be retired in coming weeks

@twrichards twrichards marked this pull request as ready for review March 4, 2026 10:49
@twrichards twrichards requested a review from a team as a code owner March 4, 2026 10:49
@twrichards twrichards merged commit f1798bd into main Mar 26, 2026
8 checks passed
@twrichards twrichards deleted the remove-s3watcher branch March 26, 2026 15:05
@gu-prout
Copy link
Copy Markdown

gu-prout Bot commented Mar 26, 2026

Seen on cropper, kahuna, media-api (merged by @twrichards 8 minutes and 49 seconds ago) Please check your changes!

@gu-prout
Copy link
Copy Markdown

gu-prout Bot commented Mar 26, 2026

Seen on metadata-editor, thrall (merged by @twrichards 8 minutes and 58 seconds ago) Please check your changes!

@gu-prout
Copy link
Copy Markdown

gu-prout Bot commented Mar 26, 2026

Seen on auth, collections (merged by @twrichards 9 minutes and 8 seconds ago) Please check your changes!

@gu-prout
Copy link
Copy Markdown

gu-prout Bot commented Mar 26, 2026

Seen on image-loader, leases, usage (merged by @twrichards 9 minutes and 19 seconds ago) Please check your changes!

1 similar comment
@gu-prout
Copy link
Copy Markdown

gu-prout Bot commented Mar 26, 2026

Seen on image-loader, leases, usage (merged by @twrichards 9 minutes and 19 seconds ago) Please check your changes!

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

Reviewers

@andrew-nowak andrew-nowak andrew-nowak approved these changes

Assignees

@twrichards twrichards

@AndyKilmory AndyKilmory

Labels

maintenance Departmental tracking: maintenance work, not a fix or a feature Seen-on-auth Seen-on-collections Seen-on-cropper Seen-on-image-loader Seen-on-kahuna Seen-on-leases Seen-on-media-api Seen-on-metadata-editor Seen-on-thrall Seen-on-usage

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@twrichards @AndyKilmory @andrew-nowak