Skip to content
This repository has been archived by the owner on Mar 16, 2021. It is now read-only.

Commit

Permalink
deploy: add README, fix typos
Browse files Browse the repository at this point in the history
The csi-hostpath-snapshotter.yaml was misspelled when creating the
file. The upstream driver-registrar RBAC file uses
"csi-driver-registrar" as service account name and therefore the
hostpath plugin should also use that instead of the (now obsolete)
"csi-node-sa" name from the Kubernetes E2E testing.

This dependency on the upstream RBAC rules should be mentioned. For
the sake of avoiding yet another copy of URLs the new README just
refers to other documentation.
  • Loading branch information
pohly committed Dec 4, 2018
1 parent 8a7f2d3 commit dd25fd6
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 1 deletion.
6 changes: 6 additions & 0 deletions deploy/hostpath/README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
The .yaml files in this directory depend on the RBAC files that were
released together the CSI sidecar containers. Check the repos and/or
release notes for the CSI sidecar containers for details.

https://kubernetes-csi.github.io/docs/ has an example that puts all
pieces together.
2 changes: 1 addition & 1 deletion deploy/hostpath/csi-hostpathplugin.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ spec:
labels:
app: csi-hostpathplugin
spec:
serviceAccountName: csi-node-sa
serviceAccountName: csi-driver-registrar
hostNetwork: true
containers:
- name: driver-registrar
Expand Down

0 comments on commit dd25fd6

Please sign in to comment.