Skip to content
This repository was archived by the owner on Mar 26, 2026. It is now read-only.
This repository was archived by the owner on Mar 26, 2026. It is now read-only.

Helm chart: container resources hardcoded at 128Mi, not configurable #13

@lukacsi

Description

@lukacsi

Problem

The Helm chart hardcodes container resource limits at 128Mi memory in deployment.yaml. There's no resources key in values.yaml to override them.

On a small homelab cluster (3 nodes, ~15 apps, ~60 secrets), the pod consistently OOMs at 128Mi. Controllers with unfiltered Secret sources load all cluster secrets into memory for join evaluation — including TLS certs, CNPG credentials, and service account tokens. An enterprise cluster with hundreds of secrets would be far worse.

128Mi is not a viable default for any real deployment.

Fix

Already implemented in lukacsi/dcontroller@e9f6e01:

# deployment.yaml — was hardcoded, now templatable
{{- with .Values.resources }}
resources:
  {{- toYaml . | nindent 10 }}
{{- end }}
# values.yaml — new default
resources:
  limits:
    cpu: 500m
    memory: 512Mi
  requests:
    cpu: 10m
    memory: 128Mi

Happy to open a PR.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions