]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Move replication_status update to init_with_rpc
authorJohn Griffith <john.griffith8@gmail.com>
Wed, 9 Mar 2016 23:43:14 +0000 (16:43 -0700)
committerJohn Griffith <john.griffith8@gmail.com>
Thu, 10 Mar 2016 18:49:34 +0000 (18:49 +0000)
commitaf941066b32a0aa875739556f1d6360f6f058405
tree46d1937d21e3988b3dc33bf977436fae20c2a8af
parentb043410f3921c6f1246fd34ebb5fc26b62a6e427
Move replication_status update to init_with_rpc

We were using host_init to read replication_status from the
driver and update the service entry in the DB.  It turns out
that on a fresh install this doesn't actually work, because
while we have multiple init methods for the backend, the
Service entry isn't actually created in a fresh deploy until
AFTER init_host.  The result was that in some cases we were
trying to update a column on a non-existent Service in the
DataBase.

This patch moves the replication_status updates for the
service into the init_with_rpc method.  That method was
just a noop stub in the parent manager class, so we just
implement it in cinder.volume.manager and do what we need
with the replication update info.

Change-Id: I18b2658e2f93959f74377ccb86ce8b01b6970c60
Closes-Bug: #1555370
cinder/volume/manager.py