Skip to content

GC: report errors, but remove erroneous silences and continue#4724

Merged
SuperQ merged 1 commit intoprometheus:mainfrom
ultrotter:GCErrors0
Nov 13, 2025
Merged

GC: report errors, but remove erroneous silences and continue#4724
SuperQ merged 1 commit intoprometheus:mainfrom
ultrotter:GCErrors0

Conversation

@ultrotter
Copy link
Contributor

Currently if errors are found GC aborts, but will keep bumping into the same errors, forever.
With this change we count the errors, but also remove the problematic entries, thus becoming able to move on with the cleanup.

This change is stacked on top of #4723

Signed-off-by: Guido Trotter <guido@hudson-trading.com>
Copy link
Contributor

@siavashs siavashs left a comment

Choose a reason for hiding this comment

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

LGTM, Thank you

Copy link
Member

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

Nice

@SuperQ SuperQ merged commit 4ea517b into prometheus:main Nov 13, 2025
7 checks passed
@SoloJacobs SoloJacobs mentioned this pull request Nov 24, 2025
@ultrotter ultrotter deleted the GCErrors0 branch December 3, 2025 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants