Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 2020317

Summary:	[rfe] store mac_addresses in lower case in neutron database
Product:	Red Hat OpenStack	Reporter:	David Hill <dhill>
Component:	openstack-neutron	Assignee:	Rodolfo Alonso <ralonsoh>
Status:	CLOSED DUPLICATE	QA Contact:	Eran Kuris <ekuris>
Severity:	low	Docs Contact:
Priority:	unspecified
Version:	16.1 (Train)	CC:	chrisw, ralonsoh, scohen
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	x86_64
OS:	All
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2021-11-15 10:20:56 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description David Hill 2021-11-04 15:20:10 UTC

Description of problem:
[rfe] store mac_addresses in lower case in neutron database

it doesn't look like we do case convertion on the macs and they end up being stored in uppercase in the database and nova does some sort of convertion to lower case (or neutron?) but not everywhere.   storing the mac in neutron in lowercase would solve this issue at the root 

Version-Release number of selected component (if applicable):
All

How reproducible:
Always

Steps to Reproduce:
1. Setting a mac_address in uppercase in a heat templates ends up storing the mac in uppercase in the database and nova live migration doesn't find a matching value because it was converted to lowercase at some point.  
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Rodolfo Alonso 2021-11-15 09:47:32 UTC

Hello:

This RFE has been handled in [1] OSP16.2. I'll backport the D/S patch submitted for this BZ.

The U/S feature [2] was solved by:
- [3] changing the n-lib API converter. Any user input will be correctly formatted to xx:xx:xx:xx:xx:xx.
- [4] reading and converting all stored MACs in the DB, and adding a check in the ML2 plugin when reading the existing MACs.

I'll mark this BZ as duplicated and I'll handle all backports in [1].

Regards.

[1]https://bugzilla.redhat.com/show_bug.cgi?id=1949632
[2]https://bugs.launchpad.net/neutron/+bug/1926273
[3]https://review.opendev.org/c/openstack/neutron-lib/+/788300
[4]https://review.opendev.org/c/openstack/neutron/+/789831

Comment 2 Rodolfo Alonso 2021-11-15 10:20:56 UTC


*** This bug has been marked as a duplicate of bug 2023242 ***