Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/21863 )
Change subject: [tests] fix failing TestWebUIDoesNotCrashCluster ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/21863/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/21863/1//COMMIT_MSG@11 PS1, Line 11: correct > nit: Should this be 'incorrect'? Nope, this should be as-is. The point of this patch is to make the test pass: when the KUDU_HOME environment variable was set and pointed to the location of the git workspace, the test scenario would fail because it's assuming the output isn't rendered with corresponding mustache template. -- To view, visit http://gerrit.cloudera.org:8080/21863 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I59b6ef64b522540620bdc622156b3516ada23f5d Gerrit-Change-Number: 21863 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yuqi Du <[email protected]> Gerrit-Comment-Date: Mon, 30 Sep 2024 15:59:25 +0000 Gerrit-HasComments: Yes
