From eb97efd4cda068716fc5540b17b926dc855d37f8 Mon Sep 17 00:00:00 2001 From: Jarek Potiuk Date: Wed, 8 Jul 2026 18:22:32 +0200 Subject: [PATCH] Run CI on chart and airflow-ctl release-branch pull requests Pull requests targeting the chart/v*-test|stable and airflow-ctl/v*-test|stable release branches received no CI at all: those branch patterns were missing from the pull_request/push branch filters of ci-amd, ci-arm and codeql-analysis, so only branch-independent bot checks reported and a PR could show all-green without any tests, static checks, or CodeQL ever running. Add the chart and airflow-ctl release-branch patterns to those triggers (the sync-enforced AMD-only trigger block in check_ci_workflows_in_sync.py is updated in the same change so the amd/arm guard keeps passing). push triggers get the -test variants only, matching the existing release-prep push convention. --- .github/workflows/ci-amd.yml | 6 ++++++ .github/workflows/ci-arm.yml | 2 ++ .github/workflows/codeql-analysis.yml | 18 ++++++++++++++++-- scripts/ci/prek/check_ci_workflows_in_sync.py | 4 ++++ 4 files changed, 28 insertions(+), 2 deletions(-) diff --git a/.github/workflows/ci-amd.yml b/.github/workflows/ci-amd.yml index ab0a1a7ff606f..bae46cfc5e952 100644 --- a/.github/workflows/ci-amd.yml +++ b/.github/workflows/ci-amd.yml @@ -37,6 +37,10 @@ on: # yamllint disable-line rule:truthy - v[0-9]+-[0-9]+-test - v[0-9]+-[0-9]+-stable - providers-[a-z]+-?[a-z]*/v[0-9]+-[0-9]+ + - chart/v[0-9]+-[0-9]+x-test + - chart/v[0-9]+-[0-9]+x-stable + - airflow-ctl/v[0-9]+-[0-9]+-test + - airflow-ctl/v[0-9]+-[0-9]+-stable types: [opened, reopened, synchronize, ready_for_review] push: # Post-merge pushes to release-prep / providers branches run on both @@ -47,6 +51,8 @@ on: # yamllint disable-line rule:truthy branches: - v[0-9]+-[0-9]+-test - providers-[a-z]+-?[a-z]*/v[0-9]+-[0-9]+ + - chart/v[0-9]+-[0-9]+x-test + - airflow-ctl/v[0-9]+-[0-9]+-test workflow_dispatch: permissions: # All other permissions are set to none by default diff --git a/.github/workflows/ci-arm.yml b/.github/workflows/ci-arm.yml index 354155192a2d5..6dccbcd32e1b4 100644 --- a/.github/workflows/ci-arm.yml +++ b/.github/workflows/ci-arm.yml @@ -40,6 +40,8 @@ on: # yamllint disable-line rule:truthy branches: - v[0-9]+-[0-9]+-test - providers-[a-z]+-?[a-z]*/v[0-9]+-[0-9]+ + - chart/v[0-9]+-[0-9]+x-test + - airflow-ctl/v[0-9]+-[0-9]+-test workflow_dispatch: permissions: # All other permissions are set to none by default diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml index ec2d9e6fd8feb..cd681028a4701 100644 --- a/.github/workflows/codeql-analysis.yml +++ b/.github/workflows/codeql-analysis.yml @@ -20,9 +20,23 @@ name: "CodeQL" on: # yamllint disable-line rule:truthy pull_request: - branches: ['main', 'v[0-9]+-[0-9]+-test', 'v[0-9]+-[0-9]+-stable'] + branches: + - main + - v[0-9]+-[0-9]+-test + - v[0-9]+-[0-9]+-stable + - chart/v[0-9]+-[0-9]+x-test + - chart/v[0-9]+-[0-9]+x-stable + - airflow-ctl/v[0-9]+-[0-9]+-test + - airflow-ctl/v[0-9]+-[0-9]+-stable push: - branches: ['main', 'v[0-9]+-[0-9]+-test', 'v[0-9]+-[0-9]+-stable'] + branches: + - main + - v[0-9]+-[0-9]+-test + - v[0-9]+-[0-9]+-stable + - chart/v[0-9]+-[0-9]+x-test + - chart/v[0-9]+-[0-9]+x-stable + - airflow-ctl/v[0-9]+-[0-9]+-test + - airflow-ctl/v[0-9]+-[0-9]+-stable schedule: - cron: '0 2 * * *' diff --git a/scripts/ci/prek/check_ci_workflows_in_sync.py b/scripts/ci/prek/check_ci_workflows_in_sync.py index d6116d856b8bf..2e5584c2431dd 100755 --- a/scripts/ci/prek/check_ci_workflows_in_sync.py +++ b/scripts/ci/prek/check_ci_workflows_in_sync.py @@ -126,6 +126,10 @@ - v[0-9]+-[0-9]+-test - v[0-9]+-[0-9]+-stable - providers-[a-z]+-?[a-z]*/v[0-9]+-[0-9]+ + - chart/v[0-9]+-[0-9]+x-test + - chart/v[0-9]+-[0-9]+x-stable + - airflow-ctl/v[0-9]+-[0-9]+-test + - airflow-ctl/v[0-9]+-[0-9]+-stable types: [opened, reopened, synchronize, ready_for_review] """