Failure on SR-IOV . Missing 'parent_addr

Bug #1535367 reported by Lenny
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
High
Vladik Romanovsky

Bug Description

Mellanox CI fails on SR-IOV hardware
1. Running nova from master commit ffa07781ab47baf096854cd6c22a3e433eab3f0d
2. Full logs http://144.76.193.39/ci-artifacts/269109/1/Nova-ML2-Sriov/
3. Reproduce:
     ./stack.sh
     neutron port-create --binding:vnic_type=direct private
     nova boot --flavor m1.small --image mellanox_eth --nic port-id=<port_id> vm1
4. port binding fails
     nova fails to find appropriate host

Tags: sr-iov
Changed in nova:
importance: Undecided → Critical
status: New → Confirmed
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/269191

Changed in nova:
assignee: nobody → Nikola Đipanov (ndipanov)
status: Confirmed → In Progress
Revision history for this message
Matt Riedemann (mriedem) wrote :

This only affects select virt drivers, so it's not critical.

Changed in nova:
importance: Critical → High
Revision history for this message
Yongli He (yongli-he) wrote :

pci-test also affected, and same fail reason, to prevent it feedback meaningless comment, it's been disabled.

Changed in nova:
assignee: Nikola Đipanov (ndipanov) → Vladik Romanovsky (vladik-romanovsky)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

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

commit abcb26c121c7643201079121ca98266079edb8b8
Author: Nikola Dipanov <email address hidden>
Date: Tue Jan 19 13:49:47 2016 +0000

    Make sure that we always have a parent_addr set

    Since enabling PF tracking in the following commit:

    50355c4595e08f293f610da32247e405b20c1c5b

    we expect all of the PciDevice objects to have a parent_addr field
    set since we use it for matching in case the (miss-)feature of allowing
    to specify a PF address to enable all VFs was used in the whitelist.

    This caused a lazy-load on the object which always fails, so this patch
    makes sure that 'parent_addr' field is set to None on the object even if
    it was not passed in by the driver.

    Closes-bug: #1535367
    Change-Id: I0ed8dd0702287859f7809a1a1a87dafe168d6e9b

Changed in nova:
status: In Progress → Fix Released
Revision history for this message
Lenny (lennyb) wrote :

We are still fail since PCIFilter returns 0 hosts.
http://144.76.193.39/ci-artifacts/269191/4/Nova-ML2-Sriov/

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.