exceptionfactory commented on a change in pull request #5787:
URL: https://github.com/apache/nifi/pull/5787#discussion_r814831662
##########
File path:
nifi-system-tests/nifi-system-test-suite/src/test/java/org/apache/nifi/tests/system/verification/ClusteredVerifiableControllerServiceSystemIT.java
##########
@@ -57,6 +57,6 @@ public void testDifferentResultsFromDifferentNodes() throws
InterruptedException
// Second verification result will be verification results
assertEquals(Outcome.SUCCESSFUL.name(),
resultList.get(1).getOutcome());
// Third verification result is for Fail On Primary Node
- assertEquals(Outcome.FAILED.name(), resultList.get(2).getOutcome());
+ // assertEquals(Outcome.FAILED.name(),
resultList.get(2).getOutcome()); // NIFI-9717
Review comment:
Thanks for the response @greyp9, that is helpful. I have seen a number
of ignored tests here and there, so that was the initial reason for flagging
the commented lines. Given that you already created a specific Jira issue and
mentioned it in the comment, it seems acceptable to leave the commented line
for now. Hopefully that issue can be addressed soon after resolving the
general system test stability issues.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
