diff options
author | melanie witt <melwittt@gmail.com> | 2018-03-19 23:10:47 +0000 |
---|---|---|
committer | melanie witt <melwittt@gmail.com> | 2018-03-20 23:54:21 +0000 |
commit | 2f7df9417bd79c00deb7acbd9febc1b730c3301a (patch) | |
tree | 33fc0b5664fe3f40861af4169097649fd90b3821 /nova/rpc.py | |
parent | 43c72f02f277d6f1083e5e9a4009d3d03d965368 (diff) | |
download | nova-2f7df9417bd79c00deb7acbd9febc1b730c3301a.tar.gz |
Remove useless run_periodic_tasks call in ClientRouter
The only periodic task (_remove_stale_clients) in ClientRouter was
removed in change I10f374adca672576058c4dbab708c040d166df47 and this
isn't the correct way to run periodic tasks repeatedly anyway (this
will only run them once upon ClientRouter.__init__), so remove the
useless call.
This also removes a functional regression test which was testing only
this specific code path, where the intended periodic task ran only
once upon init of the compute service. All other periodic tasks in
compute are disabled by default in functional tests.
Related-Bug: #1627838
Change-Id: I3079c5ae4bd60de44f04e0136978a67d13e7a809
Diffstat (limited to 'nova/rpc.py')
-rw-r--r-- | nova/rpc.py | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/nova/rpc.py b/nova/rpc.py index c99b74a0f9..7ed87dde5a 100644 --- a/nova/rpc.py +++ b/nova/rpc.py @@ -409,8 +409,6 @@ class ClientRouter(periodic_task.PeriodicTasks): # NOTE(melwitt): Cells v1 does its own serialization and won't # have a serializer available on the client object. self.serializer = getattr(default_client, 'serializer', None) - # Prevent this empty context from overwriting the thread local copy - self.run_periodic_tasks(nova.context.RequestContext(overwrite=False)) def client(self, context): transport = context.mq_connection |