Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dependency chart requires adding repo definition first #85

Open
fty4 opened this issue May 3, 2022 · 1 comment
Open

dependency chart requires adding repo definition first #85

fty4 opened this issue May 3, 2022 · 1 comment
Labels
question Further information is requested

Comments

@fty4
Copy link
Contributor

fty4 commented May 3, 2022

Hello everyone,

I just tested this action and find it very useful.

When using the ct install inside a workflow I have a problem with the dependencies.
The chart that will be installed depends on another chart and Helm expects the repository already be present.

Here is the message from the run:

Error message
Installing charts...
Version increment checking disabled.

------------------------------------------------------------------------------------------------------------------------
 Charts to be processed:
------------------------------------------------------------------------------------------------------------------------
 mychart => (version: "0.2.3", path: "./")
------------------------------------------------------------------------------------------------------------------------

Error: no repository definition for [https://chart.url/.](https://chart.url/) Please add the missing repos via 'helm repo add'
Error: Error installing charts: Error building dependencies for chart 'mychart  => (version: "0.2.3", path: "./")': Error waiting for process: exit status 1
------------------------------------------------------------------------------------------------------------------------
No chart changes detected.
------------------------------------------------------------------------------------------------------------------------
Error installing charts: Error building dependencies for chart 'mychart  => (version: "0.2.3", path: "./")': Error waiting for process: exit status 1
Error: Process completed with exit code 1.

In my local dev environment this repo to the chart is already added - but inside the runner not.
Is there any best practice how the charts can be added automatically (by ct?)?

Currently my approach would be to read out all dependencies.*.repository values from the Chart.yaml and add them in an extra step via bash.

@cpanato
Copy link
Member

cpanato commented Sep 27, 2022

if the dependent chart is public and has a release for that you can just add it in the config.

like

chart-repos:
  - ingress-nginx=https://kubernetes.github.io/ingress-nginx
  - sigstore=https://sigstore.github.io/helm-charts

@cpanato cpanato added the question Further information is requested label Sep 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants