Page MenuHomePhabricator
Differential D13942

[backup] add errorType to error logs in backup
ClosedPublic
Actions

Authored by will on Nov 14 2024, 8:45 PM.
Tags
None
Referenced Files
F3512590: D13942.id45838.diff
Sat, Dec 21, 8:28 PM
F3512589: D13942.id45833.diff
Sat, Dec 21, 8:28 PM
F3512548: D13942.id.diff
Sat, Dec 21, 8:27 PM
F3512524: D13942.diff
Sat, Dec 21, 8:26 PM
Unknown Object (File)
Tue, Dec 3, 3:45 PM
Unknown Object (File)
Mon, Dec 2, 9:52 PM
Unknown Object (File)
Mon, Dec 2, 6:13 AM
Unknown Object (File)
Sun, Dec 1, 9:50 AM
View All 45 Files
Subscribers
ashoat
tomek

Details

Reviewers
kamil
bartek
Commits
rCOMM3d2db7642854: [backup] add errorType to error logs in backup
Summary

Add errorTypes to errors logs in backup service.

This includes:

  • DDB Error
  • Auth Error
  • Blob Error
  • WS Error

Depends on D13941

Test Plan

Deployed to staging. Verified that JSON error logs on cloudwatch include errorType

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

will created this revision.Nov 14 2024, 8:45 PM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptNov 14 2024, 8:45 PM
will added a child revision: D13943: [terraform] break up alarms into separate terraform files.Nov 14 2024, 8:52 PM
will retitled this revision from [backup] add errorType logs to backup to [backup] add errorType to error logs in backup.Nov 14 2024, 8:53 PM
will mentioned this in D13944: [terraform] create backup cloudwatch alarms.Nov 14 2024, 8:57 PM
Harbormaster completed remote builds in B32674: Diff 45833.Nov 14 2024, 9:03 PM
will requested review of this revision.Nov 14 2024, 9:03 PM
bartek accepted this revision.Nov 15 2024, 2:16 AM
bartek added 1 blocking reviewer(s): kamil.
kamil accepted this revision.Nov 15 2024, 2:17 AM
kamil added 1 blocking reviewer(s): bartek.

LGTM but @bartek should review this too

bartek accepted this revision.Nov 15 2024, 2:18 AM
This revision is now accepted and ready to land.Nov 15 2024, 2:18 AM
Closed by commit rCOMM3d2db7642854: [backup] add errorType to error logs in backup (authored by will). · Explain WhyNov 15 2024, 9:53 AM
This revision was automatically updated to reflect the committed changes.
will added a commit: rCOMM3d2db7642854: [backup] add errorType to error logs in backup.
will mentioned this in rCOMM9fe6ac4e8124: [terraform] create backup cloudwatch alarms.Nov 18 2024, 4:54 AM

Revision Contents
Changeset List

PathSize
services/
backup/
src/
9 lines
database/
34 lines
17 lines
http/
handlers/
7 lines

Diff 45838

View Options

services/backup/src/constants.rs

Loading...
View Options

services/backup/src/database/mod.rs

Loading...
View Options

services/backup/src/error.rs

Loading...
View Options

services/backup/src/http/handlers/log.rs

Loading...