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

[v22.1.x] cloud_storage/tests: wait for partition metadata during topic recovery tests #6015

Merged

Conversation

vbotbuildovich
Copy link
Collaborator

@vbotbuildovich vbotbuildovich commented Aug 13, 2022

Backport from pull request: #5757.

Fixes: #6014

If a leader election happened right before we verify topic watermark,
rpk may return an error such as "not leader for partition" due to stale
metadata.

Instead of failing immediately this change makes the caller wait for
upto 1 minute so that the new leadership info is propagated to the
cluster and rpk returns the correct partition metadata.

(cherry picked from commit 90f0e12)
@vbotbuildovich vbotbuildovich added this to the v22.1.x-next milestone Aug 13, 2022
@vbotbuildovich vbotbuildovich added the kind/backport PRs targeting a stable branch label Aug 13, 2022
@abhijat abhijat marked this pull request as ready for review August 13, 2022 17:16
@abhijat abhijat merged commit b57f390 into redpanda-data:v22.1.x Aug 17, 2022
@mmedenjak mmedenjak added kind/bug Something isn't working ci-failure area/cloud-storage Shadow indexing subsystem labels Aug 23, 2022
@mmedenjak mmedenjak linked an issue Aug 23, 2022 that may be closed by this pull request
@BenPope BenPope modified the milestones: v22.1.x-next, v22.1.8 Dec 22, 2022
This pull request 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/backport PRs targeting a stable branch kind/bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[v22.1.x] Assertion Error in TopicRecoveryTest.test_fast3
5 participants