Bug 1502862

Summary: Update storage-environment.yaml so that it defaults to ceph-ansible
Product: Red Hat OpenStack Reporter: John Fulton <johfulto>
Component: openstack-tripleo-heat-templatesAssignee: Giulio Fidente <gfidente>
Status: CLOSED ERRATA QA Contact: Yogev Rabl <yrabl>
Severity: high Docs Contact:
Priority: high    
Version: 12.0 (Pike)CC: afazekas, agurenko, dbecker, jschluet, mburns, mcornea, morazi, rhel-osp-director-maint, sasha, sclewis, yrabl
Target Milestone: rcKeywords: Triaged
Target Release: 12.0 (Pike)   
Hardware: Unspecified   
OS: Unspecified   
Fixed In Version: openstack-tripleo-heat-templates-7.0.3-2.el7ost Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-12-13 22:15:46 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:

Description John Fulton 2017-10-16 20:59:32 UTC
A containerized ceph deployment which didn't pass any overrides but used only the defaults found in: 

-e /usr/share/openstack-tripleo-heat-templates/environments/ceph-ansible/ceph-ansible.yaml 

Produced a resources.WorkflowTasks_Step2_Execution Heat resource whose Mistral ansible-playbook call did not pass the osd_scenario parameter to the ansible-playbook --extra-vars run [1]. This resulted in a overcloud deployment with 0 OSDs (0 OSD containers) though the deployment succeeded and evidence of the failure was not seen later until a tempest run. 

Though ceph-ansible should better handle this scenario and fail as per BZ 1502841 [2], the defaults alone should work, provided that the ceph overcloud node has the device /dev/vdb. 

[1] http://paste.openstack.org/show/623752 
[2] https://bugzilla.redhat.com/show_bug.cgi?id=1502841

Comment 4 Giulio Fidente 2017-10-17 09:42:12 UTC
Thanks John, as a side note, we might also default storage-environment.yaml to ceph-ansible at this point.

Comment 5 Giulio Fidente 2017-10-17 09:55:42 UTC
The root cause here is that /home/stack/virt/internal.yaml is overriding the default CephAnsibleDisksConfig passing only 'devices:', see [1]

Instead of closing this BZ though, we might use it to track migration of storage-environment.yaml from puppet-ceph to ceph-ansible if QE agrees with it being a good idea.

Yogev, Sasha, Marius, what do you guys think?

1. http://cougar11.scl.lab.tlv.redhat.com/phase1-12_director-rhel-7.4-virthost-1cont_1comp_1ceph-ipv4-vxlan-ceph-containers/125/undercloud-0.tar.gz?undercloud-0/home/stack/virt/internal.yaml

Comment 15 Yogev Rabl 2017-11-15 20:11:37 UTC
verified on openstack-tripleo-heat-templates-7.0.3-3.el7ost.noarch

Comment 18 errata-xmlrpc 2017-12-13 22:15:46 UTC
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.