Commit 5c288a77 authored by Matt Freeman's avatar Matt Freeman

Fix error handling in endpoint controller

Added missing returns, subsequent statements depend on key
parent feee4834
...@@ -273,6 +273,7 @@ func (e *EndpointController) deletePod(obj interface{}) { ...@@ -273,6 +273,7 @@ func (e *EndpointController) deletePod(obj interface{}) {
podKey, err := keyFunc(obj) podKey, err := keyFunc(obj)
if err != nil { if err != nil {
glog.Errorf("Couldn't get key for object %+v: %v", obj, err) glog.Errorf("Couldn't get key for object %+v: %v", obj, err)
return
} }
glog.Infof("Pod %q was deleted but we don't have a record of its final state, so it will take up to %v before it will be removed from all endpoint records.", podKey, FullServiceResyncPeriod) glog.Infof("Pod %q was deleted but we don't have a record of its final state, so it will take up to %v before it will be removed from all endpoint records.", podKey, FullServiceResyncPeriod)
...@@ -284,6 +285,7 @@ func (e *EndpointController) enqueueService(obj interface{}) { ...@@ -284,6 +285,7 @@ func (e *EndpointController) enqueueService(obj interface{}) {
key, err := keyFunc(obj) key, err := keyFunc(obj)
if err != nil { if err != nil {
glog.Errorf("Couldn't get key for object %+v: %v", obj, err) glog.Errorf("Couldn't get key for object %+v: %v", obj, err)
return
} }
e.queue.Add(key) e.queue.Add(key)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment