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

[PR] Breeze: ensure generated Dockerfile.pmc upgrades uv to required floor [airflow]

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

   `breeze release-management check-release-files providers` generates a
   `Dockerfile.pmc` that copies the root `pyproject.toml` into the CI image
   and then runs `uv pip install`. If the CI image ships an older uv than
   `[tool.uv] required-version`, the install step fails with:
   
   ```
   error: Required uv version `>=X.Y.Z` does not match the running
   version `A.B.C`. Update `uv` by running `uv self update`.
   ```
   
   This is reproducible today: the current `ghcr.io/apache/airflow/main/ci/python3.10`
   image ships uv `0.11.6`, the pin is `>=0.11.7`, and so every PMC
   `Dockerfile.pmc` install attempt breaks out of the box.
   
   ## Fix
   
   - Inject `RUN pip install --upgrade 'uv>=X.Y.Z'` before `uv pip install`
     in `PROVIDERS_DOCKER`. The install step now works regardless of when
     the CI image was last rebuilt.
   - Put the floor in a `_PROVIDERS_DOCKER_UV_MIN_VERSION` constant with
     the existing `# sync-uv-min-version` marker, and add
     `check_release_files.py` to the `sync-uv-min-version-markers` prek
     hook's file list. When someone bumps `[tool.uv] required-version`,
     the hook rewrites this constant on the next commit.
   - Two new tests guard the change: (1) the rendered `PROVIDERS_DOCKER`
     contains the uv upgrade step and it precedes `uv pip install`;
     (2) the hard-coded floor matches the current `required-version` from
     the root `pyproject.toml` (belt-and-braces — the hook enforces this
     at commit time, the test catches accidental drift).
   
   Not auto-syncing with `AIRFLOW_UV_VERSION` on purpose — per the comment
   next to `required-version`, that value intentionally stays decoupled
   and is managed via `upgrade_important_versions.py`.
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Opus 4.7
   
   Generated-by: Claude Opus 4.7 following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)


-- 
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] Breeze: ensure generated Dockerfile.pmc upgrades uv to required floor [airflow]

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


-- 
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] Breeze: ensure generated Dockerfile.pmc upgrades uv to required floor [airflow]

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

   ### Backport successfully created: v3-2-test
   
   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/pull/65829"><img src="https://img.shields.io/badge/PR-65829-blue" alt="PR Link"></a></td>
       </tr>
   </table>


-- 
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