hpe-storage / csi-driver

A Container Storage Interface (CSI) driver from HPE
https://scod.hpedev.io
Apache License 2.0
62 stars 57 forks source link

Failed to pull a container image on quay.io #237

Closed yakiyoshi closed 3 years ago

yakiyoshi commented 3 years ago

OpenShift 4.6 HPE CSI Driver Operator v1.4.0 (via OperatorHub)

Pulling hpestorage container image which is specified HPE CSI Driver Operatorv1.4.0 was failed.

$ oc get csv
NAME                                           DISPLAY                           VERSION                 REPLACES   PHASE
hpe-csi-operator.v1.4.0                        HPE CSI Operator for Kubernetes   1.4.0                              Succeeded

And also hpe csi driver operator didn't have correct authority and couldn't deploy some clusterroles. hpe-csi-operator-sa need patch operation for storage.hpe.com API. I adding it in the clusterrole attached the SA manually.

raunakkumar commented 3 years ago

Hi @yakiyoshi Looks like there is a typo in the HPECSIDriver.yaml for the key registry. It looks to be quey.io instead of quay.io. The official document on operator hub refers to quay.io. Did you modify the value?

datamattsson commented 3 years ago

@raunakkumar this reveals the bug that we've forgotten to prefix the CSI sidecars with the registry variable.

raunakkumar commented 3 years ago

We have prefixed the side cars with registry prefix for all the hpe images. https://github.com/hpe-storage/co-deployments/blob/14cae90fe95ba6a297275e0d0f206af1da4f957b/helm/charts/hpe-csi-driver/templates/hpe-csi-controller.yaml#L99-L100

yakiyoshi commented 3 years ago

Oops. Sorry for the typo. Thank you for pointing that out.

I'v already released ocp cluster and hardwares are sharing with another. I can't use any more. If no other disscusions, I would like to close this issue.