Bug 1421811

Summary:	[DOCS] The ansible hosts file for the standalone registry is missing openshift_master_default_subdomain
Product:	OpenShift Container Platform	Reporter:	ldomb
Component:	Documentation	Assignee:	Brandi Munilla <bmcelvee>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Johnny Liu <jialiu>
Severity:	high	Docs Contact:	Vikram Goyal <vigoyal>
Priority:	high
Version:	3.4.1	CC:	aos-bugs, jokerman, mmccomas, tomckay
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:	2017-03-27 12:48:43 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 ldomb 2017-02-13 18:15:42 UTC

Document URL: https://docs.openshift.com/container-platform/3.4/install_config/install/stand_alone_registry.html

and

https://access.redhat.com/documentation/en/openshift-container-platform/3.4/single/installation-and-configuration/#install-config-installing-stand-alone-registry

Section Number and Name: 2.8.5.2. Advanced Installation for Stand-alone Registries

Example 1. All-in-one Stand-alone Registry Inventory File (ocpdoc)

Example 2.5. All-in-one Stand-alone Registry Inventory File (rhdoc)

Describe the issue: the hosts file is missing the openshift_master_default_subdomain=reg.local.domb.com which causes the install to create an internal route instead of exposing it.

Suggestions for improvement: 

Add openshift_master_default_subdomain=$appdomain to the ansible hosts file

Additional information:

Comment 1 Brandi Munilla 2017-03-09 00:56:10 UTC

Hi Wei, 

I've added 'openshift_master_default_subdomain=$appdomain' to the Ansible hosts file in PR#3905[1]. Please verify that I put it in the correct location. 

Thank you!

[1]https://github.com/openshift/openshift-docs/pull/3905

Comment 2 Johnny Liu 2017-03-09 03:10:48 UTC

The modification in the above PR seem like not in right place.

There are two places need to be updated in https://docs.openshift.com/container-platform/3.4/install_config/install/stand_alone_registry.html#registry-advanced-installation section: "Example 1" and "Example 2".

Should add the following line to "[OSEv3:vars]" section just like https://docs.openshift.com/container-platform/3.4/install_config/install/advanced_install.html#configuring-cluster-variables, but not add it onto node line.
openshift_master_default_subdomain=apps.test.example.com

Comment 3 Brandi Munilla 2017-03-15 20:24:05 UTC

Hi Johnny, 

Thank you for your feedback. I updated PR#3905[1] accordingly. Please take a look when you get a chance. 

[1]https://github.com/openshift/openshift-docs/pull/3905

Comment 4 Johnny Liu 2017-03-16 06:40:15 UTC

It looks good to me.

Comment 5 openshift-github-bot 2017-03-16 18:16:52 UTC

Commits pushed to master at https://github.com/openshift/openshift-docs

https://github.com/openshift/openshift-docs/commit/bd51966544d9e55dbb652e5b0c4eec153a70dd1f
Bug 1421811 Added subdomain file

https://github.com/openshift/openshift-docs/commit/93ca220629fab84f41670184e23fc52ae1693ca8
Merge pull request #3905 from bmcelvee/BZ1421811

Bug 1421811 Added subdomain file

Comment 6 Brandi Munilla 2017-03-27 12:48:43 UTC

Link to documentation on Customer Portal: https://access.redhat.com/documentation/en-us/openshift_container_platform/3.4/html-single/installation_and_configuration/#configuring-sdn-config-pod-network-ansible