In case of Neutron with ML2/OVN vlan-limit in OVS should be set

Bug #1918418 reported by Slawek Kaplonski
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
In Progress
Undecided
Slawek Kaplonski

Bug Description

By default other_config:vlan-limit value is set to 1 in ovs and that is fine for ML2/OVS and for ML2/OVN when vlan_transparency is disabled.
But in case when vlan_transparency is enabled, to make vlan network working with double vlan tagging and security groups it is needed to set other_config:vlan-limit to 0 (unlimited number of vlans can be processed by OVS) or eventually 2 (2 vlans can be processed).
See BZ https://bugzilla.redhat.com/show_bug.cgi?id=1904141 and https://bugzilla.redhat.com/show_bug.cgi?id=1936475 for details

Revision history for this message
Slawek Kaplonski (slaweq) wrote :
Changed in tripleo:
assignee: nobody → Slawek Kaplonski (slaweq)
status: New → In Progress
Revision history for this message
Slawek Kaplonski (slaweq) wrote :
Revision history for this message
Takashi Kajinami (kajinamit) wrote :

After discussion with Daniel and Slawek, we agree that we should add support for vlan-limit option not to puppet-ovn but to puppet-vsiwtch.
I have submitted a patch to puppet-vswitch accordingly.

 https://review.opendev.org/c/openstack/puppet-vswitch/+/779796

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

Reviewed: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/783930
Committed: https://opendev.org/openstack/tripleo-heat-templates/commit/4260d30eae0b83789d0bdb9b027be7e88176e187
Submitter: "Zuul (22348)"
Branch: stable/victoria

commit 4260d30eae0b83789d0bdb9b027be7e88176e187
Author: Slawek Kaplonski <email address hidden>
Date: Wed Mar 10 10:53:16 2021 +0000

    Set vlan-limit value depending on vlan_transparent setting

    In case when vlan_transparent in Neutron is enabled,
    other_config:vlan-limit should be set to value "0" in the openvswitch on
    all nodes.

    Related-Bug: #1918418

    Depends-On: https://review.opendev.org/c/openstack/puppet-vswitch/+/783215
    Change-Id: Id6fc08bce5673a41fd9fa5cb27f41c9786f560da
    (cherry picked from commit e6801cebc81f25b6198cf29b4a5ab7308e16d5fd)

tags: added: in-stable-victoria
wes hayutin (weshayutin)
Changed in tripleo:
milestone: none → xena-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-heat-templates (stable/train)

Reviewed: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/783932
Committed: https://opendev.org/openstack/tripleo-heat-templates/commit/71fb10fe28a8d0419436c20069f515cba70d7e63
Submitter: "Zuul (22348)"
Branch: stable/train

commit 71fb10fe28a8d0419436c20069f515cba70d7e63
Author: Slawek Kaplonski <email address hidden>
Date: Wed Mar 10 10:53:16 2021 +0000

    Set vlan-limit value depending on vlan_transparent setting

    In case when vlan_transparent in Neutron is enabled,
    other_config:vlan-limit should be set to value "0" in the openvswitch on
    all nodes.

    Related-Bug: #1918418

    Depends-On: https://review.opendev.org/c/openstack/puppet-vswitch/+/783859
    Change-Id: Id6fc08bce5673a41fd9fa5cb27f41c9786f560da
    (cherry picked from commit e6801cebc81f25b6198cf29b4a5ab7308e16d5fd)

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

Reviewed: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/783931
Committed: https://opendev.org/openstack/tripleo-heat-templates/commit/73837b8b027eafad40f49f8d03b68ef51410ccee
Submitter: "Zuul (22348)"
Branch: stable/ussuri

commit 73837b8b027eafad40f49f8d03b68ef51410ccee
Author: Slawek Kaplonski <email address hidden>
Date: Wed Mar 10 10:53:16 2021 +0000

    Set vlan-limit value depending on vlan_transparent setting

    In case when vlan_transparent in Neutron is enabled,
    other_config:vlan-limit should be set to value "0" in the openvswitch on
    all nodes.

    Related-Bug: #1918418

    Conflicts:
        deployment/ovn/ovn-controller-container-puppet.yaml

    Depends-On: https://review.opendev.org/c/openstack/puppet-vswitch/+/783858
    Change-Id: Id6fc08bce5673a41fd9fa5cb27f41c9786f560da
    (cherry picked from commit e6801cebc81f25b6198cf29b4a5ab7308e16d5fd)

tags: added: in-stable-ussuri
Changed in tripleo:
milestone: xena-1 → xena-2
Changed in tripleo:
milestone: xena-2 → xena-3
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.