You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "vatsrahul1001 (via GitHub)" <gi...@apache.org> on 2026/05/21 09:57:04 UTC

[PR] workflows: use `task-sdk` alias in publish-docs NON_PROVIDER_TOKENS [airflow]

vatsrahul1001 opened a new pull request, #67279:
URL: https://github.com/apache/airflow/pull/67279

   ## Summary
   
   `publish-docs-to-s3.yml` filters distributions through a `NON_PROVIDER_TOKENS`
   bash array, with a matching `frozenset` in `dev/registry/derive_wave_providers.py`.
   Both lists currently use the PyPI distribution name `apache-airflow-task-sdk`,
   but `docs.build_docs` itself accepts the docs-folder alias `task-sdk` (see its
   `click` choices). Invoking the workflow with either name fails:
   
   - `task-sdk`: not in `NON_PROVIDER_TOKENS`, falls through to
     `dev/registry/derive_wave_providers.py` — which does not exist on older
     release branches like `v3-2-stable`.
   - `apache-airflow-task-sdk`: passes the filter but `docs.build_docs` rejects
     the name with `Invalid value for '[apache-airflow | ... | task-sdk | ...]': 'apache-airflow-task-sdk' is not one of ...`.
   
   This PR replaces `apache-airflow-task-sdk` with `task-sdk` in both lists so
   the workflow can publish task-sdk docs end-to-end. The other references to
   `apache-airflow-task-sdk` in the repo (pyproject templates, pip install
   commands, PyPI URLs) are intentional PyPI distribution-name uses and are
   left alone.
   
   ## Repro
   
   Discovered during the Airflow 3.2.2rc1 release. Run
   https://github.com/apache/airflow/actions/runs/26216945618 fails at the
   `Build documentation` step:
   
   ```
   Invalid value for '...': 'apache-airflow-task-sdk' is not one of
   'apache-airflow', ..., 'task-sdk', ..., 'docker-stack'.
   ```
   
   ## Related
   
   Follow-up to #66438 which introduced `NON_PROVIDER_TOKENS`.
   
   ## Test plan
   
   - [ ] After merge, re-run `breeze workflow-run publish-docs --ref 3.2.2rc1 --site-env staging apache-airflow docker-stack task-sdk` and confirm both the filter step and `Build documentation` step pass.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] workflows: use `task-sdk` alias in publish-docs NON_PROVIDER_TOKENS [airflow]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #67279:
URL: https://github.com/apache/airflow/pull/67279#issuecomment-4507044923

   ### Backport failed to create: v3-2-test. View the failure log <a href='https://github.com/apache/airflow/actions/runs/26219396067'> Run details </a>
   
   Note: As of [Merging PRs targeted for Airflow 3.X](https://github.com/apache/airflow/blob/main/dev/README_AIRFLOW3_DEV.md#merging-prs-targeted-for-airflow-3x)
   the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.
   
   In matter of doubt please ask in [#release-management](https://apache-airflow.slack.com/archives/C03G9H97MM2) Slack channel.
   
   <table>
       <tr>
           <th>Status</th>
           <th>Branch</th>
           <th>Result</th>
       </tr>
       <tr>
           <td>❌</td>
           <td>v3-2-test</td>
           <td><a href="https://github.com/apache/airflow/commit/f51d9e6ed237e9c5e2730abf9f1419dac39cff9f"><img src='https://img.shields.io/badge/Commit-f51d9e6-red' alt='Commit Link'></a></td>
       </tr>
   </table>
   
   You can attempt to backport this manually by running:
   
   ```bash
   cherry_picker f51d9e6 v3-2-test
   ```
   
   This should apply the commit to the v3-2-test branch and leave the commit in conflict state marking
   the files that need manual conflict resolution.
   
   After you have resolved the conflicts, you can continue the backport process by running:
   
   ```bash
   cherry_picker --continue
   ```
   
   If you don't have cherry-picker installed, see the [installation guide](https://github.com/apache/airflow/blob/main/dev/README.md#how-to-backport-pr-with-cherry-picker-cli).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] workflows: use `task-sdk` alias in publish-docs NON_PROVIDER_TOKENS [airflow]

Posted by "vatsrahul1001 (via GitHub)" <gi...@apache.org>.
vatsrahul1001 merged PR #67279:
URL: https://github.com/apache/airflow/pull/67279


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org