Skip to content

feat: replace reconciles by WarningsPollInterval with custom event source - #367

Open
GrigoryPervakov wants to merge 1 commit into
mainfrom
cluster-observer
Open

GrigoryPervakov wants to merge 1 commit into
mainfrom
cluster-observer

Conversation

@GrigoryPervakov

@GrigoryPervakov GrigoryPervakov commented Oct 13, 2026 •

Copy link
Copy Markdown
Member

Why

The warnings-reporting feature triggers a full reconciliation every few seconds. Config reload is detected only by polling during reconciliation; this will trigger reconciliation earlier.
Keeper status can be stale if the most recent reconciliation was long ago.

What

Custom event source for the keeper/clickhouse controllers that queues reconciliations if the observable state of the cluster has changed.
EventSource caches the last result and emits an event only when the result changes.

  • ClickHouse: config revisions + replica reachability
  • Keeper: replicas roles

@GrigoryPervakov
GrigoryPervakov marked this pull request as ready for review October 20, 2026 15:05

This branch has not been deployed

No deployments
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