Integrate nested resource providers in allocation candidates

Registered by Jay Pipes

We need to account for nested resource providers in the calculation of allocation requests returned by the GET /allocation_candidates HTTP endpoint

Blueprint information

Status:
Complete
Approver:
Alex Xu
Priority:
High
Drafter:
Jay Pipes
Direction:
Approved
Assignee:
Tetsuro Nakamura
Definition:
Approved
Series goal:
Accepted for rocky
Implementation:
Implemented
Milestone target:
milestone icon rocky-3
Started by
Jay Pipes
Completed by
Matt Riedemann

Related branches

Sprints

Whiteboard

Gerrit topic: https://review.openstack.org/#q,topic:bp/nested-resource-providers-allocation-candidates,n,z

Addressed by: https://review.openstack.org/531899
    tests for alloc candidates with nested and traits

Addressed by: https://review.openstack.org/554529
    placement: resource requests for nested providers

Addressed by: https://review.openstack.org/556873
    Handle nested providers for allocation candidates

Addressed by: https://review.openstack.org/533195
    Fix comments in get_all_with_shared()

Addressed by: https://review.openstack.org/553122
    remove unnecessary short cut in placement

Addressed by: https://review.openstack.org/533396
    Fix allocation_candidates not to ignore shared RPs

Addressed by: https://review.openstack.org/556450
    Consider nested RPs in get_all_with_shared

Addressed by: https://review.openstack.org/556514
    Support shared and nested allocation candidates

Addressed by: https://review.openstack.org/559480
    Return all nested providers in tree

Addressed by: https://review.openstack.org/558045
    Return all resources in provider_summaries

Addressed by: https://review.openstack.org/559554
    Return resource providers without inventories

Addressed by: https://review.openstack.org/559555
    Consult the dictionary for root provider ids

Addressed by: https://review.openstack.org/559638
    Improve getting nested providers with capacity

Gerrit topic: https://review.openstack.org/#q,topic:bug/1760276,n,z

Addressed by: https://review.openstack.org/561842
    Support nested alloc cands without shared RPs

Approved for Rocky. -- mriedem 20180419

Gerrit topic: https://review.openstack.org/#q,topic:bp/placement-return-all-resources,n,z

Addressed by: https://review.openstack.org/567113
    Return nested providers in get_by_request

Gerrit topic: https://review.openstack.org/#q,topic:bp/nested-resource-providers-allocation-candidate,n,z

Addressed by: https://review.openstack.org/567150
    Add traits check in nested provider cadidates

Addressed by: https://review.openstack.org/567151
    Fix broken allocation candidates with traits

Addressed by: https://review.openstack.org/567508
    Support nested alloc cands with sharing providers

Addressed by: https://review.openstack.org/568737
    Add tests for _get_trees_matching_all() function

Gerrit topic: https://review.openstack.org/#q,topic:bug/1772243,n,z

Addressed by: https://review.openstack.org/569720
    Use list instead of set for duplicate check

Addressed by: https://review.openstack.org/572673
    Fix nits in nested provider allocation candidates

Gerrit topic: https://review.openstack.org/#q,topic:placement-return-all-resources,n,z

Addressed by: https://review.openstack.org/565487
    Add microversion for nested allocation candidate

Gerrit topic: https://review.openstack.org/#q,topic:nested-resource-providers-allocation-candidates,n,z

Addressed by: https://review.openstack.org/576609
    Squash: exclude nested alloc cands in older microversions

Addressed by: https://review.openstack.org/573604
    Fix nits in placement-return-all-resources series

Marking this complete for Rocky. It sounds like there still needs to be some testing done to make sure that requests < 1.29 have not regressed, but those would be bugs if we did regress something so I don't see the need to hold the blueprint open for that testing. -- mriedem 20180629

Addressed by: https://review.openstack.org/589407
    [placement] api-ref: add description for 1.29

Addressed by: https://review.openstack.org/579577
    api-ref: fix min_version for parent_provider_uuid in responses

(?)

Work Items

Dependency tree

* Blueprints in grey have been implemented.

This blueprint contains Public information 
Everyone can see this information.