[
https://issues.apache.org/jira/browse/ZOOKEEPER-2285?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15400001#comment-15400001
]
Michael Han edited comment on ZOOKEEPER-2285 at 7/29/16 9:22 PM:
-----------------------------------------------------------------
+1 LGTM - though this skipped test seems not the deciding factor that causes
pre-commit builds getting -1, but we should get this in as the test itself is
legitimate (verified the patch with a 200 test runs for this specific test on
my cluster.).
was (Author: hanm):
+1 LGTM - though this skipped test seems not the deciding factor that causes
pre-commit builds getting -1, but we should get this in as the test itself is
legitimate.
> QuorumTest's ignored test case causes wrong CI pre-commit feedback
> ------------------------------------------------------------------
>
> Key: ZOOKEEPER-2285
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2285
> Project: ZooKeeper
> Issue Type: Bug
> Components: tests
> Affects Versions: 3.5.0
> Reporter: Arshad Mohammad
> Assignee: Arshad Mohammad
> Fix For: 3.5.3
>
> Attachments: ZOOKEEPER-2285-01.patch
>
>
> # test case {{org.apache.zookeeper.test.QuorumTest.testSessionMove()}} is
> marked ignored by ZOOKEEPER-907
> # Most of the CI pre-commit feedback is -1 because of above ignored test case.
> # Test case is locally passing
> The ignore tag should be removed from testSessionMove test case.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
