Bug 1582281

Summary:	facts are gathered 2 times for client nodes on containerized deployment
Product:	[Red Hat Storage] Red Hat Ceph Storage	Reporter:	Guillaume Abrioux <gabrioux>
Component:	Ceph-Ansible	Assignee:	Guillaume Abrioux <gabrioux>
Status:	CLOSED ERRATA	QA Contact:	Vasishta <vashastr>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	3.1	CC:	adeza, aschoen, ceph-eng-bugs, ceph-qe-bugs, gmeno, hnallurv, nthomas, sankarshan, tserlin, vpoliset
Target Milestone:	rc
Target Release:	3.1
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	RHEL: ceph-ansible-3.1.0-0.1.rc4.el7cp Ubuntu: ceph-ansible_3.1.0~rc4-2redhat1	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2018-09-26 18:21:13 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 Guillaume Abrioux 2018-05-24 17:26:13 UTC

Description of problem:

facts are gathered 2 times on clients due to a leftover after a fix aiming to improve facts gathering.


How reproducible:
Launch a containerized deployment with delegate_facts: true (which is the default).


Actual results:
facts are gathered 2 times on client nodes.

Expected results:
facts are gathered only 1 time.

Additional info:

Comment 3 Guillaume Abrioux 2018-05-25 07:43:13 UTC

will be in v3.1.0rc4

Comment 8 subhash 2018-07-04 11:07:26 UTC

Verified with version ansible-2.4.5.0-1.el7ae.noarch ,
ceph-ansible-3.1.0-0.1.rc9.el7cp.noarch

Facts are gathered only once for clients ,moving to verified state

Inventory hosts file
[mons]
magna021

[osds]
magna028 devices="['/dev/sdb','/dev/sdc','/dev/sdd']" osd_scenario="collocated"
magna029 osd_auto_discovery='true' osd_scenario="collocated"
magna030 osd_auto_discovery='true' osd_scenario="collocated" dmcrypt="true"

[mgrs]
magna021

[clients]
magna031


TASK [gather facts] **********************************************************************************************************************************
task path: /usr/share/ceph-ansible/site-docker.yml:24
Wednesday 04 July 2018  10:48:58 +0000 (0:00:00.301)       0:00:00.301 ********
skipping: [magna021] => {"changed": false, "skip_reason": "Conditional result was False", "skipped": true}
skipping: [magna028] => {"changed": false, "skip_reason": "Conditional result was False", "skipped": true}
skipping: [magna029] => {"changed": false, "skip_reason": "Conditional result was False", "skipped": true}
skipping: [magna030] => {"changed": false, "skip_reason": "Conditional result was False", "skipped": true}
skipping: [magna031] => {"changed": false, "skip_reason": "Conditional result was False", "skipped": true}

TASK [gather and delegate facts] *********************************************************************************************************************
task path: /usr/share/ceph-ansible/site-docker.yml:29
Wednesday 04 July 2018  10:48:58 +0000 (0:00:00.242)       0:00:00.544 ********
^[ok: [magna021 -> magna021] => (item=magna021)
ok: [magna021 -> magna031] => (item=magna031)
ok: [magna021 -> magna028] => (item=magna028)
ok: [magna021 -> magna029] => (item=magna029)
ok: [magna021 -> magna030] => (item=magna030)

Comment 10 errata-xmlrpc 2018-09-26 18:21:13 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/RHBA-2018:2819