Skip to content

feat(kasm): add extraVolumes, extraEnv, and initContainers (upstream PR #32)#59

Open
darkhonor wants to merge 1 commit intomainfrom
upstream/pr-32-extra-volumes-env
Open

feat(kasm): add extraVolumes, extraEnv, and initContainers (upstream PR #32)#59
darkhonor wants to merge 1 commit intomainfrom
upstream/pr-32-extra-volumes-env

Conversation

@darkhonor
Copy link
Copy Markdown
Owner

Summary

  • Ports kasmtech/kasm-helm#32 by @TomGrozev
  • Adds per-component support for initContainers, extraEnv, extraVolumes, extraVolumeMounts
  • Components: api, guac, manager, proxy, rdp-gw, rdp-https-gw
  • Works alongside existing init containers and volume mounts

Why

This is the key PR for eliminating our Kustomize post-render patches. With extraVolumes/extraVolumeMounts, we can inject writable emptyDir mounts for /tmp and /opt/kasm/current/log (our issue kasmtech/kasm-helm#34 findings 1e/1f) directly through values.yaml. Also enables custom CA bundle injection without template changes.

Test plan

  • helm lint charts/kasm
  • helm template with default values — verify no extra volumes/env in output
  • helm template with extraVolumes configured — verify volumes appear in correct deployment
  • helm template with initContainers configured — verify init containers are appended
  • Verify existing functionality (existingSecret, containerSecurity) is preserved

🤖 Generated with Claude Code

Port of kasmtech/kasm-helm#32 (TomGrozev). Adds the ability to specify
custom init containers, extra volumes and volume mounts, and extra
environment variables for each component (api, proxy, guac, manager,
rdp-gw, rdp-https-gw).

This enables solving Restricted PSA requirements (writable /tmp,
/opt/kasm/current/log) and custom CA bundle injection via values
overrides instead of Kustomize post-render patches.

Upstream-PR: kasmtech/kasm-helm#32
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

1 participant