Skip to content

Conversation

@nikw9944
Copy link
Contributor

@nikw9944 nikw9944 commented Feb 3, 2026

Summary of Changes

  • Add ClickHouse integration to the controller to record successful GetConfig gRPC calls for device health tracking as per rfc12
  • New ClickhouseWriter batches events every 10 seconds and writes to a controller_grpc_getconfig_success table partitioned by month
  • ClickHouse is opt-in, configured via environment variables: CLICKHOUSE_ADDR, CLICKHOUSE_DB, CLICKHOUSE_USER, CLICKHOUSE_PASS, CLICKHOUSE_TLS_DISABLED
  • CLICKHOUSE_TLS_DISABLED is only intended for use in e2e tests and testing with a local clickhouse instance
  • Logs on startup whether ClickHouse is enabled or disabled, including connection details (addr, db, user)
  • Table is auto-created with CREATE TABLE IF NOT EXISTS so it's safe across restarts

Testing Verification

  • Tested by hand in devnet

@nikw9944 nikw9944 marked this pull request as ready for review February 3, 2026 20:18
@nikw9944 nikw9944 linked an issue Feb 3, 2026 that may be closed by this pull request
@nikw9944 nikw9944 self-assigned this Feb 3, 2026
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.

controller: publish getconfig metrics to clickhouse

2 participants