Add TCP/UDP port forwarding extension to OVN

[Flavio Fernandes]

Floating IP Port Forwarding needs to be implemented to ml2/OVN

This is a functionality gap that currently exists between ml2/ovs and ml2/ovn, mentioned in [1].

Blueprint information is available here [2].

[1]: https://review.opendev.org/#/c/658414/19/specs/ussuri/ml2ovs-ovn-convergence.rst
[2]: https://blueprints.launchpad.net/neutron/+spec/port-forwarding

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/724445
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=102c442bcfa39074787e3292818880ecf238ac61
Submitter: Zuul
Branch: master

commit 102c442bcfa39074787e3292818880ecf238ac61
Author: Flavio Fernandes <email address hidden>
Date: Wed Apr 29 20:26:00 2020 -0400

    port_forwarding: extend support for OVN usage

    This commit adds possibility to configure fip port_forwarding
    service plugin and l3 extension with devstack plugin for OVN.

    Since OVN uses API workers, this change also introduces the
    callbacks necessary in pf_plugin, so events related to port
    forwarding are sent using neutron_lib callbacks registry.

    Related-Bug: #1877447
    Change-Id: I8124fac13bf4d802d232e8b3976e6a2cebc72106

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.opendev.org/734344

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/734344
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=eb46081150a4ca4e2007cabfcadd134cbc85435e
Submitter: Zuul
Branch: master

commit eb46081150a4ca4e2007cabfcadd134cbc85435e
Author: Flavio Fernandes <email address hidden>
Date: Mon Jun 8 18:02:06 2020 -0400

    port_forwarding: extend support for OVN usage (cont.)

    This is a follow up from commit 102c442bcfa39074787e3292818880ecf238ac61

    The call to resources_rpc.ResourcesPushRpcApi() will fail in situations
    where RPC is not available. That is the case for neutron_ovn_db_sync_util.

    The changes here will only set push_api if any of the configured service
    plugins need it. This is a behavior similar to what is done for the QOS
    plugin.

    Related-Bug: #1877447
    Change-Id: I3f2e18fabf4556cd708c6e544b5aaf37f72b44df

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/740955

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.opendev.org/740993

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/741000

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/741303

[OpenStack Infra (hudson-openstack)]

Change abandoned by Flavio Fernandes (<email address hidden>) on branch: master
Review: https://review.opendev.org/723863
Reason: Abandon. This is now broken up to make review a little easier. See:
https://review.opendev.org/#/q/topic:ovn/port_forwarding+(status:open)

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/741731

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/740993
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=61bf68de3185c48363a6c6c9187773368152ffdd
Submitter: Zuul
Branch: master

commit 61bf68de3185c48363a6c6c9187773368152ffdd
Author: Flavio Fernandes <email address hidden>
Date: Tue Jul 14 10:53:54 2020 -0400

    [ovn]: port forwarding -- extend support for OVN usage (cont.)

    This is a follow up from commit eb46081150a4ca4e2007cabfcadd134cbc85435e

    In order to be used by functional tests -- or anything that does not use
    stevedore that can map from "ovn-router" value -- pf_plugin also needs to
    recognize "neutron.services.ovn_l3.plugin.OVNL3RouterPlugin" as a way for
    specifying the OVN L3 service plugin.

    Change-Id: I4227cc3ceb2a17f40df86dadd71d99198b6b26e9
    Related-Bug: #1877447

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/743772

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/741000
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=597bb0d1877f92f9300d7d337bb9920ac82cab9a
Submitter: Zuul
Branch: master

commit 597bb0d1877f92f9300d7d337bb9920ac82cab9a
Author: Flavio Fernandes <email address hidden>
Date: Tue Jul 14 12:22:47 2020 -0400

    [ovn]: port forwarding -- IDL changes

    This is a subset of the changes for implementing the floating IP
    port forwarding feature in neutron, using OVN as the backend.

    This changeset covers the additions to northbound api needed for
    handling load balancer entries created on behalf of port forwarding.

    Depends-On: https://review.opendev.org/#/c/729354/
    Change-Id: I7d0aa51468dbd2c298395c3dccd8f222e87032d8
    Partially-implements: ovn/port_forwarding
    Partial-Bug: #1877447

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/743862

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/740955
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=e2060a2f0b317aadc41443accfbdc5cafae44f44
Submitter: Zuul
Branch: master

