Bug 1567532 - Unidle handling in router should ignore headless services

[pravisankar]

• Currently, service with empty ClusterIP is ignored but headless services
  can have ClusterIP="None" and those need to be ignored as well.

Found this problem on ca-central starter cluster where router reload failed due to incorrect haproxy config:

limiter.go:137] error reloading router: exit status 1
: parsing [/var/lib/haproxy/conf/haproxy.config:63325] : 'server ept:mysql:None:3306' : could not resolve address 'None'.
: Failed to initialize server(s) addr.

None in endpoint.ID came from service.Spec.ClusterIP

[pravisankar]

@openshift/sig-networking PTAL

[pravisankar]

/retest

[imcsk8]

LGTM

[knobunc]

/lgtm

Thanks Ravi

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: knobunc, pravisankar

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• pkg/router/OWNERS [knobunc]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment