[libvirt] [PATCH 2/2] sanlock: Forbid VIR_DOMAIN_LOCK_FAILURE_RESTART

[Jiri Denemark]

https://bugzilla.redhat.com/show_bug.cgi?id=905282
https://bugzilla.redhat.com/show_bug.cgi?id=967494

When lock failure is detected by sanlock, our sanlock_helper kill script
will try to restart (shutdown followed by start) the affected domain
when RESTART action is configured for it. While shutting down kills QEMU
and removes all its leases (which is what sanlock wants to happen),
trying to start it again just hangs because libvirt tries reacquire the
locks in the failed lock space. Hence, this action cannot be supported
by sanlock driver.

Signed-off-by: Jiri Denemark <jdenemar at redhat.com>
---
 src/locking/lock_driver_sanlock.c |  2 +-
 src/locking/sanlock_helper.c      | 22 +---------------------
 2 files changed, 2 insertions(+), 22 deletions(-)

diff --git a/src/locking/lock_driver_sanlock.c b/src/locking/lock_driver_sanlock.c
index b8c86fc..624691f 100644
--- a/src/locking/lock_driver_sanlock.c
+++ b/src/locking/lock_driver_sanlock.c
@@ -786,10 +786,10 @@ virLockManagerSanlockRegisterKillscript(int sock,
         return 0;
 
     case VIR_DOMAIN_LOCK_FAILURE_POWEROFF:
-    case VIR_DOMAIN_LOCK_FAILURE_RESTART:
     case VIR_DOMAIN_LOCK_FAILURE_PAUSE:
         break;
 
+    case VIR_DOMAIN_LOCK_FAILURE_RESTART:
     case VIR_DOMAIN_LOCK_FAILURE_IGNORE:
     case VIR_DOMAIN_LOCK_FAILURE_LAST:
         virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
diff --git a/src/locking/sanlock_helper.c b/src/locking/sanlock_helper.c
index c913cc3..08102eb 100644
--- a/src/locking/sanlock_helper.c
+++ b/src/locking/sanlock_helper.c
@@ -91,33 +91,13 @@ main(int argc, char **argv)
             ret = EXIT_SUCCESS;
         break;
 
-    case VIR_DOMAIN_LOCK_FAILURE_RESTART:
-        if (virDomainIsPersistent(dom)) {
-            if ((virDomainDestroy(dom) == 0 ||
-                 virDomainIsActive(dom) == 0) &&
-                virDomainCreate(dom) == 0)
-                ret = EXIT_SUCCESS;
-        } else {
-            xml = virDomainGetXMLDesc(dom,
-                                      VIR_DOMAIN_XML_SECURE |
-                                      VIR_DOMAIN_XML_INACTIVE);
-
-            if (!xml ||
-                (virDomainDestroy(dom) < 0 &&
-                 virDomainIsActive(dom) != 0))
-                goto cleanup;
-            virDomainFree(dom);
-            if ((dom = virDomainCreateXML(conn, xml, 0)))
-                ret = EXIT_SUCCESS;
-        }
-        break;
-
     case VIR_DOMAIN_LOCK_FAILURE_PAUSE:
         if (virDomainSuspend(dom) == 0)
             ret = EXIT_SUCCESS;
         break;
 
     case VIR_DOMAIN_LOCK_FAILURE_DEFAULT:
+    case VIR_DOMAIN_LOCK_FAILURE_RESTART:
     case VIR_DOMAIN_LOCK_FAILURE_IGNORE:
     case VIR_DOMAIN_LOCK_FAILURE_LAST:
         fprintf(stderr, _("unsupported failure action: '%s'\n"),
-- 
1.9.1