[GCP DWS] Fix None issue when no provision timeout is provided

sky/provision/gcp/instance_utils.py

@@ -1081,16 +1081,17 @@ def create_instances(
                 run_duration=managed_instance_group_config['run_duration'])
             cls.wait_for_operation(operation, project_id, zone=zone)
 
+        provision_timeout = managed_instance_group_config.get('provision_timeout')
+        if provision_timeout is None:
+            provision_timeout = constants.DEFAULT_MANAGED_INSTANCE_GROUP_PROVISION_TIMEOUT
         # This will block the provisioning until the nodes are ready, which
         # makes the failover not effective. We rely on the request timeout set
         # by user to trigger failover.
         mig_utils.wait_for_managed_group_to_be_stable(
             project_id,
             zone,
             managed_instance_group_name,
-            timeout=managed_instance_group_config.get(
-                'provision_timeout',
-                constants.DEFAULT_MANAGED_INSTANCE_GROUP_PROVISION_TIMEOUT))
+            timeout=provision_timeout)
 
         pending_running_instance_names = cls._add_labels_and_find_head(
             cluster_name, project_id, zone, labels, potential_head_instances)

sky/provision/gcp/mig_utils.py

@@ -207,3 +207,4 @@ def wait_for_managed_group_to_be_stable(project_id: str, zone: str,
     except subprocess.CalledProcessError as e:
         stderr = e.stderr.decode('ascii')
         logger.info(stderr)
+        raise