Comment #2 on issue 1136 by [email protected]: RAPI does not support forceful disk deactivation
https://code.google.com/p/ganeti/issues/detail?id=1136

commit 205ed2896985f2904e09fc9977b3d091940d4990
Author: Klaus Aehlig <[email protected]>
Date:   Tue Oct 27 15:32:26 2015 +0100

    Support force option for deactivate disks on RAPI

    Signed-off-by: Klaus Aehlig <[email protected]>
    Reviewed-by: Helga Velroyen <[email protected]>

diff --git a/lib/rapi/client.py b/lib/rapi/client.py
index 65f82ab..b6ef576 100644
--- a/lib/rapi/client.py
+++ b/lib/rapi/client.py
@@ -964,7 +964,7 @@ class GanetiRapiClient(object): # pylint: disable=R0904
                              ("/%s/instances/%s/activate-disks" %
(GANETI_RAPI_VERSION, instance)), query, None)

-  def DeactivateInstanceDisks(self, instance, reason=None):
+  def DeactivateInstanceDisks(self, instance, reason=None, force=False):
     """Deactivates an instance's disks.

     @type instance: string
@@ -976,6 +976,7 @@ class GanetiRapiClient(object): # pylint: disable=R0904

     """
     query = []
+    _AppendForceIf(query, force)
     _AppendReason(query, reason)
     return self._SendRequest(HTTP_PUT,
                              ("/%s/instances/%s/deactivate-disks" %
diff --git a/lib/rapi/rlib2.py b/lib/rapi/rlib2.py
index 8fee8fb..34b4124 100644
--- a/lib/rapi/rlib2.py
+++ b/lib/rapi/rlib2.py
@@ -1395,6 +1395,7 @@ class R_2_instances_name_deactivate_disks(baserlib.OpcodeResource):
     """
     return ({}, {
       "instance_name": self.items[0],
+      "force": self.useForce(),
       })




--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

Reply via email to