Skip to content

feat(helm): support custom vm.args via RELEASE_VM_ARGS#3735

Merged
djwhitt merged 1 commit into
feat/helmfrom
davidwhittington/helm-vm-args
Jul 24, 2026
Merged

feat(helm): support custom vm.args via RELEASE_VM_ARGS#3735
djwhitt merged 1 commit into
feat/helmfrom
davidwhittington/helm-vm-args

Conversation

@djwhitt

@djwhitt djwhitt commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Stacked on feat/helm (#3719). Adds custom Erlang VM args support to the chart — gap found comparing against Chase's logflare-k8s/init setup in o11y-team-tools.

Why

Chase's setup sets RELEASE_VM_ARGS with +Q 1048576, and flags that on Kubernetes it's effectively required: containerd commonly runs pods with nofile=infinity, and without +Q the BEAM sizes its preallocated port table from that limit → ~2GB RSS at boot.

What

  • New opt-in vmArgs value (default includes +Q 1048576). When set, it's rendered into a {{fullname}}-vm-args ConfigMap, mounted at /etc/logflare/vm.args (subPath, so it coexists with the cert-files mount under /etc/logflare), and wired via RELEASE_VM_ARGS. Set vmArgs: "" to use the image's baked-in args.
  • A checksum/vm-args pod annotation rolls the Deployment when the args change.
  • helm-unittest coverage (renders/omits ConfigMap, sets/omits RELEASE_VM_ARGS, mount).

Adds an opt-in `vmArgs` value rendered into a ConfigMap, mounted at
/etc/logflare/vm.args, and wired via RELEASE_VM_ARGS. Defaults to a set
including `+Q 1048576`, which caps the BEAM's preallocated port table — needed
on Kubernetes, where containerd runs pods with nofile=infinity and the VM would
otherwise size the table from that (~2GB RSS at boot).

Mounted with subPath so it coexists with the cert-files mount under
/etc/logflare. A checksum/vm-args pod annotation rolls the Deployment when the
args change. Set vmArgs to "" to fall back to the image's baked-in vm.args.
Adds helm-unittest coverage.
@djwhitt
djwhitt merged commit c7ae510 into feat/helm Jul 24, 2026
2 checks passed
@djwhitt
djwhitt deleted the davidwhittington/helm-vm-args branch July 24, 2026 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants