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

[Slawek Kaplonski]

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

[Slawek Kaplonski (slaweq)]

Related patches https://review.opendev.org/c/openstack/puppet-ovn/+/779713 and https://review.opendev.org/c/openstack/tripleo-heat-templates/+/779714

[Slawek Kaplonski (slaweq)]

For puppet-ovn valid patch is https://review.opendev.org/c/openstack/puppet-ovn/+/779421

[Takashi Kajinami (kajinamit)]

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

[OpenStack Infra (hudson-openstack)]

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)

[OpenStack Infra (hudson-openstack)]

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)

[OpenStack Infra (hudson-openstack)]

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)