-
Notifications
You must be signed in to change notification settings - Fork 193
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
Cluster and endpoint information should be cleaned up #636
Comments
Makes sense, I'd propose a mechanism similar to : https://github.com/submariner-io/submariner/blob/master/pkg/cableengine/syncer/syncer.go#L125 With a bigger TTL time |
This issue has been automatically marked as stale because it has not had activity for 60 days. It will be closed if no further activity occurs. Please make a comment if this issue/pr is still valid. Thank you for your contributions. |
bump |
This issue has been automatically marked as stale because it has not had activity for 60 days. It will be closed if no further activity occurs. Please make a comment if this issue/pr is still valid. Thank you for your contributions. |
bump |
This issue has been automatically marked as stale because it has not had activity for 60 days. It will be closed if no further activity occurs. Please make a comment if this issue/pr is still valid. Thank you for your contributions. |
bump |
This issue has been automatically marked as stale because it has not had activity for 60 days. It will be closed if no further activity occurs. Please make a comment if this issue/pr is still valid. Thank you for your contributions. |
bump |
This issue has been automatically marked as stale because it has not had activity for 60 days. It will be closed if no further activity occurs. Please make a comment if this issue/pr is still valid. Thank you for your contributions. |
bump |
@skitt this one is one year old now. If it's important, let's prioritize for this/next sprint |
This issue has been automatically marked as stale because it has not had activity for 60 days. It will be closed if no further activity occurs. Please make a comment if this issue/pr is still valid. Thank you for your contributions. |
We talked about this on the PR scrub today. It still seems important, and it seems related-but-different from the ongoing Uninstall Epic (submariner-io/enhancements#50). This is more about destroyed/missing clusters. |
Closing in favor of submariner-io/enhancements#39, which tracks the same thing. |
What would you like to be added:
Some way to expire broker CRs which are no longer relevant.
Why is this needed:
Currently, clusters which stop running remain in the broker, which causes new clusters to try to set up tunnels to them, unsuccessfully. This includes previous incarnations of re-joining clusters, which means clusters can end up spending significant amounts of time trying to set tunnels up to themselves...
The text was updated successfully, but these errors were encountered: