Skip all Neutron DHCP Agent ports in networking-ovn

Bug #1848521 reported by Daniel Alvarez on 2019-10-17
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
networking-ovn
Undecided
Daniel Alvarez

Bug Description

When creating OVN localports (for Metadata), we need to skip those ports created by Neutron DHCP Agent (if any):

    It looks like sometimes, the device_id for ports created
    by Neutron DHCP Agent can be in the form of:

    - dhcp-$hostuuid-$networkid
    - 'reserved_dhcp_port' (DEVICE_ID_RESERVED_DHCP_PORT)

    Current code is only taking the first form into account when
    skipping Neutron DHCP Agent ports.

We need to take both into account or we may have broken metadata service in setups where Neutron DHCP Agent is used for other purposes (baremetal provisioning for example).

Changed in networking-ovn:
assignee: nobody → Daniel Alvarez (dalvarezs)

Fix proposed to branch: master
Review: https://review.opendev.org/689174

Changed in networking-ovn:
status: New → In Progress

Reviewed: https://review.opendev.org/689174
Committed: https://git.openstack.org/cgit/openstack/networking-ovn/commit/?id=15bf8f265b14c3a1e2c89a17a77cee69cad22b9d
Submitter: Zuul
Branch: master

commit 15bf8f265b14c3a1e2c89a17a77cee69cad22b9d
Author: Daniel Alvarez <email address hidden>
Date: Thu Oct 17 16:18:17 2019 +0200

    Exclude all device_ids that belong to Neutron DHCP Agent

    It looks like sometimes, the device_id for ports created
    by Neutron DHCP Agent can be in the form of:

    - dhcp-$hostuuid-$networkid
    - 'reserved_dhcp_port' (DEVICE_ID_RESERVED_DHCP_PORT)

    Current code is only taking the first form into account when
    skipping Neutron DHCP Agent ports. This patch is changing it
    to include both forms.

    Closes-Bug: #1848521
    Co-Authored-By: Lucas Alvares Gomes <email address hidden>
    Change-Id: Ifbfc551ac68dcc5d3d39a155f7642f2f2d9272c4
    Signed-off-by: Daniel Alvarez <email address hidden>

Changed in networking-ovn:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers