IPv6 ip port mappings need to be enclosed on [ ]

Bug #2057471 reported by Fernando Royo
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ovsdbapp
Fix Released
High
Terry Wilson

Bug Description

API methods to manage the add/delete of ip port mappings to OVN LB ip_port_mapping field (lb_add_ip_port_mapping/lb_del_ip_port_mapping) work fine for IPv4 but for IPv6 new items need to be added enclosed as [IPv6]:PORT (use square brackets for IPv6).

Current code is parsing IPv6 format [1] and [2], so first solution about pass already as string including square brackets was discarded because netaddr library will except with a not valid format.

[1] https://github.com/openstack/ovsdbapp/blob/7a242c6ed8618052b65e54306a049c9667ca893c/ovsdbapp/schema/ovn_northbound/commands.py#L1679
[2] https://github.com/openstack/ovsdbapp/blob/7a242c6ed8618052b65e54306a049c9667ca893c/ovsdbapp/schema/ovn_northbound/commands.py#L1695

Changed in ovsdbapp:
importance: Undecided → High
assignee: nobody → Terry Wilson (otherwiseguy)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ovsdbapp (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/ovsdbapp/+/914414

Changed in ovsdbapp:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ovsdbapp (master)

Reviewed: https://review.opendev.org/c/openstack/ovsdbapp/+/914414
Committed: https://opendev.org/openstack/ovsdbapp/commit/8d206dd63d7bbf9ed23c8368f4692c59c0fdab99
Submitter: "Zuul (22348)"
Branch: master

commit 8d206dd63d7bbf9ed23c8368f4692c59c0fdab99
Author: Terry Wilson <email address hidden>
Date: Tue Mar 26 17:12:25 2024 -0500

    Handle IPv6 addresses for LB IP port mappings

    The ovn-nb manpage states that for v6 addresses, the IPs need to
    be enclosed in brackets for Load_Balancer.ip_port_mappings.

    Closes-Bug: #2057471

    Signed-off-by: Terry Wilson <email address hidden>
    Change-Id: If93037f35723c56d3da635e1a7aa42c092c8e7d6

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

Fix proposed to branch: stable/2024.1
Review: https://review.opendev.org/c/openstack/ovsdbapp/+/915143

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

Fix proposed to branch: stable/2023.2
Review: https://review.opendev.org/c/openstack/ovsdbapp/+/915144

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

Fix proposed to branch: stable/2023.1
Review: https://review.opendev.org/c/openstack/ovsdbapp/+/915145

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on ovsdbapp (stable/zed)

Change abandoned by "Elod Illes <email address hidden>" on branch: stable/zed
Review: https://review.opendev.org/c/openstack/ovsdbapp/+/915132
Reason: stable/zed branch of openstack/ovsdbapp is about to be deleted. To be able to do that, all open patches need to be abandoned. Please cherry pick the patch to unmaintained/zed if you want to further work on this patch.

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.