Jenkins Prometheus dashboard KPI and layout updates #721
Conversation
| "plotType": "LINE", | ||
| "targetAxis": "Y1", | ||
| "timeSeriesQuery": { | ||
| "prometheusQuery": "(vm_memory_total_max{job=\"jenkins-metrics\",${Cluster},${Location},${Namespace}} - vm_memory_total_used{job=\"jenkins-metrics\",${Cluster},${Location},${Namespace}}) / vm_memory_total_max{job=\"jenkins-metrics\",${Cluster},${Location},${Namespace}} * 100.0" | ||
| "prometheusQuery": "(vm_memory_total_max{job=\"jenkins-metrics\",${Cluster},${Location},${Namespace}} - vm_memory_total_used{job=\"jenkins-metrics\",${Cluster},${Location},${Namespace}}) / vm_memory_total_max{job=\"jenkins-metrics\",${Cluster},${Location},${Namespace}} * 100.0", |
There was a problem hiding this comment.
I noticed in a test environment we have, the job here is named jenkins instead of jenkins-metrics so this chart was blank. I'm not honestly sure if there's a standard name. What do you think of a looser regex match like job=~"jenkins.*" which at least catches both of our setups?
There was a problem hiding this comment.
I'm not sure it's a standard, but in the documentation the PodMonitoring resource name is jenkins, whereas in the environment on our end, we named it jenkins-metrics. I think that with the documentation having jenkins in the example yaml, it's somewhat safe to assume that the word jenkins will be in the name somewhere. Does opting for job=~".*jenkins.*" make more sense in your opinion?
There was a problem hiding this comment.
@johnbryan I've pushed up my suggestion, let me know if you think it makes sense. Hope all is well!
There was a problem hiding this comment.
Looks great, thanks for the fix! (and apologies for slow replies here, I had some vacation time recently)
| "plotType": "LINE", | ||
| "targetAxis": "Y1", | ||
| "timeSeriesQuery": { | ||
| "prometheusQuery": "vm_cpu_load{job=\"jenkins-metrics\",${Cluster},${Location},${Namespace}}" | ||
| "prometheusQuery": "vm_cpu_load{job=\"jenkins-metrics\",${Cluster},${Location},${Namespace}}", |
There was a problem hiding this comment.
same comment on job here
algchoo
force-pushed
the
dashboard/jenkins-kpi-layout-update
branch
from
0e8406f to
85e0035
Compare
Description
This PR updates the layout and includes new KPIs for the Jenkins Prometheus dashboard.
Changes
Updated screenshot
Additional KPIs
Jobs
Build Queue