]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Handle the case where az is disabled/removed
authorJoshua Harlow <harlowja@yahoo-inc.com>
Mon, 9 Jun 2014 22:32:17 +0000 (15:32 -0700)
committerJoshua Harlow <harlowja@yahoo-inc.com>
Tue, 17 Jun 2014 22:23:38 +0000 (15:23 -0700)
commit415a4ba29b12e51b646c79e19bb0a63c80c1157f
treef04024b9a941746211237ed73d67493cac6f901f
parent2355c7935054b6f6fbc7e641d289a0efd52a8c63
Handle the case where az is disabled/removed

Instead of retaining a local in-memory list of
availability zones that is never purged and therefore
never updates itself if availability zones are removed
we should instead cache those zones for a given period
of time (defaulting to one hour) and on cache expiry
refetch the enabled availability zones.

DocImpact: adds a new configuration that defines the
default availability zone cache expiry time in seconds.

This will enable the engine that runs the create volume
request to in a future change remove the need to have
a functor which proxies the validation logic around
availability zones.

Change-Id: I8e45845b66ba62313248ee9cbeb301a5fe54a08b
cinder/tests/test_volume.py
cinder/volume/api.py
etc/cinder/cinder.conf.sample