Use uv's venv path

Signed-off-by: Zack Cerza <zack@cerza.org>

diff --git a/bootstrap b/bootstrap
index cd6638a0a18bf1f846fa54b2990f0bb84433bbe7..2145e17ccb33e3f53c4153a83eb8e4ed1d733193 100755 (executable)
--- a/bootstrap
+++ b/bootstrap
@@ -86,6 +86,8 @@ fi
 command -v uv > /dev/null || pipx ensurepath
 PATH=$PATH:$HOME/.local/bin
 uv sync --frozen --all-extras
+# To avoid crashing older dispatchers
+ln -sf .venv virtualenv
 
 # Install ansible collections
 uv run ansible-galaxy install -r requirements.yml

diff --git a/containers/teuthology-dev/teuthology.sh b/containers/teuthology-dev/teuthology.sh
index 373f6efb8bf64c64260aaf0ff16e36d8f561d47e..49219d20556c6e3e6cf6a6c52b44f2970d019cb4 100755 (executable)
--- a/containers/teuthology-dev/teuthology.sh
+++ b/containers/teuthology-dev/teuthology.sh
@@ -1,6 +1,6 @@
 #!/usr/bin/bash
 set -e
-source /teuthology/virtualenv/bin/activate
+PATH=$PATH:/teuthology/.venv/bin
 set -x
 cat /run/secrets/id_rsa > $HOME/.ssh/id_rsa
 if [ -n "$TEUTHOLOGY_TESTNODES" ]; then

diff --git a/docs/docker-compose/teuthology/teuthology.sh b/docs/docker-compose/teuthology/teuthology.sh
index 78770e323cb43e70d33ba91548ccc3e49ea77cc0..c9ed897e23e90f936d07faef7928c655ab88339d 100755 (executable)
--- a/docs/docker-compose/teuthology/teuthology.sh
+++ b/docs/docker-compose/teuthology/teuthology.sh
@@ -4,7 +4,7 @@ set -e
 if [ -n "$SSH_PRIVKEY_FILE" ]; then
     echo "$SSH_PRIVKEY" > $HOME/.ssh/$SSH_PRIVKEY_FILE
 fi
-source /teuthology/virtualenv/bin/activate
+PATH=$PATH:/teuthology/.venv/bin
 set -x
 if [ -n "$TESTNODES" ]; then
     for node in $(echo $TESTNODES | tr , ' '); do

diff --git a/teuthology/dispatcher/__init__.py b/teuthology/dispatcher/__init__.py
index 59f8ae3279d8e27509e2322d6cc32c0ea8bf6815..f64246f2a303c1ef0d8a3d24c11bf136d51933fb 100644 (file)
--- a/teuthology/dispatcher/__init__.py
+++ b/teuthology/dispatcher/__init__.py
@@ -304,7 +304,7 @@ def prep_job(job_config, log_file_path, archive_dir):
         )
         raise SkipJob()
 
-    teuth_bin_path = os.path.join(teuth_path, 'virtualenv', 'bin')
+    teuth_bin_path = os.path.join(teuth_path, '.venv', 'bin')
     if not os.path.isdir(teuth_bin_path):
         raise RuntimeError("teuthology branch %s at %s not bootstrapped!" %
                            (teuthology_branch, teuth_bin_path))

diff --git a/teuthology/dispatcher/test/test_dispatcher.py b/teuthology/dispatcher/test/test_dispatcher.py
index 58f58cf9cf1f970997744c2cfbdc0cbbcefe12f0..4b15e35376ea36110d02c6fc02e1252c4036e744 100644 (file)
--- a/teuthology/dispatcher/test/test_dispatcher.py
+++ b/teuthology/dispatcher/test/test_dispatcher.py
@@ -71,7 +71,7 @@ class TestDispatcher(object):
         )
         assert got_config['teuthology_branch'] == 'main'
         m_fetch_teuthology.assert_called_once_with(branch='main', commit='teuth_hash')
-        assert teuth_bin_path == '/teuth/path/virtualenv/bin'
+        assert teuth_bin_path == '/teuth/path/.venv/bin'
         m_fetch_qa_suite.assert_called_once_with('main', 'suite_hash')
         assert got_config['suite_path'] == '/suite/path'
 

diff --git a/teuthology/repo_utils.py b/teuthology/repo_utils.py
index db6e87a8c7138cfffdd7264da51e70d944790e54..bb959b5c226da4ca5f877ad213c0f9bedd0f9fea 100644 (file)
--- a/teuthology/repo_utils.py
+++ b/teuthology/repo_utils.py
@@ -464,7 +464,7 @@ def bootstrap_teuthology(dest_path):
         if returncode != 0:
             for line in out.split("\n"):
                 log.warning(line.strip())
-            venv_path = os.path.join(dest_path, 'virtualenv')
+            venv_path = os.path.join(dest_path, '.venv')
             log.info("Removing %s", venv_path)
             shutil.rmtree(venv_path, ignore_errors=True)
             raise BootstrapError("Bootstrap failed!")