DevFW-CICD/ingress-nginx-helm
16
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

warn when ConfigMap is missing or unparsable instead of erroring

Browse source
This commit is contained in:
Bryan Hanner 2019-10-11 16:48:46 -07:00 committed by Bryan Hanner
parent 8fd17045e6
commit 9957d30048
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
internal/ingress/controller/controller.go
View file

@ -253,12 +253,12 @@ func (n *NGINXController) getStreamServices(configmapName string, proto apiv1.Pr
klog.V(3).Infof("Obtaining information about %v stream services from ConfigMap %q", proto, configmapName) klog.V(3).Infof("Obtaining information about %v stream services from ConfigMap %q", proto, configmapName)
_, _, err := k8s.ParseNameNS(configmapName) _, _, err := k8s.ParseNameNS(configmapName)
if err != nil { if err != nil {
klog.Errorf("Error parsing ConfigMap reference %q: %v", configmapName, err) klog.Warningf("Error parsing ConfigMap reference %q: %v", configmapName, err)
return []ingress.L4Service{} return []ingress.L4Service{}
} }
configmap, err := n.store.GetConfigMap(configmapName) configmap, err := n.store.GetConfigMap(configmapName)
if err != nil { if err != nil {
klog.Errorf("Error getting ConfigMap %q: %v", configmapName, err) klog.Warningf("Error getting ConfigMap %q: %v", configmapName, err)
return []ingress.L4Service{} return []ingress.L4Service{}
} }