KeDeng has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/21909 )

Change subject: [security-itest] stop polluting TEST_TMPDIR with logs
......................................................................

[security-itest] stop polluting TEST_TMPDIR with logs

By default, KuduTest-based scenarios have their FLAGS_log_dir pointed to
the TEST_TMPDIR directory, so if master/tserver is run using the
"kudu master run"/"kudu tserver run" CLI tool, the generated log files
aren't cleaned up.  This patch addresses the issue by customizing the
--log_dir flag to keep all the logs under the test's directory which
is automatically removed once the scenario successfully completes.

This patch addresses KUDU-3615, at least partially.

Change-Id: Ib70b8a477beeab82af0ee6b4099114d5482f12cf
Reviewed-on: http://gerrit.cloudera.org:8080/21909
Tested-by: Alexey Serbin <[email protected]>
Reviewed-by: KeDeng <[email protected]>
---
M src/kudu/integration-tests/security-itest.cc
1 file changed, 14 insertions(+), 0 deletions(-)

Approvals:
  Alexey Serbin: Verified
  KeDeng: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib70b8a477beeab82af0ee6b4099114d5482f12cf
Gerrit-Change-Number: 21909
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: KeDeng <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>

Reply via email to