Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

feat: add some metrics for twisted's threadpool #1094

Merged
merged 1 commit into from
Dec 7, 2017
Merged

Conversation

pjenvey
Copy link
Member

@pjenvey pjenvey commented Dec 2, 2017

emitted every 15 seconds per datadog's preferred minimum

Closes #1093

# type: (IMetrics, Optional[str]) -> None
"""Emit metrics on twisted's thread pool.

Only meant to be ran as a LoopingCall (TimerService).
Copy link
Member

Choose a reason for hiding this comment

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

nit: "run"

emitted every 15 seconds per datadog's preferred minimum

Closes #1093
@codecov-io
Copy link

codecov-io commented Dec 6, 2017

Codecov Report

Merging #1094 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1094      +/-   ##
==========================================
+ Coverage    99.7%   99.73%   +0.02%     
==========================================
  Files          58       58              
  Lines        9284     9506     +222     
==========================================
+ Hits         9257     9481     +224     
+ Misses         27       25       -2
Impacted Files Coverage Δ
autopush/tests/test_metrics.py 100% <100%> (ø) ⬆️
autopush/main.py 86.74% <100%> (+0.24%) ⬆️
autopush/metrics.py 100% <100%> (ø) ⬆️
autopush/tests/test_db.py 100% <0%> (ø) ⬆️
autopush/web/health.py 100% <0%> (ø) ⬆️
autopush/tests/test_integration.py 100% <0%> (ø) ⬆️
autopush/memusage.py 100% <0%> (ø) ⬆️
autopush/diagnostic_cli.py 100% <0%> (ø) ⬆️
autopush/db.py 99.79% <0%> (+0.36%) ⬆️
autopush/web/registration.py 100% <0%> (+0.44%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 82b1fbd...6a968c6. Read the comment docs.

@bbangert bbangert merged commit bec183c into master Dec 7, 2017
@bbangert bbangert deleted the feat/1093 branch December 7, 2017 21:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants