Skip to content

Commit 17b6957

Browse files
Zuulopenstack-gerrit
authored andcommitted
Merge "Use CONF.long_rpc_timeout in post_live_migration_at_destination"
2 parents ec0c45a + f628dc9 commit 17b6957

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

nova/compute/rpcapi.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -695,7 +695,9 @@ def post_live_migration_at_destination(self, ctxt, instance,
695695
block_migration, host):
696696
version = '5.0'
697697
cctxt = self.router.client(ctxt).prepare(
698-
server=host, version=version)
698+
server=host, version=version,
699+
call_monitor_timeout=CONF.rpc_response_timeout,
700+
timeout=CONF.long_rpc_timeout)
699701
return cctxt.call(ctxt, 'post_live_migration_at_destination',
700702
instance=instance, block_migration=block_migration)
701703

nova/tests/unit/compute/test_rpcapi.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -342,9 +342,11 @@ def test_live_migration_abort(self):
342342
migration_id='1', version='5.0')
343343

344344
def test_post_live_migration_at_destination(self):
345+
self.flags(long_rpc_timeout=1234)
345346
self._test_compute_api('post_live_migration_at_destination', 'call',
346347
instance=self.fake_instance_obj,
347-
block_migration='block_migration', host='host', version='5.0')
348+
block_migration='block_migration', host='host', version='5.0',
349+
timeout=1234, call_monitor_timeout=60)
348350

349351
def test_pause_instance(self):
350352
self._test_compute_api('pause_instance', 'cast',

0 commit comments

Comments
 (0)