[Fluidstack] Support opening ports and service (#3294)

* Allow ports for fluidstack

* expose open_ports

* fix api for cleanup_ports

* fix

sky/clouds/fluidstack.py

@@ -50,9 +50,6 @@ class Fluidstack(clouds.Cloud):
         clouds.CloudImplementationFeatures.CUSTOM_DISK_TIER:
             'Custom disk tiers'
             f' is not supported in {_REPR}.',
-        clouds.CloudImplementationFeatures.OPEN_PORTS:
-            'Opening ports'
-            f'is not supported in {_REPR}.',
     }
     # Using the latest SkyPilot provisioner API to provision and check status.
     PROVISIONER_VERSION = clouds.ProvisionerVersion.SKYPILOT

sky/provision/fluidstack/__init__.py

@@ -3,6 +3,7 @@
 from sky.provision.fluidstack.config import bootstrap_instances
 from sky.provision.fluidstack.instance import cleanup_ports
 from sky.provision.fluidstack.instance import get_cluster_info
+from sky.provision.fluidstack.instance import open_ports
 from sky.provision.fluidstack.instance import query_instances
 from sky.provision.fluidstack.instance import run_instances
 from sky.provision.fluidstack.instance import stop_instances

sky/provision/fluidstack/instance.py

@@ -342,6 +342,17 @@ def query_instances(
 
 def cleanup_ports(
     cluster_name_on_cloud: str,
+    ports: List[str],
+    provider_config: Optional[Dict[str, Any]] = None,
+) -> None:
+    del cluster_name_on_cloud, ports, provider_config
+
+
+def open_ports(
+    cluster_name_on_cloud: str,
+    ports: List[str],
     provider_config: Optional[Dict[str, Any]] = None,
 ) -> None:
     del cluster_name_on_cloud, provider_config
+    logger.debug(f'Skip opening ports {ports} for Fluidstack instances, as all '
+                 'ports are open by default.')

sky/utils/controller_utils.py

@@ -144,6 +144,7 @@ def _get_cloud_dependencies_installation_commands(
         'pip install google-api-python-client>=2.69.0 google-cloud-storage '
         '> /dev/null 2>&1',
         f'{gcp.GOOGLE_SDK_INSTALLATION_COMMAND}',
+        # fluidstack does not need to install any cloud dependencies.
     ]
     # k8s and ibm doesn't support open port and spot instance yet, so we don't
     # install them for either controller.