Added some listner needsModification debug info and removed some unnecessary hostname debugs

Author: bigkraig

pkg/alb/ls/listener.go

@@ -94,7 +94,7 @@ func (l *Listener) Reconcile(rOpts *ReconcileOptions) error {
 			return err
 		}
 		rOpts.Eventf(api.EventTypeNormal, "MODIFY", "%v listener modified", *l.ls.current.Port)
-		l.logger.Infof("Completed Listener modification. ARN: %s | Port: %s | Proto: %s.",
+		l.logger.Infof("Completed Listener modification. ARN: %s | Port: %v | Proto: %s.",
 			*l.ls.current.ListenerArn, *l.ls.current.Port, *l.ls.current.Protocol)
 
 	default:
@@ -200,16 +200,22 @@ func (l *Listener) needsModification(rOpts *ReconcileOptions) bool {
 	case lsc == nil && lsd == nil:
 		return false
 	case lsc == nil:
+		l.logger.Debugf("Current is nil")
 		return true
 	case !util.DeepEqual(lsc.Port, lsd.Port):
+		l.logger.Debugf("Port needs to be changed (%v != %v)", log.Prettify(lsc.Port), log.Prettify(lsd.Port))
 		return true
 	case !util.DeepEqual(lsc.Protocol, lsd.Protocol):
+		l.logger.Debugf("Protocol needs to be changed (%v != %v)", log.Prettify(lsc.Protocol), log.Prettify(lsd.Protocol))
 		return true
 	case !util.DeepEqual(lsc.Certificates, lsd.Certificates):
+		l.logger.Debugf("Certificates needs to be changed (%v != %v)", log.Prettify(lsc.Certificates), log.Prettify(lsd.Certificates))
 		return true
 	case !util.DeepEqual(lsc.DefaultActions, lsd.DefaultActions):
+		l.logger.Debugf("DefaultActions needs to be changed (%v != %v)", log.Prettify(lsc.DefaultActions), log.Prettify(lsd.DefaultActions))
 		return true
 	case !util.DeepEqual(lsc.SslPolicy, lsd.SslPolicy):
+		l.logger.Debugf("SslPolicy needs to be changed (%v != %v)", log.Prettify(lsc.SslPolicy), log.Prettify(lsd.SslPolicy))
 		return true
 	}
 	return false

pkg/alb/rs/rule.go

@@ -103,8 +103,7 @@ func (r *Rule) Reconcile(rOpts *ReconcileOptions) error {
 			log.Prettify(r.rs.current.Conditions))
 
 	case *r.rs.desired.IsDefault: // rule is default (attached to listener), do nothing
-		r.logger.Debugf("Found desired rule that is a default and is already created with its respective listener. Rule: %s",
-			log.Prettify(r.rs.desired))
+		// r.logger.Debugf("Found desired rule that is a default and is already created with its respective listener. Rule: %s", log.Prettify(r.rs.desired))
 		r.rs.current = r.rs.desired
 
 	case r.rs.current == nil: // rule doesn't exist and should be created

pkg/albingress/albingress.go

@@ -225,7 +225,7 @@ func (a *ALBIngress) Hostnames() ([]api.LoadBalancerIngress, error) {
 		return nil, fmt.Errorf("No ALB hostnames for ingress")
 	}
 
-	a.logger.Debugf("Ingress library requested hostname list and we returned %s", *a.loadBalancer.Hostname())
+	// a.logger.Debugf("Ingress library requested hostname list and we returned %s", *a.loadBalancer.Hostname())
 	return hostnames, nil
 }