Skip to content

KAFKA-15664: Add 3.4 Streams upgrade system tests#14601

Merged
mimaison merged 1 commit into
apache:trunkfrom
mimaison:kafka-15664
Oct 23, 2023
Merged

KAFKA-15664: Add 3.4 Streams upgrade system tests#14601
mimaison merged 1 commit into
apache:trunkfrom
mimaison:kafka-15664

Conversation

@mimaison

Copy link
Copy Markdown
Member

Extracted the 3.4 changes from #13860

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

@mimaison

Copy link
Copy Markdown
Member Author

@mjsax It looks much better now. I ran the Streams system tests with this PR and just got a few flaky failures. All tests passed at least once in "only" 2 runs.

@mjsax mjsax added streams tests Test fixes (including flaky tests) labels Oct 23, 2023

@mjsax mjsax left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice! Thanks a ton! LGTM.

Should we trigger a system test run on this branch before merging?

@mimaison

Copy link
Copy Markdown
Member Author

When I ran the Streams system tests in our CI I used this branch, so merging to trunk.

@mimaison
mimaison merged commit 9c77c17 into apache:trunk Oct 23, 2023
@mimaison
mimaison deleted the kafka-15664 branch October 23, 2023 08:34
mimaison added a commit to mimaison/kafka that referenced this pull request Oct 23, 2023
Reviewers: Luke Chen <showuon@gmail.com>,  Matthias J. Sax <mjsax@apache.org>
mimaison added a commit to mimaison/kafka that referenced this pull request Oct 23, 2023
Reviewers: Luke Chen <showuon@gmail.com>,  Matthias J. Sax <mjsax@apache.org>
@mimaison

mimaison commented Oct 23, 2023

Copy link
Copy Markdown
Member Author

Backported to 3.5 and 3.6

Running the system tests on 3.4, then I'll backport to 3.4 too

mimaison added a commit that referenced this pull request Oct 23, 2023
Reviewers: Luke Chen <showuon@gmail.com>,  Matthias J. Sax <mjsax@apache.org>
@mjsax

mjsax commented Oct 23, 2023

Copy link
Copy Markdown
Member

We don't need to backported to 3.4 branch -- In 3.4 branch we test this implicitly by upgrading to DEV_VERSION.

@mimaison

Copy link
Copy Markdown
Member Author

Ah ok, thanks for clarifying.

mjsax pushed a commit to confluentinc/kafka that referenced this pull request Nov 22, 2023
Reviewers: Luke Chen <showuon@gmail.com>,  Matthias J. Sax <mjsax@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

streams tests Test fixes (including flaky tests)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants