Bug 1563781

Summary:	Docs are missing template_service_broker_selector
Product:	OpenShift Container Platform	Reporter:	Jared Hocutt <jhocutt>
Component:	Documentation	Assignee:	Michael Burke <mburke>
Status:	CLOSED WONTFIX	QA Contact:	Vikram Goyal <vigoyal>
Severity:	medium	Docs Contact:	Vikram Goyal <vigoyal>
Priority:	unspecified
Version:	3.9.0	CC:	aos-bugs, jokerman, mburke, mmccomas
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2018-04-30 18:54: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 Jared Hocutt 2018-04-04 16:47:29 UTC

Document URL:

https://docs.openshift.com/container-platform/3.9/install_config/install/advanced_install.html


Section Number and Name: N/A


Describe the issue: 

The OpenShift documentation has no mention of the template_service_broker_selector option.

An assumption is made that a node is going to have the label region=infra. This can be overridden by setting template_service_broker_selector={'role': 'foo'}, but this is not documented.

During a default installation of OpenShift 3.7 and 3.9, if a node does not have a label region=infra, this will cause the install to fail.


Suggestions for improvement:

Document the template_service_broker_selector option to prevent users ending up with a failed install of OpenShift.


Additional information:

Comment 1 Michael Burke 2018-04-26 14:41:36 UTC

@jared This issue seems to have been brought up via https://github.com/openshift/openshift-docs/pull/7507/files. 

Can you take a look at Eric's change to see if it meets your understanding of the option.

Comment 2 Jared Hocutt 2018-04-26 14:44:07 UTC

(In reply to Michael Burke from comment #1)
> @jared This issue seems to have been brought up via
> https://github.com/openshift/openshift-docs/pull/7507/files. 
> 
> Can you take a look at Eric's change to see if it meets your understanding
> of the option.

Yep, that looks good to me. Thanks!

Comment 3 Michael Burke 2018-04-30 18:54:13 UTC

Closing as duplicate. Change was implemented vix https://github.com/openshift/openshift-docs/pull/7507

Comment 4 Michael Burke 2018-04-30 18:54:48 UTC

bugathon