Mirantis OpenStack

"live_migration" call in Liberty doesn't have backwards compatibility with Kilo

Bug #1576053 reported by Roman Sokolkov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Status tracked in 10.0.x
10.0.x
Fix Released
High
Timofey Durakov
Mirantis OpenStack 10.0
8.0.x
Invalid
High
Sergii Rizvan
Mirantis OpenStack 8.0-updates

Bug Description

For upgrades we need fix for upstream bug: https://bugs.launchpad.net/nova/+bug/1576048

Description from upstrem bug:

"live_migration" call in Liberty doesn't have backwards compatibility with Kilo.

Liberty control plane and Kilo compute nodes.

Doing live-migration and getting error below on compute node

TypeError: live_migration() got an unexpected keyword argument 'migration'

Workaround(On controller): http://paste.openstack.org/show/495612/

Roman Podoliaka (rpodolyaka)
Changed in mos:
assignee: nobody → Timofey Durakov (tdurakov)
milestone: none → 9.0
status: New → Confirmed
Revision history for this message
Roman Podoliaka (rpodolyaka) wrote :

This is needed at least for fuel-octane, although I'm not sure we claim we support this case.

Changed in mos:
importance: Undecided → High
Revision history for this message
Bug Checker Bot (bug-checker) wrote : Autochecker

(This check performed automatically)
Please, make sure that bug description contains the following sections filled in with the appropriate data related to the bug you are describing:

actual result

version

steps to reproduce

For more detailed information on the contents of each of the listed sections see https://wiki.openstack.org/wiki/Fuel/How_to_contribute#Here_is_how_you_file_a_bug

tags: added: need-info
Revision history for this message
Timofey Durakov (tdurakov) wrote :

Fix for master is already merged in upstream, stable/liberty and stable/mitaka patches are on review, going to get it them with daily patch sync, once they merged

Revision history for this message
Roman Podoliaka (rpodolyaka) wrote :

Patches in upstream:

https://review.openstack.org/#/c/312813/
https://review.openstack.org/#/c/312812/

Changed in mos:
status: Confirmed → In Progress
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/nova (9.0/mitaka)

Fix proposed to branch: 9.0/mitaka
Change author: Wenzhi Yu <wenzhi_yu@163.com>
Review: https://review.fuel-infra.org/21640

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/nova (9.0/mitaka)

Reviewed: https://review.fuel-infra.org/21640
Submitter: Pkgs Jenkins <email address hidden>
Branch: 9.0/mitaka

Commit: 8ce07e486446b63f0cf1a63a561dd4c0b53c9592
Author: Wenzhi Yu <wenzhi_yu@163.com>
Date: Fri Jun 3 14:21:55 2016

Make compute rpcapi 'live_migration' backward compatible

This commit[1] added compute rpcapi version 4.2 which introduced a new
argument 'migration' when calling 'live_migration' method, but the code
did not ensure the backward compatibility with older version compute
node.

When calling 'live_migration' method with 'migration' argument on older
version(like Kilo) of nova compute node, a "TypeError: live_migration()
got an unexpected keyword argument 'migration'" will be threw up.

[1]https://github.com/openstack/nova/commit/2f4e64a7cf81de4a22fe2fc4210b1f795b1a3db4

Closes-Bug: #1576053

(cherry picked from commit 34b8336a41e2896f0639c402f64a05247f475fea)
Change-Id: Ie5c498d34e7c4c6573d611abd592c25f1bcb4f44

Revision history for this message
Roman Podoliaka (rpodolyaka) wrote :

https://review.fuel-infra.org/#/c/21640/ merged ^

Changed in mos:
status: In Progress → Fix Committed
Revision history for this message
Timur Nurlygayanov (tnurlygayanov) wrote :

The verification requires to setup complex environment, I've checked the code and fix is presented in MOS 9.0 #495. Marked as Fix Released.

Changed in mos:
status: Fix Committed → Fix Released
Revision history for this message
Sergii Rizvan (srizvan) wrote :

Closed as Invalid for 8.0 because the fix was consumed via sync with stable/liberty https://review.fuel-infra.org/#/c/22251/

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.