Don't retry when we get a config load error.
All the SetConfig errors we return are permanent until something in the configuration
of either metallb or the configmap is updated. So, there's no point in retrying,
just mark the config sad, log warnings in the k8s event log, and move on.
@@ -231,7 +231,8 @@ func (c *Client) sync(key interface{}) error {
if err := c.controller.SetConfig(cfg); err != nil {
configStale.Set(1)
- return err
+ c.events.Eventf(cm, v1.EventTypeWarning, "InvalidConfig", "%s", err)
+ return nil
}
configLoaded.Set(1)
