message_forwarding: Don't block cross-pool migrations for unbootable VMs

forward_to_suitable_host picks a host that can boot up the VM, erroring out
with NO_HOSTS_AVAILABLE otherwise. In case of a non-live VM that can't boot
anywhere in the source pool (because it requires a larger amount of memory, for
example), this would block its migration to a suitable pool altogether.

Catch the error and launch the migration from any host that can see the VM's
SRs instead of blocking it.

Signed-off-by: Andrii Sultanov <andriy.sultanov@vates.tech>

Author: last-genius

ocaml/xapi/message_forwarding.ml

@@ -2672,13 +2672,14 @@ functor
           Client.VM.migrate_send ~vm ~dest ~live ~vdi_map ~vif_map ~options
             ~vgpu_map
         in
+        let host = List.assoc Xapi_vm_migrate._host dest |> Ref.of_string in
+        let cross_pool = not (Db.is_valid_ref __context host) in
         let migration_type =
           if Xapi_vm_lifecycle_helpers.is_live ~__context ~self:vm then
-            let host = List.assoc Xapi_vm_migrate._host dest |> Ref.of_string in
-            if Db.is_valid_ref __context host then
-              `Live_intrapool host
-            else
+            if cross_pool then
               `Live_interpool
+            else
+              `Live_intrapool host
           else
             `Non_live
         in
@@ -2700,12 +2701,22 @@ functor
           | `Live_interpool ->
               forward_internal_async ()
               (* resources on the destination will be reserved separately *)
-          | `Non_live ->
+          | `Non_live -> (
               let snapshot = Db.VM.get_record ~__context ~self:vm in
-              fst
-                (forward_to_suitable_host ~local_fn ~__context ~vm ~snapshot
-                   ~host_op:`vm_migrate ~remote_fn ()
-                )
+              try
+                fst
+                  (forward_to_suitable_host ~local_fn ~__context ~vm ~snapshot
+                     ~host_op:`vm_migrate ~remote_fn ()
+                  )
+              with
+              | Api_errors.Server_error (code, _)
+              when code = Api_errors.no_hosts_available && cross_pool
+              ->
+                (* If non-live VM can't start anywhere in the pool, allow
+                   cross-pool migrations, with any host that can see its
+                   SRs acting as the sender *)
+                forward_to_access_srs ~local_fn ~__context ~vm ~remote_fn
+            )
           | `Live_intrapool host ->
               (* reserve resources on the destination host, then forward the call to the source. *)
               let snapshot = Db.VM.get_record ~__context ~self:vm in