[OVN migration] qr- interfaces and trunk subports aren't cleaned after migration to ML2/OVN

Bug #1946479 reported by Slawek Kaplonski
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Slawek Kaplonski

Bug Description

After migration from ML2/OVS+DVR to ML2/OVN qr- interfaces (router ports) aren't cleaned from the nodes even if they aren't needed in ML2/OVN case at all.
The same is with trunk's subports (spi- and spt- patch ports which connects br-int and trunk bridge).

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/+/813186

Changed in neutron:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/813187

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

Reviewed: https://review.opendev.org/c/openstack/neutron/+/813186
Committed: https://opendev.org/openstack/neutron/commit/5092f3fb98b4c6bf8334de0e71548ddd073f6234
Submitter: "Zuul (22348)"
Branch: master

commit 5092f3fb98b4c6bf8334de0e71548ddd073f6234
Author: Slawek Kaplonski <email address hidden>
Date: Fri Oct 8 15:04:10 2021 +0200

    [OVN Migration] Remove qr and dhcp ports from the nodes

    After migration from ML2/OVS to ML2/OVN tap ports which were created
    by the DHCP agents and router ports (like e.g. qr- or qg-) which were
    created by the L3 agents aren't needed at all and should be deleted.
    Previously those ports were set to be DOWN only. With this patch
    all such ports will be simply deleted from the openvswitch.

    Related-bug: #1946479
    Change-Id: I74cd5820389c86819c6884d3d61c9b2f7907cc88

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/+/815236

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/+/815237

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/+/815238

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/+/815239

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

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

commit bf6831e0d2ad33ff751f06d568abe6cedff4d717
Author: Slawek Kaplonski <email address hidden>
Date: Fri Oct 8 15:06:19 2021 +0200

    [OVN Migration] Remove trunk's subports from the nodes

    After migration from ML2/OVS to ML2/OVN trunk subports which were created
    by the neutron-ovs-agent to connect br-int with trunk bridges (tbr-)
    aren't needed anymore and should be deleted.

    Closes-Bug: #1946479
    Change-Id: Ib1e3b78597ebdde1aa9d2b242e2005a05a7db89f

Changed in neutron:
status: In Progress → Fix Released
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/+/815236
Committed: https://opendev.org/openstack/neutron/commit/a8a099f069b7a92876b75db462d0265187cb07e7
Submitter: "Zuul (22348)"
Branch: stable/xena

commit a8a099f069b7a92876b75db462d0265187cb07e7
Author: Slawek Kaplonski <email address hidden>
Date: Fri Oct 8 15:04:10 2021 +0200

    [OVN Migration] Remove qr and dhcp ports from the nodes

    After migration from ML2/OVS to ML2/OVN tap ports which were created
    by the DHCP agents and router ports (like e.g. qr- or qg-) which were
    created by the L3 agents aren't needed at all and should be deleted.
    Previously those ports were set to be DOWN only. With this patch
    all such ports will be simply deleted from the openvswitch.

    Related-bug: #1946479
    Change-Id: I74cd5820389c86819c6884d3d61c9b2f7907cc88
    (cherry picked from commit 5092f3fb98b4c6bf8334de0e71548ddd073f6234)

tags: added: in-stable-xena
tags: added: in-stable-wallaby
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/+/815237
Committed: https://opendev.org/openstack/neutron/commit/9a3f0b37606435388d66ed3823e4226924988915
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit 9a3f0b37606435388d66ed3823e4226924988915
Author: Slawek Kaplonski <email address hidden>
Date: Fri Oct 8 15:04:10 2021 +0200

    [OVN Migration] Remove qr and dhcp ports from the nodes

    After migration from ML2/OVS to ML2/OVN tap ports which were created
    by the DHCP agents and router ports (like e.g. qr- or qg-) which were
    created by the L3 agents aren't needed at all and should be deleted.
    Previously those ports were set to be DOWN only. With this patch
    all such ports will be simply deleted from the openvswitch.

    Related-bug: #1946479
    Change-Id: I74cd5820389c86819c6884d3d61c9b2f7907cc88
    (cherry picked from commit 5092f3fb98b4c6bf8334de0e71548ddd073f6234)

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/+/815238
Committed: https://opendev.org/openstack/neutron/commit/fa102252a543e731f17f91fe627a70ddf1c1e3d6
Submitter: "Zuul (22348)"
Branch: stable/victoria

commit fa102252a543e731f17f91fe627a70ddf1c1e3d6
Author: Slawek Kaplonski <email address hidden>
Date: Fri Oct 8 15:04:10 2021 +0200

    [OVN Migration] Remove qr and dhcp ports from the nodes

    After migration from ML2/OVS to ML2/OVN tap ports which were created
    by the DHCP agents and router ports (like e.g. qr- or qg-) which were
    created by the L3 agents aren't needed at all and should be deleted.
    Previously those ports were set to be DOWN only. With this patch
    all such ports will be simply deleted from the openvswitch.

    Related-bug: #1946479
    Change-Id: I74cd5820389c86819c6884d3d61c9b2f7907cc88
    (cherry picked from commit 5092f3fb98b4c6bf8334de0e71548ddd073f6234)

