Skip to content
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

CI Failure (MetricCheckFailed) in RaftAvailabilityTest. test_leadership_transfer #19953

Closed
michael-redpanda opened this issue Jun 22, 2024 · 1 comment · Fixed by #20378
Closed

CI Failure (MetricCheckFailed) in RaftAvailabilityTest. test_leadership_transfer #19953

michael-redpanda opened this issue Jun 22, 2024 · 1 comment · Fixed by #20378
Assignees
@mmaslankaprv
Labels
area/raft ci-failure kind/bug Something isn't working

Comments

@michael-redpanda
Copy link
Contributor

michael-redpanda commented Jun 22, 2024 •
edited by jira bot
Loading

https://buildkite.com/redpanda/redpanda/builds/50477

Module: rptest.tests.raft_availability_test
Class:  RaftAvailabilityTest
Method: test_leadership_transfer

test_id:    rptest.tests.raft_availability_test.RaftAvailabilityTest.test_leadership_transfer
status:     FAIL
run time:   23.475 seconds


    MetricCheckFailed('vectorized_raft_leadership_changes_total', 0.0, 1.0)
Traceback (most recent call last):
  File "/opt/.ducktape-venv/lib/python3.10/site-packages/ducktape/tests/runner_client.py", line 184, in _do_run
    data = self.run_test()
  File "/opt/.ducktape-venv/lib/python3.10/site-packages/ducktape/tests/runner_client.py", line 276, in run_test
    return self.test_context.function(self.test)
  File "/root/tests/rptest/services/cluster.py", line 105, in wrapped
    r = f(self, *args, **kwargs)
  File "/root/tests/rptest/tests/raft_availability_test.py", line 402, in test_leadership_transfer
    metric_check.expect([
  File "/root/tests/rptest/services/metrics_check.py", line 141, in expect
    raise error
rptest.services.metrics_check.MetricCheckFailed: MetricCheckFailed<vectorized_raft_leadership_changes_total old=0.0 new=1.0>

JIRA Link: CORE-4272
@michael-redpanda michael-redpanda added kind/bug Something isn't working ci-failure labels Jun 22, 2024
@piyushredpanda piyushredpanda mentioned this issue Jun 23, 2024
Merged
@travisdowns travisdowns added area/raft ci-ignore Automatic ci analysis tools ignore this issue labels Jun 23, 2024
@travisdowns
Copy link
Member

ci-ignore because links need to be the BK build link or PT is unhappy

This was referenced Jun 25, 2024
Merged
Merged
@oleiman oleiman mentioned this issue Jun 27, 2024
Merged
8 tasks
@mmaslankaprv mmaslankaprv self-assigned this Jun 27, 2024
@mmaslankaprv mmaslankaprv removed the ci-ignore Automatic ci analysis tools ignore this issue label Jun 27, 2024
mmaslankaprv added a commit to mmaslankaprv/redpanda that referenced this issue Jun 27, 2024
@mmaslankaprv
tests: fixed querying new leader node in leadership transfer test
b455334 
The test was using an api expecting the node ordinal to query for the
node that was elected as a leader. The query was based on the node id.
This lead to the validation being invalid.

Fixes: redpanda-data#19953

Signed-off-by: Michał Maślanka <michal@redpanda.com>
@mmaslankaprv mmaslankaprv mentioned this issue Jun 27, 2024
Merged
7 tasks
vbotbuildovich pushed a commit to vbotbuildovich/redpanda that referenced this issue Jun 27, 2024
@mmaslankaprv @vbotbuildovich
tests: fixed querying new leader node in leadership transfer test
1c236e4 
The test was using an api expecting the node ordinal to query for the
node that was elected as a leader. The query was based on the node id.
This lead to the validation being invalid.

Fixes: redpanda-data#19953

Signed-off-by: Michał Maślanka <michal@redpanda.com>
(cherry picked from commit b455334)
@vbotbuildovich vbotbuildovich mentioned this issue Jun 27, 2024
Closed
vbotbuildovich pushed a commit to vbotbuildovich/redpanda that referenced this issue Jun 27, 2024
@mmaslankaprv @vbotbuildovich
tests: fixed querying new leader node in leadership transfer test
f890b8d 
The test was using an api expecting the node ordinal to query for the
node that was elected as a leader. The query was based on the node id.
This lead to the validation being invalid.

Fixes: redpanda-data#19953

Signed-off-by: Michał Maślanka <michal@redpanda.com>
(cherry picked from commit b455334)
@vbotbuildovich vbotbuildovich mentioned this issue Jun 27, 2024
Closed
@ballard26 ballard26 mentioned this issue Jun 28, 2024
Merged
7 tasks
@mmaslankaprv mmaslankaprv mentioned this issue Jun 28, 2024
Merged
7 tasks
@bharathv bharathv mentioned this issue Jun 28, 2024
Merged
7 tasks
@mmaslankaprv mmaslankaprv mentioned this issue Jul 4, 2024
Merged
7 tasks
@ztlpn ztlpn mentioned this issue Jul 17, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mmaslankaprv mmaslankaprv

Labels
area/raft ci-failure kind/bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

tests: fixed querying new leader node in leadership transfer test mmaslankaprv/redpanda
3 participants
@travisdowns @mmaslankaprv @michael-redpanda