Remove binding:profile modification from Mellanox ML2 driver

Bug #1360762 reported by Irena Berezovsky
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
Irena Berezovsky

Bug Description

Mellanox networking solution requires knowledge of physical network during VIF plugging in nova.
Temporary work-around for Icehouse release was to use binding:profile to pass it.
During Juno release, patch to use binding:vif_details was added both on neutron and nova code.
vif _details dictionary on port:binding is used to pass physical_network information to nova and nova code gets physical network from vif_details.
It is about time to remove the work-around from the neutron code.

Tags: mellanox ml2
Changed in neutron:
assignee: nobody → Irena Berezovsky (irenab)
Itzik Brown (itzikb1)
Changed in neutron:
status: New → Confirmed
tags: added: mellanox ml2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

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

Changed in neutron:
status: Confirmed → In Progress
Kyle Mestery (mestery)
Changed in neutron:
milestone: none → juno-3
importance: Undecided → Low
Thierry Carrez (ttx)
Changed in neutron:
milestone: juno-3 → juno-rc1
Revision history for this message
Shiv Haris (shh) wrote :

irenab: i guess this is all taken care of. please confirm.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/116612
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=a8b13e0ecd67ebf07fc865f9faf07a279e577a55
Submitter: Jenkins
Branch: master

commit a8b13e0ecd67ebf07fc865f9faf07a279e577a55
Author: Irena Berezovsky <email address hidden>
Date: Mon Aug 25 13:56:38 2014 +0300

    Remove binding:profile update from Mellanox ML2 MD

    Remove temporary work-around that populates physical_network in
    binding:profile. Now when both neutron and nova use binding:vif_details
    to pass attributes to plug neutron port, this work-around is not nequired.
    Remove related temporary code from ML2 plugin.

    Change-Id: Ib37203af396f31080b1b27b500a571c20917f5a4
    Closes-Bug: 1360762

Changed in neutron:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in neutron:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: juno-rc1 → 2014.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.