Skip to content

Remove "catprocio" for imgsrv monitoring#979

Merged
rrotter merged 2 commits intoproductionfrom
remove-imgsrv-catprocio
Apr 20, 2026
Merged

Remove "catprocio" for imgsrv monitoring#979
rrotter merged 2 commits intoproductionfrom
remove-imgsrv-catprocio

Conversation

@aelkiss
Copy link
Copy Markdown
Member

@aelkiss aelkiss commented Apr 14, 2026

We never really made use of the results of this and it was always a bit of a hack.

Should merge & deploy this first: hathitrust/babel#200

We never really made use of the results of this and it was always a bit
of a hack.
@aelkiss aelkiss requested a review from rrotter April 16, 2026 14:13
@aelkiss aelkiss marked this pull request as ready for review April 16, 2026 14:13
@aelkiss
Copy link
Copy Markdown
Member Author

aelkiss commented Apr 16, 2026

@rrotter hathitrust/babel#200 is merged & deployed; so we can go ahead and deploy this whenever after you review & approve.

@rrotter
Copy link
Copy Markdown
Member

rrotter commented Apr 17, 2026

git rm files/imgsrv/catprocio?

Copy link
Copy Markdown
Member

@rrotter rrotter left a comment

Choose a reason for hiding this comment

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

I'll merge this on Monday morning (2026-04-20)

@rrotter rrotter merged commit 9ba3d28 into production Apr 20, 2026
7 checks passed
@rrotter rrotter deleted the remove-imgsrv-catprocio branch April 20, 2026 13:02
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