]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
coraid: allow setting default repository
authorRoman Bogorodskiy <rbogorodskiy@mirantis.com>
Wed, 1 Oct 2014 08:15:14 +0000 (08:15 +0000)
committerRoman Bogorodskiy <rbogorodskiy@mirantis.com>
Mon, 6 Oct 2014 15:33:35 +0000 (15:33 +0000)
commita368376053484b382f93a622c38483981b9e8347
tree58eaaa090b2c52554b63ac08c41ab01391dbf726
parent393c2c58949a5a7b19daa545e9f23065a71d7003
coraid: allow setting default repository

Currently, the Coraid driver requires storing Storage Repository
name in volume type key named 'coraid_repository' or different key name
if a value of the 'coraid_repository_key' option changed in the cinder
configuration file.

In case of creating a volume without volume type associated or creating
a volume with volume type that doesn't have an appropriate key with
the repository name, volume creation fails.

That is an inconvenient behaviour, especially in case of using that with
Tempest that creates new volume types but obviously doesn't know
that it should add keys with the repository name.

In order to fix that, introduce a 'coraid_default_repository' config
option that allows to set a default repository to use if one is not
specified in the volume type key.

In case if neither volume type nor config file provide a repository
name, throw CoraidException with description of the problem.

DocImpact

Change-Id: Ib205553cdee0af0036a774f6926433f4452ec023
cinder/tests/test_coraid.py
cinder/volume/drivers/coraid.py
etc/cinder/cinder.conf.sample