Hello Kudu Jenkins, Adar Dembo, Todd Lipcon,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/11886
to look at the new patch set (#2).
Change subject: run-test: don't report leaks that don't fail test
......................................................................
run-test: don't report leaks that don't fail test
The presence of issues like
https://github.com/google/sanitizers/issues/757 means that LeakSanitizer
can report false positives in workloads with high concurrency.
Generally, tests will fail in the face of real leaks, but in some test,
particularly those that use an ExternalMiniCluster, the test may report
leaks, but not actually crash (e.g. see GenericServiceImpl::CheckLeaks).
We currently inject errors into JUnit XML report if any leaks are found.
This patch updates this to only inject these errors if the test also
failed.
Change-Id: I1f199795c48bd9b6106110aae132ec165eb0f647
---
M build-support/run-test.sh
1 file changed, 6 insertions(+), 4 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/86/11886/2
--
To view, visit http://gerrit.cloudera.org:8080/11886
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I1f199795c48bd9b6106110aae132ec165eb0f647
Gerrit-Change-Number: 11886
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <[email protected]>
