OpenShift follow-up for NVBug 6218358#2166
Conversation
Document RC9 QA findings: prebuilt ffmpeg image on restricted-v2, openshift-restricted values profile, internal registry pull secrets, optional NIM LD_LIBRARY_PATH overrides, and Omni caption smoke-test request shape. Cross-link from deployment-options and prerequisites-support-matrix.
Greptile SummaryThis PR moves the OpenShift deployment guide out of the Helm README into a new canonical page (
|
| Filename | Overview |
|---|---|
| docs/docs/extraction/openshift.md | New canonical OpenShift guide covering restricted-v2/PSA-restricted overrides, prebuilt ffmpeg image, internal registry pull secrets, full env lists for optional NIMs, and install examples for service-only and NIM Operator flows. |
| nemo_retriever/helm/README.md | OpenShift section collapsed to a one-paragraph stub with cross-link; air-gapped snippet now sets name: "" to clear the ngc-secret default; "see" → "refer to" wording pass; OpenShift ffmpeg note added to the audio/video section. |
| docs/docs/extraction/deployment-options.md | ffmpeg paragraph now names OpenShift restricted-v2 as a blocked path and links to openshift.md; minor "see" → "refer to" wording clean-up. |
| docs/docs/extraction/prerequisites-support-matrix.md | ffmpeg bullet trimmed to a single line pointing to audio-video.md; "see" → "refer to" wording pass across footnotes and notes. |
| docs/mkdocs.yml | Added "OpenShift deployment" nav entry under "3. Deployment options", pointing at the new extraction/openshift.md page. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[User: Install on OpenShift] --> B{Audio/Video needed?}
B -- No --> C[Use openshift-restricted.yaml\nservice.installFfmpeg=false]
B -- Yes --> D[Build custom ffmpeg image\nFROM nrl-service + apt install ffmpeg]
D --> E[Push to internal registry or NGC]
E --> F[Set service.image.repository/tag\nin openshift-restricted.yaml]
C --> G{In-cluster NIMs?}
F --> G
G -- No --> H[helm install\n--set nims.enabled=false\n--set persistence.enabled=false]
G -- Yes --> I[Install NIM Operator + GPU Operator\nthen helm install with NIM Operator flags]
H --> J[oc get pods\nVerify SCC-assigned numeric UID]
I --> K[oc get pods\noc get nimservice]
K --> L{ASR/Omni CrashLoop?}
L -- Yes --> M[Add LD_LIBRARY_PATH overrides\nto nimOperator.audio.env /\nnimOperator.nemotron_3_nano_omni.env]
L -- No --> N[Ready]
M --> N
J --> N
Reviews (12): Last reviewed commit: "Merge branch 'main' into docs/6218358-op..." | Re-trigger Greptile
Move Omni smoke-test note out of the chart admonition; use neutral link label to helm README anchor.
Add explicit #1-service-image anchor and Parakeet step-4 OpenShift caveat for installFfmpeg.
Resolve conflict in helm README OpenShift smoke-test example: keep service.image repository/tag overrides from the PR branch.
Replace matrix smoke-test prose with a one-line helm link per page roles. Show full nimOperator env lists in OpenShift LD_LIBRARY_PATH examples so Helm list overrides do not drop chart defaults.
Move Helm, OpenShift, and Omni smoke-test guidance out of the support matrix so deployment details stay on audio-video.md and the Helm README.
Extract restricted-v2 install procedures from the Helm README into docs/docs/extraction/openshift.md per review feedback. Helm README keeps a stub with cross-links; deployment-options and mkdocs nav point to the new page. Remove QA phrasing and fix ngcImagePullSecret name clearing.
Rewrite service-only and NIM Operator install intros as user actions instead of validation or QA repro language, per review on PR 2166.
Replace the removed QA validation paragraph with user-facing pod and NIMService checks plus guidance when optional ASR or Omni NIMs fail.
Clarify internal-registry pull secret wording and replace see-with-link CTAs with refer to so check-nrl-doc-leakage passes on PR 2166 files.
Replace via/once wording, drop code from headings, disambiguate caption links, and fix split refer-to link CTAs in the Helm README.
Clarify air-gap ngcImagePullSecret.name clearing and add copy-paste-safe LD_LIBRARY_PATH env examples with explicit chart defaults.
Summary
docs/docs/extraction/openshift.mdas the canonical OpenShift install guide (restricted-v2 / PSA restricted):openshift-restricted.yamlprofile, prebuilt ffmpeg service image, internal registry pull secrets (ngcImagePullSecret.name: ""), optional NIMLD_LIBRARY_PATHoverrides, direct Omni NIM/v1/chat/completionsexample, service-only and NIM Operator install flows, and post-install pod/NIMServicechecks.nemo_retriever/helm/README.mdto a short#openshift-deploymentstub plus cross-links intoopenshift.md; chart values and generic Kubernetes guidance stay in the Helm README.prerequisites-support-matrix.md(Software Requirements ffmpeg bullet → one line toaudio-video.md; removed matrix Omni smoke-test cross-link).deployment-options.mdanddocs/mkdocs.ymlnav to point atopenshift.md.NVBugs
6218358 — OpenShift Helm chart SCC/PodSecurity documentation follow-up (comment #6).
PR scope check
maindocs/docs/extraction/openshift.md(new),docs/docs/extraction/deployment-options.md,docs/docs/extraction/prerequisites-support-matrix.md,docs/mkdocs.yml,nemo_retriever/helm/README.md.cursor/, link-audit artifacts, chart/code changes)Test plan
openshift.mdrenders on docs site (mkdocs nav entry under Deployment options)#openshift-deploymentstub links toopenshift.md