images for node disk manager are wrong about charts HOT 8 CLOSED

@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.

from charts.

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

from charts.

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.

from charts.

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

from charts.

@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.

from charts.

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

from charts.

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

from charts.

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

from charts.