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

Incorrect Documentation/Misleading Warning for postgresql.resources #31467

Open
jonchun opened this issue Jan 19, 2025 · 1 comment
Open

Incorrect Documentation/Misleading Warning for postgresql.resources #31467

jonchun opened this issue Jan 19, 2025 · 1 comment
Assignees
Labels
kubeapps tech-issues The user has a technical issue about an application triage Triage is needed

Comments

@jonchun
Copy link
Contributor

jonchun commented Jan 19, 2025

Name and Version

kubeapps-17.1.1

What architecture are you using?

amd64

What steps will reproduce the bug?

  1. Deploy kubeapps with default settings.
  2. Set the following values:
postgresql:
  resourcesPreset: none
  resources:
    requests:
      cpu: "1"
      memory: 512Mi
    limits:
      memory: 1024Mi

What do you see instead?

This does NOT work as documented. The solution is in #31265 which is to set it under the primary: key (or readReplicas according to https://github.com/bitnami/charts/blob/main/bitnami/postgresql/values.yaml)
e.g.

postgresql:
  primary:
    resourcesPreset: none
    resources:
      requests:
        cpu: "1"
        memory: 512Mi
      limits:
        memory: 3072Mi

While that resolves the issue, there is still a misleading warning that shows up warning about a key that has no effect.

WARNING: There are "resources" sections in the chart not set. Using "resourcesPreset" is not recommended for production. For production installations, please set the following values according to your workload needs:
  - postgresql.resources
@jonchun jonchun added the tech-issues The user has a technical issue about an application label Jan 19, 2025
@github-actions github-actions bot added the triage Triage is needed label Jan 19, 2025
@carrodher
Copy link
Member

Thank you for bringing this issue to our attention. We appreciate your involvement! If you're interested in contributing a solution, we welcome you to create a pull request. The Bitnami team is excited to review your submission and offer feedback. You can find the contributing guidelines here.

Your contribution will greatly benefit the community. Feel free to reach out if you have any questions or need assistance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kubeapps tech-issues The user has a technical issue about an application triage Triage is needed
Projects
None yet
Development

No branches or pull requests

2 participants