Bug 1416356
Summary: | Overcloud deployment using external NFS storage for Cinder fails | ||||||
---|---|---|---|---|---|---|---|
Product: | Red Hat OpenStack | Reporter: | Marius Cornea <mcornea> | ||||
Component: | openstack-tripleo-heat-templates | Assignee: | Angus Thomas <athomas> | ||||
Status: | CLOSED ERRATA | QA Contact: | lkuchlan <lkuchlan> | ||||
Severity: | urgent | Docs Contact: | |||||
Priority: | urgent | ||||||
Version: | 11.0 (Ocata) | CC: | abishop, acanan, agurenko, cschwede, dbecker, egafford, eharney, emacchi, jschluet, mburns, mcornea, morazi, pgrist, rhel-osp-director-maint, scohen, tbarron, tshefi | ||||
Target Milestone: | rc | Keywords: | Triaged | ||||
Target Release: | 11.0 (Ocata) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Whiteboard: | |||||||
Fixed In Version: | openstack-tripleo-heat-templates-6.0.0-0.10.el7ost | Doc Type: | If docs needed, set a value | ||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2017-05-17 19:43:11 UTC | Type: | Bug | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Bug Depends On: | |||||||
Bug Blocks: | 1394019 | ||||||
Attachments: |
|
Description
Marius Cornea
2017-01-25 10:56:10 UTC
Hi all, This appears to be a simple type error: the Cinder module requires CinderNfsShares to be a list, such that it can be joined. The environment file provided a string; wrapping this string in [] should solve this immediate problem. I have also posted a review for tripleo-heat-templates to change the defaults for this parameter to an empty list (from an empty string.) Thanks, Elise Note: I requested the blocker? flag be removed from this bug, as the fix is configuration, and any use of the NFS backend will require setting CinderNfsServers. Still, it would be best if we were to accept the default change patch prior to the next release. Cheers, Elise (In reply to Elise Gafford from comment #5) > Note: I requested the blocker? flag be removed from this bug, as the fix is > configuration, and any use of the NFS backend will require setting > CinderNfsServers. Still, it would be best if we were to accept the default > change patch prior to the next release. > > Cheers, > Elise Hi Elise, The following CinderNfsServers values used to work in OSP10: CinderNfsServers: '10.0.0.254:/srv/nfs/cinder' CinderNfsServers: "[fd00:fd00:fd00:3000::1]:/srv/nfs/cinder" Could you please let me know what are the correct formats for OSP11 so I can test them? Thanks! We also need this info to determine what we need to do in case of OSP10->OSP11 upgrade, if we need to change this parameters during upgrade or not. Proposed upstream fix: https://review.openstack.org/448208 @Marius, can you test that fix if possible? Like this? On the undercloud: 1. cp -r /usr/share/openstack-tripleo-heat-templates . 2. Modify puppet/services/cinder-volume.yaml like this: https://review.openstack.org/#/c/448208/1/puppet/services/cinder-volume.yaml 3. Deploy with overcloud deploy ... --templates openstack-tripleo-heat-templates ... The options you have been using should continue to work then. (In reply to Christian Schwede (cschwede) from comment #9) > Proposed upstream fix: https://review.openstack.org/448208 > > @Marius, can you test that fix if possible? Like this? > > On the undercloud: > > 1. cp -r /usr/share/openstack-tripleo-heat-templates . > > 2. Modify puppet/services/cinder-volume.yaml like this: > https://review.openstack.org/#/c/448208/1/puppet/services/cinder-volume.yaml > > 3. Deploy with > overcloud deploy ... --templates openstack-tripleo-heat-templates ... > > The options you have been using should continue to work then. Yes, I can confirm it works fine with the existing parameters after applying the patch. Thanks! Should I mark this bug as a blocker to make sure we keep track of the upstream patch landing in downstream in time for the release? @Marius: yes, adding the blocker flag helps - this looks like a clear regression. Thanks for verifying the patch! Backport merged upstream on stable/ocata. Created attachment 1269401 [details]
Example storage yaml file used.
Verified on version:
openstack-tripleo-heat-templates-6.0.0-3.el7ost.noarch
Attaching yaml file for refute reference.
Cinder create volumes works.
Volume is created on NFS backend.
[stack@undercloud-0 ~]$ cinder list
+--------------------------------------+-----------+-----------+------+-------------+----------+-------------+
| ID | Status | Name | Size | Volume Type | Bootable | Attached to |
+--------------------------------------+-----------+-----------+------+-------------+----------+-------------+
| 531df6a7-2864-4818-b215-6b5e8057685a | available | nfsvolume | 1 | - | false | |
+--------------------------------------+-----------+-----------+------+-------------+----------+-------------+
On nfs server
# ll
total 0
-rw-r--r--. 1 root root 0 פבר 16 14:23 ins_cinder_export
-rw-rw----. 1 cinder cinder 1073741824 אפר 6 16:45 volume-531df6a7-2864-4818-b215-6b5e8057685a
On overcloud_deploy.sh (overcloud deploy command) add refrance to your storage yaml file.
See comment 14 ^ Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHEA-2017:1245 |