Allow multiple IPv6 ports on router from same network on ml2/ovs+vxlan+dvr

Bug #2002800 reported by Fernando Royo
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Critical
Fernando Royo

Bug Description

On a recent change [1], some additional checks was added to avoid ports overlapping cidrs on a router. On this change was also added a check to do not attached more than one port IPv6 from same network, but this check need to allow multiple ports when a deployment is done using ml2/ovs+xvlan+dvr, because two ports are added:

- one with device_owner as network:router_interface_distributed
- another one with device_owner as network:router_centralized_snat

[1] https://review.opendev.org/c/openstack/neutron/+/859143

Changed in neutron:
assignee: nobody → Fernando Royo (froyoredhat)
description: updated
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/+/870079

Changed in neutron:
status: New → In Progress
Revision history for this message
LIU Yulong (dragon889) wrote :

Seems a bug which breaks the exsiting feature.

Changed in neutron:
importance: Undecided → Critical
Revision history for this message
LIU Yulong (dragon889) wrote :

And [1] had spread to many stable branches.
[1] https://review.opendev.org/c/openstack/neutron/+/859143

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

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

commit e2097157bba6f18c4e30069ad76f9e1409ea1c0a
Author: Fernando Royo <email address hidden>
Date: Fri Jan 13 12:22:08 2023 +0100

    Allow multiple IPv6 ports on router from same network ml2/ovs+dvr

    On a recent change [1], some additional checks was added to avoid
    ports overlapping cidrs on a router. On this change was also added
    a check to do not attach more than one port IPv6 from same network,
    but this check need to allow multiple ports when a deployment is
    done using ml2/ovs+xvlan+dvr and the router has an external gateway
    configured, because two ports are added:

    - one with device_owner as network:router_interface_distributed
    - another one with device_owner as network:router_centralized_snat

    Also an small improvement is done for just run this check over the
    existing ports on the router for the same network_id.

    [1] https://review.opendev.org/c/openstack/neutron/+/859143

    Closes-Bug: #2002800
    Change-Id: I765a7b41e7e84f42a3180dfd15e3a41a8e085284

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

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

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

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

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

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

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

Reviewed: https://review.opendev.org/c/openstack/neutron/+/870657
Committed: https://opendev.org/openstack/neutron/commit/6226fa9fd63538c698e6d41a01bc39c3da94ba6f
Submitter: "Zuul (22348)"
Branch: stable/zed

commit 6226fa9fd63538c698e6d41a01bc39c3da94ba6f
Author: Fernando Royo <email address hidden>
Date: Fri Jan 13 12:22:08 2023 +0100

    Allow multiple IPv6 ports on router from same network ml2/ovs+dvr

    On a recent change [1], some additional checks was added to avoid
    ports overlapping cidrs on a router. On this change was also added
    a check to do not attach more than one port IPv6 from same network,
    but this check need to allow multiple ports when a deployment is
    done using ml2/ovs+xvlan+dvr and the router has an external gateway
    configured, because two ports are added:

    - one with device_owner as network:router_interface_distributed
    - another one with device_owner as network:router_centralized_snat

    Also an small improvement is done for just run this check over the
    existing ports on the router for the same network_id.

    [1] https://review.opendev.org/c/openstack/neutron/+/859143

    Closes-Bug: #2002800
    Change-Id: I765a7b41e7e84f42a3180dfd15e3a41a8e085284
    (cherry picked from commit e2097157bba6f18c4e30069ad76f9e1409ea1c0a)

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

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

commit ddb40f9d6df5d549004d0a314961aa9fbc729965
Author: Fernando Royo <email address hidden>
Date: Fri Jan 13 12:22:08 2023 +0100

    Allow multiple IPv6 ports on router from same network ml2/ovs+dvr

    On a recent change [1], some additional checks was added to avoid
    ports overlapping cidrs on a router. On this change was also added
    a check to do not attach more than one port IPv6 from same network,
    but this check need to allow multiple ports when a deployment is
    done using ml2/ovs+xvlan+dvr and the router has an external gateway
    configured, because two ports are added:

    - one with device_owner as network:router_interface_distributed
    - another one with device_owner as network:router_centralized_snat

    Also an small improvement is done for just run this check over the
    existing ports on the router for the same network_id.

    [1] https://review.opendev.org/c/openstack/neutron/+/859143

    Closes-Bug: #2002800
    Change-Id: I765a7b41e7e84f42a3180dfd15e3a41a8e085284
    (cherry picked from commit e2097157bba6f18c4e30069ad76f9e1409ea1c0a)

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

Reviewed: https://review.opendev.org/c/openstack/neutron/+/870658
Committed: https://opendev.org/openstack/neutron/commit/97e86e51aee0573dbeea31fb648ac3c31132c507
Submitter: "Zuul (22348)"
Branch: stable/yoga

commit 97e86e51aee0573dbeea31fb648ac3c31132c507
Author: Fernando Royo <email address hidden>
Date: Fri Jan 13 12:22:08 2023 +0100

    Allow multiple IPv6 ports on router from same network ml2/ovs+dvr

    On a recent change [1], some additional checks was added to avoid
    ports overlapping cidrs on a router. On this change was also added
    a check to do not attach more than one port IPv6 from same network,
    but this check need to allow multiple ports when a deployment is
    done using ml2/ovs+xvlan+dvr and the router has an external gateway
    configured, because two ports are added:

    - one with device_owner as network:router_interface_distributed
    - another one with device_owner as network:router_centralized_snat

    Also an small improvement is done for just run this check over the
    existing ports on the router for the same network_id.

    [1] https://review.opendev.org/c/openstack/neutron/+/859143

    Closes-Bug: #2002800
    Change-Id: I765a7b41e7e84f42a3180dfd15e3a41a8e085284
    (cherry picked from commit e2097157bba6f18c4e30069ad76f9e1409ea1c0a)

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

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

commit 7574e153c08bf97d3d39cc4341ec6a37ff2d760b
Author: Fernando Royo <email address hidden>
Date: Fri Jan 13 12:22:08 2023 +0100

    Allow multiple IPv6 ports on router from same network ml2/ovs+dvr

    On a recent change [1], some additional checks was added to avoid
    ports overlapping cidrs on a router. On this change was also added
    a check to do not attach more than one port IPv6 from same network,
    but this check need to allow multiple ports when a deployment is
    done using ml2/ovs+xvlan+dvr and the router has an external gateway
    configured, because two ports are added:

    - one with device_owner as network:router_interface_distributed
    - another one with device_owner as network:router_centralized_snat

    Also an small improvement is done for just run this check over the
    existing ports on the router for the same network_id.

    [1] https://review.opendev.org/c/openstack/neutron/+/859143

    Closes-Bug: #2002800
    Change-Id: I765a7b41e7e84f42a3180dfd15e3a41a8e085284
    (cherry picked from commit e2097157bba6f18c4e30069ad76f9e1409ea1c0a)

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

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

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

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

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

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

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

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

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

This issue was fixed in the openstack/neutron wallaby-eom release.

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

Other bug subscribers

Remote bug watches

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