helm / chart-releaser

Hosting Helm Charts via GitHub Pages and Releases
Apache License 2.0
681 stars 113 forks source link

docker image and chart dependencies #114

Open gimler opened 3 years ago

gimler commented 3 years ago

Remote dependencies does not work with docker image

Chart.yaml

dependencies:
- name: redis-ha
  version: "4.12.9"
  repository: "https://dandydeveloper.github.io/charts"

command running from docker image

cr package charts/sap-event-ticketing/

error

Error: no repository definition for https://dandydeveloper.github.io/charts

workaround add helm and repo than it works

apk add --no-cache -X http://dl-cdn.alpinelinux.org/alpine/edge/testing helm
helm repo add dandydev https://dandydeveloper.github.io/charts

Solution 1: Install helm into the docker image

Solution 2: let chart-releaser fetch the repo data to

~/.cache/helm/repository/dandydev-charts.txt
~/.cache/helm/repository/dandydev-index.yaml

Solution 3: Install helm into the docker image and use helm package instate of cr package with --repository-config option. Use ~/.config/helm/repositories.yaml as example

I would prefer to install helm into the docker image.

gcaracuel commented 3 years ago

Another solution is what chart-testing does including a config parameter to setup those repositories: 'chart-repos'.

It does work nice and allows users to setup internal only repos as explained in their README