Page MenuHomePhabricator

Fix Varnishkafka delivery error icinga warning
Closed, ResolvedPublic

Description

Varnishkafka generates warnings about delivery errors. At least sometimes, perhaps always, the descriptions of the alerts say (null).

varnishkafka-icinga2.png (1,143×333 px, 68 KB)

  • Determine if these are genuine warnings.
  • Improve alert descriptions.

Event Timeline

Ottomata claimed this task.
Ottomata raised the priority of this task from to Medium.
Ottomata updated the task description. (Show Details)
Ottomata added projects: acl*sre-team, Analytics.
Ottomata changed Security from none to None.
Ottomata subscribed.

I want to move this monitoring into graphite via logster. First step, make varnishkafka instance naming consistent:

https://gerrit.wikimedia.org/r/#/c/177546/.

Merging that and then running this:

salt -E 'cp.*|amssq.*' cmd.run '
remove_bad_name_varnishkafka() {
    name=$1
    echo "Removing $name varnishkafka instance."
    service varnishkafka-$name stop &&
    update-rc.d -f varnishkafka-$name remove && 
    rm -fv /etc/ganglia/conf.d/varnishkafka-$name.pyconf /etc/varnishkafka/$name.conf;
}
puppet agent -t &&
test -f /etc/init/varnishkafka-frontend.conf && remove_bad_name_varnishkafka frontend
test -f /etc/init/varnishkafka-$(hostname).conf && remove_bad_name_varnishkafka $(hostname)
'

since this is going to be many metrics and frequently updated we should consider running a local statsd (txstatsd in our case for now) which in turn will consolidate and push to graphite. this for example has been done for swift statsd to reduce the statsd traffic. of course the approach works only if statsd traffic can be aggregated after being processed by statsd (e.g. if the metrics are machine-specific)

I misunderstood "many metrics", what I meant with the previous suggestion applied only if the statsd traffic was going to be reduced by a local txstatsd, this doesn't seem to be the case however.

note however that a local txstatsd is going to be more efficient anyways because it'll batch metrics and push them to graphite via pickle

Ottomata mentioned this in Unknown Object (Diffusion Commit).Dec 29 2014, 4:33 PM

I turned on a check_graphite check for this today. I'm going to let this be for a few days, and then hopefully turn off the check_ganglia ones next week.

re-closing per:

< ottomata> mutante: i think that is a new issue, that one was for check_ganglia. this one is from graphite, and the data is really not in graphite

created T92965 instead