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-18280:fix e2e TestSecurityRollingUpgrade.test_rolling_upgrade_sasl_mechanism_phase_one #18308

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from

Conversation

TaiJuWu
Copy link
Contributor

@TaiJuWu TaiJuWu commented Dec 24, 2024

JIRA: https://issues.apache.org/jira/projects/KAFKA/issues/KAFKA-18280

In KRaft mode, miniKDC will start two services so client side token and server side token is unmatched after node restart.

In this patch, we do not unassign MiniKDC to avoid creating another service.

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 triage PRs from the community tests Test fixes (including flaky tests) small Small PRs labels Dec 24, 2024
@TaiJuWu
Copy link
Contributor Author

TaiJuWu commented Dec 24, 2024

Result

SESSION REPORT (ALL TESTS)
ducktape version: 0.12.0
session_id:       2024-12-23--021
run time:         4 minutes 12.166 seconds
tests run:        1
passed:           1
flaky:            0
failed:           0
ignored:          0
================================================================================
test_id:    kafkatest.tests.core.security_rolling_upgrade_test.TestSecurityRollingUpgrade.test_rolling_upgrade_sasl_mechanism_phase_one.new_client_sasl_mechanism=PLAIN.metadata_quorum=ISOLATED_KRAFT
status:     PASS
run time:   4 minutes 11.980 seconds
--------------------------------------------------------------------------------

@TaiJuWu TaiJuWu marked this pull request as ready for review December 24, 2024 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-approved small Small PRs tests Test fixes (including flaky tests) triage PRs from the community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants