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
improv: check for k8s backing before running sync #27269
improv: check for k8s backing before running sync #27269
Conversation
ce12572
to
3c4803e
Compare
/test |
Signed-off-by: kwakubiney <kebiney@hotmail.com>
3c4803e
to
90f5fc8
Compare
/test |
@@ -95,14 +103,6 @@ func (epSync *EndpointSynchronizer) RunK8sCiliumEndpointSync(e *endpoint.Endpoin | |||
return fmt.Errorf("Kubernetes apiserver is not available") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this condition (k8s connectivity) change as well, or can we also hoist it out of the controller?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd look at it but looks like PR has been merged?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hah! Nevermind :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just one possible small improvement. Thanks!
This change ensures that the synchronisation controller does not run at all when the endpoint has no
CEP
name, which implies that it also has no pod backing.Fixes: #27128