openstack/deb-heat
Code Issues Proposed changes

Cinder volume attachment attributes should not be cached

Browse Source 
The Cinder volume 'attachments' and 'attachments_list' attributes are
dynamically built lists and must be queried from the Cinder API on each
request. There are several factors that may change the contents of this
list that are unknown to Heat.  Caching should be prevented for these
attributes.

Closes-Bug: #1703605
Related-Bug: #1703387
Co-Authored-By: Paul Bourke <paul.bourke@oracle.com>
Change-Id: I1a646e567b2d88f2acf09b3077c205c5b061d4bc
This commit is contained in:
Gerry Buteau 2017-07-11 10:09:15 -04:00 committed by Paul Bourke
parent dcf2463f1f
commit 9269787d39
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
heat/engine/resources/openstack/cinder/volume.py
View File

@ -223,6 +223,7 @@ class CinderVolume(vb.BaseVolume, sh.SchedulerHintsMixin):
_('A string representation of the list of attachments of the '
'volume.'),
type=attributes.Schema.STRING,
cache_mode=attributes.Schema.CACHE_NONE,
support_status=support.SupportStatus(
status=support.DEPRECATED,
message=_('Use property %s.') % ATTACHMENTS_LIST,
@ -236,6 +237,7 @@ class CinderVolume(vb.BaseVolume, sh.SchedulerHintsMixin):
ATTACHMENTS_LIST: attributes.Schema(
_('The list of attachments of the volume.'),
type=attributes.Schema.LIST,
cache_mode=attributes.Schema.CACHE_NONE,
support_status=support.SupportStatus(version='9.0.0'),
),
MULTI_ATTACH_ATTR: attributes.Schema(