Skip to content

csi: Use default csi node hostpath - #1073

Open
hajiler wants to merge 1 commit into
agent-substrate:mainfrom
hajiler:volume-mount-path
Open

csi: Use default csi node hostpath#1073
hajiler wants to merge 1 commit into
agent-substrate:mainfrom
hajiler:volume-mount-path

Conversation

@hajiler

@hajiler hajiler commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Mount kubelet-pods host path to atelet for CSI node operations. This hostpath is the default for the average CSI driver, and will enable compatibility with default CSI node daemonsets without additional customization.

Volumes are mounted to the node at the following path: /var/lib/kubelet/pods/[worker-pod-uuid]/substrate-volumes/[actor-uuid]/[vol-name]". By mounting the directory in substrate-volumes, this avoids interferences from the volume reconstruction feature, which uses var/lib/kubelet/pods/*/volumes/ for discovery.

TAG=agy
CONV=945df49f-7efa-4dc1-a9e4-03ecbbed676f

#232

It's a good idea to open an issue first for discussion.

  • [X ] Tests pass
  • Appropriate changes to documentation are included in the PR

@hajiler
hajiler force-pushed the volume-mount-path branch from 1e43568 to 8a4d40d Compare August 19, 2026 21:14
@mattcary

Copy link
Copy Markdown

lgtm from a CSI driver perspective

…/lib/kubelet/pods

TAG=agy
CONV=945df49f-7efa-4dc1-a9e4-03ecbbed676f
@hajiler
hajiler force-pushed the volume-mount-path branch from 8a4d40d to dde9e00 Compare August 21, 2026 16:15
@hajiler
hajiler marked this pull request as ready for review August 24, 2026 16:51
@msau42

Copy link
Copy Markdown
Collaborator

We discussed offline, but I don't think we want to depend on "/var/lib/kubelet/pods/[worker-pod-uuid]/substrate-volumes/[actor-uuid]/[vol-name]" because we won't be resilient to worker pod crashes/restarts/upgrades, etc.

I suggest exploring something under "/var/lib/kubelet/plugins"

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.

3 participants