From dd4d1a347730195456c3d71ec857cbfc72e78826 Mon Sep 17 00:00:00 2001 From: Sebastian Rittau Date: Wed, 27 Aug 2025 16:05:01 +0200 Subject: [PATCH 1/2] [CI] Move installing 3rd party pks into extra step --- .github/workflows/daily.yml | 12 ++++++++---- .github/workflows/stubtest_third_party.yml | 21 +++++++++++++++------ 2 files changed, 23 insertions(+), 10 deletions(-) diff --git a/.github/workflows/daily.yml b/.github/workflows/daily.yml index b59ee0d89a26..a00d887fd8fa 100644 --- a/.github/workflows/daily.yml +++ b/.github/workflows/daily.yml @@ -76,7 +76,7 @@ jobs: stubs/**/METADATA.toml - name: Install dependencies run: pip install -r requirements-tests.txt - - name: Run stubtest + - name: Install required system packages shell: bash run: | PACKAGES=$(python tests/get_stubtest_system_requirements.py) @@ -86,8 +86,6 @@ jobs: printf "Installing APT packages:\n $(echo $PACKAGES | sed 's/ /\n /g')\n" sudo apt-get update -q && sudo apt-get install -qy $PACKAGES fi - - PYTHON_EXECUTABLE="xvfb-run python" else if [ "${{ runner.os }}" = "macOS" ] && [ -n "$PACKAGES" ]; then printf "Installing Homebrew packages:\n $(echo $PACKAGES | sed 's/ /\n /g')\n" @@ -98,7 +96,13 @@ jobs: printf "Installing Chocolatey packages:\n $(echo $PACKAGES | sed 's/ /\n /g')\n" choco install -y $PACKAGES fi - + fi + - name: Run stubtest + shell: bash + run: | + if [ "${{ runner.os }}" = "Linux" ]; then + PYTHON_EXECUTABLE="xvfb-run python" + else PYTHON_EXECUTABLE="python" fi diff --git a/.github/workflows/stubtest_third_party.yml b/.github/workflows/stubtest_third_party.yml index 12ba22e2b85b..6dc88301a7d7 100644 --- a/.github/workflows/stubtest_third_party.yml +++ b/.github/workflows/stubtest_third_party.yml @@ -48,7 +48,7 @@ jobs: stubs/**/METADATA.toml - name: Install dependencies run: pip install -r requirements-tests.txt - - name: Run stubtest + - name: Determine changed stubs shell: bash run: | # This only runs stubtest on changed stubs, because it is much faster. @@ -59,18 +59,17 @@ jobs: (while read stub; do [ -d "stubs/$stub" ] && echo "$stub" || true; done) } STUBS=$(find_stubs || echo '') - + echo "Changed stubs: $STUBS" + echo "STUBS=$STUBS" >> $GITHUB_ENV + - name: Install required system packages + run: | if [ -n "$STUBS" ]; then - echo "Testing $STUBS..." PACKAGES=$(python tests/get_stubtest_system_requirements.py $STUBS) - if [ "${{ runner.os }}" = "Linux" ]; then if [ -n "$PACKAGES" ]; then printf "Installing APT packages:\n $(echo $PACKAGES | sed 's/ /\n /g')\n" sudo apt-get update -q && sudo apt-get install -qy $PACKAGES fi - - PYTHON_EXECUTABLE="xvfb-run python" else if [ "${{ runner.os }}" = "macOS" ] && [ -n "$PACKAGES" ]; then printf "Installing Homebrew packages:\n $(echo $PACKAGES | sed 's/ /\n /g')\n" @@ -81,7 +80,17 @@ jobs: printf "Installing Chocolatey packages:\n $(echo $PACKAGES | sed 's/ /\n /g')\n" choco install -y $PACKAGES fi + fi + fi + - name: Run stubtest + shell: bash + run: | + if [ -n "$STUBS" ]; then + echo "Testing $STUBS..." + if [ "${{ runner.os }}" = "Linux" ]; then + PYTHON_EXECUTABLE="xvfb-run python" + else PYTHON_EXECUTABLE="python" fi From a123daab1fc5f81c7b7f22c4c705bdad8e55e438 Mon Sep 17 00:00:00 2001 From: Sebastian Rittau Date: Wed, 27 Aug 2025 16:09:39 +0200 Subject: [PATCH 2/2] Fix shell --- .github/workflows/stubtest_third_party.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/stubtest_third_party.yml b/.github/workflows/stubtest_third_party.yml index 6dc88301a7d7..bdfe81f568c2 100644 --- a/.github/workflows/stubtest_third_party.yml +++ b/.github/workflows/stubtest_third_party.yml @@ -62,6 +62,7 @@ jobs: echo "Changed stubs: $STUBS" echo "STUBS=$STUBS" >> $GITHUB_ENV - name: Install required system packages + shell: bash run: | if [ -n "$STUBS" ]; then PACKAGES=$(python tests/get_stubtest_system_requirements.py $STUBS)