Opened 11 years ago

Closed 11 years ago

#346 closed enhancement (fixed)

Graph sql connection count with munin

Reported by: adehnert Owned by:
Priority: normal Milestone:
Component: sql Keywords:
Cc:

Description

#328 added nagios monitoring of used connections. In tuning those thresholds (and for fun), it would be nice if munin graphed the number of (total and max per-user) connections to sql as well.

Change History (7)

comment:1 Changed 11 years ago by adehnert

p-k has this for total connections, but does not appear to graph max per-user connections, which is the alert that fires more often. Adding total connections to k-s/s-b seems questionably worthwhile, but adding per-user to p-k is probably still worthwhile.

(Since this was apparently unclear, this is not "one number for each user", which achernya correctly points out would be a privacy issue and impossible to read. This is a single number, which is the number of connections the current heaviest user has.)

comment:2 Changed 11 years ago by adehnert

It looks like munin (unlike nagios) gets connection counts from SHOW GLOBAL STATUS, which means that fixing this probably involves more significant changes than "just copy a line and change it slighly", unfortunately.

comment:3 Changed 11 years ago by adehnert

s-b now has total connections. k-s is effectively dead. Nothing has per-user connections yet.

comment:4 Changed 11 years ago by adehnert

I think for per-user connections we probably want to build off of https://github.com/munin-monitoring/contrib/blob/master/plugins/mysql/mysql_aggregate_.

comment:5 Changed 11 years ago by adehnert

Okay, munin is now graphing per-user connections on p-k --- http://sipb-noc.mit.edu/munin/sql.mit.edu/primary-key/mysql_aggregate_user_conns.html.

comment:6 Changed 11 years ago by adehnert

I set this up on s-b as well (https://sipb-noc.mit.edu/munin/sql.mit.edu/sliced-bread/mysql_aggregate_user_conns.html), and submitted a pull request for the modified mysql_aggregate_ plugin (https://github.com/munin-monitoring/contrib/pull/330), so I think this is done.

comment:7 Changed 11 years ago by adehnert

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.