Merge pull request #565 from NVIDIA/am/no-copies

Do not use --copies for venv creation

Author: amaslenn

src/cloudai/systems/slurm/slurm_installer.py

@@ -263,7 +263,7 @@ def _create_venv(self, item: PythonExecutable) -> InstallStatusResult:
             logging.debug(msg)
             return InstallStatusResult(True, msg)
 
-        cmd = ["python", "-m", "venv", "--copies", str(venv_path)]
+        cmd = ["python", "-m", "venv", str(venv_path)]
         logging.debug(f"Creating venv using cmd: {' '.join(cmd)}")
         result = subprocess.run(cmd, capture_output=True, text=True)
         logging.debug(f"venv creation STDOUT:\n{result.stdout}\nSTDERR:\n{result.stderr}")

tests/test_slurm_installer.py

@@ -188,9 +188,7 @@ def test_venv_created(self, installer: SlurmInstaller, git: GitRepo):
             mock_run.return_value = CompletedProcess(args=[], returncode=0)
             res = installer._create_venv(py)
         assert res.success
-        mock_run.assert_called_once_with(
-            ["python", "-m", "venv", "--copies", str(venv_path)], capture_output=True, text=True
-        )
+        mock_run.assert_called_once_with(["python", "-m", "venv", str(venv_path)], capture_output=True, text=True)
 
     @pytest.mark.parametrize("failure_on_venv_creation,reqs_install_failure", [(True, False), (False, True)])
     def test_error_creating_venv(