Last modified: 2014-07-03 12:57:54 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T68174, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 66174 - Zuul: statsd metrics for Gerrit events are broken
Zuul: statsd metrics for Gerrit events are broken
Status: RESOLVED WONTFIX
Product: Wikimedia
Classification: Unclassified
Continuous integration (Other open bugs)
wmf-deployment
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-06-05 09:29 UTC by Antoine "hashar" Musso (WMF)
Modified: 2014-07-03 12:57 UTC (History)
7 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Antoine "hashar" Musso (WMF) 2014-06-05 09:29:42 UTC
Whenever a Gerrit event is received by Zuul, it increments a counter in statsd. The format is:

 gerrit.event.<event type>

For some reason the metrics are broken and stuck to 1. Apparently since March 2014.

I give it a try with:

 echo "gerrit.test.hashar:5|c"| nc -w1 -q0 -u statsd.eqiad.wmnet 8125

Nothing is created.


I guess we want to drop the whole gerrit.* metric namespace and try again.  CCing Ori since he knows about statsd.
Comment 1 Greg Grossmeier 2014-06-05 17:44:09 UTC
See also the related scap bug: https://bugzilla.wikimedia.org/show_bug.cgi?id=62667
Comment 2 Bryan Davis 2014-06-05 18:12:37 UTC
Matt Walker has been talking to ops about problems that sound related in using txstatsd. I think there are plans in the works to either fix txstatsd or find a new statsd implementation. I don't know if there is an RT ticket or bug to track that yet.
Comment 3 Antoine "hashar" Musso (WMF) 2014-06-06 20:22:45 UTC
Maybe we can drop the whole gerrit. namespace and hope that it clears things up?

I have no need for the history, the main purpose is to display a graph of patchset/comment added on the Zuul status page though I removed it.  A live example at openstack http://status.openstack.org/zuul/ (browse to the bottom).
Comment 4 Antoine "hashar" Musso (WMF) 2014-07-03 12:57:54 UTC
I have dropped the graph from https://integration.wikimedia.org/zuul/ which was the only use case.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links