helmfile icon indicating copy to clipboard operation
helmfile copied to clipboard

[Feature Request] helmfile supports pulling charts from an OCI repository via helm’s experimental support

Open onpaj opened this issue 5 years ago • 8 comments

Helmfile can pull charts from Azure Container Registry using Helm 2.0 approach via az acr helm, which is deprecated. Suggested approach to work with Azure Container Registry is by using Helm 3.0 OCI artifacts support (helm chart push, helm chart pull).

onpaj avatar Aug 07 '20 10:08 onpaj

@onpaj Hey! Sounds good, but honestly I'm not at all familiar with the OCI repository support.

Does Helmfile need to helm chart pull and then helm chart export instead of just helm fetch for this? Are there anything else we need to care about?

mumoshu avatar Aug 08 '20 06:08 mumoshu

I'm interested in adding this feature. Followings are from the research I made.

  • helm OCI don't support semver & signed release yet. 1
  • helm OCI requires login to registry. We'll need to add username & password to the repositories. 2
  • helm chart export need to save the chart somewhere outside the helm cache dir 2. Maybe this need to be configurable?
  • Since helm OCI requires totally different commands, maybe it's better to have different syntax to refer to the chart. There's already some discussion on helm. 3

johnlinvc avatar Sep 21 '20 07:09 johnlinvc

  • 1 to add this to helmfile. In the meantime, here is what we use: https://gitlab.com/-/snippets/2018727 It is a pretty naive bash script that pulls charts from a registry so that they can be used as "local charts"

guillaq avatar Sep 24 '20 15:09 guillaq

I need this to enable OCI registries in jenkins-x so will probably take a look at creating a PR for this (assuming it doesn't become a rabbit hole ;) )

WIP PR here - #1629

chrismellard avatar Dec 23 '20 10:12 chrismellard

Actually we are so eagerly waiting for this feature, any update on including this feature! I can also try to help

varunpalekar avatar Jan 11 '21 21:01 varunpalekar

HI @varunpalekar . Dev has been done on the linked PR. Just awaiting a review.

chrismellard avatar Jan 11 '21 23:01 chrismellard

@chrismellard is this ticket already closed? I guess, as the PR #1629 is merged?!

aeimer avatar Sep 08 '21 12:09 aeimer

  • Related #2112

scottrigby avatar Mar 21 '22 22:03 scottrigby