Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

make /var/lib/kubelet configurable #668

Closed
pohly opened this issue Jun 4, 2020 · 1 comment · Fixed by #698
Closed

make /var/lib/kubelet configurable #668

pohly opened this issue Jun 4, 2020 · 1 comment · Fixed by #698
Assignees
Labels
0.8 needs to be fixed in 0.8.x

Comments

@pohly
Copy link
Contributor

pohly commented Jun 4, 2020

Not all clusters use /var/lib/kubelet as state dir for kublet (kubernetes-csi/csi-driver-host-path#71). To work on such clusters, we (ideally) need to determine the state dir automatically (not sure how to do that) or make it configurable.

This can be done already via kustomize, but also needs to work for the operator.

@pohly pohly added the future needs to be fixed in some future release label Jun 4, 2020
@pohly pohly added 0.8 needs to be fixed in 0.8.x and removed future needs to be fixed in some future release labels Jun 18, 2020
@pohly
Copy link
Contributor Author

pohly commented Jun 18, 2020

We want to stabilize the operator API and therefore have to get this included there.

avalluri added a commit to avalluri/pmem-CSI that referenced this issue Jul 24, 2020
Not all clusters use /var/lib/kubelet as state directory for kubelet
(kubernetes-csi/csi-driver-host-path#71). There is no way for the operator
to auto-detect this path. Hence, added a new field to deployment spec that
could be used to provide the kubelet path to driver deployment.

FIXES: intel#668
avalluri added a commit to avalluri/pmem-CSI that referenced this issue Aug 18, 2020
Not all clusters use /var/lib/kubelet as state directory for kubelet
(kubernetes-csi/csi-driver-host-path#71). There is no way for the operator
to auto-detect this path. Hence, added a new field to deployment spec that
could be used to provide the kubelet path to driver deployment.

FIXES: intel#668
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.8 needs to be fixed in 0.8.x
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants