Skip to content

chore!: Rename metric loki_log_flushes - #21765

Merged
chaudum merged 1 commit into
mainfrom
chaudum/rename-log_flushes_metric
May 5, 2026
Merged

chore!: Rename metric loki_log_flushes#21765
chaudum merged 1 commit into
mainfrom
chaudum/rename-log_flushes_metric

Conversation

@chaudum

@chaudum chaudum commented May 5, 2026

Copy link
Copy Markdown
Contributor

Rename metric loki_log_flushes to loki_internal_log_flushes to be consistent with loki_internal_log_messages_total.


Note

Medium Risk
Medium risk because it’s a breaking Prometheus metric rename that will break existing dashboards/alerts unless updated; runtime behavior is otherwise unchanged.

Overview
Renames the buffered logger flush histogram metric from loki_log_flushes to loki_internal_log_flushes in pkg/util/log/log.go to align with the loki_internal_* naming scheme.

Updates the upgrade guide to document the metric rename as a breaking change so operators can adjust dashboards/alerts accordingly.

Reviewed by Cursor Bugbot for commit d192758. Bugbot is set up for automated code reviews on this repo. Configure here.

Rename metric `loki_log_flushes` to `loki_internal_log_flushes` to be
consistent with `loki_internal_log_messages_total`.
@chaudum
chaudum requested a review from a team as a code owner May 5, 2026 08:24
@chaudum
chaudum enabled auto-merge (squash) May 5, 2026 08:24
@chaudum chaudum added the 4.0 Loki 4.0 label May 5, 2026
@github-actions

github-actions Bot commented May 5, 2026

Copy link
Copy Markdown
Contributor

💻 Deploy preview deleted (chore!: Rename metric loki_log_flushes).

@ashwanthgoli ashwanthgoli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@chaudum
chaudum merged commit 67b213e into main May 5, 2026
89 checks passed
@chaudum
chaudum deleted the chaudum/rename-log_flushes_metric branch May 5, 2026 11:24
sophiewaldman pushed a commit that referenced this pull request May 15, 2026
Rename metric loki_log_flushes to loki_internal_log_flushes to be consistent with loki_internal_log_messages_total.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2 participants