Skip to content

feat: expose hyperdx initContainers, volumes, volumeMounts passthrough#218

Merged
dhable merged 3 commits into
mainfrom
girish/expose-hyperdx-deployment-hooks
May 20, 2026
Merged

feat: expose hyperdx initContainers, volumes, volumeMounts passthrough#218
dhable merged 3 commits into
mainfrom
girish/expose-hyperdx-deployment-hooks

Conversation

@nohupped
Copy link
Copy Markdown
Contributor

Summary

Adds three optional passthrough fields under hyperdx.deployment so users can inject standard Kubernetes pod-spec extension into the HyperDX Deployment

  • hyperdx.deployment.initContainers: appended after wait-for-mongodb when mongodb.enabled is true; rendered standalone when MongoDB is disabled
  • hyperdx.deployment.volumes: rendered at spec.template.spec.volumes
  • hyperdx.deployment.volumeMounts: rendered on the app container

Unit tests included

Motivation

Several common deployment patterns currently require forking this chart, for eg:

  • Fetching a TLS CA bundle at startup (e.g., AWS RDS/DocumentDB CA via a small init container)
  • Mounting SAML IdP signing certs from a ConfigMap at /etc/hyperdx/idp-cert.pem
  • Attaching external secrets, license files, or config snippets from ConfigMaps/Secrets

Exposing these standard hooks keeps these workflows in user values files instead of chart forks.

Backward compatibility

Backward compatible. Defaults are empty lists. The existing should include initContainers when mongodb.enabled is true test continues to pass unchanged, confirming no behavior change for current users.

Changeset included.

nohupped added 2 commits May 20, 2026 23:42
… passthrough

Adds three optional fields under hyperdx.deployment:
  - initContainers: appended after wait-for-mongodb when mongodb.enabled,
    or used alone when mongodb is disabled
  - volumes: rendered at pod spec level
  - volumeMounts: rendered on the app container

Defaults are empty lists, so existing values files render byte-identical YAML with no behavior change. Enables common Kubernetes patterns such as fetching certificates at startup, mounting external secrets, or attaching ConfigMap files without forking the chart.

Includes unit tests covering defaults, user-provided values, ordering when both mongodb and user init containers are set, and full passthrough of fields like subPath and readOnly.
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 20, 2026

🦋 Changeset detected

Latest commit: 231b640

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
helm-charts Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Copy Markdown
Collaborator

@wrn14897 wrn14897 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dhable dhable enabled auto-merge (squash) May 20, 2026 16:47
@dhable dhable merged commit 63458a5 into main May 20, 2026
5 checks passed
@dhable dhable deleted the girish/expose-hyperdx-deployment-hooks branch May 20, 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.

3 participants