Skip to content

MINOR: remove AlterUserScramCredentialsRequestTest.testAlterNotController#17757

Merged
chia7712 merged 1 commit into
apache:trunkfrom
cmccabe:remove_AlterUserScramCredentialsRequestTest_testAlterNotController
Nov 12, 2024
Merged

MINOR: remove AlterUserScramCredentialsRequestTest.testAlterNotController#17757
chia7712 merged 1 commit into
apache:trunkfrom
cmccabe:remove_AlterUserScramCredentialsRequestTest_testAlterNotController

Conversation

@cmccabe

@cmccabe cmccabe commented Nov 11, 2024

Copy link
Copy Markdown
Contributor

Remove AlterUserScramCredentialsRequestTest.testAlterNotController since
the scenario it tests is ZK-only.

@github-actions github-actions Bot added core Kafka Broker tests Test fixes (including flaky tests) small Small PRs labels Nov 11, 2024
…ller

Remove AlterUserScramCredentialsRequestTest.testAlterNotController since
the scenario it tests is ZK-only.
@cmccabe cmccabe force-pushed the remove_AlterUserScramCredentialsRequestTest_testAlterNotController branch from 5b3eb7f to 4e9984b Compare November 11, 2024 18:26

@chia7712 chia7712 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.

the failed tests are unrelated to this PR, since this PR only delete a test case.

@chia7712 chia7712 merged commit 668299e into apache:trunk Nov 12, 2024
@cmccabe cmccabe deleted the remove_AlterUserScramCredentialsRequestTest_testAlterNotController branch November 12, 2024 20:41
chiacyu pushed a commit to chiacyu/kafka that referenced this pull request Nov 30, 2024
…ller (apache#17757)

Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
tedyu pushed a commit to tedyu/kafka that referenced this pull request Jan 6, 2025
…ller (apache#17757)

Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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