commit e2060a2f0b317aadc41443accfbdc5cafae44f44
Author: Flavio Fernandes <email address hidden>
Date: Tue Jul 14 08:57:52 2020 -0400

    [ovn]: port forwarding -- documentation

    This is a subset of the changes for implementing the floating IP
    port forwarding feature in neutron, using OVN as the backend.

    This changeset covers the documentation updates for the feature,
    as well as a high-level description of how OVN implements it.

    Partially-implements: ovn/port_forwarding
    Partially-implements: blueprint portforwarding-description
    Partial-Bug: #1877447

    Change-Id: I2059a011f650dd7070a74dc6107aab2b15ca7104

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/741303
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=d74f409c82fb73601b859cd6b243bbbc97b49153
Submitter: Zuul
Branch: master

commit d74f409c82fb73601b859cd6b243bbbc97b49153
Author: Flavio Fernandes <email address hidden>
Date: Wed Jul 15 14:26:31 2020 -0400

    [ovn]: port forwarding -- core changes

    This is a subset of the changes for implementing the floating IP
    port forwarding feature in neutron, using OVN as the backend.

    This changeset covers the core implementation for portforwarding/drivers/ovn,
    mech_driver, ovn-router as well as a subset of tests.

    Port forwarding support in ovn_db_sync is not included here to facilitate review.
    That, as well as all other supporting changes, are under the ovn/port_forwarding topic:
    https://review.opendev.org/#/q/topic:ovn/port_forwarding+(status:open+OR+status:merged)

    Depends-On: https://review.opendev.org/#/c/726478/
    Partially-implements: ovn/port_forwarding
    Partial-Bug: #1877447

    Change-Id: I019fe11ac1ddcf2304f3f144c62d52667fc11dce

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/743862
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=605de6d96854767edd6ed5a1512e3e46b7b0c311
Submitter: Zuul
Branch: master

commit 605de6d96854767edd6ed5a1512e3e46b7b0c311
Author: Flavio Fernandes <email address hidden>
Date: Wed Jul 29 21:38:51 2020 -0400

    [ovn]: port forwarding -- Functional test for IDL changes

    This is a subset of the changes for implementing the floating IP
    port forwarding feature in neutron, using OVN as the backend.

    This changeset introduces functional tests for the IDL functions
    added to the NB API for supporting port forwarding.

    Depends-On: https://review.opendev.org/#/c/741000/
    Change-Id: I01f54ea4a21eb635f0bf87a15694929ed1f3a643
    Partially-implements: ovn/port_forwarding
    Partial-Bug: #1877447

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/743772
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=361b4853180b84b1d8ba38a6967de4c254aacb0c
Submitter: Zuul
Branch: master

commit 361b4853180b84b1d8ba38a6967de4c254aacb0c
Author: Flavio Fernandes <email address hidden>
Date: Wed Jul 29 10:24:11 2020 -0400

    [ovn]: port forwarding -- feature support under ovn_db_sync

    This is a subset of the changes for implementing the floating IP
    port forwarding feature in neutron, using OVN as the backend.

    This changeset updates ovn_db_sync utility and its tests to
    ensure that floating ip port forwarding can be repaired.

    Depends-On: https://review.opendev.org/#/c/741303/
    Change-Id: I7a158173252e73e081914f242133634c41de7999
    Partially-implements: ovn/port_forwarding
    Partial-Bug: #1877447

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/747283

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/747284

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/747285

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/747286

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/747287

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/747289

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/747299

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/747300

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/747301

[OpenStack Infra (hudson-openstack)]

