Skip to content

Commit

Permalink
agent-flow-mixin: remove unnecessary sum by (le) for native histogr…
Browse files Browse the repository at this point in the history
…ams (#6611)

Native histograms do not have an `le` label, so there is no point in
specifying `sum by (le)`.

This change was tested and makes no visual difference to the panels.
  • Loading branch information
rfratto authored Mar 5, 2024
1 parent 6fff3c3 commit 2df4296
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions operations/agent-flow-mixin/dashboards/controller.libsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -219,15 +219,15 @@ local filename = 'agent-flow-controller.json';
panel.withQueries([
panel.newQuery(
expr=|||
histogram_quantile(0.99, sum by (le) (rate(agent_component_evaluation_seconds{cluster="$cluster",namespace="$namespace"}[$__rate_interval])))
histogram_quantile(0.99, sum(rate(agent_component_evaluation_seconds{cluster="$cluster",namespace="$namespace"}[$__rate_interval])))
or
histogram_quantile(0.99, sum by (le) (rate(agent_component_evaluation_seconds_bucket{cluster="$cluster",namespace="$namespace"}[$__rate_interval])))
|||,
legendFormat='99th percentile',
),
panel.newQuery(
expr=|||
histogram_quantile(0.50, sum by (le) (rate(agent_component_evaluation_seconds{cluster="$cluster",namespace="$namespace"}[$__rate_interval])))
histogram_quantile(0.50, sum(rate(agent_component_evaluation_seconds{cluster="$cluster",namespace="$namespace"}[$__rate_interval])))
or
histogram_quantile(0.50, sum by (le) (rate(agent_component_evaluation_seconds_bucket{cluster="$cluster",namespace="$namespace"}[$__rate_interval])))
|||,
Expand Down Expand Up @@ -287,7 +287,7 @@ local filename = 'agent-flow-controller.json';
panel.withQueries([
panel.newQuery(
expr=|||
sum by (le) (increase(agent_component_evaluation_seconds{cluster="$cluster", namespace="$namespace"}[$__rate_interval]))
sum(increase(agent_component_evaluation_seconds{cluster="$cluster", namespace="$namespace"}[$__rate_interval]))
or ignoring (le)
sum by (le) (increase(agent_component_evaluation_seconds_bucket{cluster="$cluster", namespace="$namespace"}[$__rate_interval]))
|||,
Expand All @@ -312,7 +312,7 @@ local filename = 'agent-flow-controller.json';
panel.withQueries([
panel.newQuery(
expr=|||
sum by (le) (increase(agent_component_dependencies_wait_seconds{cluster="$cluster", namespace="$namespace"}[$__rate_interval]))
sum(increase(agent_component_dependencies_wait_seconds{cluster="$cluster", namespace="$namespace"}[$__rate_interval]))
or ignoring (le)
sum by (le) (increase(agent_component_dependencies_wait_seconds_bucket{cluster="$cluster", namespace="$namespace"}[$__rate_interval]))
|||,
Expand Down

0 comments on commit 2df4296

Please sign in to comment.