kafka icon indicating copy to clipboard operation
kafka copied to clipboard

KAFKA-17609:[1/4] Changes needed to convert system tests to use KRaft and remove ZK

Open bbejeck opened this issue 1 year ago • 5 comments

This is part one of a multi-pr effort to convert Kafka Streams system tests to KRaft. I decided to break down the changes into multiple PRs to reduce the review load

Committer Checklist (excluded from commit message)

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

bbejeck avatar Sep 25 '24 18:09 bbejeck

Thanks Bill. Made a pass. Can you also trigger a system test run and share the link?

mjsax avatar Sep 26 '24 02:09 mjsax

Thanks Bill. Made a pass. Can you also trigger a system test run and share the link?

Passing system tests for ApplicationUpgradeTest Passing system tests for BrokerBounceTest

Also in the PR description

bbejeck avatar Sep 30 '24 21:09 bbejeck

@mjsax all comments addressed

bbejeck avatar Oct 01 '24 19:10 bbejeck

I kicked off a new branch builder for streams_application_upgrade_test and I'll post the link here when it completes.

bbejeck avatar Oct 01 '24 19:10 bbejeck

System test results 29 passed, 3 failed - I'll look into the failures.

bbejeck avatar Oct 01 '24 22:10 bbejeck

Merged #17275 into trunk

bbejeck avatar Nov 05 '24 16:11 bbejeck