opendev/system-config
Code Issues Proposed changes

prod-playbook: use job name for stats

Browse Source 
Because "." is a field separator for graphite, we're incorrectly
nesting the results.

A better idea seems to be to store these stats under the job name.
That's going to be more helpful when looking up in Zuul build results
anyway.

Follow-on to I90dfb7a25cb5ab08403c89ef59ea21972cf2aae2

Change-Id: Icbb57fd23d8b90f52bc7a0ea5fa80f389ab3892e
This commit is contained in:
Ian Wienand 2022-03-10 16:18:50 +11:00
parent b7cdaa7fce
commit d87ce0e35f
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
playbooks/zuul/run-production-playbook.yaml
View File

@ -32,8 +32,8 @@
# delta is in string format h:m:s.sss; convert to ms for statsd
{% set delta = _run.delta.split(':') %}
{% set delta_ms = ((delta[0]|int * 60 * 60 * 1000) + (delta[1]|int * 60 * 1000) + (delta[2]|float * 1000)) | int %}
echo 'bridge.ansible.{{ playbook_name }}.runtime:{{ delta_ms }}|ms' | nc -w 1 -u graphite.opendev.org 8125
echo 'bridge.ansible.{{ playbook_name }}.rc:{{ _run.rc }}|g' | nc -w 1 -u graphite.opendev.org 8125
echo 'bridge.ansible.{{ zuul.job }}.runtime:{{ delta_ms }}|ms' | nc -w 1 -u graphite.opendev.org 8125
echo 'bridge.ansible.{{ zuul.job }}.rc:{{ _run.rc }}|g' | nc -w 1 -u graphite.opendev.org 8125
args:
executable: '/bin/bash'