images for node disk manager are wrong

Question

images for node disk manager are wrong

Closed this issue 4 years ago · 8 comments

Hello,

helm chart tries to reference node-disk-manager:v0.9.0 but one gets report that it is unauthorized
if one changes image to node-disk-manager-amd64 it gets pulled

Answer 1 · 2020-10-31T09:03:28.000Z

@ErroKokkarda Did you use the latest charts.? Because the default image tag points to 0.9.0

Also , it is recommended to use the openebs/node-disk-manager:0.9.0, (without the v prefix). Because this is a multiarch image and going forward we will be building and pushing multiarch images only.

Answer 2 · 2020-10-31T14:51:00.000Z

I used this on rancher and helm charts once checking them out it seems try to pull from quay.io/openebs/node-disk-manager
for 2.x version of openebs
once deploying 1.x openebs worked

your suggestion does work openebs/node-disk-manager:0.9.0 is an image I can pull from local
but not quay.io/openebs/node-disk-manager:0.9.0

thanks. I'm happy with that answer and I might check from where does rancher pull these helm charts and image be parametrized

we can close this if you want but if your charts reference quay.io then others can't pull it

Answer 3 · 2020-10-31T15:06:48.000Z

We are not pushing multiarch images to quay as quay did not support multiarch images at the time.

Also because of the frequent outages we moved our default registry from quay to docker hub.

I might check from where does rancher pull these helm charts and image be parametrized

thanks @ErroKokkarda . Can you update that in the issue once you get the information, so that we can fix it in the charts.

Answer 4 · 2020-10-31T15:13:18.000Z

I think they come from here
https://github.com/rancher/charts/blob/master/charts/openebs/v2.1.0/questions.yml
they maintain their library charts from here I guess but their interface might but defaults that reference quay

Answer 5 · 2020-10-31T17:17:15.000Z

@prateekpandey14 , PTAL on this. I think we should update the default in the questions section to docker hub instead of quay.

The charts are not using quay, but from the options the images are taken from quay.

Answer 6 · 2020-11-02T06:49:16.000Z

@ErroKokkarda @akhilerm sure i will take a look and update the images repository.

Answer 7 · 2020-11-02T07:36:57.000Z

@ErroKokkarda PR rancher/charts#854 has been raised to fix the image. thanks for creating an issue.

Answer 8 · 2020-12-04T06:37:50.000Z

Closing this issue , the PR rancher/charts#854 is merged and fix will available in openebs 2.2.0 release.