Bug 1498908

Summary:	openshift-installer image should support an inventory directory in addition to flat files.
Product:	OpenShift Container Platform	Reporter:	Eric Sauer <esauer>
Component:	Installer	Assignee:	Luke Meyer <lmeyer>
Status:	CLOSED ERRATA	QA Contact:	Wenkai Shi <weshi>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	3.6.1	CC:	aos-bugs, jokerman, mmccomas
Target Milestone:	---
Target Release:	3.7.0
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Enhancement
Doc Text:	Feature: Enable installer to use an inventory specified as a directory rather than just a single file. Reason: Customers with production clusters often find it easier to manage their inventory as multiple files in a directory. Result: This adds a parameter INVENTORY_DIR to the openshift-ansible image such that the user can indicate that ansible-playbook should use a mounted inventory directory.	Story Points:	---
Clone Of:		Environment:
Last Closed:	2017-11-28 22:15:02 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 Eric Sauer 2017-10-05 14:17:56 UTC

Description of problem:

The current ose-ansible (or origin-ansible) image only supports passing a single file to -i like `ansible-playbook -i /path/to/hosts playbook.yml`. This prevents a user from any kind of more advanced usage of ansible inventories, such as a hybrid dynamic/static style inventory, which is becoming fairly common in the field.

I would like to add support for an INVENTORY_DIR variable that handles a directory instead of a single file.

Additional info:
Pull request addressing feature: https://github.com/openshift/openshift-ansible/pull/5644

Comment 1 Eric Sauer 2017-10-05 14:52:28 UTC

Here's a test of the PR to show that it works: http://pastebin.test.redhat.com/522141

Comment 3 Wenkai Shi 2017-10-19 10:03:49 UTC

Verified with ose-ansible image tag v3.6.173.0.56-1(f4b5ddbbde9b), it's support INVENTORY_DIR variable now.

Comment 6 errata-xmlrpc 2017-11-28 22:15:02 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/RHSA-2017:3188