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-4574: Ignore test_zk_security_upgrade until KIP-101 lands #2742

Closed
wants to merge 1 commit into from

Conversation

ijuma
Copy link
Contributor

@ijuma ijuma commented Mar 27, 2017

The transient failures make it harder to spot real failures and we can live without what is being tested (adding security to ZK via a rolling upgrade) until KIP-101 lands.

@asfbot
Copy link

asfbot commented Mar 27, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.11/2412/
Test PASSed (JDK 8 and Scala 2.11).

@asfbot
Copy link

asfbot commented Mar 27, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/2408/
Test PASSed (JDK 8 and Scala 2.12).

@asfbot
Copy link

asfbot commented Mar 27, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.10/2408/
Test PASSed (JDK 7 and Scala 2.10).

Copy link
Contributor

@apurvam apurvam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@junrao
Copy link
Contributor

junrao commented Mar 28, 2017

@ijuma : Thanks for the patch. LGTM

@ijuma
Copy link
Contributor Author

ijuma commented Mar 29, 2017

The test is ignored in https://jenkins.confluent.io/job/system-test-kafka-branch-builder-2/235/console, merging to trunk.

@asfgit asfgit closed this in 4ce65d6 Mar 29, 2017
@ijuma ijuma deleted the disable-zk-upgrade-test branch September 5, 2017 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants