]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Ensure replication functions check driver status
authorWalter A. Boring IV <walter.boring@hp.com>
Thu, 1 Oct 2015 23:04:04 +0000 (16:04 -0700)
committerWalter A. Boring IV (hemna) <walter.boring@hpe.com>
Thu, 1 Oct 2015 23:44:08 +0000 (23:44 +0000)
commitbc4a60d742bc537d2b2eebc6b754182a09baa79d
tree0dfe50e41f8c35abafc987e8c19674a625585806
parent792c24bf360ca816055e6f492f67a41f1ce7e359
Ensure replication functions check driver status

This patch adds the missing checks to ensure
the driver's are initialized to both
enable_replication and disable_replication.

This patch also adds comment blocks to the
failover_replication as well as list_replication_targets
functions to explain why they are both are not
checking the driver for initialization.  One of the main
use cases for those 2 functions happen precisely when the
driver can't talk to the primary backend, but might be
able to talk to the secondary or target backends.

Change-Id: I4052e0ff4d2977ecfba940db56dba24c063bbbb6
Closes-Bug: #1501936
cinder/tests/unit/test_volume.py
cinder/volume/manager.py