yahoo-eng-team team mailing list archive

[OpenStack Infra <1549068@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.openstack.org/284025
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=e147a63cb396ebaadd7c841a305cf3a52be1d373
Submitter: Jenkins
Branch:    master

commit e147a63cb396ebaadd7c841a305cf3a52be1d373
Author: Zhenguo Niu <niuzhenguo@xxxxxxxxxx>
Date:   Wed Feb 24 17:14:59 2016 +0800

    [Ironic]Match vif-pif mac address before setting 'vif_port_id'
    
    When booting an ironic instance with multi networks, the ironic port
    'vif_port_id' may mismatch with the corresponding neutron port.
    
    Closes-Bug: #1549068
    Co-Authored-By: Sivaramakrishna Garimella (sivaramakrishna.garimella@xxxxxx)
    
    Change-Id: Id5f033136283987eef8de4ce2f6be256e48cdbf8


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

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1549068

Title:
  Ironic vif_port_id mismatch with neutron port id if specified multi
  networks

Status in Ironic:
  Won't Fix
Status in OpenStack Compute (nova):
  Fix Released

Bug description:
  when booting an ironic instance with multi networks, we create the
  network resources with the macs from ironic node ports, and then set
  the neutron port id back to ironic port's extra/vif_port_id, but the
  current logic will lead the vif-pif ids pair mismatch.

  code:

  for vif, pif in zip(network_info, ports):
    port_id = six.text_type(vif['id'])
    patch = [{'op': 'add',
         'path': '/extra/vif_port_id',
         'value':  port_id}]
    self.ironicclient.call("port.update", pif.uuid, patch)

  we should check whether the mac addresses match between vif and pif
  before setting the 'vif_port_id'.

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