Skip to content

Support customizable telemetry data retention policy#10366

Draft
bnaecker wants to merge 1 commit intomainfrom
ben/add-clickhouse-retention-policy
Draft

Support customizable telemetry data retention policy#10366
bnaecker wants to merge 1 commit intomainfrom
ben/add-clickhouse-retention-policy

Conversation

@bnaecker
Copy link
Copy Markdown
Collaborator

@bnaecker bnaecker commented May 4, 2026

  • Add clickhouse-admin types for represeting the telemetry retention policy and the disk usage of the database tables or oximeter timeseries specifically.
  • Add clickhouse-admin-server APIs for manipulating the policy and listing the database and oximeter-timeseries usage.

@bnaecker bnaecker marked this pull request as draft May 4, 2026 06:00
@bnaecker
Copy link
Copy Markdown
Collaborator Author

bnaecker commented May 4, 2026

This still needs a good bit of work, but I'd like to get early eyes on it. This is supposed to resolve #10357 by letting us set a customizable retention policy (in days) on the tables in the oximeter database. It does that through new endpoints on the clickhouse-admin server(s), and follow-up commits will add omdb subcommands to exercising those endpoints.

Like I said, this needs work, but I really want some kind of guardrails and procedure around manipulating the retention policy at our customer sites, rather than manually writing SQL against the database. Hopefully this can be made good enough by the time we want to release R19.3.

@bnaecker bnaecker requested review from jgallagher and jmcarp May 4, 2026 06:14
@bnaecker bnaecker force-pushed the ben/add-clickhouse-retention-policy branch from e9d9ca7 to 78ba9ed Compare May 4, 2026 06:15
- Add clickhouse-admin types for represeting the telemetry retention
  policy and the disk usage of the database tables or oximeter
  timeseries specifically.
- Add clickhouse-admin-server APIs for manipulating the policy and
  listing the database and oximeter-timeseries usage.
- Add `omdb` subcommands for exercising the new APIs
@bnaecker bnaecker force-pushed the ben/add-clickhouse-retention-policy branch from 78ba9ed to e09811f Compare May 4, 2026 06:23
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