Skip to content

Conversation

@dobesv
Copy link

@dobesv dobesv commented Aug 7, 2025

I found that when using --split-cluster you should also use direct connections, otherwise it just only connects to the same replica and reports its metrics as if all the replicas have exactly the same metric values.

I found that when using `--split-cluster` you should also use direct connections, otherwise it just only connects to the same replica and reports its metrics as if all the replicas have exactly the same metric values.
@dobesv dobesv requested a review from a team as a code owner August 7, 2025 19:49
@dobesv dobesv requested review from BupycHuk and JiriCtvrtka and removed request for a team August 7, 2025 19:49
@it-percona-cla
Copy link

it-percona-cla commented Aug 7, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ idoqo
❌ dobesv
You have signed the CLA already but the status is still pending? Let us recheck it.

@JiriCtvrtka
Copy link
Contributor

@dobesv could you sign CLA? Thank you.

@codecov
Copy link

codecov bot commented Oct 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.82%. Comparing base (dc46ed5) to head (66c3df0).
⚠️ Report is 145 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1134      +/-   ##
==========================================
- Coverage   70.88%   64.82%   -6.07%     
==========================================
  Files          28       29       +1     
  Lines        3569     2428    -1141     
==========================================
- Hits         2530     1574     -956     
+ Misses        904      713     -191     
- Partials      135      141       +6     
Flag Coverage Δ
agent 64.82% <ø> (-6.07%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dobesv
Copy link
Author

dobesv commented Oct 24, 2025

I don't mind ... but when I try, I get an error:

image

Maybe easier if you just recreate the same trivial change in a new PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants