Skip to content

docs(arch): update Kubernetes supervisor delivery to reflect init-container pattern#1215

Open
mesutoezdil wants to merge 1 commit intoNVIDIA:mainfrom
mesutoezdil:docs/fix-kubernetes-supervisor-delivery-arch
Open

docs(arch): update Kubernetes supervisor delivery to reflect init-container pattern#1215
mesutoezdil wants to merge 1 commit intoNVIDIA:mainfrom
mesutoezdil:docs/fix-kubernetes-supervisor-delivery-arch

Conversation

@mesutoezdil
Copy link
Copy Markdown
Contributor

Summary

PR #1154 replaced the Kubernetes hostPath volume approach for supervisor binary injection with an init container + emptyDir pattern. Two architecture docs were not updated to reflect this change:

  • architecture/build-containers.md: Kubernetes row in the supervisor delivery table described the old approach ("Sandbox pod image or Kubernetes driver pod template configuration").
  • architecture/podman-driver.md: Compared Podman's OCI image volume mechanism against "the Kubernetes hostPath approach", which no longer applies.

Related Issue

Follows #1154.

Changes

  • architecture/build-containers.md: Updated the Kubernetes supervisor delivery row to describe the init container + emptyDir pattern and reference supervisor_image / supervisor_image_pull_policy.
  • architecture/podman-driver.md: Replaced the reference to "Kubernetes hostPath approach" with "Kubernetes init-container approach".

Testing

Documentation change only. No code changes.

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)

…tainer pattern

PR NVIDIA#1154 replaced the hostPath volume approach with an init container
that copies the supervisor binary into a shared emptyDir. Two architecture
docs still referenced the old hostPath pattern.

Signed-off-by: mesutoezdil <mesudozdil@gmail.com>
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 6, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants