update mac address for neutron + contrail doesn't work

Bug #1712359 reported by Vasyl Saienko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Medium
Vasyl Saienko

Bug Description

Contrail update mac address only when vnic_type is baremetal or port doesn't have host_id [0].
On Ironic side we do not update neither host_id/device_owner or vnic_type when update mac_address on port [1].

[0] https://github.com/Juniper/contrail-controller/blob/e754a997d7d75ba205e17610ff566b649fc1d0bc/src/config/vnc_openstack/vnc_openstack/neutron_plugin_db.py#L2128

[1] https://github.com/openstack/ironic/blob/94c1cabf09ebe8da3feb7b15611b9c9ea8d34960/ironic/common/neutron.py#L124-L142

Vasyl Saienko (vsaienko)
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (master)

Fix proposed to branch: master
Review: https://review.openstack.org/496249

Changed in ironic:
assignee: nobody → Vasyl Saienko (vsaienko)
status: New → In Progress
Vladyslav Drok (vdrok)
Changed in ironic:
importance: Undecided → Medium
Vasyl Saienko (vsaienko)
description: updated
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.openstack.org/496249
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=e35f230e1a9374cebd6aa43761da2a49e7d5c204
Submitter: Jenkins
Branch: master

commit e35f230e1a9374cebd6aa43761da2a49e7d5c204
Author: Vasyl Saienko <email address hidden>
Date: Tue Aug 22 17:30:36 2017 +0300

    Rework update_port_address logic

    Some neutron backends like contrail require port is unbound before one
    can change its mac_address or vnic_type is set to baremeteal.
    Rework update_port_address for Neutron case to use separate steps for:
      1. Unbind port
      2. Update mac address
      3. Bind port

    Change-Id: I9938fa565eef7e0f1403f6a7c55922115128158c
    Closes-Bug: #1712359

Changed in ironic:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to ironic (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/506314

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to ironic (master)

Reviewed: https://review.openstack.org/506314
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=a1c5ce9f6b1f4f13cd953a9f447018e740b01e4c
Submitter: Jenkins
Branch: master

commit a1c5ce9f6b1f4f13cd953a9f447018e740b01e4c
Author: Ruby Loo <email address hidden>
Date: Thu Sep 21 13:05:50 2017 -0400

    [reno] update for MAC address update fix

    This updates the release note for the MAC-address-update fix
    from e35f230e1a9374cebd6aa43761da2a49e7d5c204.

    Change-Id: I539a43257e63715074fce07496e795a6bf5a6953
    Related-Bug: #1712359

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ironic 9.2.0

This issue was fixed in the openstack/ironic 9.2.0 release.

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.