List port by mac address is case sensitive

Bug #1843428 reported by Martin Midolesov on 2019-09-10
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Medium
Brian Haley

Bug Description

Searching for a MAC address in neutron should not be case-sensitive.
For example in case of executing openstack port list --mac-address A:B:C should show results for a:b:c. This is found in queens release.

Bernard Cafarelli (bcafarel) wrote :

I need to dig into where this is actually handled, but case-insensitive search work here:
(devstack master)
$ openstack port list --mac-address FA:16:3e:67:9b:4F -f value -c mac_address
fa:16:3e:67:9b:4f
(RHOSP 13, so queens release)
$ openstack port list --mac-address FA:16:3e:b4:A7:c3 -f value -c id
f9c080ef-be07-4ff4-b3ee-d866b0029a43
Sorry I do not have a vanilla queens env around to confirm that last bit

So this was probably fixed at some point and you may just be missing a backport

No I already fixed that. I am supposed to open a bug. I will upload fix shortly.

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

Changed in neutron:
assignee: nobody → Martin Midolesov (martinmidolesov)
status: New → In Progress
Miguel Lavalle (minsel) on 2019-09-15
Changed in neutron:
importance: Undecided → Medium
Changed in neutron:
assignee: Martin Midolesov (martinmidolesov) → Brian Haley (brian-haley)

Reviewed: https://review.opendev.org/681390
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=ffe5943e657ef32bf4c509a8965cf03a4703be1d
Submitter: Zuul
Branch: master

commit ffe5943e657ef32bf4c509a8965cf03a4703be1d
Author: mmidolesov <email address hidden>
Date: Wed Sep 11 00:09:45 2019 -0700

    Make port list by mac case insesitive

    This fix removes the case sensitivity when performing
    port list by mac address criteria.

    Closes-Bug: #1843428
    Change-Id: I7ab6934ea333467d2efc5da0471b1af82ebb44b8

Changed in neutron:
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