From 171e88ac7da0e999f2e94d45cb324cd408c562ac Mon Sep 17 00:00:00 2001 From: Patrick Donnelly Date: Mon, 20 Jan 2020 11:23:09 -0800 Subject: [PATCH] qa: log warning on scrub error Instead of printing the (useless) traceback, just print a warning about ignoring the failure. The traceback makes it harder to search for the real problem in the teuthology log. Fixes: https://tracker.ceph.com/issues/43718 Signed-off-by: Patrick Donnelly (cherry picked from commit b7454e423620e829e7417cdfca1faf5cd91dec3f) Conflicts: qa/tasks/mon_thrash.py - whereas master has "self.manager.raw_cluster_cmd('mon', 'scrub')" in the try block, in nautilus it is only "self.manager.raw_cluster_cmd('scrub')" --- qa/tasks/mon_thrash.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/qa/tasks/mon_thrash.py b/qa/tasks/mon_thrash.py index 3076c936b2b42..9c18ee316724b 100644 --- a/qa/tasks/mon_thrash.py +++ b/qa/tasks/mon_thrash.py @@ -299,8 +299,8 @@ class MonitorThrasher: self.log('triggering scrub') try: self.manager.raw_cluster_cmd('scrub') - except Exception: - log.exception("Saw exception while triggering scrub") + except Exception as e: + log.warning("Ignoring exception while triggering scrub: %s", e) if self.thrash_delay > 0.0: self.log('waiting for {delay} secs before continuing thrashing'.format( -- 2.39.5