Skip to content

feat: Scout monitoring for celery workers#6342

Merged
rjsparks merged 2 commits intoietf-tools:mainfrom
jennifer-richards:scout-celery
Sep 21, 2023
Merged

feat: Scout monitoring for celery workers#6342
rjsparks merged 2 commits intoietf-tools:mainfrom
jennifer-richards:scout-celery

Conversation

@jennifer-richards
Copy link
Copy Markdown
Member

This matches what was deployed via patch on 2023-09-20.

We might want to consider renaming the environment variables we're using to configure Scout to something more distinct, but at least wanted to put up a PR matching what was deployed.

@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 20, 2023

Codecov Report

Merging #6342 (720217e) into main (32bb4ef) will increase coverage by 0.00%.
Report is 34 commits behind head on main.
The diff coverage is 87.83%.

@@           Coverage Diff           @@
##             main    #6342   +/-   ##
=======================================
  Coverage   88.69%   88.69%           
=======================================
  Files         290      290           
  Lines       40390    40422   +32     
=======================================
+ Hits        35823    35852   +29     
- Misses       4567     4570    +3     
Files Changed Coverage Δ
ietf/celeryapp.py 0.00% <0.00%> (ø)
ietf/help/views.py 63.63% <ø> (+8.46%) ⬆️
ietf/ipr/models.py 88.95% <ø> (ø)
ietf/urls.py 80.00% <ø> (-0.56%) ⬇️
ietf/ipr/views.py 87.96% <95.45%> (+0.24%) ⬆️
ietf/doc/models.py 88.98% <100.00%> (ø)
ietf/doc/utils.py 87.38% <100.00%> (+0.23%) ⬆️
ietf/doc/views_review.py 95.16% <100.00%> (+0.20%) ⬆️
ietf/doc/views_search.py 89.69% <100.00%> (+0.40%) ⬆️
ietf/ipr/feeds.py 93.33% <100.00%> (+0.22%) ⬆️
... and 3 more

... and 3 files with indirect coverage changes

@rjsparks rjsparks merged commit b08d26d into ietf-tools:main Sep 21, 2023
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 25, 2023
@jennifer-richards jennifer-richards deleted the scout-celery branch July 12, 2024 14:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants