Skip to content

[release-1.4] fix(release): sync quickstart vllm images#2522

Merged
k8s-ci-robot merged 3 commits intorelease-1.4from
release-1.4-quickstart-image-sync
Mar 9, 2026
Merged

[release-1.4] fix(release): sync quickstart vllm images#2522
k8s-ci-robot merged 3 commits intorelease-1.4from
release-1.4-quickstart-image-sync

Conversation

@danehans
Copy link
Copy Markdown
Contributor

@danehans danehans commented Mar 8, 2026

What type of PR is this?
/kind bug

What this PR does / why we need it:
Fix the release-1.4 quickstart model server manifest drift introduced by the release automation.

This change does two things:

  • restores the release-1.4 vLLM GPU and simulator manifests to the image versions already used on main
  • updates hack/release-quickstart.sh so make release no longer rewrites the vLLM image tags and only normalizes imagePullPolicy for release branches

This keeps the release branch on the intended quickstart image versions while preventing future release commits from downgrading them again.

Which issue(s) this PR fixes:
Fixes #2521

Does this PR introduce a user-facing change?:

NONE

Signed-off-by: Daneyon Hansen <daneyon.hansen@solo.io>
@k8s-ci-robot k8s-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Mar 8, 2026
@k8s-ci-robot k8s-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Mar 8, 2026
danehans added 2 commits March 8, 2026 15:46
Signed-off-by: Daneyon Hansen <daneyon.hansen@solo.io>
Signed-off-by: Daneyon Hansen <daneyon.hansen@solo.io>
@danehans danehans changed the title [release-1.4] fix(release): sync quickstart vllm images with main [release-1.4] fix(release): sync quickstart vllm images Mar 8, 2026
Copy link
Copy Markdown
Contributor

@nirrozenbaum nirrozenbaum left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Mar 9, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: danehans, nirrozenbaum

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [danehans,nirrozenbaum]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot merged commit 4833908 into release-1.4 Mar 9, 2026
11 checks passed
@danehans danehans deleted the release-1.4-quickstart-image-sync branch March 10, 2026 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/bug Categorizes issue or PR as related to a bug. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants