Skip to content

Fix vLLM image reference and add pull secret verification#28

Merged
aneeshkp merged 1 commit intoopendatahub-io:mainfrom
aneeshkp:docs/verify-pull-secret
Feb 26, 2026
Merged

Fix vLLM image reference and add pull secret verification#28
aneeshkp merged 1 commit intoopendatahub-io:mainfrom
aneeshkp:docs/verify-pull-secret

Conversation

@aneeshkp
Copy link
Copy Markdown
Contributor

@aneeshkp aneeshkp commented Feb 26, 2026

Summary

  • Fix incorrect vLLM image reference from registry.redhat.io/rhaiis-tech-preview/vllm-openai-rhel9
    to registry.redhat.io/rhaiis/vllm-cuda-rhel9 which is the actual image used in the charts
  • Add vLLM image pull verification step to README alongside the existing Sail Operator check

Changes

  • README.md — add podman pull registry.redhat.io/rhaiis/vllm-cuda-rhel9:latest verification step
  • docs/deploying-llm-d-on-managed-kubernetes.md — fix vLLM image in pull secret verification
    and component reference table

Test plan

  • Verify podman pull registry.redhat.io/rhaiis/vllm-cuda-rhel9:latest works with valid credentials

Summary by CodeRabbit

  • Documentation
    • Updated verification steps to reference Sail Operator deployment.
    • Added new verification command for vLLM container image access.
    • Updated vLLM container image references across deployment documentation.

Signed-off-by: Aneesh Puttur <aneeshputtur@gmail.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to data retention organization setting

📥 Commits

Reviewing files that changed from the base of the PR and between 1494329 and 6bac248.

📒 Files selected for processing (2)
  • README.md
  • docs/deploying-llm-d-on-managed-kubernetes.md

📝 Walkthrough

Walkthrough

The PR updates documentation to reference an updated vLLM container image registry and adds a verification command for the Sail Operator. Image registry references are changed from a tech-preview registry to a CUDA-based production registry across two documentation files.

Changes

Cohort / File(s) Summary
Documentation Updates
README.md, docs/deploying-llm-d-on-managed-kubernetes.md
Updated vLLM container image registry references from registry.redhat.io/rhaiis-tech-preview/vllm-openai-rhel9:latest to registry.redhat.io/rhaiis/vllm-cuda-rhel9:latest in multiple locations. Added a new verification command in README to pull the vLLM image and clarified the verification section as "Sail Operator" specific.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main changes: fixing the vLLM image reference and adding pull secret/image verification steps across README and documentation files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Copy link
Copy Markdown
Member

@anishasthana anishasthana left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Feb 26, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aneeshkp, anishasthana

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 [aneeshkp,anishasthana]

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

@aneeshkp aneeshkp merged commit e540cfe into opendatahub-io:main Feb 26, 2026
2 checks passed
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.

2 participants