Bug 1846396

Summary: Kuryr blocks the usage of tcp and udp listeners on the same port for ovn-octavia provider
Product: OpenShift Container Platform Reporter: Luis Tomas Bolivar <ltomasbo>
Component: NetworkingAssignee: Luis Tomas Bolivar <ltomasbo>
Networking sub component: kuryr QA Contact: GenadiC <gcheresh>
Status: CLOSED ERRATA Docs Contact:
Severity: low    
Priority: medium CC: rlobillo
Version: 4.4   
Target Milestone: ---   
Target Release: 4.6.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Feature: Not blocking having several listeners (on different protocols) on the same port for ovn-octavia driver. Reason: This was not supported previously on the ovn octavia driver, but now it is supported and there is no need to block it Result: Several listeners on different protocols can be exposed in the same port. This means that we can have, for instance, the dns service to expose port 53 in both tcp and udp protocols when using ovn-octavia
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-10-27 16:06:37 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1846452    

Description Luis Tomas Bolivar 2020-06-11 13:41:27 UTC
Ovn-octavia provider now supports listeners on the same port
for different protocols and kuryr should not block its usage

Comment 3 rlobillo 2020-07-15 08:56:19 UTC
Verified on 4.6.0-0.nightly-2020-07-14-112216 over OSP16 (RHOS-16.1-RHEL-8-20200701.n.0) with OVN.

No admission controllers observed on openshift-kuryr namespace with normal operation:

# openshfit-kuryr namespace:

[stack@undercloud-0 ~]$ oc get all -n openshift-kuryr
NAME                                    READY   STATUS    RESTARTS   AGE
pod/kuryr-cni-ck4ph                     1/1     Running   0          16h
pod/kuryr-cni-g54kg                     1/1     Running   0          17h
pod/kuryr-cni-n8bcj                     1/1     Running   0          16h
pod/kuryr-cni-n9t42                     1/1     Running   0          17h
pod/kuryr-cni-ntmvp                     1/1     Running   0          16h
pod/kuryr-cni-splb2                     1/1     Running   0          17h
pod/kuryr-controller-6586c48bb4-ql4bc   1/1     Running   3          17h

NAME                       TYPE        CLUSTER-IP   EXTERNAL-IP   PORT(S)    AGE
service/kuryr-cni          ClusterIP   None         <none>        9655/TCP   17h
service/kuryr-controller   ClusterIP   None         <none>        9654/TCP   17h

NAME                       DESIRED   CURRENT   READY   UP-TO-DATE   AVAILABLE   NODE SELECTOR   AGE
daemonset.apps/kuryr-cni   6         6         6       6            6           <none>          17h

NAME                               READY   UP-TO-DATE   AVAILABLE   AGE
deployment.apps/kuryr-controller   1/1     1            1           17h

NAME                                          DESIRED   CURRENT   READY   AGE
replicaset.apps/kuryr-controller-6586c48bb4   1         1         1       17h


# test namespace:

[stack@undercloud-0 ~]$ oc get all -n test
NAME                      READY   STATUS    RESTARTS   AGE
pod/demo                  1/1     Running   0          75s
pod/demo-allowed-caller   1/1     Running   0          75s
pod/demo-caller           1/1     Running   0          75s

NAME           TYPE        CLUSTER-IP      EXTERNAL-IP   PORT(S)   AGE
service/demo   ClusterIP   172.30.243.41   <none>        80/TCP    75s
[stack@undercloud-0 ~]$ oc rsh pod/demo-caller curl 172.30.243.41
demo: HELLO! I AM ALIVE!!!

# OVN-Octavia provider on use:

(overcloud) [stack@undercloud-0 ~]$ openstack loadbalancer provider list
+---------+-------------------------------------------------+
| name    | description                                     |
+---------+-------------------------------------------------+
| amphora | The Octavia Amphora driver.                     |
| octavia | Deprecated alias of the Octavia Amphora driver. |
| ovn     | Octavia OVN driver.                             |
+---------+-------------------------------------------------+
(overcloud) [stack@undercloud-0 ~]$ openstack loadbalancer show test/demo
+---------------------+--------------------------------------+
| Field               | Value                                |
+---------------------+--------------------------------------+
| admin_state_up      | True                                 |
| created_at          | 2020-07-15T08:49:55                  |
| description         |                                      |
| flavor_id           | None                                 |
| id                  | 1b5b92ce-d3f2-4743-aa49-e9b4388b4937 |
| listeners           | e6059208-dd96-4ca5-8ee4-e487f4f95c30 |
| name                | test/demo                            |
| operating_status    | ONLINE                               |
| pools               | 0fecc65d-3328-46ec-b068-8c614fb4cca5 |
| project_id          | 75ebd27eb4e9406293cc9240a7cdf220     |
| provider            | ovn                                  |
| provisioning_status | ACTIVE                               |
| updated_at          | 2020-07-15T08:50:13                  |
| vip_address         | 172.30.243.41                        |
| vip_network_id      | 29d56b48-8976-43fd-a038-467b2e6de714 |
| vip_port_id         | 387e1016-e4d6-4bac-8fa6-d3733695bb91 |
| vip_qos_policy_id   | None                                 |
| vip_subnet_id       | 1123213a-a956-41b8-b704-c59fe3d8b7bd |
+---------------------+--------------------------------------+

Comment 5 errata-xmlrpc 2020-10-27 16:06:37 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (OpenShift Container Platform 4.6 GA Images), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2020:4196