VMware drivers are no longer passing quantum port-id to vCenter

Bug #1183452 reported by dan wendlandt
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Critical
dan wendlandt
Grizzly
Fix Released
Critical
Yaguang Tang
VMwareAPI-Team
Fix Released
Critical
Unassigned

Bug Description

to work with quantuam, the vCenter driver passes an iface-id to vCenter.
I need to investigate this more. Its a one-line work around, but I need to do some more investigation to figure out what the "right" way to do the fix is.

Tags: vmware
dan wendlandt (danwent)
Changed in nova:
assignee: nobody → dan wendlandt (danwent)
tags: added: vmware
Revision history for this message
dan wendlandt (danwent) wrote :

actually, the fix is quite simple. commit 70c659059b8fbef811ce79700aecb01c60242ebd broke this, but all we have to do is change one line to pass in vif['id'] rather than vif.get_meta('iface_id') which is never populated with anything useful.

tags: added: grizzly-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

Changed in nova:
status: New → In Progress
Revision history for this message
dan wendlandt (danwent) wrote :

please set this to priority 'high'

Changed in nova:
importance: Undecided → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/30545
Committed: http://github.com/openstack/nova/commit/c028e54f6be75d1cd335550cf5291ddf94f973e3
Submitter: Jenkins
Branch: master

commit c028e54f6be75d1cd335550cf5291ddf94f973e3
Author: Dan Wendlandt <email address hidden>
Date: Sat May 25 14:22:23 2013 -0700

    correctly set iface-id in vmware driver

    bug 1183452

    Commit 70c659059b8fbef811ce79700aecb01c60242ebd updated the vmware
    driver to use the new vif model, but was incorrect in how it tried to
    grab the iface-id and set it in the vmx file spec. This patch fixes
    that issue and adds a check for this in the existing unit tests.

    Change-Id: I756539871d1844a828f9a0a295fc9fa3e59610f7

Changed in nova:
status: In Progress → Fix Committed
Revision history for this message
Sina Sadeghi (sina-sa) wrote :

Hi Dan,

I can confirm that the patch fixes the issue for us.

Thanks very much.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/grizzly)

Fix proposed to branch: stable/grizzly
Review: https://review.openstack.org/33145

Changed in nova:
importance: High → Critical
milestone: none → havana-2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/grizzly)

Reviewed: https://review.openstack.org/33145
Committed: http://github.com/openstack/nova/commit/4bb6263e91e27f6023ff05cf718a648d340a4143
Submitter: Jenkins
Branch: stable/grizzly

commit 4bb6263e91e27f6023ff05cf718a648d340a4143
Author: Dan Wendlandt <email address hidden>
Date: Sat May 25 14:22:23 2013 -0700

    correctly set iface-id in vmware driver

    bug 1183452

    Commit 70c659059b8fbef811ce79700aecb01c60242ebd updated the vmware
    driver to use the new vif model, but was incorrect in how it tried to
    grab the iface-id and set it in the vmx file spec. This patch fixes
    that issue and adds a check for this in the existing unit tests.

    Change-Id: I756539871d1844a828f9a0a295fc9fa3e59610f7
    (cherry picked from commit c028e54f6be75d1cd335550cf5291ddf94f973e3)

tags: added: in-stable-grizzly
Thierry Carrez (ttx)
Changed in nova:
status: Fix Committed → Fix Released
Alan Pevec (apevec)
tags: removed: grizzly-backport-potential in-stable-grizzly
Tracy Jones (tjones-i)
tags: added: vmware-co-preferred
tags: removed: vmware-co-preferred
Tracy Jones (tjones-i)
Changed in openstack-vmwareapi-team:
status: New → Fix Released
Tracy Jones (tjones-i)
Changed in openstack-vmwareapi-team:
importance: Undecided → Critical
Thierry Carrez (ttx)
Changed in nova:
milestone: havana-2 → 2013.2
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.