Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/10865 )

Change subject: [tools] fix glog output colorizing for kudu CLI tests
......................................................................

[tools] fix glog output colorizing for kudu CLI tests

This patch adds the '--nocolorlogtostderr' flag to every invocation
of the kudu CLI tool run by the kudu::tools::RunKuduTool() function.

Prior to this fix, the following tests would fail if running in the
environment where the GLOG_colorlogtostderr variable is set to 1:

  ToolTest.TestTopLevelHelp
  ToolTest.TestModeHelp
  ToolTest.TestActionHelp
  ToolTest.TestActionMissingRequiredArg

Change-Id: I6635955182e8fdbb4406c9909a7a09a7d3a55955
Reviewed-on: http://gerrit.cloudera.org:8080/10865
Reviewed-by: Adar Dembo <a...@cloudera.com>
Tested-by: Kudu Jenkins
---
M src/kudu/tools/tool_test_util.cc
1 file changed, 6 insertions(+), 0 deletions(-)

Approvals:
  Adar Dembo: Looks good to me, approved
  Kudu Jenkins: Verified

--
To view, visit http://gerrit.cloudera.org:8080/10865
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I6635955182e8fdbb4406c9909a7a09a7d3a55955
Gerrit-Change-Number: 10865
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins

Reply via email to