Skip azure compat tests

python/tests/compat/conftest.py

@@ -23,8 +23,15 @@ def is_running_on_windows():
 
 def run_shell_command(command : List[Union[str, os.PathLike]], cwd : Optional[os.PathLike] = None) -> subprocess.CompletedProcess:
     logger.info(f"Executing command: {command}")
-    # shell=True is required for running the correct python executable on Windows
-    result = subprocess.run(command, cwd=cwd, capture_output=True, shell=is_running_on_windows())
+    result = None
+    if is_running_on_windows():
+        # shell=True is required for running the correct python executable on Windows
+        result = subprocess.run(command, cwd=cwd, capture_output=True, shell=True)
+    else:
+        # On linux we need shell=True for conda feedstock runners (because otherwise they fail to expand path variables)
+        # But to correctly work with shell=True we need a single command string.
+        command_string = ' '.join(command)
+        result = subprocess.run(command_string, cwd=cwd, capture_output=True, shell=True, stdin=subprocess.DEVNULL)
     if result.returncode != 0:
         logger.warning(f"Command failed, stdout: {str(result.stdout)}, stderr: {str(result.stderr)}")
     return result
@@ -140,15 +147,14 @@ def assert_read(self, sym : str, df) -> None:
     scope="session",
     params=[
         pytest.param("1.6.2", marks=VENV_COMPAT_TESTS_MARK),
-        pytest.param("4.5.0", marks=VENV_COMPAT_TESTS_MARK),
+        pytest.param("4.5.1", marks=VENV_COMPAT_TESTS_MARK),
     ] # TODO: Extend this list with other old versions
 )
 def old_venv(request):
     version = request.param
     path = os.path.join("venvs", version)
-    # The requirements_file needs to be relative to the [path] we use for the venv.
-    # Absolute paths break some Azure CI runners on conda forge
-    requirements_file = os.path.join("..", "..", "tests", "compat", f"requirements-{version}.txt")
+    compat_dir = os.path.dirname(os.path.abspath(__file__))
+    requirements_file = os.path.join(compat_dir, f"requirements-{version}.txt")
     with Venv(path, requirements_file, version) as old_venv:
         yield old_venv
 
@@ -176,10 +182,13 @@ def arctic_uri(request):
 
 @pytest.fixture()
 def old_venv_and_arctic_uri(old_venv, arctic_uri):
-    # TODO: Replace 4.5.0 with 4.5.1 when it is released to re-enable both mongo and lmdb.
-    if Version(old_venv.version) <= Version("4.5.0") and arctic_uri.startswith("mongo"):
-        pytest.skip("Mongo storage backend has a desctruction bug present until 4.5.0, which can cause flaky segfaults.")
+    if arctic_uri.startswith("mongo"):
+        # TODO: We tought the bug was fixed in 4.5.1 but it's still present.
+        pytest.skip("Mongo storage backend has a desctruction bug, which can cause flaky segfaults.")
     if Version(old_venv.version) <= Version("4.5.0") and arctic_uri.startswith("lmdb"):
         pytest.skip("LMDB storage backend has a desctruction bug present until 4.5.0, which can cause flaky segfaults.")
+    if arctic_uri.startswith("azure"):
+        # TODO: Once #1979 is understood and fixed reenable azure on versions which have the fix.
+        pytest.skip("Azure storage backend has probable a desctruction bug, which can cause flaky segfaults.")
 
     return old_venv, arctic_uri

python/tests/compat/requirements-4.5.1.txt

@@ -0,0 +1,3 @@
+arcticdb==4.5.1
+numpy<2
+pyarrow