Skip to content

[v24.2.x] tests/balancer: ignore reconciliation stuck log line #26126

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
piyushredpanda merged 1 commit into from
Jun 17, 2025
Merged

[v24.2.x] tests/balancer: ignore reconciliation stuck log line #26126

piyushredpanda merged 1 commit into from
Jun 17, 2025
+9 −1

Conversation

@vbotbuildovich
Copy link
Collaborator

@vbotbuildovich vbotbuildovich commented May 13, 2025

Backport of PR #26103

@mmaslankaprv @vbotbuildovich
tests/balancer: ignore reconciliation stuck log line
b6bb96c 
When partition balancer detection of unavailable node is tested the node
may be suspended for a long time. In this situation `controller_backend`
can detect stuck reconciliation as the last reconciliation timestamp
might have been taken before the node was suspended. The test should
still pass in this case.

Added appropriate pattern not to treat stuck reconciliation log line as
a test failure.

Signed-off-by: Michał Maślanka <michal@redpanda.com>
(cherry picked from commit 6dd2615)
@vbotbuildovich vbotbuildovich added this to the v24.2.x-next milestone May 13, 2025
@vbotbuildovich vbotbuildovich added the kind/backport PRs targeting a stable branch label May 13, 2025
@vbotbuildovich
Copy link
Collaborator Author

vbotbuildovich commented May 13, 2025 •
edited
Loading

Retry command for Build#65918

please wait until all jobs are finished before running the slash command

/ci-repeat 1
tests/rptest/tests/partition_balancer_test.py::PartitionBalancerTest.test_transfer_controller_leadership
tests/rptest/tests/e2e_shadow_indexing_test.py::ShadowIndexingManyPartitionsTest.test_many_partitions_recovery

@vbotbuildovich
Copy link
Collaborator Author

vbotbuildovich commented May 13, 2025

CI test results

test results on build#65918
test_class test_method test_arguments test_kind job_url test_status passed reason
ShadowIndexingManyPartitionsTest test_many_partitions_recovery ducktape https://buildkite.com/redpanda/redpanda/builds/65918#0196ca97-b469-44f4-80ce-738660826578 FLAKY 20/21 upstream reliability is '100.0'. current run reliability is '95.23809523809523'. drift is 4.7619 and the allowed drift is set to 0. The test should FAIL
PartitionBalancerTest test_transfer_controller_leadership ducktape https://buildkite.com/redpanda/redpanda/builds/65918#0196ca97-b468-40f5-8504-ad0e60483544 FLAKY 16/21 upstream reliability is '100.0'. current run reliability is '76.19047619047619'. drift is 23.80952 and the allowed drift is set to 0. The test should FAIL

@piyushredpanda piyushredpanda merged commit f49a910 into redpanda-data:v24.2.x Jun 17, 2025
14 of 17 checks passed
@piyushredpanda piyushredpanda modified the milestones: v24.2.x-next, v24.2.25 Jun 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mmaslankaprv mmaslankaprv mmaslankaprv approved these changes

Assignees

No one assigned

Labels

kind/backport PRs targeting a stable branch

Projects

None yet

Milestone

v24.2.25

Development

Successfully merging this pull request may close these issues.

3 participants

@vbotbuildovich @mmaslankaprv @piyushredpanda