k8s-at-home / charts

⚠️ Deprecated : Helm charts for applications you run at home
https://docs.k8s-at-home.com
Apache License 2.0
1.45k stars 623 forks source link

[homeassistant] fix pvc for postgresql … #1520

Closed Brice187 closed 2 years ago

Brice187 commented 2 years ago

Signed-off-by: Lars Kerick Brice187@users.noreply.github.com

Description of the change

The creation of the PVC for bitnami postgresql helm chart does not work in the current implementation. Fixed values.yaml template

Benefits PVC gets created correctly following the rules of parent repository

Possible drawbacks

No known drawbacks

Additional information

Checklist

Brice187 commented 2 years ago

It is really hard to contribute… I think I’ll give up :( #thisismysecond attempt for a really small fix :(

onedr0p commented 2 years ago

https://github.com/k8s-at-home/charts/runs/6065272541?check_suite_focus=true#step:3:76

Don't give up, just read why the workflow failed ^

 annotations:
   artifacthub.io/changes: |
     - kind: changed
-      description: "Upgraded values.yaml to fix issue for postgresql pvc creation"
\ No newline at end of file
+      description: "Upgraded values.yaml to fix issue for postgresql pvc creation"
Brice187 commented 2 years ago

Already spend more than 30 minutes. Don't want waste more time

onedr0p commented 2 years ago

No problem, thanks for trying!