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

Backport pr 14434 v23.2.x 808 #15175

Merged

Conversation

graphcareful
Copy link
Contributor

@graphcareful graphcareful commented Nov 28, 2023

Backport

Issue: #14492

which depends on: 5eb3a91

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v23.2.x
  • v23.1.x
  • v22.3.x

Release Notes

  • none

Allows for testing the behavior of topics with different cleanup policies.

(cherry picked from commit 5eb3a91)
@graphcareful graphcareful requested review from savex and removed request for a team November 28, 2023 14:58
@graphcareful graphcareful changed the base branch from dev to v23.2.x November 28, 2023 14:58
Rob Blafford added 2 commits November 28, 2023 10:00
- Adds a retry when calling rpk consume, it may be the case due to a
concurrent node failure that rpk will not retry and return an error,
causing the test to fail

- Fixes: redpanda-data#14151

(cherry picked from commit fc3837e)
- PR redpanda-data#14434 was merged too early before edits were made, this change
represents the intended edits that were not force-pushed before merge.

(cherry picked from commit 1f12a7f)
@graphcareful graphcareful force-pushed the backport-pr-14434-v23.2.x-808 branch from 363a088 to 6476432 Compare November 28, 2023 15:00
@graphcareful graphcareful merged commit 480d2e4 into redpanda-data:v23.2.x Nov 28, 2023
24 checks passed
@graphcareful graphcareful deleted the backport-pr-14434-v23.2.x-808 branch November 28, 2023 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants