cinder volume attachments are no longer available

Bug #1703387 reported by Gerry Buteau
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Murano
Fix Released
Undecided
zhurong

Bug Description

attachments_list seems to be unavailable in CinderVolume.yaml while building the resource definition. This functionality was originally added by the following change, but is now no longer working.

https://review.openstack.org/#/c/451909/

Revision history for this message
Ellen Batbouta (ellen-batbouta) wrote :

The attachments_list output from buildResourceDefinition was removed as part of:

https://review.openstack.org/#/c/480848/6

For our Oracle database package, we do need this information.

Changed in murano:
assignee: nobody → Gerry Buteau (gerry.buteau)
Revision history for this message
Gerry Buteau (gerry.buteau) wrote :

The root of this problem appears to be some recent changes in HEAT involving resource/attribute caching. This may be why the murano volume attachment tests were failing originally.

In addition to the change ellen-batbouta mentioned, there will need to be a patch for the heat project that indicates that the attachments_list attribute should not be cached for cinder volumes:

diff --git a/heat/engine/resources/openstack/cinder/volume.py b/heat/engine/resources/openstack/cinder/volume.py
index 23b05af..4f0589f 100644
--- a/heat/engine/resources/openstack/cinder/volume.py
+++ b/heat/engine/resources/openstack/cinder/volume.py
@@ -236,6 +236,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(

Revision history for this message
Gerry Buteau (gerry.buteau) wrote :

The heat patch to fix volume attachments has been submitted:

https://review.openstack.org/#/c/482663/

This murano patch can now be reverted:

https://review.openstack.org/#/c/480848

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to murano (master)

Fix proposed to branch: master
Review: https://review.openstack.org/485001

Changed in murano:
assignee: Gerry Buteau (gerry.buteau) → zhurong (zhu-rong)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to murano (master)

Reviewed: https://review.openstack.org/485001
Committed: https://git.openstack.org/cgit/openstack/murano/commit/?id=e667abdb8415c3150264e93208af979c3c8f0b40
Submitter: Jenkins
Branch: master

commit e667abdb8415c3150264e93208af979c3c8f0b40
Author: zhurong <email address hidden>
Date: Wed Jul 19 09:21:58 2017 +0800

    Make cinder volume attachments available

    This commit is revert from Ib996bf50e1b7d542b98cf9d7125b824771c143d6,

    Change-Id: Ie866833d205c119254b811654426c586c338d7c8
    Closes-Bug: #1703387

Changed in murano:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/murano 4.0.0.0b3

This issue was fixed in the openstack/murano 4.0.0.0b3 development milestone.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.