Change abandoned by Slawek Kaplonski (<email address hidden>) on branch: stable/ussuri
Review: https://review.opendev.org/747287
Reason: As we discussed in the team and with stable-maint-team, we will not backport this patch to stable branches.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Slawek Kaplonski (<email address hidden>) on branch: stable/ussuri
Review: https://review.opendev.org/747285
Reason: As we discussed in the team and with stable-maint-team, we will not backport this patch to stable branches.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Slawek Kaplonski (<email address hidden>) on branch: stable/ussuri
Review: https://review.opendev.org/747284
Reason: As we discussed in the team and with stable-maint-team, we will not backport this patch to stable branches.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Slawek Kaplonski (<email address hidden>) on branch: stable/ussuri
Review: https://review.opendev.org/747283
Reason: As we discussed in the team and with stable-maint-team, we will not backport this patch to stable branches.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Slawek Kaplonski (<email address hidden>) on branch: stable/ussuri
Review: https://review.opendev.org/747289
Reason: As we discussed in the team and with stable-maint-team, we will not backport this patch to stable branches.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Slawek Kaplonski (<email address hidden>) on branch: stable/ussuri
Review: https://review.opendev.org/747286
Reason: As we discussed in the team and with stable-maint-team, we will not backport this patch to stable branches.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Slawek Kaplonski (<email address hidden>) on branch: stable/ussuri
Review: https://review.opendev.org/747299
Reason: As we discussed in the team and with stable-maint-team, we will not backport this patch to stable branches.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Slawek Kaplonski (<email address hidden>) on branch: stable/ussuri
Review: https://review.opendev.org/747300
Reason: As we discussed in the team and with stable-maint-team, we will not backport this patch to stable branches.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Slawek Kaplonski (<email address hidden>) on branch: stable/ussuri
Review: https://review.opendev.org/747301
Reason: As we discussed in the team and with stable-maint-team, we will not backport this patch to stable branches.

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: stable/train
Review: https://review.opendev.org/749841

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: stable/train
Review: https://review.opendev.org/749843

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: stable/train
Review: https://review.opendev.org/749844

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/741731
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=0deec6621e6485e0a7c219953850fd14b83a7c41
Submitter: Zuul
Branch: master

commit 0deec6621e6485e0a7c219953850fd14b83a7c41
Author: Flavio Fernandes <email address hidden>
Date: Fri Jul 17 17:58:17 2020 -0400

    [ovn]: port forwarding -- functional tests for maintenance task

    This is a subset of the changes for implementing the floating IP
    port forwarding feature in neutron, using OVN as the backend.

    This changeset adds functional tests in maintenance task to ensure
    that floating ips as well as port forwarding are properly handled.

    Depends-On: https://review.opendev.org/#/c/741303/
    Change-Id: I0671ed8d73ca8c0e3ba4aded81f395e20957d598
    Partially-implements: ovn/port_forwarding
    Partial-Bug: #1877447

[OpenStack Infra (hudson-openstack)]

Change abandoned by Flavio Fernandes (<email address hidden>) on branch: master
Review: https://review.opendev.org/753366
Reason: Now done via https://review.opendev.org/#/c/753419/

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: stable/train
Review: https://review.opendev.org/757134

[OpenStack Infra (hudson-openstack)]

Change abandoned by Flavio Fernandes (<email address hidden>) on branch: stable/train
Review: https://review.opendev.org/749844

[OpenStack Infra (hudson-openstack)]

Change abandoned by Flavio Fernandes (<email address hidden>) on branch: stable/train
Review: https://review.opendev.org/749843

[OpenStack Infra (hudson-openstack)]

Change abandoned by Flavio Fernandes (<email address hidden>) on branch: stable/train
Review: https://review.opendev.org/749841

[OpenStack Infra (hudson-openstack)]

Change abandoned by Flavio Fernandes (<email address hidden>) on branch: stable/train
Review: https://review.opendev.org/757134

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/tripleo-heat-templates 14.1.0 release.