va.gov-team
va.gov-team copied to clipboard
Fix 'cannot overwrite table' warnings when building vets-API manifest
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[])

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