grep pip list first

skypilot-org · Nov 26, 2023 · ccc81b0 · ccc81b0
1 parent 57410ac
commit ccc81b0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/sky/templates/sky-serve-controller.yaml.j2 b/sky/templates/sky-serve-controller.yaml.j2
@@ -11,7 +11,8 @@ setup: |
   {%- endfor %}
 
   # Install serve dependencies.
-  pip install uvicorn fastapi 2>&1 > /dev/null
+  pip list | grep uvicorn > /dev/null 2>&1 || pip install uvicorn > /dev/null 2>&1
+  pip list | grep fastapi > /dev/null 2>&1 || pip install fastapi > /dev/null 2>&1
 
 file_mounts:
   {{remote_task_yaml_path}}: {{local_task_yaml_path}}

diff --git a/sky/utils/controller_utils.py b/sky/utils/controller_utils.py
@@ -152,7 +152,7 @@ def _get_cloud_dependencies_installation_commands(
         commands.append(
             'pip list | grep azure-cli > /dev/null 2>&1 || '
             'pip install azure-cli>=2.31.0 azure-core azure-identity>=1.13.0 '
-            'azure-mgmt-network > /dev/null')
+            'azure-mgmt-network > /dev/null 2>&1')
     return commands