bcgov / common-object-management-service

A microservice for managing access control to S3 Objects
https://bcgov.github.io/common-object-management-service/
Apache License 2.0
6 stars 9 forks source link

Helm Chart can't be packaged to .tgz #84

Closed pbolduc closed 1 year ago

pbolduc commented 1 year ago

Describe the bug

Trying to package the helm chart complains about missing dependencies. When deploying, we package up the charts as .tgz files and commit to our repository. Linting the package also produces the warning.

To Reproduce

Steps to reproduce the behavior:

  1. Clone the repository
  2. Open a command prompt in the repository directory
  3. Change into the charts folder
  4. Run helm lint coms
==> Linting coms
[INFO] Chart.yaml: icon is recommended
[WARNING] C:\Users\Phil\source\repos\github\bcgov\common-object-management-service\charts\coms: chart directory is missing these dependencies: patroni

1 chart(s) linted, 0 chart(s) failed
  1. Clone the repository
  2. Open a command prompt in the repository directory
  3. Change into the charts folder
  4. Run helm package coms
Error: found in Chart.yaml, but missing in charts/ directory: patroni

Expected behavior

Should be able to package the helm as a .tgz. It would also be useful to have a github page that allows adding the helm repo without cloning.

Screenshots

n/a

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context

pbolduc commented 1 year ago

Seems you need to run helm dependency build in the coms directory before this can work.