Bug 1421440

Summary: tripleo-ansible-inventory does not report ceph
Product: Red Hat OpenStack Reporter: Amit Ugol <augol>
Component: openstack-tripleo-validationsAssignee: Florian Fuchs <flfuchs>
Status: CLOSED ERRATA QA Contact: Arik Chernetsky <achernet>
Severity: high Docs Contact:
Priority: high    
Version: 11.0 (Ocata)CC: apannu, asimonel, augol, beth.white, flfuchs, jjoyce, jrist, jschluet, opavlenk, shardy, slinaber, tvignaud
Target Milestone: Upstream M2Keywords: Triaged
Target Release: 12.0 (Pike)   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: openstack-tripleo-validations-7.1.1-0.20170621111847.fb7346f.el7ost Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-12-13 21:08:54 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:

Description Amit Ugol 2017-02-12 11:10:42 UTC
Description of problem:
Ceph nodes are not reported when using tripleo-ansible-inventory --list
This worked well in previous rhos11 versions and as a tech preview in rhos10.

Version-Release number of selected component (if applicable):
openstack-tripleo-validations-5.3.1-0.20170125194508.6b928f1.el7ost.noarch

How reproducible:
100%

Steps to Reproduce:
1. deploy with ceph nodes (note that locally I am calling it 'ceph'
2. source stackrc
3. run tripleo-ansible-inventory --list

Actual results:
[stack@undercloud-0 ~]$ tripleo-ansible-inventory --list 2> /dev/null | tr "{}" "\n"

"overcloud": 
"children": ["controller", "compute"], "vars": 
"ansible_ssh_user": "heat-admin"

, "controller": ["192.168.24.12"], "undercloud": 
"hosts": ["localhost"], "vars": 
"overcloud_horizon_url": "http://10.0.0.105:80/dashboard", "overcloud_admin_password": "bBNw4aJwbkzh6Eqht9eXVF9CM", "ansible_connection": "local"

, "compute": ["192.168.24.17"]


Expected results:
"children" should have ceph.
ceph section needs to have ceph stuff.


Additional info:
Deployment command:
openstack overcloud deploy --debug \
--templates \
--libvirt-type kvm \
--ntp-server clock.redhat.com \
--control-scale 1 \
--control-flavor controller \--compute-scale 1 \
--compute-flavor compute \
--environment-file /usr/share/openstack-tripleo-heat-templates/environments/services/sahara.yaml \
--environment-file /usr/share/openstack-tripleo-heat-templates/environments/cinder-backup.yaml \
--ceph-storage-scale 1 \
--ceph-storage-flavor ceph \
-e /usr/share/openstack-tripleo-heat-templates/environments/storage-environment.yaml \
-e /home/stack/virt/ceph.yaml \
-e /home/stack/virt/network/network-environment.yaml \
-e /usr/share/openstack-tripleo-heat-templates/environments/network-isolation.yaml \
-e /home/stack/virt/hostnames.yml \
-e /home/stack/virt/debug.yaml \
--log-file overcloud_deployment_23.log

Comment 4 Steven Hardy 2017-04-11 12:52:51 UTC
This is because we have hard-coded role references like here:

https://github.com/openstack/tripleo-validations/blob/master/scripts/tripleo-ansible-inventory#L141

self.fetch_stack_resources('Controller') etc

The fix is to retrieve the roles_data.yaml from the plan container, then iterate over all roles.

AFAICS this has never worked, so I'm not sure the "This worked well.." comment is accurate, unless something got lost in translation when we did this initial commit to tripleo-validations?

https://github.com/openstack/tripleo-validations/commit/f614ef744fc2949ea8911683662ae50896a8f398

Comment 6 Florian Fuchs 2017-05-11 13:25:21 UTC
Merged upstream for pike

Comment 7 Thierry Vignaud 2017-06-27 12:47:28 UTC
Can you add the gerrit id please so that we can check if we've the relevant change?
Thanks

Comment 10 Florian Fuchs 2017-09-05 11:22:42 UTC
(In reply to Thierry Vignaud from comment #7)
> Can you add the gerrit id please so that we can check if we've the relevant
> change?

Yes, the one attached to the ticket contains the relevant change:

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

Comment 14 errata-xmlrpc 2017-12-13 21:08:54 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.

https://access.redhat.com/errata/RHEA-2017:3462