Filippo Giunchedi has uploaded a new change for review. https://gerrit.wikimedia.org/r/174691
Change subject: merge job timing stats together ...................................................................... merge job timing stats together having per-status job timings yields too many metrics, particularly because statsd will emit derived metrics for timings (percentiles, min/max, etc) Change-Id: I7cfc876e134d51fbc6f969bfec5b87f21cb26517 --- M zuul/scheduler.py 1 file changed, 3 insertions(+), 3 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/integration/zuul refs/changes/91/174691/1 diff --git a/zuul/scheduler.py b/zuul/scheduler.py index 2cd4d46..08c11bd 100644 --- a/zuul/scheduler.py +++ b/zuul/scheduler.py @@ -502,12 +502,12 @@ try: if statsd and build.pipeline: jobname = build.job.name.replace('.', '_') - key = 'zuul.pipeline.%s.job.%s.%s' % (build.pipeline.name, - jobname, build.result) + key = 'zuul.pipeline.%s.job.%s' % (build.pipeline.name, + jobname) if build.result in ['SUCCESS', 'FAILURE'] and build.start_time: dt = int((build.end_time - build.start_time) * 1000) statsd.timing(key, dt) - statsd.incr(key) + statsd.incr(key + '.' + build.result) key = 'zuul.pipeline.%s.all_jobs' % build.pipeline.name statsd.incr(key) except: -- To view, visit https://gerrit.wikimedia.org/r/174691 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I7cfc876e134d51fbc6f969bfec5b87f21cb26517 Gerrit-PatchSet: 1 Gerrit-Project: integration/zuul Gerrit-Branch: master Gerrit-Owner: Filippo Giunchedi <fgiunch...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits