This is an automated email from the ASF dual-hosted git repository.

potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new 73f273517ae Skip registry-derive in publish-docs for non-provider 
distributions (#66438)
73f273517ae is described below

commit 73f273517aecba72b4141e82557fb669518cdeb2
Author: Jarek Potiuk <[email protected]>
AuthorDate: Tue May 5 22:00:44 2026 +0200

    Skip registry-derive in publish-docs for non-provider distributions (#66438)
    
    The "Derive registry trigger inputs" step in publish-docs-to-s3.yml
    runs `python3 dev/registry/derive_wave_providers.py` unconditionally
    in the Build Info job. The downstream `update-registry` job only
    runs when `registry-providers != ''` or `registry-full-build ==
    'true'`, so for non-provider distributions (apache-airflow,
    apache-airflow-ctl, apache-airflow-task-sdk, helm-chart,
    docker-stack) the script is a no-op anyway.
    
    Skipping the script for those distributions also lets release tags
    cut before #66100 (when the script was added) publish docs without
    the workflow tripping on the missing file. Surfaced while
    publishing docs for airflow-ctl/0.1.4 (cut from 0.1.4rc3 on
    2026-04-22).
    
    The non-provider list is kept in sync with NON_PROVIDER_TOKENS in
    dev/registry/derive_wave_providers.py.
    
    closes: #66436
---
 .github/workflows/publish-docs-to-s3.yml | 34 +++++++++++++++++++++++++++++++-
 1 file changed, 33 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/publish-docs-to-s3.yml 
b/.github/workflows/publish-docs-to-s3.yml
index 53ff431a8ec..eeb23ec2e3f 100644
--- a/.github/workflows/publish-docs-to-s3.yml
+++ b/.github/workflows/publish-docs-to-s3.yml
@@ -127,7 +127,39 @@ jobs:
         env:
           INCLUDE_DOCS: ${{ inputs.include-docs }}
           REF: ${{ inputs.ref }}
-        run: python3 dev/registry/derive_wave_providers.py
+        shell: bash
+        # The registry rebuild is provider-targeted. For non-provider
+        # distributions (apache-airflow, airflow-ctl, helm-chart, etc.) the
+        # update-registry job below is a no-op anyway, so skip the derive
+        # step — that lets release tags cut before
+        # `dev/registry/derive_wave_providers.py` was added (#66100) still
+        # publish docs without the workflow tripping on the missing file.
+        # Tracked at #66436.
+        run: |
+          # Keep the non-provider list in sync with NON_PROVIDER_TOKENS in
+          # dev/registry/derive_wave_providers.py.
+          NON_PROVIDER_TOKENS=("apache-airflow" "apache-airflow-ctl" 
"apache-airflow-task-sdk" "helm-chart" "docker-stack")
+          has_provider=false
+          for token in $INCLUDE_DOCS; do
+            is_non_provider=false
+            for np in "${NON_PROVIDER_TOKENS[@]}"; do
+              if [[ "$token" == "$np" ]]; then
+                is_non_provider=true
+                break
+              fi
+            done
+            if ! $is_non_provider; then
+              has_provider=true
+              break
+            fi
+          done
+          if ! $has_provider; then
+            echo "Non-provider distribution(s) only ('${INCLUDE_DOCS}') — 
skipping registry-derive."
+            echo "registry-providers=" >> "${GITHUB_OUTPUT}"
+            echo "registry-full-build=false" >> "${GITHUB_OUTPUT}"
+            exit 0
+          fi
+          python3 dev/registry/derive_wave_providers.py
       - name: "Input parameters summary"
         shell: bash
         id: parameters

Reply via email to