]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Update list_replication_targets
authorJohn Griffith <john.griffith8@gmail.com>
Tue, 24 Nov 2015 21:46:19 +0000 (14:46 -0700)
committerJohn Griffith <john.griffith8@gmail.com>
Tue, 24 Nov 2015 21:46:52 +0000 (14:46 -0700)
commitc30ce044b713113317bbeb1c3df2593bd6a6ec2b
tree4cce2830b6c8bc6e5e4f11bd605915433b2da03e
parent2c3b6e9912b6dc60b78585effaff6219377e5d8d
Update list_replication_targets

We were planning to let the list targets call be a
free for all, and just be a generic dict of info.

This probably isn't such a great idea, inparticular since
we have a required identifier field for each replication
device it makes more sense to just use that in the response
and leave the rest of the info internal.

In the future if there's a need we can always implement a
show command that displays all the dirty details.

This patch changes the identifier name to 'target_device_id'
to be more self describing, and updates the docstrings and
devref docs to reflect the changes and agreed upon changes.

Change-Id: If14130f5d5bb2e6df4478bd0e14a1f33d706bf78
cinder/volume/driver.py
cinder/volume/manager.py
doc/source/devref/replication.rst