Page MenuHomeSoftware Heritage

celery: add 2 statsd probes for the runner and listener
ClosedPublic

Authored by douardda on Dec 4 2019, 10:58 AM.

Details

Summary
  • runner: counting the number of scheduled tasks,
  • listener: counting the number of processed events.

Related to T2133

Diff Detail

Repository
rDSCH Scheduling utilities
Branch
t2133
Lint
Lint Skipped
Unit
Unit Tests Skipped
Build Status
Buildable 9504
Build 13985: tox-on-jenkinsJenkins
Build 13984: arc lint + arc unit

Event Timeline

rebased + rename statsd probes (-scheduler/+total)

ardumont added a subscriber: ardumont.

Looks good to me

swh/scheduler/celery_backend/listener.py
53

That could be the occasion to also have a counter for unhandled types (i have no idea how noisy that would be).
swh_listener_unhandled_event_total or something?

This revision is now accepted and ready to land.Dec 4 2019, 3:33 PM
swh/scheduler/celery_backend/listener.py
53

not sure we want to engage the fight with celery on this. Who knows how many event type may exists here?

ok, let's keep _scheduler_ in probe names

also fix a stupid "typo"...