opf / helm-charts

OPF helm chart repository
https://charts.openproject.org
GNU General Public License v3.0
12 stars 22 forks source link

Fix whitespace generation in s3 secret #49

Closed lukasredev closed 9 months ago

lukasredev commented 9 months ago

If you run the chart install with: helm upgrade --create-namespace --namespace openproject --install openproject openproject/openproject --dry-run -f values.yaml

and you have defined s3 in your values.yaml:

s3:
  enabled: true
  endpoint: "minio.local"
  auth:
    existingSecret: "minio-credentials"
    secretKeys:
      accessKeyId: "accessKeyId"
      secretAccessKey: "secretAccessKey"

it fails with the following error: Helm upgrade failed: YAML parse error on openproject/templates/secret_s3.yaml: error converting YAML to JSON: yaml: line 15: mapping values are not allowed in this context

this is due to a whitespace formatting error in secret_s3.yaml which this pull request fixes.

changeset-bot[bot] commented 9 months ago

🦋 Changeset detected

Latest commit: b0254dbcfc7a2e7d6850ca0b52094ed86af68261

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | ------------------------ | ----- | | @openproject/helm-charts | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR