Merge branch 'faster-spot-queue' of github.com:skypilot-org/skypilot …

…into faster-spot-queue
skypilot-org · Mar 11, 2024 · 36e4865 · 36e4865
2 parents 6e34249 + c1e224d
commit 36e4865
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/sky/backends/backend_utils.py b/sky/backends/backend_utils.py
@@ -2312,10 +2312,10 @@ def is_controller_accessible(
         error_msg = non_existent_message
     elif (controller_status == status_lib.ClusterStatus.INIT or
           need_connection_check):
-        # We check the access to controller in INIT state or failed to fetch the
-        # status, as its lock can be hold by another `sky spot launch` or
-        # `sky serve up`, but still have head_ip available. In those cases,
-        # we can allow the access to the controller.
+        # Check ssh connection if (1) controller is in INIT state, or (2) we failed to fetch the
+        # status, both of which can happen when controller's status lock is held by another `sky spot launch` or
+        # `sky serve up`. If we have controller's head_ip available and it is ssh-reachable,
+        # we can allow access to the controller.
         ssh_credentials = ssh_credential_from_yaml(handle.cluster_yaml,
                                                    handle.docker_user,
                                                    handle.ssh_user)