-
Notifications
You must be signed in to change notification settings - Fork 24
Conversation
updated node plugin template which is required for kubernetes v1.12.1 Signed-off-by: Madhu Rajanna <mrajanna@redhat.com>
Fixes: #26 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe this Fixes #26 as opposed to 30
@JohnStrunk yeah it fixes #26 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this all that is required to get GCS to work on 1.12? I needed a few more changes to the RBAC rules as well to get even the provisioner working.
@@ -164,16 +164,21 @@ spec: | |||
args: | |||
- "--v=5" | |||
- "--csi-address=$(ADDRESS)" | |||
- "--kubelet-registration-path=$(DRIVER_REG_SOCK_PATH)" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The registration socket is csi.sock
as I see it. If that is the case, ADDRESS already provides the path, so we could use that. We do not need an additional volume for the registration socket, and have a separate env variable it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is an issue opened in driver-registrar to combine both flags, once it's done we can reconsider this and use only one volume
current implementation is done as per CSI doc registrar-sidecar-configuratio
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the doc link. Makes it a little more clear.
env: | ||
- name: ADDRESS | ||
value: /plugin/csi.sock | ||
- name: DRIVER_REG_SOCK_PATH | ||
value: /var/lib/kubelet/plugins/org.gluster.glusterfs/csi.sock |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we really do need a separate variable for the registration socket, then shouldn't the value here be /registration/org.gluster.glusterfs/csi.sock
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the driver-registrar expose a Unix domain socket to handle Kubelet Plugin Registration.
this socket MUST be surfaced on the host in the kubelet plugin registration directory (in addition to the CSI driver socket)
Tested this out. Can confirm it works. Merging. |
updated node plugin template which is required for kubernetes v1.12.1.
Signed-off-by: Madhu Rajanna mrajanna@redhat.com
output
kubectl version
pod status
mount info