From 3700ecfa6d91cb519f956266aaff30c409ecb737 Mon Sep 17 00:00:00 2001 From: youben11 Date: Wed, 28 May 2025 10:47:47 +0100 Subject: [PATCH] fix(ci): use venv for concrete macos tests --- .../concrete_compiler_test_macos_cpu.yml | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/.github/workflows/concrete_compiler_test_macos_cpu.yml b/.github/workflows/concrete_compiler_test_macos_cpu.yml index 9215d899d..5c263258c 100644 --- a/.github/workflows/concrete_compiler_test_macos_cpu.yml +++ b/.github/workflows/concrete_compiler_test_macos_cpu.yml @@ -48,16 +48,18 @@ jobs: persist-credentials: 'false' - name: Setup rust toolchain for concrete-cpu uses: ./.github/actions/setup_rust_toolchain_for_concrete_cpu - - name: Set python variables - run: | - { - echo "PYTHON_EXEC=$(which ${{ env.python }})" - } >> "${GITHUB_ENV}" - name: Install dependencies run: | - brew install ninja ccache zstd - ${{ env.pip }} install -r ${{ env.concrete-compiler-dir }}/lib/Bindings/Python/requirements_dev.txt - ${{ env.pip }} install -r ${{ env.concrete-compiler-dir }}/../llvm-project/mlir/python/requirements.txt + # We don't start macos instances on every run + # brew install ninja ccache zstd + ${{ env.python }} -m venv .testenv + . .testenv/bin/activate + pip install -r ${{ env.concrete-compiler-dir }}/lib/Bindings/Python/requirements_dev.txt + pip install -r ${{ env.concrete-compiler-dir }}/../llvm-project/mlir/python/requirements.txt + pip install pytest + { + echo "PYTHON_EXEC=$(which python)" + } >> "${GITHUB_ENV}" - name: Build compiler run: | set -e @@ -84,6 +86,7 @@ jobs: - name: Test run: | set -e + . .testenv/bin/activate cd ${{ env.concrete-compiler-dir }} export CONCRETE_COMPILER_DATAFLOW_EXECUTION_ENABLED=OFF make MINIMAL_TESTS=${{ env.MINIMAL_TESTS }} Python3_EXECUTABLE="${PYTHON_EXEC}" run-tests