mgr/orchestrator: use result property in Completion classes

The _Completion class defined a get_result() method, but the orchestrator
CLI and Rook module both just expected a result property, so let's run with
the property version.

Signed-off-by: Tim Serong <tserong@suse.com>

diff --git a/src/pybind/mgr/orchestrator.py b/src/pybind/mgr/orchestrator.py
index 43eee7da630d7c414586bb3dc8ade433dc72f1e8..a66ff53ffd322aefa18480b9b3292c91de18568f 100644 (file)
--- a/src/pybind/mgr/orchestrator.py
+++ b/src/pybind/mgr/orchestrator.py
@@ -7,7 +7,8 @@ Please see the ceph-mgr module developer's guide for more information.
 
 
 class _Completion(object):
-    def get_result(self):
+    @property
+    def result(self):
         """
         Return the result of the operation that we were waited
         for.  Only valid after calling Orchestrator.wait() on this

diff --git a/src/pybind/mgr/rook/module.py b/src/pybind/mgr/rook/module.py
index aadbdd50535fc1026535b77d664c6cfb4c474ad1..bbc0ceebb7a3be2546d2d5e88fd2078504043de6 100644 (file)
--- a/src/pybind/mgr/rook/module.py
+++ b/src/pybind/mgr/rook/module.py
@@ -33,7 +33,7 @@ class RookReadCompletion(orchestrator.ReadCompletion):
     def __init__(self, cb):
         super(RookReadCompletion, self).__init__()
         self.cb = cb
-        self.result = None
+        self._result = None
         self._complete = False
 
         self.message = "<read op>"
@@ -42,12 +42,16 @@ class RookReadCompletion(orchestrator.ReadCompletion):
         global all_completions
         all_completions.append(self)
 
+    @property
+    def result(self):
+        return self._result
+
     @property
     def is_complete(self):
         return self._complete
 
     def execute(self):
-        self.result = self.cb()
+        self._result = self.cb()
         self._complete = True