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

KAFKA-18076: Remove isZkMigrationTest and related code #17922

Merged
merged 3 commits into from
Nov 23, 2024

Conversation

Yunyung
Copy link
Contributor

@Yunyung Yunyung commented Nov 23, 2024

Remove the following code and its related code

  1. TestInfoUtils.isZkMigrationTest
  2. QuorumTestHarness.isZkMigrationTest
  3. DeleteTopicsRequestTest.testNotController

CreateTopicsRequestTest.testNotController will be removed in #17913

JIRA: https://issues.apache.org/jira/browse/KAFKA-18076

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 23, 2024
@chia7712
Copy link
Contributor

@Yunyung could you please fix conflicts?

@Yunyung
Copy link
Contributor Author

Yunyung commented Nov 23, 2024

@Yunyung could you please fix conflicts?

Done:)

@chia7712
Copy link
Contributor

the failed test is handled by #17924

@chia7712 chia7712 merged commit 8a90ca0 into apache:trunk Nov 23, 2024
6 of 8 checks passed
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