Skip to content

ci: Pin uv to 0.12.7 - #6831

Open
haoxu0 wants to merge 1 commit into
feast-dev:masterfrom
haoxu0:ci/pin-uv-0.12.7
Open

haoxu0 wants to merge 1 commit into
feast-dev:masterfrom
haoxu0:ci/pin-uv-0.12.7

Conversation

@haoxu0

@haoxu0 haoxu0 commented Sep 12, 2026

Copy link
Copy Markdown
Collaborator

What this PR does / why we need it:

Pins uv to 0.12.7 in all GitHub Actions workflows that install it through setup-uv.

Newer uv releases reject the locked PyPI hashes when --torch-backend cpu selects the equivalent local-version torchvision wheel (0.28.0+cpu), causing dependency installation to fail across CI. uv 0.12.7 preserves the behavior under which the current lock files were generated, restoring CI without regenerating or upgrading the dependency graph.

The step label is also changed from "Install the latest version of uv" to "Install uv" so it accurately describes the pinned installation.

Which issue(s) this PR fixes:

N/A

Checks

  • I've made sure the tests are passing.
  • My commits are signed off (git commit -s)
  • My PR title follows conventional commits format

Testing Strategy

  • Unit tests
  • Integration tests
  • Manual tests
  • Testing is not required for this change

Validated that all 12 setup-uv steps pin 0.12.7, parsed every changed workflow as YAML, and ran a successful uv 0.12.7 dry-run of the Python 3.11 x86_64 Linux dependency sync with --torch-backend cpu (449 packages resolved, including torchvision==0.28.0+cpu). Repository pre-commit checks also passed.

Misc

This is intentionally a minimal CI stabilization change. It does not modify pyproject.toml, requirement locks, Pixi configuration, or dependency versions.

Signed-off-by: HaoXuAI <sduxuhao@gmail.com>
@haoxu0
haoxu0 requested a lite review from Copilot September 12, 2026 01:07
@codecov-commenter

codecov-commenter commented Sep 12, 2026

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 47.10%. Comparing base (81e1546) to head (73e314c).
⚠️ Report is 2 commits behind head on master.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6831      +/-   ##
==========================================
+ Coverage   47.08%   47.10%   +0.01%     
==========================================
  Files         419      419              
  Lines       51877    51879       +2     
  Branches     7525     7525              
==========================================
+ Hits        24428    24436       +8     
+ Misses      25700    25693       -7     
- Partials     1749     1750       +1     
Flag Coverage Δ
go-feature-server 30.58% <ø> (ø)
python-unit 48.41% <ø> (+0.01%) ⬆️
see 3 files with indirect coverage changes

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b7a8928...73e314c. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@haoxu0 haoxu0 changed the title ci: pin uv to 0.12.7 ci: Pin uv to 0.12.7 Sep 12, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

Fork workflow templates still use unpinned uv and should receive the same pin.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Pins uv to 0.12.7 in active GitHub Actions workflows to prevent locked-hash dependency failures.

Changes:

  • Pins 12 setup-uv steps to uv 0.12.7.
  • Updates installation labels.
  • Fork workflow templates remain unpinned.
File summaries
File Description
.github/workflows/unit_tests.yml Pins uv in unit-test jobs
.github/workflows/smoke_tests.yml Pins uv for smoke tests
.github/workflows/registry-rest-api-tests.yml Pins uv for registry tests
.github/workflows/pr_remote_rbac_integration_tests.yml Pins uv for remote RBAC tests
.github/workflows/pr_local_integration_tests.yml Pins uv for local RBAC tests
.github/workflows/pr_integration_tests.yml Pins uv in integration jobs
.github/workflows/nightly-ci.yml Pins uv for nightly CI
.github/workflows/master_only.yml Pins uv for master-only CI
.github/workflows/linter.yml Pins uv for linting
.github/workflows/dbt-integration-tests.yml Pins uv for dbt tests
Review details
  • Files reviewed: 10/10 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

- name: Install uv
uses: astral-sh/setup-uv@v5
with:
version: "0.12.7"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants