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

Kubernetes 1.22 and v1beta1 versus v1 #63

Open
gnunn1 opened this issue Oct 21, 2021 · 4 comments
Open

Kubernetes 1.22 and v1beta1 versus v1 #63

gnunn1 opened this issue Oct 21, 2021 · 4 comments

Comments

@gnunn1
Copy link

gnunn1 commented Oct 21, 2021

I'm trying to run this in OpenShift 4.9 and am running into some issues due to the usage of v1beta1 APIs instead of v1. The helm chart was easy to update to v1 however once the driver is running I see in the lvm-provisioner log that it is expected to find CSINode on v1beta1 instead of v1

E1021 19:09:47.474514 1 reflector.go:156] k8s.io/client-go/informers/factory.go:135: Failed to list *v1beta1.CSINode: the server could not find the requested resource

Any thoughts on how to work around this?

@majst01
Copy link
Contributor

majst01 commented Oct 22, 2021

We are not aware how openshift manages api versioning and which k8s version is implemented in openshift 4.9.

Current k8s client.go master branch (v1.23alpha) still implements v1beta1.CSINode listers
https://github.com/kubernetes/client-go/blob/master/listers/storage/v1beta1/csinode.go#L33

@gnunn1
Copy link
Author

gnunn1 commented Oct 22, 2021

You can view the OpenShift documentation which notes the removal of the vibeta1 APIs as being driven by kubernetes 1.22 here:

https://docs.openshift.com/container-platform/4.9/release_notes/ocp-4-9-release-notes.html#ocp-4-9-removed-kube-1-22-apis

It links to the kubernetes docs which notes the removal of these APIs here:

https://kubernetes.io/docs/reference/using-api/deprecation-guide/#storage-resources-v122

@mwennrich
Copy link
Contributor

If you like to try it out, there is an updated (but not yet released) version of the helm-chart here:
https://github.com/metal-stack/helm-charts/tree/master/charts/csi-driver-lvm

contains (beside other updates):
https://github.com/metal-stack/helm-charts/blob/master/charts/csi-driver-lvm/templates/common.yaml#L1-L6

@gnunn1
Copy link
Author

gnunn1 commented Oct 24, 2021

Thanks, that help chart does work fine though I did need to add the anyuid and privilege SCCs (OpenShift's version of PSP) to the controller and plugin service accounts. After that it works great.

One question, are you supposed to be able to just past the devicePattern, if I comment out vgName it complains that it is a required value and always seems to prefer it. It's not a big deal to create an LVM PV and VG ahead of time though.

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

No branches or pull requests

3 participants