source: Drop spec.host check from OCP Route impl

On OpenShift, a route resource cannot have the
`spec.host` field be set to the empty string. This commit clarifies
the FQDN template logic in the OCP Route source implementation
by removing the explicit `spec.host` check and adding comments.

Signed-off-by: Stephen Greene <sgreene@redhat.com>

source/ocproute.go

@@ -141,8 +141,12 @@ func (ors *ocpRouteSource) Endpoints(ctx context.Context) ([]*endpoint.Endpoint,
 
 		orEndpoints := endpointsFromOcpRoute(ocpRoute, ors.ignoreHostnameAnnotation)
 
-		// apply template if host is missing on OpenShift Route
-		if (ors.combineFQDNAnnotation || len(orEndpoints) == 0) && ors.fqdnTemplate != nil {
+		// apply template if combineFQDNAnnotation is set
+		// and if we have a fqdnTemplate.
+		// A route's spec.host field cannot be empty, so checking
+		// that field when determining whether or not to use the fqdn
+		// template is futile.
+		if ors.combineFQDNAnnotation && ors.fqdnTemplate != nil {
 			oEndpoints, err := ors.endpointsFromTemplate(ocpRoute)
 			if err != nil {
 				return nil, err