yahoo-eng-team team mailing list archive

[OpenStack Infra <1926273@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.opendev.org/c/openstack/neutron-lib/+/788300
Committed: https://opendev.org/openstack/neutron-lib/commit/9d3cac1d69a143c0a44230144760dd86c52d6ac7
Submitter: "Zuul (22348)"
Branch:    master

commit 9d3cac1d69a143c0a44230144760dd86c52d6ac7
Author: Rodolfo Alonso Hernandez <ralonsoh@xxxxxxxxxx>
Date:   Tue Apr 27 13:43:31 2021 +0000

    Modify "port.mac_address" API definition
    
    Added a convertion method in the "port.mac_address" parameter. This
    convertion will return a sanitized MAC address formatted as:
      xx:xx:xx:xx:xx:xx
    
    This patch is not creating a new extension (that should be the
    regular process):
    - To avoid the Neutron extension explosion.
    - Because this method will not validate the MAC address but
      sanitize it, if valid. Other services or projects will expect
      a MAC address with the defined format.
    - This sanitization won't change the MAC address value.
    - This sanitization will be always enabled (no extension needed).
    
    Closes-Bug: #1926273
    
    Change-Id: I95d596c84716835076365981376fb23166c6b23c


** Changed in: neutron
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1926273

Title:
  Port can be created with an invalid MAC address

Status in neutron:
  Fix Released

Bug description:
  For example:

    $ openstack port create --network private --mac-address \
        11:22:33:4:5:66 port200
    ...
    | mac_address             | 11:22:33:4:5:66
    ...

  The ML2 plugin should check the validity of the MAC address provided.

  Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1949632

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1926273/+subscriptions