neutron

ovn: DVR on VLAN networks does not work

Bug #1920976 reported by Michal Nasiadka
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
neutron
Fix Released
High
Michal Nasiadka

Bug Description

OVN deployment with DVR router, two networks attached:
- external (type VLAN)
- internal (type VLAN)

OVN configured to use external_ids:ovn-chassis-mac-mappings - to allow VLAN backed DVR:

https://github.com/ovn-org/ovn/commit/1fed74cfc1a1e3b29cf86eab2e96048813019b57

Neutron is setting reside-on-redirect-chassis on the router ports - which essentially makes the router centralized on the network nodes - and disabling DVR.
With neutron/ml2.conf setting of distributing floating ips - it makes them inaccessible - clearing reside-on-redirect-chassis from lrp options makes the functionality working.

Revision history for this message
Michal Nasiadka (mnasiadka) wrote :

Fixing in https://review.opendev.org/c/openstack/neutron/+/782250

tags: added: ovn
Lajos Katona (lajos-katona)
Changed in neutron:
assignee: nobody → Michal Nasiadka (mnasiadka)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 18.0.0.0rc1

This issue was fixed in the openstack/neutron 18.0.0.0rc1 release candidate.

Lajos Katona (lajos-katona)
Changed in neutron:
importance: Undecided → High
status: New → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/ussuri)

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/c/openstack/neutron/+/789868

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/ussuri)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/789868
Committed: https://opendev.org/openstack/neutron/commit/2611e10a1e3fc79ded5a9bee5f5f7abfb47f81f8
Submitter: "Zuul (22348)"
Branch: stable/ussuri

commit 2611e10a1e3fc79ded5a9bee5f5f7abfb47f81f8
Author: Michał Nasiadka <email address hidden>
Date: Mon Mar 22 13:57:01 2021 +0100

    ovn: Do not set reside-on-redirect-chassis on distributed FIP

    Since [1] and [2] OVN supports DVR on VLAN networks with usage of
    external-ids:ovn-chassis-mac-mappings, but reside-on-redirect-chassis
    needs to be unset when distributed floating IP is enabled.

    [1]: https://github.com/ovn-org/ovn/commit/1fed74cfc1a1e3b29cf86eab2e96048813019b57
    [2]: https://github.com/ovn-org/ovn/commit/522911269f1422e1274cbfbe53035a3bb8f573eb

    Closes-Bug: #1920976
    Change-Id: I68b9a586d1d7ec75820fedcb8e51d1c49ae871ae
    (cherry picked from commit 828765412789a15a5f375060cfab7628c65e9d5d)
    (cherry picked from commit b956411b6c398a51a7e385f560a2c72aa29b0edd)

tags: added: in-stable-ussuri
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 16.3.2

This issue was fixed in the openstack/neutron 16.3.2 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 17.1.2

This issue was fixed in the openstack/neutron 17.1.2 release.

Bernard Cafarelli (bcafarel)
tags: added: neutron-proactive-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/826912

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (stable/xena)

Related fix proposed to branch: stable/xena
Review: https://review.opendev.org/c/openstack/neutron/+/828727

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (stable/victoria)

Related fix proposed to branch: stable/victoria
Review: https://review.opendev.org/c/openstack/neutron/+/828728

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (stable/wallaby)

Related fix proposed to branch: stable/wallaby
Review: https://review.opendev.org/c/openstack/neutron/+/828729

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (stable/ussuri)

Related fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/c/openstack/neutron/+/828730

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (master)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/826912
Committed: https://opendev.org/openstack/neutron/commit/5bef86847790f60bf2cb87903db8b9ef88051623
Submitter: "Zuul (22348)"
Branch: master

commit 5bef86847790f60bf2cb87903db8b9ef88051623
Author: Lucas Alvares Gomes <email address hidden>
Date: Fri Jan 28 15:57:45 2022 +0000

    [OVN] Migrate "reside-on-redirect-chassis" for distributed FIP

    This patch complement the changes made at
    828765412789a15a5f375060cfab7628c65e9d5d by creating a maintenance task
    resposible for migrating the "reside-on-redirect-chassis" option value
    for existing router ports when the deployment is updated.

    Change-Id: I0c623a101357226ec0b4e3e42b829303846b6f6b
    Related-Bug: #1920976
    Signed-off-by: Lucas Alvares Gomes <email address hidden>

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/xena)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/828727
Committed: https://opendev.org/openstack/neutron/commit/f20a01588675bcea8fd5e8dd18d5a6a97edcc755
Submitter: "Zuul (22348)"
Branch: stable/xena

