[blob] Add logs to cleanup task
ClosedPublic
Actions

Authored by bartek on Oct 9 2023, 5:40 AM.

Details

Reviewers

michal
varun
• jon

Commits

rCOMM80af6b4fe855: [blob] Add logs to cleanup task

Summary

Added logs to cleanup task to help debugging. I didn't add them previously because I didn't want to bloat the (already a bit complicated) diffs with logic.

Depends on D9413

Test Plan

RUST_LOG=blob=trace cargo run -- cleanup displayed the logs.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Oct 9 2023, 5:40 AM

bartek held this revision as a draft.

Herald added a reviewer: • jon. · View Herald TranscriptOct 9 2023, 5:40 AM

Herald added subscribers: will, tomek, ashoat. · View Herald Transcript

bartek added a child revision: D9415: [terraform] Add blob cleanup cron task.Oct 9 2023, 5:42 AM

bartek published this revision for review.Oct 9 2023, 6:03 AM

Harbormaster completed remote builds in B23089: Diff 31791.Oct 9 2023, 6:04 AM

michal accepted this revision.Oct 10 2023, 6:37 AM

This revision is now accepted and ready to land.Oct 10 2023, 6:37 AM

Rebase on feedback

Harbormaster completed remote builds in B23178: Diff 31922.Oct 11 2023, 3:20 AM

Closed by commit rCOMM80af6b4fe855: [blob] Add logs to cleanup task (authored by bartek). · Explain WhyOct 16 2023, 1:40 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMM80af6b4fe855: [blob] Add logs to cleanup task.

Revision Contents
Changeset List

Path

Size

services/

blob/

src/

service.rs

55 lines

Diff 31791

View Options

services/blob/src/service.rs

[blob] Add logs to cleanup taskClosedPublicActions