Bug 1659142

Summary:	[DOCS] [RFE] Add Ability to Set projectRequestMessage Through Inventory Variable
Product:	OpenShift Container Platform	Reporter:	Jack Ottofaro <jack.ottofaro>
Component:	Documentation	Assignee:	Kathryn Alexander <kalexand>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Weihua Meng <wmeng>
Severity:	low	Docs Contact:	Vikram Goyal <vigoyal>
Priority:	low
Version:	3.11.0	CC:	aalevy, aos-bugs, jokerman, mmccomas, rhowe, vigoyal, wmeng
Target Milestone:	---	Keywords:	Reopened
Target Release:	3.11.0
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:	2019-06-18 16:47:45 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 Jack Ottofaro 2018-12-13 16:43:24 UTC

Description of problem:

Support setting projectRequestMessage through a playbook variable that can be referenced in an inventory file.

Customer feels the current method of requiring an administrator to manually modify master-config.yaml and restart api services to be cumbersome.

Comment 1 Jack Ottofaro 2018-12-13 16:46:52 UTC

Related to bug 1659141.

Comment 5 Ryan Howe 2019-03-03 20:37:50 UTC

This is already possible in 3.11 setting osm_project_request_message variable in the inventory file. 

https://github.com/openshift/openshift-ansible/blob/release-3.11/inventory/hosts.example#L334
https://github.com/openshift/openshift-ansible/blob/release-3.11/roles/openshift_control_plane/templates/master.yaml.v1.j2#L177

Closing bug.

Comment 6 Jack Ottofaro 2019-04-11 14:47:40 UTC

Re-opening as a documentation bug. Document use of setting osm_project_request_message variable in the inventory file.

Comment 7 Jack Ottofaro 2019-05-08 18:39:21 UTC

Any updates?

Comment 8 Vikram Goyal 2019-05-09 01:31:20 UTC

(In reply to Jack Ottofaro from comment #7)
> Any updates?

As soon as 4.1 is out. :)

Comment 15 Kathryn Alexander 2019-06-07 19:05:18 UTC

The PR's here: https://github.com/openshift/openshift-docs/pull/15265

Jianlin, will you PTAL? The values are based on the information from https://docs.openshift.com/container-platform/3.11/install_config/master_node_configuration.html#master-node-config-project-config

Comment 16 Weihua Meng 2019-06-12 07:01:15 UTC

LGTM.

Comment 17 Kathryn Alexander 2019-06-12 14:48:00 UTC

I'm waiting for a peer review before I merge.

Comment 18 Kathryn Alexander 2019-06-18 16:47:45 UTC

This change is live on docs.openshift, eg https://docs.openshift.com/container-platform/3.11/install/configuring_inventory_file.html#configuring-project and https://docs.openshift.com/container-platform/3.6/install_config/install/advanced_install.html