Comment 5 for bug 1732731

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

Reviewed: https://review.openstack.org/519601
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=042bc07fd87f3649929ab00429ba369f5378f07e
Submitter: Zuul
Branch: master

commit 042bc07fd87f3649929ab00429ba369f5378f07e
Author: Balazs Gibizer <email address hidden>
Date: Fri Dec 15 11:09:18 2017 +0100

    Test alloc_cands with indirectly sharing RPs

    There are two new test cases proposed in this patch about shared RPs.
    The generic setup in both cases are the following:

           CN1 (VCPU, MEMORY_MB)
         / \
        /agg1 \agg2
       / \
      SS1 ( SS2 (
       DISK_GB) IPV4_ADDRESS
                  SRIOV_NET_VF)

    In the first case we only request resources provided by SS1 and SS2
    but not CN1. In the second case we also request resources from CN1
    besides requesting resources from SS1 and SS2.

    The first case had been broken before nested providers are supported.
    But this is fixed now thanks to the following commit, which enables
    getting nested providers as well as sharing providers.
    patch: https://review.openstack.org/#/c/567508/
    commit: 068c56f55dffb60e7dd15b35001c280660c79229

    Closes-Bug: #1732731
    Change-Id: Iaf7993773b8f1f5cf3cb83c161b4c88d9368297b
    Co-Authored-By: Tetsuro Nakamura <email address hidden>