Skip to content

KAFKA-15737: KRaft Support in ConsumerBounceTest#17770

Closed
frankvicky wants to merge 3 commits into
apache:trunkfrom
frankvicky:KAFKA-15737
Closed

KAFKA-15737: KRaft Support in ConsumerBounceTest#17770
frankvicky wants to merge 3 commits into
apache:trunkfrom
frankvicky:KAFKA-15737

Conversation

@frankvicky

Copy link
Copy Markdown
Contributor

JIRA: KAFKA-15737

Since we are removing zk, it's time to migrate the test to KRaft.

Committer Checklist (excluded from commit message)

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

@github-actions github-actions Bot added core Kafka Broker tests Test fixes (including flaky tests) small Small PRs labels Nov 12, 2024
@frankvicky frankvicky force-pushed the KAFKA-15737 branch 3 times, most recently from c17ccb2 to d54a7d1 Compare November 13, 2024 02:27
@frankvicky

frankvicky commented Nov 13, 2024

Copy link
Copy Markdown
Contributor Author

I have looped ConsumerBounceTest on my local machine, and it looks good except for testConsumptionWithBrokerFailures, where the interval between two poll calls may exceed max.poll.interval.ms, causing the coordinator to think the consumer is down and remove it from the group.

@frankvicky frankvicky force-pushed the KAFKA-15737 branch 2 times, most recently from 6574790 to ec8485f Compare November 18, 2024 10:00
@chia7712

Copy link
Copy Markdown
Member

testConsumptionWithBrokerFailures

any update for this test case?

@frankvicky

Copy link
Copy Markdown
Contributor Author

This is fixed by #17997

@frankvicky frankvicky closed this Dec 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-approved core Kafka Broker small Small PRs tests Test fixes (including flaky tests)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants