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

tests: allow logging unsupported api version entry when upgrading #6580

Merged
merged 1 commit into from
Oct 3, 2022

Conversation

mmaslankaprv
Copy link
Member

@mmaslankaprv mmaslankaprv commented Sep 30, 2022

Cover letter

Older redpanda versions may not support some of the APIs in versions that the client currently use. Added allowed error log entry to tolerate this situation during an upgrade test.

Fixes: #5868

Describe in plain language the motivation (bug, feature, etc.) behind the change in this PR and how the included commits address it.

Fixes #ISSUE-NUMBER, Fixes #ISSUE-NUMBER, ...

Backport Required

  • not a bug fix
  • issue does not exist in previous branches
  • papercut/not impactful enough to backport
  • v22.2.x
  • v22.1.x
  • v21.11.x

UX changes

Describe in plain language how this PR affects an end-user. What topic flags, configuration flags, command line flags, deprecation policies etc are added/changed.

Release notes

  • none

@ZeDRoman
Copy link
Contributor

Why we can just ignore this error?
Which request causes this error?
I thought if we have such error inside log, we would see same error in response to request, but test doesn't fail if it gots response error.

@mmaslankaprv
Copy link
Member Author

we ignore the error as we didn't backported this change to previous version and this is an upgrade test

Older redpanda versions may not support some of the APIs in versions
that the client currently use. Added allowed error log entry to
tolerate this situation during an upgrade test.

Fixes: redpanda-data#5868

Signed-off-by: Michal Maslanka <michal@redpanda.com>
@mmaslankaprv mmaslankaprv merged commit e3e7cee into redpanda-data:dev Oct 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unsupported kafka api v3 (leave group) in PartitionMovementUpgradeTest.test_basic_upgrade
4 participants