Skip to content

Commit

Permalink
Merge pull request #16344 from vbotbuildovich/backport-pr-16336-v23.3…
Browse files Browse the repository at this point in the history
….x-71

[v23.3.x] Aggregate partition label in `ntp_level_probe`
  • Loading branch information
piyushredpanda committed Jan 30, 2024
2 parents 2706ccf + 8cd3c23 commit e6b7b4d
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion src/v/archival/probe.cc
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,9 @@ void ntp_level_probe::setup_ntp_metrics(const model::ntp& ntp) {
partition_label(ntp.tp.partition()),
};

auto aggregate_labels = std::vector<sm::label>{
sm::shard_label, partition_label};

_metrics.add_group(
prometheus_sanitize::metrics_name("ntp_archiver"),
{
Expand All @@ -69,7 +72,7 @@ void ntp_level_probe::setup_ntp_metrics(const model::ntp& ntp) {
labels),
},
{},
std::vector<sm::label>{sm::shard_label});
aggregate_labels);
}

void ntp_level_probe::setup_public_metrics(const model::ntp& ntp) {
Expand Down

0 comments on commit e6b7b4d

Please sign in to comment.