Skip to content

Commit

Permalink
Mixins: draw graphs at full resolution
Browse files Browse the repository at this point in the history
Remove "intervalFactor: 2", which corresponded to "Resolution 1/2" in
the Grafana UI. This feature has now been dropped from the UI, and I
can't see why were ever using it - why would you not want the full
resolution for your panels?

Left `awx-mixin/dashboards/clustertable.libsonnet` untouched because
those are table queries not timeseries.
  • Loading branch information
bboreham committed Aug 22, 2022
1 parent a2ba95a commit f3b3d58
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 13 deletions.
1 change: 0 additions & 1 deletion envoy-mixin/dashboards.libsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,6 @@ local template = import 'grafonnet/template.libsonnet';
{
expr: 'sum by (status) (label_replace(rate(' + selector + '[$__interval]), "status", "${1}xx", "envoy_response_code_class", "(.*)"))',
format: 'time_series',
intervalFactor: 2,
legendFormat: '{{status}}',
refId: 'A',
step: 10,
Expand Down
7 changes: 0 additions & 7 deletions grafana-builder/grafana.libsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -248,7 +248,6 @@
legendLink: legendLink,
expr: ql.q,
format: 'time_series',
intervalFactor: 2,
legendFormat: ql.l,
step: 10,
}
Expand All @@ -265,7 +264,6 @@
expr: query,
format: 'time_series',
instant: true,
intervalFactor: 2,
refId: 'A',
},
],
Expand Down Expand Up @@ -328,7 +326,6 @@
expr: qs[i],
format: 'table',
instant: true,
intervalFactor: 2,
legendFormat: '',
step: 10,
refId: std.char(65 + i),
Expand Down Expand Up @@ -404,7 +401,6 @@
"status", "${1}", "%s", "([a-z]+)"))
||| % [selector, statusLabelName, statusLabelName],
format: 'time_series',
intervalFactor: 2,
legendFormat: '{{status}}',
refId: 'A',
step: 10,
Expand All @@ -418,23 +414,20 @@
{
expr: 'histogram_quantile(0.99, sum(rate(%s_bucket%s[$__rate_interval])) by (le)) * %s' % [metricName, selector, multiplier],
format: 'time_series',
intervalFactor: 2,
legendFormat: '99th Percentile',
refId: 'A',
step: 10,
},
{
expr: 'histogram_quantile(0.50, sum(rate(%s_bucket%s[$__rate_interval])) by (le)) * %s' % [metricName, selector, multiplier],
format: 'time_series',
intervalFactor: 2,
legendFormat: '50th Percentile',
refId: 'B',
step: 10,
},
{
expr: 'sum(rate(%s_sum%s[$__rate_interval])) * %s / sum(rate(%s_count%s[$__rate_interval]))' % [metricName, selector, multiplier, metricName, selector],
format: 'time_series',
intervalFactor: 2,
legendFormat: 'Average',
refId: 'C',
step: 10,
Expand Down
2 changes: 0 additions & 2 deletions jaeger-mixin/dashboards.libsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -10,15 +10,13 @@ local g = (import 'grafana-builder/grafana.libsonnet') + {
{
expr: expr(selectorErr),
format: 'time_series',
intervalFactor: 2,
legendFormat: 'error',
refId: 'A',
step: 10,
},
{
expr: expr(selectorTotal) + ' - ' + expr(selectorErr),
format: 'time_series',
intervalFactor: 2,
legendFormat: 'success',
refId: 'B',
step: 10,
Expand Down
3 changes: 0 additions & 3 deletions mixin-utils/utils.libsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,6 @@ local g = import 'grafana-builder/grafana.libsonnet';
sumBy: sumByHisto,
},
format: 'time_series',
intervalFactor: 2,
legendFormat: '%(legend)s99th Percentile' % legend,
refId: 'A',
step: 10,
Expand All @@ -80,7 +79,6 @@ local g = import 'grafana-builder/grafana.libsonnet';
sumBy: sumByHisto,
},
format: 'time_series',
intervalFactor: 2,
legendFormat: '%(legend)s50th Percentile' % legend,
refId: 'B',
step: 10,
Expand All @@ -94,7 +92,6 @@ local g = import 'grafana-builder/grafana.libsonnet';
sumBy: sumBy,
},
format: 'time_series',
intervalFactor: 2,
legendFormat: '%(legend)sAverage' % legend,
refId: 'C',
step: 10,
Expand Down

0 comments on commit f3b3d58

Please sign in to comment.