Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1852448

Summary: GCP UPI 05_control_plane.yaml and 06_worker.yaml has a redundant key/value pair region: '${REGION}'
Product: OpenShift Container Platform Reporter: Yang Yang <yanyang>
Component: DocumentationAssignee: Kathryn Alexander <kalexand>
Status: CLOSED CURRENTRELEASE QA Contact: Yang Yang <yanyang>
Severity: low Docs Contact: Vikram Goyal <vigoyal>
Priority: low    
Version: 4.4CC: aos-bugs, jokerman, kalexand
Target Milestone: ---   
Target Release: 4.4.z   
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: 2020-08-04 17:54:42 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 Yang Yang 2020-06-30 12:42:49 UTC
Document URL: 
https://docs.openshift.com/container-platform/4.4/installing/installing_gcp/installing-gcp-user-infra.html
https://docs.openshift.com/container-platform/4.3/installing/installing_gcp/installing-gcp-user-infra.html
https://docs.openshift.com/container-platform/4.2/installing/installing_gcp/installing-gcp-user-infra.html

Section Number and Name: 
Creating the control plane machines in GCP
Creating additional worker machines in GCP

Describe the issue: 
05_control_plane.py and 06_worker.py do not have variable region, but 05_control_plane.yaml and 06_worker.yaml have

Suggestions for improvement: 
Suggest to remove region: '${REGION}' from 05_control_plane.yaml and 06_worker.yaml


Additional information:

Comment 1 Kathryn Alexander 2020-07-31 19:46:28 UTC
Trevor removed the variable from the workers on https://github.com/openshift/openshift-docs/pull/23500,

The PR to remove the variable from the control plane is here: https://github.com/openshift/openshift-docs/pull/24384

Yang Yang, will you PTAL?

Comment 2 Yang Yang 2020-08-03 13:43:08 UTC
The fix looks good to me. Moving it to verified state.

Comment 3 Kathryn Alexander 2020-08-03 17:33:31 UTC
Thanks! I've merged the change and am waiting for it to go live.