Skip to content

Comments

WFPREV-41 : Add alarms to cloudfront#1066

Merged
sukhpalp merged 2 commits intomainfrom
WFPREV-41
Jan 8, 2026
Merged

WFPREV-41 : Add alarms to cloudfront#1066
sukhpalp merged 2 commits intomainfrom
WFPREV-41

Conversation

@sukhpalp
Copy link
Collaborator

@sukhpalp sukhpalp commented Jan 5, 2026

No description provided.

Copy link
Contributor

@dhlevi dhlevi left a comment

Choose a reason for hiding this comment

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

Alarms look good, but the thresholds might need some tweaking based on what we see in behaviour.

Worth adding a check/alarm/canary for some other things too, like inability to hit the OAuth service (early on-prem outage warning) or for outages to any BCGW connections. This could also be added to the API healthcheck (not the readiness check for instance spin down!) for ease of checking with a simple response package. Could use a lambda

@sukhpalp
Copy link
Collaborator Author

sukhpalp commented Jan 8, 2026

Alarms look good, but the thresholds might need some tweaking based on what we see in behaviour.

Worth adding a check/alarm/canary for some other things too, like inability to hit the OAuth service (early on-prem outage warning) or for outages to any BCGW connections. This could also be added to the API healthcheck (not the readiness check for instance spin down!) for ease of checking with a simple response package. Could use a lambda

Split this into another ticket because the changes would be bigger than the scope of this PR. WFPREV-887

@sukhpalp sukhpalp merged commit 48ded2a into main Jan 8, 2026
5 of 6 checks passed
@sukhpalp sukhpalp deleted the WFPREV-41 branch January 8, 2026 22:18
sukhpalp added a commit that referenced this pull request Jan 27, 2026
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