This is an automated email from the ASF dual-hosted git repository.
mck pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/cassandra-dtest.git
The following commit(s) were added to refs/heads/trunk by this push:
new 7ed2daf Fix `--keep-failed-test-dir` on skipped dtests
7ed2daf is described below
commit 7ed2daf38699fa9555feb9049c1c27a410f1520e
Author: Mick Semb Wever <[email protected]>
AuthorDate: Thu Jan 21 14:58:53 2021 +0100
Fix `--keep-failed-test-dir` on skipped dtests
patch by Mick Semb Wever; reviewed by Tomek Lasica for CASSANDRA-16397
---
dtest_setup.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/dtest_setup.py b/dtest_setup.py
index b3f4838..f613ab2 100644
--- a/dtest_setup.py
+++ b/dtest_setup.py
@@ -348,7 +348,8 @@ class DTestSetup(object):
def cleanup_cluster(self, request=None):
with log_filter('cassandra'): # quiet noise from driver when nodes
start going down
- if self.dtest_config.keep_test_dir or
(self.dtest_config.keep_failed_test_dir and request and
request.node.rep_call.failed):
+ test_failed = request and hasattr(request.node, 'rep_call') and
request.node.rep_call.failed
+ if self.dtest_config.keep_test_dir or
(self.dtest_config.keep_failed_test_dir and test_failed):
self.cluster.stop(gently=self.dtest_config.enable_jacoco_code_coverage)
else:
# when recording coverage the jvm has to exit normally
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
