Support server move operations with ports having resource request

Registered by Balazs Gibizer on 2019-03-20

The Nova REST API microversion 2.72, added in Stein, supports creating server with ports having resource request. However nova rejects moving such servers. This bp aims to add support for the following operations on such servers:
* resize
* cold migrate
* live migrate
* evacuate
* unshelve after shelve offload

Blueprint information

Status:
Complete
Approver:
Eric Fried
Priority:
Medium
Drafter:
Balazs Gibizer
Direction:
Approved
Assignee:
Balazs Gibizer
Definition:
Approved
Series goal:
Accepted for train
Implementation:
Implemented
Milestone target:
milestone icon train-3
Started by
Balazs Gibizer on 2019-05-30
Completed by
Matt Riedemann on 2019-09-14

Related branches

Sprints

Whiteboard

Gerrit topic: https://review.openstack.org/#/q/topic:bp/support-server-move-operations-with-ports-having-resource-request

Addressed by: https://review.openstack.org/652608
    Server move operations with ports having resource request

Gerrit topic: https://review.opendev.org/#/q/topic:support-server-move-operations-with-ports-having-resource-request

Addressed by: https://review.opendev.org/655107
    Remove unused param from _fill_provider_mapping

Addressed by: https://review.opendev.org/655108
    Move _fill_provider_mapping to the scheduler_utils

Addressed by: https://review.opendev.org/655109
    prepare func test env for moving servers with bandwidth

Addressed by: https://review.opendev.org/655110
    allow getting resource request of every bound ports of an instance

Addressed by: https://review.opendev.org/655111
    Pass network API to the conducor's MigrationTask

Addressed by: https://review.opendev.org/655112
    handle port allocation during migration

Addressed by: https://review.opendev.org/655113
    func test for migrate server with ports having resource request

Addressed by: https://review.opendev.org/655114
    Extend NeutronFixture to handle migrations

Gerrit topic: https://review.opendev.org/#/q/topic:bp/support-server-move-operations-with-ports-having-resource-request

Addressed by: https://review.opendev.org/652608
    Server move operations with ports having resource request

Addressed by: https://review.opendev.org/655721
    Add request_spec to server move RPC calls

Addressed by: https://review.opendev.org/656422
    update allocation in binding profile during migrate

Gerrit topic: https://review.opendev.org/#/q/topic:bp/support-move-ops-with-qos-ports

Addressed by: https://review.opendev.org/671497
    Allow migrating server with port resource request

Addressed by: https://review.opendev.org/676138
    Make _rever_allocation nested allocation aware

Addressed by: https://review.opendev.org/676140
    Func test for revert migration with bandwidth

Addressed by: https://review.opendev.org/676972
    Func test for migrate re-schedule with bandwidth

Addressed by: https://review.opendev.org/676980
    Support migrating SRIOV port with bandwidth

Addressed by: https://review.opendev.org/678827
    Do not query allocations twice in finish_revert_resize

Addressed by: https://review.opendev.org/679019
    Allow resizing server with port resource request

Addressed by: https://review.opendev.org/678841
    Rename the nova-grenade-live-migration job to nova-grenade-multinode

Addressed by: https://review.opendev.org/679210
    Add cold migrate and resize to nova-grenade-multinode

Addressed by: https://review.opendev.org/679629
    Nice to have test coverage for If1f465112b8e9b0304b8b5b864b985f72168d839

Addressed by: https://review.opendev.org/679651
    Refactor MigrationTask._execute

Addressed by: https://review.opendev.org/679653
    unit test: do not fill rp mapping for failed re-schedule

Addressed by: https://review.opendev.org/680394
    Add min service level check for migrate with bandwidth

Addressed by: https://review.opendev.org/680395
    Add bandwidth min service level check of source compute

Addressed by: https://review.opendev.org/680396
    Add bandwidth min service level check of source compute

Addressed by: https://review.opendev.org/680810
    Follow up for 220fa02ee916728e241503084b14984bab4b0c3b

Addressed by: https://review.opendev.org/680991
    Extract pf$N literals as constants from func test

Addressed by: https://review.opendev.org/680998
    Improve dest service level func tests

Addressed by: https://review.opendev.org/681238
    Shrink the race window in confirm resize func test

Addressed by: https://review.opendev.org/681490
    Follow up for Ib50b6b02208f5bd2972de8a6f8f685c19745514c

Addressed by: https://review.opendev.org/681513
    Skip querying resource request if no qos port

Addressed by: https://review.opendev.org/681667
    Make SRIOV computes non symmetric in func test

Addressed by: https://review.opendev.org/682389
    Follow up for the bandwidth series

Addressed by: https://review.opendev.org/682876
    Trigger real BuildAbortException during migrate with bandwidth

Addressed by: https://review.opendev.org/684253
    Remove upgrade specific info from user facing exception text

(?)

Work Items

Dependency tree

* Blueprints in grey have been implemented.

This blueprint contains Public information 
Everyone can see this information.