tags: added: in-stable-victoria
tags: added: in-stable-ussuri
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/+/815239
Committed: https://opendev.org/openstack/neutron/commit/0c5a5ef921ac8e8b6d207484f720b7ec40a32b33
Submitter: "Zuul (22348)"
Branch: stable/ussuri

commit 0c5a5ef921ac8e8b6d207484f720b7ec40a32b33
Author: Slawek Kaplonski <email address hidden>
Date: Fri Oct 8 15:04:10 2021 +0200

    [OVN Migration] Remove qr and dhcp ports from the nodes

    After migration from ML2/OVS to ML2/OVN tap ports which were created
    by the DHCP agents and router ports (like e.g. qr- or qg-) which were
    created by the L3 agents aren't needed at all and should be deleted.
    Previously those ports were set to be DOWN only. With this patch
    all such ports will be simply deleted from the openvswitch.

    Related-bug: #1946479
    Change-Id: I74cd5820389c86819c6884d3d61c9b2f7907cc88
    (cherry picked from commit 5092f3fb98b4c6bf8334de0e71548ddd073f6234)

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

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

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

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

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

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

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/+/815435

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

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

commit e91b0d07e3db9200914e2070fb056725494d924f
Author: Slawek Kaplonski <email address hidden>
Date: Fri Oct 8 15:06:19 2021 +0200

    [OVN Migration] Remove trunk's subports from the nodes

    After migration from ML2/OVS to ML2/OVN trunk subports which were created
    by the neutron-ovs-agent to connect br-int with trunk bridges (tbr-)
    aren't needed anymore and should be deleted.

    Closes-Bug: #1946479
    Change-Id: Ib1e3b78597ebdde1aa9d2b242e2005a05a7db89f
    (cherry picked from commit bf6831e0d2ad33ff751f06d568abe6cedff4d717)

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

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

commit 53ca3851bfba762d039cb689dbb0c4a8c47f1ec9
Author: Slawek Kaplonski <email address hidden>
Date: Fri Oct 8 15:06:19 2021 +0200

    [OVN Migration] Remove trunk's subports from the nodes

    After migration from ML2/OVS to ML2/OVN trunk subports which were created
    by the neutron-ovs-agent to connect br-int with trunk bridges (tbr-)
    aren't needed anymore and should be deleted.

    Closes-Bug: #1946479
    Change-Id: Ib1e3b78597ebdde1aa9d2b242e2005a05a7db89f
    (cherry picked from commit bf6831e0d2ad33ff751f06d568abe6cedff4d717)

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

Reviewed: https://review.opendev.org/c/openstack/neutron/+/815434
Committed: https://opendev.org/openstack/neutron/commit/4adf4086586b653bd97d58bf0c11fc37b5b6fa1e
Submitter: "Zuul (22348)"
Branch: stable/victoria

commit 4adf4086586b653bd97d58bf0c11fc37b5b6fa1e
Author: Slawek Kaplonski <email address hidden>
Date: Fri Oct 8 15:06:19 2021 +0200

    [OVN Migration] Remove trunk's subports from the nodes

    After migration from ML2/OVS to ML2/OVN trunk subports which were created
    by the neutron-ovs-agent to connect br-int with trunk bridges (tbr-)
    aren't needed anymore and should be deleted.

    Closes-Bug: #1946479
    Change-Id: Ib1e3b78597ebdde1aa9d2b242e2005a05a7db89f
    (cherry picked from commit bf6831e0d2ad33ff751f06d568abe6cedff4d717)

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

Reviewed: https://review.opendev.org/c/openstack/neutron/+/815435
Committed: https://opendev.org/openstack/neutron/commit/753be029a9d5714f98c7f84fbde06b82a0fd6fb3
Submitter: "Zuul (22348)"
Branch: stable/ussuri

commit 753be029a9d5714f98c7f84fbde06b82a0fd6fb3
Author: Slawek Kaplonski <email address hidden>
Date: Fri Oct 8 15:06:19 2021 +0200

    [OVN Migration] Remove trunk's subports from the nodes

    After migration from ML2/OVS to ML2/OVN trunk subports which were created
    by the neutron-ovs-agent to connect br-int with trunk bridges (tbr-)
    aren't needed anymore and should be deleted.

    Closes-Bug: #1946479
    Change-Id: Ib1e3b78597ebdde1aa9d2b242e2005a05a7db89f
    (cherry picked from commit bf6831e0d2ad33ff751f06d568abe6cedff4d717)

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

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

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

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

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

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

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

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

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

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

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers