worker: log Ingress resource type

Signed-off-by: Alexander Yastrebov <alexander.yastrebov@zalando.de>

kubernetes/adapter.go

@@ -102,9 +102,9 @@ type Ingress struct {
 	resourceType     ingressType
 }
 
-// String returns a string representation of the Ingress instance containing the namespace and the resource name.
+// String returns a string representation of the Ingress instance containing the type, namespace and the resource name.
 func (i *Ingress) String() string {
-	return fmt.Sprintf("%s/%s", i.Namespace, i.Name)
+	return fmt.Sprintf("%s %s/%s", i.resourceType, i.Namespace, i.Name)
 }
 
 // ConfigMap is the ingress-controller's representation of a Kubernetes

kubernetes/adapter_test.go

@@ -455,7 +455,7 @@ func TestNewIngressFromKube(tt *testing.T) {
 			} else {
 				assert.NoError(t, err)
 				assert.Equal(t, tc.ingress, got, "mapping from kubernetes ingress to adapter failed")
-				assert.Equal(t, got.String(), fmt.Sprintf("%s/%s", tc.ingress.Namespace, tc.ingress.Name), "wrong value from String()")
+				assert.Equal(t, got.String(), fmt.Sprintf("%s %s/%s", tc.ingress.resourceType, tc.ingress.Namespace, tc.ingress.Name), "wrong value from String()")
 			}
 		})
 	}

worker.go

@@ -401,12 +401,7 @@ func matchIngressesToLoadBalancers(
 
 		if ingress.CertificateARN != "" {
 			if !certs.CertificateExists(ingress.CertificateARN) {
-				log.Errorf(
-					"Failed to find certificate '%s' for ingress '%s/%s'",
-					ingress.CertificateARN,
-					ingress.Namespace,
-					ingress.Name,
-				)
+				log.Errorf("Failed to find certificate %s for %s", ingress.CertificateARN, ingress)
 				continue
 			}
 			certificateARNs = []string{ingress.CertificateARN}
@@ -576,12 +571,12 @@ func updateIngress(kubeAdapter *kubernetes.Adapter, lb *loadBalancer, problems *
 		for _, ing := range ingresses {
 			if err := kubeAdapter.UpdateIngressLoadBalancer(ing, dnsName); err != nil {
 				if err == kubernetes.ErrUpdateNotNeeded {
-					log.Debugf("Ingress update not needed %v with DNS name %q", ing, dnsName)
+					log.Debugf("Update not needed for %s with DNS name %s", ing, dnsName)
 				} else {
-					problems.Add("failed to update ingress %q: %w", ing, err)
+					problems.Add("failed to update %s: %w", ing, err)
 				}
 			} else {
-				log.Infof("Updated ingress %v with DNS name %q", ing, dnsName)
+				log.Infof("Updated %s with DNS name %s", ing, dnsName)
 			}
 		}
 	}