Closed
Bug 1006031
Opened 11 years ago
Closed 7 years ago
SQL Connection Pool Monitoring for Sync 1.5
Categories
(Cloud Services :: Operations: Metrics/Monitoring, task)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bobm, Assigned: bobm)
Details
(Whiteboard: [qa+])
Add SQL Connection pool monitoring for Sync 1.5 production similar to what is in place for Sync 1.1.
Updated•11 years ago
|
Whiteboard: [qa+]
Comment 1•11 years ago
|
||
Bob, please let me know what you need from the code side here, if anything.
| Assignee | ||
Comment 2•11 years ago
|
||
(In reply to Ryan Kelly [:rfkelly] from comment #1)
> Bob, please let me know what you need from the code side here, if anything.
Does the Sync 1.5 code output SQL pool events to statsd like the Sync 1.1 code does? I'm thinking of Connection Pool recycling events, Connection Pool Max Backlog Reached, and Connection Pool Timings. Perhaps most of these are already in the app log?
Flags: needinfo?(rfkelly)
Comment 3•11 years ago
|
||
The request summary log line should include a timer "syncstorage.storage.sql.pool.get" giving the total time spent waiting for a connection from the pool. This will already be going into heka as part of the json logging data.
The recycling and max-backlog events will be output to the json app log as separate messages, in fact I think :jbonacci has noted them in some of his debug work in the past. But I can probably make them easier to collect, by including them in the summary log in the same manner as the timer. I filed Bug 1053050 to investigate this.
Does including all this stuff into a single json log line make sense? Or would you prefer they go out as individual messages as they occur?
Flags: needinfo?(rfkelly)
| Assignee | ||
Comment 4•9 years ago
|
||
Removing the infrastructure limitation on this bug.
If all the information is in the app logs now, we can probably build a dashboard for looking at these as is.
Group: services-infra
| Assignee | ||
Updated•7 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•