commit d564266a019cb009ece1a63e5d544698f2bf74d1
Author: Matt Riedemann <email address hidden>
Date: Wed Aug 30 15:05:44 2017 -0400
Create allocations against forced dest host during evacuate
If a host is specified during an evacuate with the force=True
flag, conductor bypasses the scheduler. Since the scheduler
is what creates the "doubled up" allocation on the destination
node, and the scheduler is bypassed in the force case, we have
to create the allocations against the destination node in
conductor directly.
The unit tests cover the failure scenarios. The functional
test covers the happy path.
This is a short-term backportable fix. Long-term we'll likely
want to call the scheduler even in the 'force' scenario but pass
a flag to the scheduler to tell it to skip the filters but still
create the allocation on the destination node so we don't have
to duplicate that in conductor.
Reviewed: https:/ /review. openstack. org/499399 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=d564266a019 cb009ece1a63e5d 544698f2bf74d1
Committed: https:/
Submitter: Jenkins
Branch: master
commit d564266a019cb00 9ece1a63e5d5446 98f2bf74d1
Author: Matt Riedemann <email address hidden>
Date: Wed Aug 30 15:05:44 2017 -0400
Create allocations against forced dest host during evacuate
If a host is specified during an evacuate with the force=True
flag, conductor bypasses the scheduler. Since the scheduler
is what creates the "doubled up" allocation on the destination
node, and the scheduler is bypassed in the force case, we have
to create the allocations against the destination node in
conductor directly.
The unit tests cover the failure scenarios. The functional
test covers the happy path.
This is a short-term backportable fix. Long-term we'll likely
want to call the scheduler even in the 'force' scenario but pass
a flag to the scheduler to tell it to skip the filters but still
create the allocation on the destination node so we don't have
to duplicate that in conductor.
Change-Id: I6590f0eda4ec49 96543ad40d8c264 0b83fc3dd9d
Partial-Bug: #1713786