minio / directpv

Kubernetes CSI driver for Direct Attached Storage :minidisc:
https://directpv.io
GNU Affero General Public License v3.0
598 stars 88 forks source link

Error; unable to parse minor version 0; strconv.ParseUint: parsing "28+": invalid syntax - OKD / Openshift #902

Closed parseltongued closed 6 months ago

parseltongued commented 6 months ago

Describe the bug execute the following cli: kubectl directpv install --openshift but with error message:

Error; unable to parse minor version 0; strconv.ParseUint: parsing "28+": invalid syntax

To Reproduce On bastion server of my okd cluster, tried both kubectl krew and kubectl-directpv binaries: kubectl directpv install --openshift

Expected behavior min.io direct-cv driver installed successfully

Screenshots and logs image

Deployment information (please complete the following information):

balamurugana commented 6 months ago

@parseltongued

  1. is it OKD or Openshift?
  2. The error Error; unable to parse minor version 0; strconv.ParseUint: parsing "28+": invalid syntax is legitimate. Minor version should not have +
parseltongued commented 6 months ago

@parseltongued

  1. is it OKD or Openshift?
  2. The error Error; unable to parse minor version 0; strconv.ParseUint: parsing "28+": invalid syntax is legitimate. Minor version should not have +

Hi @balamurugana

Thanks for your prompt reply.

  1. This is an okd cluster, should I follow Openshift installation procedure? (I thought it's interchangeable)
  2. Thanks for validating this issue persist. I'm new to OKD / Openshift & direct-pv, would appreciate any help to remediate this. Tried adding --kube-version "1.28.2" in the kubectl directpv command doesn't work.

I've installed this okd cluster via a self hosted quay mirror-registry.

./oc version Client Version: 4.15.0-0.okd-2024-03-10-010116 Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3 Server Version: 4.15.0-0.okd-2024-03-10-010116 Kubernetes Version: v1.28.2-3598+6e2789bbd58938-dirty

./kubectl version Client Version: v1.28.2 Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3 Server Version: v1.28.2-3598+6e2789bbd58938-dirty

Thank you!

balamurugana commented 6 months ago

What error are you getting with --kube-version argument?

parseltongued commented 6 months ago

What error are you getting with --kube-version argument?

its the same error: [adm@repo01 okd]$ kubectl directpv install --kube-version "1.28.2" --registry "repo01.okd4.test.dev:8443" --openshift

░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 0%

Error; unable to parse minor version 0; strconv.ParseUint: parsing "28+": invalid syntax

[adm@repo01 okd]$ kubectl directpv install --kube-version "1.28.2-359" --registry "repo01.okd4.test.dev:8443" --openshift

░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 0%

Error; unable to parse minor version 0; strconv.ParseUint: parsing "28+": invalid syntax

balamurugana commented 6 months ago

For non-standard k8s version, use --dry-run flag along with --kube-version flag to generate manifest and apply it manually.

parseltongued commented 6 months ago

applied yaml manually with thanks.

iamlinix commented 4 months ago

For non-standard k8s version, use --dry-run flag along with --kube-version flag to generate manifest and apply it manually.

Hi @balamurugana , kindly please elaborate on what do you mean by

use --dry-run flag along with --kube-version flag to generate manifest and apply it manually

coz directpv install does not have a --dry-run option, and I'm having this exactly issue here too:

Error; unable to parse minor version 0; strconv.ParseUint: parsing "21+": invalid syntax

and according to Kubernetes' official comments, the suffix + is quite possible:

gitMinor string // minor version, numeric possibly followed by "+"

=======================================================

Ahhh, NVM, found the combination of --kube-version, --legacy and -o yaml options which worked for me.