percona / percona-helm-charts

Collection of Helm charts for Percona Kubernetes Operators.
https://www.percona.com/software/percona-kubernetes-operators
Other
110 stars 150 forks source link

K8SPG-527 - Fix pg-db cluster.yaml resources placeholder syntax error #304

Closed TAbhay closed 3 months ago

TAbhay commented 3 months ago

While installing the pg-db cluster using helm, instance was not able to allocation resources.

Error: client.go:134: [debug] creating 2 resource(s) error: installation failed: 1 error occurred: * perconapgcluster.pgv2.percona.com "pg-db" is invalid: [spec.instances[0].resources.limits.cpu: invalid value: "$instance.resourc": spec.instances[0].resources.limits.cpu in body should match '^(\+|-)?(([0-9]+(\.[0-9])?)|(\.[0-9]+))(([kmgtpe]i)|[numkmgtpe]|([ee](\+|.[0-9])?)|(\.[0-9]+))))?$', spec.instances[0].resources.limits.memory: invalid value: "$instance.resources.limits.memory": spec.instances.limits.memory in body should match '^(\+|-)?(([0-9]+(\.[0-9])?)|(\.[0-9]+))(([kmgtpe]i)|[numkmgtpe]|([ee](\+|-)?(([0-9]+(\.[0-9])?)|(\., spec.backups.pgbackrest.repos: required value]

Reason: Placeholder syntax error in cluster.yaml of pg-db/charts/templates/cluster.yaml

Here, I have attached the screenshots for issue & fix. Issue: issue Fix: fix

it-percona-cla commented 3 months ago

CLA assistant check
All committers have signed the CLA.

TAbhay commented 3 months ago

@tplavcic Please review it. Thank you 🙂

tplavcic commented 3 months ago

@TAbhay Thank you very much for the contribution! It is now merged.