Skip to content

feat(clickstack): surface ClickHouse table TTL as configurable values#250

Open
ZeynelKoca wants to merge 1 commit into
ClickHouse:mainfrom
ZeynelKoca:feat/expose-table-ttl
Open

feat(clickstack): surface ClickHouse table TTL as configurable values#250
ZeynelKoca wants to merge 1 commit into
ClickHouse:mainfrom
ZeynelKoca:feat/expose-table-ttl

Conversation

@ZeynelKoca

Copy link
Copy Markdown

What

Surfaces ClickHouse table retention (TTL) as documented, configurable values in the clickstack chart:

  • Defaults HYPERDX_OTEL_EXPORTER_TABLES_TTL (720h) in hyperdx.config so the global retention knob is discoverable and tunable in values.yaml.
  • Documents the per-signal overrides — HYPERDX_OTEL_EXPORTER_LOGS_TTL / _TRACES_TTL / _METRICS_TTL / _SESSIONS_TTL — and HYPERDX_OTEL_EXPORTER_RECONCILE_TABLE_TTL, inline where the rest of the collector env lives.

Why

Data retention is usually a compliance requirement (SOC 2, ISO 27001, HIPAA, PCI-DSS): security-relevant logs — and often traces — must be kept for 6–12 months, while high-volume metrics stay short to control storage cost. That's a per-signal retention policy. Today an operator has to know the exact collector env-var names and hand-inject them; this makes the retention controls first-class and self-documenting. It also complements the per-signal / reconcile capability added collector-side in hyperdxio/hyperdx#2709.

Changes

  • charts/clickstack/values.yaml — add HYPERDX_OTEL_EXPORTER_TABLES_TTL: "720h" to hyperdx.config, with a documented block for the per-signal overrides + reconcile flag.
  • charts/clickstack/tests/app-configmap_test.yaml — assert the default TTL renders and that per-signal / reconcile overrides pass through.
  • Changeset (helm-charts, minor).

Compatibility / sequencing

Testing

  • helm lint charts/clickstack — clean.
  • helm template — renders HYPERDX_OTEL_EXPORTER_TABLES_TTL: "720h" in clickstack-config.
  • helm unittest charts/clickstack181/181 pass (incl. 2 new config assertions).

@changeset-bot

changeset-bot Bot commented Jul 23, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 0a08d3e

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

@ZeynelKoca
ZeynelKoca marked this pull request as ready for review July 23, 2026 08:05
@ZeynelKoca
ZeynelKoca requested a review from a team as a code owner July 23, 2026 08:05
@github-actions github-actions Bot added the external Opened by an external contributor label Jul 23, 2026

@wrn14897 wrn14897 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM! Thanks for the contribution

@ZeynelKoca

Copy link
Copy Markdown
Author

@wrn14897 Thanks! Just to be clear, this PR exposes new configurations created in hyperdxio/hyperdx#2709 so it should be blocked for merge until the hdx PR is merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external Opened by an external contributor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants