Skip to content

Commit 01354a4

Browse files
authored
Fix provider registry never updating after docs publish (#65188)
The include-docs input uses short provider names like "amazon" and "common.ai", but the extraction logic only matched full package names like "apache-airflow-providers-amazon". This meant REGISTRY_PROVIDERS was always empty and the registry update job was always skipped. Handle both short names (converting dots to hyphens) and full package names, while skipping non-provider packages.
1 parent eb994d1 commit 01354a4

1 file changed

Lines changed: 10 additions & 2 deletions

File tree

.github/workflows/publish-docs-to-s3.yml

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -167,15 +167,23 @@ jobs:
167167
echo "airflow-base-version=no-airflow" >> ${GITHUB_OUTPUT}
168168
fi
169169
# Extract provider IDs for registry update
170-
# include-docs contains package names like "apache-airflow-providers-amazon"
171-
# Strip the "apache-airflow-providers-" prefix to get provider IDs
170+
# include-docs contains short names like "amazon", "common.ai", "openlineage"
171+
# or full package names like "apache-airflow-providers-amazon"
172172
REGISTRY_PROVIDERS=""
173173
for pkg in ${INCLUDE_DOCS}; do
174174
case "${pkg}" in
175175
apache-airflow-providers-*)
176176
PROVIDER_ID="${pkg#apache-airflow-providers-}"
177177
REGISTRY_PROVIDERS="${REGISTRY_PROVIDERS:+${REGISTRY_PROVIDERS} }${PROVIDER_ID}"
178178
;;
179+
apache-airflow|all-providers|apache-airflow-providers|helm-chart|docker-stack)
180+
# Not a provider package, skip
181+
;;
182+
*)
183+
# Short name like "amazon" or "common.ai" -- convert dots to hyphens
184+
PROVIDER_ID="${pkg//./-}"
185+
REGISTRY_PROVIDERS="${REGISTRY_PROVIDERS:+${REGISTRY_PROVIDERS} }${PROVIDER_ID}"
186+
;;
179187
esac
180188
done
181189
echo "registry-providers=${REGISTRY_PROVIDERS}" >> ${GITHUB_OUTPUT}

0 commit comments

Comments
 (0)