Merge ~twom/launchpad:stats-in-buildd-manager into launchpad:master
Proposed by
Tom Wardill
Status: | Merged |
---|---|
Approved by: | Tom Wardill |
Approved revision: | aef55cbb6e7e9d12113a9995466204785ac577ff |
Merge reported by: | Otto Co-Pilot |
Merged at revision: | not available |
Proposed branch: | ~twom/launchpad:stats-in-buildd-manager |
Merge into: | launchpad:master |
Diff against target: |
392 lines (+218/-5) 10 files modified
constraints.txt (+1/-0) lib/lp/buildmaster/interactor.py (+2/-2) lib/lp/buildmaster/manager.py (+40/-0) lib/lp/buildmaster/tests/mock_slaves.py (+3/-1) lib/lp/buildmaster/tests/test_manager.py (+90/-1) lib/lp/services/config/schema-lazr.conf (+6/-1) lib/lp/services/statsd/__init__.py (+35/-0) lib/lp/services/statsd/tests/__init__.py (+0/-0) lib/lp/services/statsd/tests/test_lp_statsd.py (+40/-0) setup.py (+1/-0) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Colin Watson (community) | Approve | ||
Ioana Lasc (community) | Approve | ||
Review via email: mp+389011@code.launchpad.net |
Commit message
Add statsd totals for builder stats by processor
Description of the change
It'd be useful to graph the builder states over time, by architecture. Create a statsd client that is configured from settings (with a dummy one to prevent network traffic if not).
Use this in builddmanager in an extra (tunable) loop to prevent impingement on other manager work.
To post a comment you must log in.
looks good to me but i'd get a review from Colin as well.