opensearch-project / helm-charts

:wheel_of_dharma: A community repository for Helm Charts of OpenSearch Project.
https://opensearch.org/docs/latest/opensearch/install/helm/
Apache License 2.0
176 stars 236 forks source link

[BUG][opensearch] `helm lint` fails if using `extraObjects` #606

Closed cyanidium closed 1 month ago

cyanidium commented 1 month ago

Describe the bug When running helm lint and creating any extra objects using the extraObjects value, helm lint fails with:

[ERROR] templates/extraManifests.yaml: unable to parse YAML: invalid Yaml document separator: apiVersion: apps/v1

To Reproduce Steps to reproduce the behavior:

  1. Create a values.yaml with extraObjects defined
  2. Run helm list -f values.yaml charts/opensearch from the repo top level
  3. See error

Expected behavior There should be no errors during linting.

Chart Name opensearch

Screenshots If applicable, add screenshots to help explain your problem.

Host/Environment (please complete the following information):

Additional context This is a known issue with helm and only affects linting, not the applied templates: https://github.com/helm/helm/issues/10149

prudhvigodithi commented 1 month ago

[Triage] Thanks @cyanidium we can get the PR's reviewed and merged for this issue. @peterzhuamazon