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

ducktape: Fix topic recovery e2e test #5736

Merged
merged 1 commit into from
Aug 2, 2022
Merged

Conversation

Lazin
Copy link
Contributor

@Lazin Lazin commented Jul 29, 2022

Cover letter

Fixes #5474 'Not all data is uploaded to S3 bucket'.
The problem is caused by the fact that we're generating huge number of
segments in this test (message size was 10000, and number of messages
was 100000). This commit decreases the message size to 5000 and
increases the timeout for upload to 10 minutes.

Backport Required

  • not a bug fix
  • papercut/not impactful enough to backport
  • v22.2.x
  • v22.1.x
  • v21.11.x

UX changes

N/A

Release notes

  • none

Features

N/A

Improvements

N/A

Fixes redpanda-data#5474 'Not all data is uploaded to S3 bucket'.
The problem is caused by the fact that we're generating huge number of
segments in this test (message size was 10000, and number of messages
was 100000). This commit decreases the message size to 5000 and
increases the timeout for upload to 10 minutes.
@mmedenjak mmedenjak added ci-failure area/cloud-storage Shadow indexing subsystem kind/bug Something isn't working labels Jul 30, 2022
@dotnwat
Copy link
Member

dotnwat commented Jul 30, 2022

/ci-repeat 10

@dotnwat
Copy link
Member

dotnwat commented Jul 30, 2022

/ci-repeat 9

@Lazin
Copy link
Contributor Author

Lazin commented Aug 1, 2022

/ci-repeat 5

Copy link
Contributor

@LenaAn LenaAn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM as soon as the test doesn't fail via ci-repeat

@Lazin Lazin added the ci-repeat-5 repeat tests 5x concurrently to check for flakey tests; self-cancelling label Aug 1, 2022
@vbotbuildovich vbotbuildovich removed the ci-repeat-5 repeat tests 5x concurrently to check for flakey tests; self-cancelling label Aug 1, 2022
Copy link
Contributor

@abhijat abhijat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Lazin
Copy link
Contributor Author

Lazin commented Aug 2, 2022

Merging since the failures are not related.

@Lazin Lazin merged commit 831c838 into redpanda-data:dev Aug 2, 2022
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 this pull request may close these issues.

Not all data is uploaded to S3 bucket in EndToEndTopicRecovery.test_restore
6 participants