commit f20a01588675bcea8fd5e8dd18d5a6a97edcc755
Author: Lucas Alvares Gomes <email address hidden>
Date: Fri Jan 28 15:57:45 2022 +0000

    [OVN] Migrate "reside-on-redirect-chassis" for distributed FIP

    This patch complement the changes made at
    828765412789a15a5f375060cfab7628c65e9d5d by creating a maintenance task
    resposible for migrating the "reside-on-redirect-chassis" option value
    for existing router ports when the deployment is updated.

    Conflicts:
      neutron/common/ovn/constants.py
      neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/maintenance.py
      neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_maintenance.py

    Change-Id: I0c623a101357226ec0b4e3e42b829303846b6f6b
    Related-Bug: #1920976
    Signed-off-by: Lucas Alvares Gomes <email address hidden>
    (cherry picked from commit 5bef86847790f60bf2cb87903db8b9ef88051623)

tags: added: in-stable-xena
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/wallaby)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/828729
Committed: https://opendev.org/openstack/neutron/commit/d96819cce425ab77b21080aeed79d5a12917f378
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit d96819cce425ab77b21080aeed79d5a12917f378
Author: Lucas Alvares Gomes <email address hidden>
Date: Fri Jan 28 15:57:45 2022 +0000

    [OVN] Migrate "reside-on-redirect-chassis" for distributed FIP

    This patch complement the changes made at
    828765412789a15a5f375060cfab7628c65e9d5d by creating a maintenance task
    resposible for migrating the "reside-on-redirect-chassis" option value
    for existing router ports when the deployment is updated.

    Conflicts:
      neutron/common/ovn/constants.py
      neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/maintenance.py
      neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_maintenance.py

    Change-Id: I0c623a101357226ec0b4e3e42b829303846b6f6b
    Related-Bug: #1920976
    Signed-off-by: Lucas Alvares Gomes <email address hidden>
    (cherry picked from commit 5bef86847790f60bf2cb87903db8b9ef88051623)

tags: added: in-stable-wallaby
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/ussuri)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/828730
Committed: https://opendev.org/openstack/neutron/commit/558b60dc44a739a1a286ce819f5a1c9206c9fdeb
Submitter: "Zuul (22348)"
Branch: stable/ussuri

commit 558b60dc44a739a1a286ce819f5a1c9206c9fdeb
Author: Lucas Alvares Gomes <email address hidden>
Date: Fri Jan 28 15:57:45 2022 +0000

    [OVN] Migrate "reside-on-redirect-chassis" for distributed FIP

    This patch complement the changes made at
    828765412789a15a5f375060cfab7628c65e9d5d by creating a maintenance task
    resposible for migrating the "reside-on-redirect-chassis" option value
    for existing router ports when the deployment is updated.

    Conflicts:
      neutron/common/ovn/constants.py
      neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/maintenance.py
      neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_maintenance.py

    Change-Id: I0c623a101357226ec0b4e3e42b829303846b6f6b
    Related-Bug: #1920976
    Signed-off-by: Lucas Alvares Gomes <email address hidden>
    (cherry picked from commit 5bef86847790f60bf2cb87903db8b9ef88051623)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/victoria)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/828728
Committed: https://opendev.org/openstack/neutron/commit/59499e14990029e5aab1ba158246bc97a26c6cf3
Submitter: "Zuul (22348)"
Branch: stable/victoria

commit 59499e14990029e5aab1ba158246bc97a26c6cf3
Author: Lucas Alvares Gomes <email address hidden>
Date: Fri Jan 28 15:57:45 2022 +0000

    [OVN] Migrate "reside-on-redirect-chassis" for distributed FIP

    This patch complement the changes made at
    828765412789a15a5f375060cfab7628c65e9d5d by creating a maintenance task
    resposible for migrating the "reside-on-redirect-chassis" option value
    for existing router ports when the deployment is updated.

    Conflicts:
      neutron/common/ovn/constants.py
      neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/maintenance.py
      neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_maintenance.py

    Change-Id: I0c623a101357226ec0b4e3e42b829303846b6f6b
    Related-Bug: #1920976
    Signed-off-by: Lucas Alvares Gomes <email address hidden>
    (cherry picked from commit 5bef86847790f60bf2cb87903db8b9ef88051623)

tags: added: in-stable-victoria
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.