yahoo-eng-team team mailing list archive

[OpenStack Infra <1550018@xxxxxxxxxxxxxxxxxx>]

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

commit 9ab6840af64d6af58baf51040afe86a3324221e5
Author: Sylvain Bauza <sbauza@xxxxxxxxxx>
Date:   Thu Feb 25 22:57:28 2016 +0100

    Fix Cells RPC API by accepting a RequestSpec arg
    
    Since cells v1 adds a proxy to a call to ComputeTaskAPI for live_migrate
    and rebuild_instance, we need to augment the Cells RPC API to accept
    the request_spec argument for both methods.
    
    That said, since cells v1 is quite feature-freeze, it's okay to not pass
    the RequestSpec down to the pipe because the cell conductor will then
    act as if the instance was not having a Spec object, which is a compat
    behaviour we want to keep for a very long time.
    
    Closes-Bug: #1550018
    
    Change-Id: I1c2aaedfc83e4e97f53c9c504e4585b1f0510d4b


** Changed in: nova
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1550018

Title:
  Cells v1 is borked for live-migrate and evacuate

Status in OpenStack Compute (nova):
  Fix Released

Bug description:
  So, we recently added a new feature for passing a RequestSpec object from the API down to the conductor. 
  Unfortunately, the CI (and the bug reporter who was the jerk who wrote the code) missed that Cells V1 provides a Proxy for the ComputeTaskAPI calls we do.
  That means that the signature between ComputeTaskAPI methods and (RPC) CellsAPI needs to be the same for live-migrate and evacuate.

  Unshelve was also modified, but Cells V1 has a different behaviour
  (not calling ComputeTaskAPI) so we're good.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1550018/+subscriptions