cilium / cilium / 42530

39%

push

travis-ci-com

ingress: Fix issue with port number is not used

This commit is to fix the issue that the port number of backend service
is not used i.e. regardless of port values, all the endpoints of ingress
backend are routable via Ingress.

Before

```
$ kg ingress basic-ingress -o yaml
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: basic-ingress
  namespace: default
spec:
  ingressClassName: cilium
  rules:
  - http:
      paths:
      - backend:
          service:
            name: details
            port:
              number: 9999
        path: /details
        pathType: Prefix
      - backend:
          service:
            name: productpage
            port:
              number: 9080
        path: /
        pathType: Prefix

$ curl http://$lb/details/1
{"id":1,"author":"William Shakespeare","year":1595,"type":"paperback","pages":200,"publisher":"PublisherA","language":"English","ISBN-10":"1234567890","ISBN-13":"123-1234567890"}
```

After

```
$ curl http://$lb/details/1
no healthy upstream

// Change invalid port number 9999 to 9080
$ curl http://$lb/details/1
{"id":1,"author":"William Shakespeare","year":1595,"type":"paperback","pages":200,"publisher":"PublisherA","language":"English","ISBN-10":"1234567890","ISBN-13":"123-1234567890"}
```

Signed-off-by: Tam Mach <tam.mach@cilium.io>

63 of 63 new or added lines in 1 file covered. (100.0%)

47706 of 141344 relevant lines covered (33.75%)

3750.46 hits per line