Skip to content

fix: refresh Automodel artifact pins - #815

Merged
svvarom merged 1 commit into
mainfrom
fix-gpu-wheel-pins/svvarom
Jul 21, 2026
Merged

fix: refresh Automodel artifact pins#815
svvarom merged 1 commit into
mainfrom
fix-gpu-wheel-pins/svvarom

Conversation

@svvarom

@svvarom svvarom commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Why

PR #762 changed both Automodel artifact producers, including the Python 3.13 OpenCV/decord wheel set, but WHEELS_TAG and BASE_TAG_AUTOMODEL still pointed to the older f0756dd... build. The release source therefore consumed artifacts that did not match its Dockerfiles and failed while installing the expected Python 3.13 wheel.

What

  • Pin both Automodel artifact inputs to the validated full 2c1a9ef... source SHA.

Validation

No public runtime API changes.

@svvarom
svvarom requested review from a team as code owners July 21, 2026 16:26
@github-actions github-actions Bot added the fix label Jul 21, 2026
@coderabbitai

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The Docker bake configuration updates the fixed commit tag used for automodel base and wheel image references.

Changes

Docker artifact pin update

Layer / File(s) Summary
Docker bake image pin update
docker-bake.hcl
BASE_TAG_AUTOMODEL and WHEELS_TAG now use commit 2c1a9ef... instead of f0756dd....

Suggested reviewers: mckornfield, crookedstorm

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title matches the main change: refreshing Automodel artifact pins to a newer validated SHA.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-gpu-wheel-pins/svvarom

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor
Suite Lines Covered Line Rate Branch Rate
Unit Tests 26384/34035 77.5% 61.8%
Integration Tests 15172/32660 46.5% 18.7%

Signed-off-by: Swarom Muley <smuley@nvidia.com>
@svvarom
svvarom force-pushed the fix-gpu-wheel-pins/svvarom branch from d496ba1 to 51474bd Compare July 21, 2026 16:37
@svvarom svvarom self-assigned this Jul 21, 2026
@svvarom svvarom changed the title fix: guard Automodel artifact pins fix: refresh Automodel artifact pins Jul 21, 2026

@crookedstorm crookedstorm 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.

If this fixes the build sequence, I'm all for it!

@svvarom
svvarom added this pull request to the merge queue Jul 21, 2026
Merged via the queue into main with commit 55120bc Jul 21, 2026
64 checks passed
@svvarom
svvarom deleted the fix-gpu-wheel-pins/svvarom branch July 21, 2026 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants