Skip to content

scrape: switch scrape manager to AppenderV2#17978

Merged
bwplotka merged 1 commit intomainfrom
bwplotka/switch-attempt
Feb 2, 2026
Merged

scrape: switch scrape manager to AppenderV2#17978
bwplotka merged 1 commit intomainfrom
bwplotka/switch-attempt

Conversation

@bwplotka
Copy link
Member

Related to #17632

Trying #17959 after fanout fix.

Does this PR introduce a user-facing change?

NONE

@bwplotka bwplotka requested a review from a team as a code owner January 30, 2026 17:40
@bwplotka bwplotka requested review from kakkoyun and removed request for a team January 30, 2026 17:40
@bwplotka
Copy link
Member Author

/prombench main

@prombot
Copy link
Contributor

prombot commented Jan 30, 2026

⏱️ Welcome to Prometheus Benchmarking Tool. ⏱️

Compared versions: PR-17978 and main

After the successful deployment (check status here), the benchmarking results can be viewed at:

Available Commands:

  • To restart benchmark: /prombench restart main
  • To stop benchmark: /prombench cancel
  • To print help: /prombench help

Base automatically changed from bwplotka/fanout to main February 2, 2026 07:04
Signed-off-by: bwplotka <bwplotka@gmail.com>
@bwplotka bwplotka force-pushed the bwplotka/switch-attempt branch from eaccc61 to 8109ba3 Compare February 2, 2026 08:54
@bwplotka
Copy link
Member Author

bwplotka commented Feb 2, 2026

IMO it looks great

Resources looks good (24h avg)

image

CPU sometimes gets higher but max 1.6% avg (last 3h)

image

But it depends what unlucky period you see, here is 1.2% less CPU for V2:

image

I think the variance (also on query latencies) is still due to prombench load testing is not equal. Here is when V2 is "slower" on queries:

image

Here is when V1 is slower:

image

Overall, I think we should be good to switch 🎉

Copy link
Member

@krajorama krajorama left a comment

Choose a reason for hiding this comment

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

LGTM, measurements look within spec

@bwplotka bwplotka merged commit 7429a75 into main Feb 2, 2026
71 of 72 checks passed
@bwplotka bwplotka deleted the bwplotka/switch-attempt branch February 2, 2026 12:12
@bwplotka
Copy link
Member Author

bwplotka commented Feb 2, 2026

/prombench cancel

@prombot
Copy link
Contributor

prombot commented Feb 2, 2026

Benchmark cancel is in progress.

wbollock pushed a commit to wbollock/prometheus that referenced this pull request Feb 6, 2026
Signed-off-by: bwplotka <bwplotka@gmail.com>
Signed-off-by: Will Bollock <wbollock@linode.com>
wbollock pushed a commit to wbollock/prometheus that referenced this pull request Feb 6, 2026
Signed-off-by: bwplotka <bwplotka@gmail.com>
wbollock pushed a commit to wbollock/prometheus that referenced this pull request Feb 6, 2026
Signed-off-by: bwplotka <bwplotka@gmail.com>
wbollock pushed a commit to wbollock/prometheus that referenced this pull request Feb 6, 2026
Signed-off-by: bwplotka <bwplotka@gmail.com>
wbollock pushed a commit to wbollock/prometheus that referenced this pull request Feb 6, 2026
Signed-off-by: bwplotka <bwplotka@gmail.com>
wbollock pushed a commit to wbollock/prometheus that referenced this pull request Feb 6, 2026
Signed-off-by: bwplotka <bwplotka@gmail.com>
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