Skip to content

Conversation

@nader-ziada
Copy link
Collaborator

Implement automatic detection of cluster state changes (API groups, OpenShift status) with configurable polling and debounce windows. The cluster state watcher runs in the background, invalidates the discovery cache periodically, and triggers a reload callback when changes are detected after a debounce period.

  • Add clusterStateWatcher to monitor API groups and OpenShift status
  • Implement debounced reload to avoid excessive reloads during changes
  • Add WatchClusterState method to Manager with configurable intervals
  • Integrate cluster state watching in kubeconfig and single cluster providers

closed PR #493 to split into this PR and future follow up

@nader-ziada
Copy link
Collaborator Author

@manusa

@nader-ziada nader-ziada changed the title feat(kubernetes): add cluster state monitoring with debounced reload feat(kubernetes): add cluster state monitoring with reload Nov 25, 2025
@manusa manusa self-assigned this Nov 28, 2025
Implement automatic detection of cluster state changes (API groups, OpenShift status) with configurable polling and debounce windows. The cluster state watcher runs in the
background, invalidates the discovery cache periodically, and triggers a reload callback when changes are detected after a debounce period.

- Add clusterStateWatcher to monitor API groups and OpenShift status
- Implement debounced reload to avoid excessive reloads during changes
- Add WatchClusterState method to Manager with configurable intervals
- Integrate cluster state watching in kubeconfig and single cluster providers

Signed-off-by: Nader Ziada <nziada@redhat.com>
@manusa manusa force-pushed the dynamic-server-reload-1 branch from 8aa4a4e to 15fb2d8 Compare November 28, 2025 09:22
@manusa manusa added this to the 0.1.0 milestone Nov 28, 2025
@manusa manusa self-requested a review November 28, 2025 09:23
Copy link
Member

@manusa manusa left a comment

Choose a reason for hiding this comment

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

LGTM, thx!

Please don't continue with the next PR.
I'm currently working on a refactor to move the watch logic out of the manager and into each provider.

Should be ready in a couple of hours.

@manusa manusa merged commit b706778 into containers:main Nov 28, 2025
6 checks passed
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.

2 participants