Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix issue of leader selection pressure #102

Merged
merged 1 commit into from
Sep 13, 2024
Merged

Conversation

lmxia
Copy link
Contributor

@lmxia lmxia commented Sep 4, 2024

Fixes #99

We will have 2 daemonsets when cluster has more than 10 nodes, it's should be configurable but now hard coded.

@lmxia lmxia requested a review from Airren September 4, 2024 03:31
@lmxia
Copy link
Contributor Author

lmxia commented Sep 4, 2024

One daemonset, check if node is controll-plane in code.

@lmxia lmxia force-pushed the feat/leader-selection-cease branch from 1ae33d7 to 3c2633f Compare September 7, 2024 06:51
@lmxia
Copy link
Contributor Author

lmxia commented Sep 10, 2024

Tested ok, tunnel connection will be auto recovered inner cluster.

@lmxia lmxia merged commit 00bed00 into main Sep 13, 2024
2 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.

optimize inner-cluster-tunnel-controller lease election
2 participants