va.gov-team icon indicating copy to clipboard operation
va.gov-team copied to clipboard

Fix 'cannot overwrite table' warnings when building vets-API manifest

Open RachalCassity opened this issue 1 year ago • 1 comments

Summary

There are warnings when templating the helm charts locally, in the vsp-infra-application-manifests](https://github.com/department-of-veterans-affairs/vsp-infra-application-manifests) repository. This could be fixed by replacing every hash with an array in the parent helm chart, values.yaml. This

Note: These warnings are not seen in ArgoCD. This is strictly a helm issue.

coalesce.go:286: warning: cannot overwrite table with non table for vets-api.vets-api.clamav.volumeMounts (map[])

Screenshot 2024-02-28 at 4.16.45 PM.png

How to reproduce

# Go to vsp-infra-application-manifests
$ cd ../vsp-infra-application-manifests

# Go to the dev helm charts
$ cd apps/vets-api/dev

# Remove the charts directory and Chart.lock, if present.
$ rm -rf charts
$ rm Chart.lock

# Build the helm charts
$ helm dependency build
$ **Clear out terminal
$ helm template .

The warnings will be at the top of the manifests.

Tasks

  • [x] Fix the warnings in dev
  • [ ] Fix the warnings in staging
  • [ ] Fix the warnings in sandbox
  • [ ] Fix the warnings in prod

RachalCassity avatar Feb 28 '24 22:02 RachalCassity