qa/tasks: Fixed AttributeError: can't set attribute

Fixes: https://tracker.ceph.com/issues/42636
Signed-off-by: Jos Collin <jcollin@redhat.com>

diff --git a/qa/tasks/ceph_manager.py b/qa/tasks/ceph_manager.py
index bf7d131a6d74f122970621d6ac4b4efc5c4afec4..ed5b2c92624ee042f8ae5e0266b13a9baf84be0a 100644 (file)
--- a/qa/tasks/ceph_manager.py
+++ b/qa/tasks/ceph_manager.py
@@ -967,7 +967,7 @@ class OSDThrasher(Thrasher):
             self._do_thrash()
         except Exception as e:
             # See _run exception comment for MDSThrasher
-            self.exception = e
+            self.set_thrasher_exception(e)
             self.logger.exception("exception:")
             # Allow successful completion so gevent doesn't see an exception.
             # The DaemonWatchdog will observe the error and tear down the test.

diff --git a/qa/tasks/mds_thrash.py b/qa/tasks/mds_thrash.py
index fc5a5514fefa1a6d60d2ef8c39f191446ce5afb0..d8c74250dd53590967f893fd0e1714530fedde58 100644 (file)
--- a/qa/tasks/mds_thrash.py
+++ b/qa/tasks/mds_thrash.py
@@ -136,7 +136,7 @@ class MDSThrasher(Thrasher, Greenlet):
             #   File "/usr/lib/python2.7/traceback.py", line 13, in _print
             #     file.write(str+terminator)
             # 2017-02-03T14:34:01.261 CRITICAL:root:IOError
-            self.exception = e
+            self.set_thrasher_exception(e)
             self.logger.exception("exception:")
             # allow successful completion so gevent doesn't see an exception...
 

diff --git a/qa/tasks/mon_thrash.py b/qa/tasks/mon_thrash.py
index d82141574e221b1df80fc890b8b0c32df6195a8e..16680538b5d9d097251ec852bedd3c0aec6e0eea 100644 (file)
--- a/qa/tasks/mon_thrash.py
+++ b/qa/tasks/mon_thrash.py
@@ -232,7 +232,7 @@ class MonitorThrasher(Thrasher):
             self._do_thrash()
         except Exception as e:
             # See _run exception comment for MDSThrasher
-            self.exception = e
+            self.set_thrasher_exception(e)
             self.logger.exception("exception:")
             # Allow successful completion so gevent doesn't see an exception.
             # The DaemonWatchdog will observe the error and tear down the test.

diff --git a/qa/tasks/rbd_mirror_thrash.py b/qa/tasks/rbd_mirror_thrash.py
index 2dbe43ceea071c6cdd13231a9ac4d99dbcc3dd55..d7fdf5607f9fdaeb6d95ca2c0008531e8ee21da4 100644 (file)
--- a/qa/tasks/rbd_mirror_thrash.py
+++ b/qa/tasks/rbd_mirror_thrash.py
@@ -86,7 +86,7 @@ class RBDMirrorThrasher(Thrasher, Greenlet):
             self.do_thrash()
         except Exception as e:
             # See _run exception comment for MDSThrasher
-            self.exception = e
+            self.set_thrasher_exception(e)
             self.logger.exception("exception:")
             # Allow successful completion so gevent doesn't see an exception.
             # The DaemonWatchdog will observe the error and tear down the test.

diff --git a/qa/tasks/thrasher.py b/qa/tasks/thrasher.py
index b34c747703adc24bb2f8d70bb553aa047c2cbdb2..0ea1bf0ee9bc4d9001850eb8e528464ae99d238f 100644 (file)
--- a/qa/tasks/thrasher.py
+++ b/qa/tasks/thrasher.py
@@ -11,6 +11,5 @@ class Thrasher(object):
     def exception(self):
         return self._exception
 
-    @exception.setter
-    def exception(self, e):
+    def set_thrasher_exception(self, e):
         self._exception = e