neutron-ovn-db-sync-util still recreates Address Sets/ACLs even if Port Groups are supported

Bug #1798028 reported by Daniel Alvarez
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
networking-ovn
Fix Released
High
Daniel Alvarez

Bug Description

When sync'ing OVN database from Neutron one, the sync util script won't take into account if port groups are supported in the schema or not recreating Address Sets and ACLs using them.

The sync script should be fixed to detect if Port Groups are supported and in this case, create the resources accordingly. This is especially important when migrating from ML2/OVS to ML2/OVN clouds where this tool is used to create the initial NB database.

Changed in networking-ovn:
assignee: nobody → Daniel Alvarez (dalvarezs)
importance: Undecided → High
Changed in networking-ovn:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to networking-ovn (master)

Reviewed: https://review.openstack.org/599426
Committed: https://git.openstack.org/cgit/openstack/networking-ovn/commit/?id=ebd5b06f554d06d7e4b2fe78dca1eaf5cd007c86
Submitter: Zuul
Branch: master

commit ebd5b06f554d06d7e4b2fe78dca1eaf5cd007c86
Author: Daniel Alvarez <email address hidden>
Date: Mon Sep 3 15:45:11 2018 +0000

    Fix ovn db sync tool to support Port Groups

    If port groups are supported, the sync script will still create
    address sets and ACLs based on the old implementation.
    This patch is taking into account whether Port Groups are supported
    and create the right resources accordingly.

    Closes-Bug: #1798028

    Change-Id: I529d82b28be9bb93649f19034027c41ee1aff08b
    Signed-off-by: Daniel Alvarez <email address hidden>

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

Fix proposed to branch: stable/rocky
Review: https://review.openstack.org/611786

tags: added: networking-ovn-proactive-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to networking-ovn (stable/rocky)

Reviewed: https://review.openstack.org/611786
Committed: https://git.openstack.org/cgit/openstack/networking-ovn/commit/?id=6c2db89715e98a9f235c686a403ec8d274a512c4
Submitter: Zuul
Branch: stable/rocky

commit 6c2db89715e98a9f235c686a403ec8d274a512c4
Author: Daniel Alvarez <email address hidden>
Date: Mon Sep 3 15:45:11 2018 +0000

    Fix ovn db sync tool to support Port Groups

    If port groups are supported, the sync script will still create
    address sets and ACLs based on the old implementation.
    This patch is taking into account whether Port Groups are supported
    and create the right resources accordingly.

    Closes-Bug: #1798028

    Change-Id: I529d82b28be9bb93649f19034027c41ee1aff08b
    Signed-off-by: Daniel Alvarez <email address hidden>
    (cherry picked from commit ebd5b06f554d06d7e4b2fe78dca1eaf5cd007c86)

tags: added: in-stable-rocky
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/networking-ovn 6.0.0.0b1

This issue was fixed in the openstack/networking-ovn 6.0.0.0b1 development milestone.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to networking-ovn (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.opendev.org/681566

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to networking-ovn (stable/queens)

Reviewed: https://review.opendev.org/681566
Committed: https://git.openstack.org/cgit/openstack/networking-ovn/commit/?id=8181ca0dd12c52b968af580210a68cbe96ad15d9
Submitter: Zuul
Branch: stable/queens

commit 8181ca0dd12c52b968af580210a68cbe96ad15d9
Author: Daniel Alvarez <email address hidden>
Date: Mon Sep 3 15:45:11 2018 +0000

    Fix ovn db sync tool to support Port Groups

    If port groups are supported, the sync script will still create
    address sets and ACLs based on the old implementation.
    This patch is taking into account whether Port Groups are supported
    and create the right resources accordingly.

    Closes-Bug: #1798028

    Change-Id: I529d82b28be9bb93649f19034027c41ee1aff08b
    Signed-off-by: Daniel Alvarez <email address hidden>
    (cherry picked from commit ebd5b06f554d06d7e4b2fe78dca1eaf5cd007c86)

tags: added: in-stable-queens
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/networking-ovn 4.0.4

This issue was fixed in the openstack/networking-ovn 4.0.4 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/networking-ovn 5.1.0

This issue was fixed in the openstack/networking-ovn 5.1.0 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.