Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v23.3.x] Add partition aggregation to some metrics #16094

Merged
merged 4 commits into from
Jan 19, 2024

Conversation

savex
Copy link
Contributor

@savex savex commented Jan 12, 2024

Backport of PR #15966

Add a comment explaining that compaction_ratio metric cannot be
aggregated as it is a ratio, for which a sum is meaningless.

(cherry picked from commit 1ab66f9)
When metrics aggregation is turned on, we want to aggregate away the
partition labels on most metrics: this wasn't occurring in the
consensus object for two metrics: leader_for and reconfiguration
changes in progress.

This change enable aggregation on the partition label for these metrics
if aggregation is turned on.

Issue #15811.
Issue redpanda-data/core-internal#677.

(cherry picked from commit 04b1535)
When metrics aggregation is turned on, we want to aggregate away the
partition labels on most metrics: this wasn't occurring in the
partition probe object.

This change enable aggregation on the partition label for almost all
metrics in the partition problem, with only two excluded:

vectorized_cluster_partition_leader_id
vectorized_cluster_partition_under_replicated_replicas

Issue #15811.
Issue redpanda-data/core-internal#677.

(cherry picked from commit 27e6e2b)
When metrics aggregation is turned on, we want to aggregate away the
partition labels on most metrics: this wasn't occurring in the
tx metrics.

This change enable aggregation on the partition label for almost all
three metrics in this probe.

Issue #15811.
Issue redpanda-data/core-internal#677.

(cherry picked from commit 65c0a89)
@savex savex added this to the v23.3.x-next milestone Jan 12, 2024
@savex savex added the kind/backport PRs targeting a stable branch label Jan 12, 2024
@savex
Copy link
Contributor Author

savex commented Jan 12, 2024

/ci-repeat

@savex
Copy link
Contributor Author

savex commented Jan 17, 2024

Created while investigating GH workflow errors. cc: @travisdowns

@piyushredpanda piyushredpanda marked this pull request as ready for review January 19, 2024 14:03
@piyushredpanda piyushredpanda merged commit 5f40317 into v23.3.x Jan 19, 2024
19 checks passed
@piyushredpanda piyushredpanda deleted the vbotbuildovich/backport-15966-v23.3.x-162 branch January 19, 2024 14:03
@piyushredpanda piyushredpanda modified the milestones: v23.3.x-next, v23.3.6 Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/redpanda kind/backport PRs targeting a stable branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants