Page MenuHomeSoftware Heritage

Add basic stats to tasks
ClosedPublic

Authored by olasd on Jan 24 2019, 5:52 PM.

Details

Summary

This just increments a counter for started tasks, ended tasks, tasks that failed
with an exception. It also registers a timer for every task run.

The way the celery worker name is pulled is a bit hideous and there must be a
better way, but I couldn't find one...

Test Plan

Need to figure out the proper way to inject a statsd in "unit" tests,
but at least the Docker changes allow you to see what the data would look like.

Diff Detail

Repository
rDSCH Scheduling utilities
Branch
feature/statsd
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 4023
Build 5281: tox-on-jenkinsJenkins
Build 5280: arc lint + arc unit

Event Timeline

ardumont added a subscriber: ardumont.
ardumont added inline comments.
swh/scheduler/task.py
1

2019 ;)

This revision is now accepted and ready to land.Jan 25 2019, 9:38 AM
olasd marked an inline comment as done.Feb 6 2019, 1:07 PM
This revision was automatically updated to reflect the committed changes.