Fixed attribute logic

[bigkraig]

There were a few problems with how load balancer attributes were managed, I also cleaned up some logging messages during the reconcile and removed a duplicate method for configuring the idle timeout. Resolves #440

[ianau74]

How does one set the connection timeout of the ALB?

In the docs, https://kubernetes-sigs.github.io/aws-alb-ingress-controller/guide/ingress/annotation/#load-balancer-attributes it says you're supposed to use:

alb.ingress.kubernetes.io/load-balancer-attributes:idle_timeout.timeout_seconds: "120"

But, that shows:
metadata.annotations: Invalid value: "alb.ingress.kubernetes.io/load-balancer-attributes:idle_timeout.timeout_seconds": name part must consist of alphanumeric characters, '-', '' or '.', and must start and end with an alphanumeric character (e.g. 'MyName', or 'my.name', or '123-abc', regex used for validation is '([A-Za-z0-9][-A-Za-z0-9.]*)?[A-Za-z0-9]')

[ianau74]

Ah, nevermind. The correct syntax is:
alb.ingress.kubernetes.io/load-balancer-attributes: idle_timeout.timeout_seconds=3600