Skip to content

Commit

Permalink
fix: Add custom all value to notification dashboard
Browse files Browse the repository at this point in the history
Shows bugged exported services as for example "null"
  • Loading branch information
adinhodovic committed Nov 7, 2023
1 parent cf6eae2 commit 055a0c3
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
4 changes: 2 additions & 2 deletions dashboards/argo-cd-notifications.libsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ local tsLegend = tsOptions.legend;
query.withSort(1) +
query.generalOptions.withLabel('Job') +
query.selectionOptions.withMulti(true) +
query.selectionOptions.withIncludeAll(true) +
query.selectionOptions.withIncludeAll(true, '.*') +
query.refresh.onLoad() +
query.refresh.onTime(),

Expand Down Expand Up @@ -85,7 +85,7 @@ local tsLegend = tsOptions.legend;
increase(
argocd_notifications_deliveries_total{
%s
exported_service =~"$exported_service",
exported_service=~"$exported_service",
}[$__rate_interval]
)
)
Expand Down
3 changes: 2 additions & 1 deletion dashboards_out/argo-cd-notifications-overview.json
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@
"type": "prometheus",
"uid": "$datasource"
},
"expr": "sum(\n round(\n increase(\n argocd_notifications_deliveries_total{\n namespace=~'$namespace',\njob=~'$job',\n\n exported_service =~\"$exported_service\",\n }[$__rate_interval]\n )\n )\n) by (job, exported_service, succeeded)\n",
"expr": "sum(\n round(\n increase(\n argocd_notifications_deliveries_total{\n namespace=~'$namespace',\njob=~'$job',\n\n exported_service=~\"$exported_service|\",\n }[$__rate_interval]\n )\n )\n) by (job, exported_service, succeeded)\n",
"legendFormat": "{{ exported_service }} - Succeeded: {{ succeeded }}"
}
],
Expand Down Expand Up @@ -138,6 +138,7 @@
"type": "query"
},
{
"allValue": ".*",
"datasource": {
"type": "prometheus",
"uid": "${datasource}"
Expand Down

0 comments on commit 055a0c3

Please sign in to comment.