Skip to content

Catch KeyError if there is no monitoring switch to remove#573

Merged
stephen-riggs merged 1 commit intomainfrom
session-delete-prom
Apr 14, 2025
Merged

Catch KeyError if there is no monitoring switch to remove#573
stephen-riggs merged 1 commit intomainfrom
session-delete-prom

Conversation

@stephen-riggs
Copy link
Contributor

When pressing the delete button on sessions we sometimes get a key error from prometheus

@codecov
Copy link

codecov bot commented Apr 11, 2025

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 30.19%. Comparing base (e150506) to head (d1ffc9a).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #573      +/-   ##
==========================================
- Coverage   30.20%   30.19%   -0.01%     
==========================================
  Files          79       79              
  Lines       10500    10503       +3     
  Branches     1401     1401              
==========================================
  Hits         3171     3171              
- Misses       7223     7226       +3     
  Partials      106      106              
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@stephen-riggs stephen-riggs merged commit bdfb0c7 into main Apr 14, 2025
17 checks passed
@stephen-riggs stephen-riggs deleted the session-delete-prom branch April 14, 2025 08:08
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