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.

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

Assertion Error in TopicRecoveryTest.test_fast3 #5737

Closed
r-vasquez opened this issue Jul 29, 2022 · 5 comments · Fixed by #5757
Closed

Assertion Error in TopicRecoveryTest.test_fast3 #5737

r-vasquez opened this issue Jul 29, 2022 · 5 comments · Fixed by #5757
Assignees
Labels
area/cloud-storage Shadow indexing subsystem ci-failure kind/bug Something isn't working

Comments

@r-vasquez
Copy link
Contributor

Saw in: https://buildkite.com/redpanda/redpanda/builds/13268#_

test_id:    rptest.tests.topic_recovery_test.TopicRecoveryTest.test_fast3
--
  | status:     FAIL
  | run time:   2 minutes 31.334 seconds
  |  
  |  
  | AssertionError()
  | Traceback (most recent call last):
  | File "/usr/local/lib/python3.10/dist-packages/ducktape/tests/runner_client.py", line 135, in run
  | data = self.run_test()
  | File "/usr/local/lib/python3.10/dist-packages/ducktape/tests/runner_client.py", line 227, in run_test
  | return self.test_context.function(self.test)
  | File "/root/tests/rptest/services/cluster.py", line 35, in wrapped
  | r = f(self, *args, **kwargs)
  | File "/root/tests/rptest/tests/topic_recovery_test.py", line 1219, in test_fast3
  | self.do_run(test_case)
  | File "/root/tests/rptest/tests/topic_recovery_test.py", line 1121, in do_run
  | test_case.after_restart_validation()
  | File "/root/tests/rptest/tests/topic_recovery_test.py", line 613, in after_restart_validation
  | self._produce_and_verify(topic)
  | File "/root/tests/rptest/tests/topic_recovery_test.py", line 188, in _produce_and_verify
  | assert new_hw is not None
  | AssertionError
@abhijat
Copy link
Contributor

abhijat commented Aug 1, 2022

the high watermark is supposed to come from rpk topic describe which returned NOT_LEADER_FOR_PARTITION

[DEBUG - 2022-07-29 18:08:38,819 - rpk - _execute - lineno:539]: Executing command: ['/var/lib/buildkite-agent/builds/buildkite-amd64-xfs-builders-i-05aa9624bd7904f95-1/redpanda/redpanda/vbuild/debug/clang/dist/local/redpanda/bin/rpk', 'topic', '--brokers', 'docker-rp-18:9092,docker-rp-17:9092,docker-rp-5:9092', 'describe', 'panda-topic-1', '-p']
[DEBUG - 2022-07-29 18:08:38,871 - rpk - _execute - lineno:552]: PARTITION  LEADER  EPOCH  REPLICAS  LOG-START-OFFSET          HIGH-WATERMARK
0          1       4      [1 2 3]   NOT_LEADER_FOR_PARTITION  NOT_LEADER_FOR_PARTITION
1          3       5      [1 2 3]   NOT_LEADER_FOR_PARTITION  NOT_LEADER_FOR_PARTITION

@r-vasquez
Copy link
Contributor Author

@dotnwat
Copy link
Member

dotnwat commented Aug 13, 2022

@r-vasquez looks like this was closed yesterday and your report came in 21 hours ago. do you think that maybe the fix wasn't merged when your test ran?

@dotnwat
Copy link
Member

dotnwat commented Aug 13, 2022

cc @abhijat

@abhijat
Copy link
Contributor

abhijat commented Aug 17, 2022

The failed assertion seen in https://buildkite.com/redpanda/redpanda/builds/14085#018293a4-8447-4943-b185-7d601b019a43:

assert new_hw is not None

is from the old version of the test. This assertion is no longer there in code with this PR. I have created a backport PR for v22.2.x and v22.1.x branches.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/cloud-storage Shadow indexing subsystem ci-failure kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants