Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Postgresql Resource Settings are being ignored and stay default with HELM Chart Version 15.0.2 #7702

Open
ILZ1105 opened this issue Apr 16, 2024 · 3 comments
Labels
kind/bug An issue that reports a defect in an existing feature

Comments

@ILZ1105
Copy link

ILZ1105 commented Apr 16, 2024

Describe the bug
Postgresql Resource Settings are being ignored and stay default with HELM Chart Version 15.0.2

To Reproduce
Steps to reproduce the behavior:

  1. Use either e resourcepreset or dial in your required resources in the part for postgres
  2. Deploy the chart (done via kubeapps in my case)
  3. Check the resources of the pod or the respective statefulset
  4. The values are still the defaults:
    resources:
    limits:
    cpu: 150m
    ephemeral-storage: 1Gi
    memory: 192Mi
    requests:
    cpu: 100m
    ephemeral-storage: 50Mi
    memory: 128Mi

Expected behavior
It uses the respective resources as set manually or used via the resourcepresets.

Screenshots
None

Desktop (please complete the following information):

  • Kubeapps Version 2.10.0
  • Kubernetes version 1.26.5
  • Package version Helm Chart version 15.0.2

Additional context
None

@ILZ1105 ILZ1105 added the kind/bug An issue that reports a defect in an existing feature label Apr 16, 2024
@github-project-automation github-project-automation bot moved this to 🗂 Backlog in Kubeapps Apr 16, 2024
@antgamdia
Copy link
Contributor

Mmm, interesting. I locally ran into some issues with postgresql recently, so perhaps this is the root cause. I'd rather report it in the bitnami/charts repository, as it is them who actually maintain the official chart.

@ILZ1105
Copy link
Author

ILZ1105 commented Apr 16, 2024

Good point, I listed the respective bug there: bitnami/charts#25183

@eroldan
Copy link

eroldan commented Aug 30, 2024

I stepped into this problem today and did some investigation. The problem is that the subchart bitnami/postgresql does not honor the postgresql.resources value. Probably because expanding the functionality, now the k8s resources need to be setup in postgres.primary.resources. For example a values.yaml:

postgresql:
  primary:
    resources:
      requests:
        cpu: 2000m
        memory: 512Mi
      limits:
        cpu: 3000m
        memory: 1024Mi

So, I consider the bug to be in the documentation ( https://github.com/vmware-tanzu/kubeapps/blob/main/chart/kubeapps/README.md?plain=1#L589 )

The upstream chart bitnami/postgresql may have a bug indeed: not keeping previous compatibility if postgresql.resource was ever accepted as a value.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug An issue that reports a defect in an existing feature
Projects
Status: 🗂 Backlog
Development

No branches or pull requests

3 participants