Bug 1657326

Summary: [DOCS] Deploying Registry is missing variable that must be manually defined.
Product: OpenShift Container Platform Reporter: Ryan Howe <rhowe>
Component: DocumentationAssignee: Ashley Hardin <ahardin>
Status: CLOSED CURRENTRELEASE QA Contact: Yadan Pei <yapei>
Severity: medium Docs Contact: Vikram Goyal <vigoyal>
Priority: unspecified    
Version: 3.11.0CC: aos-bugs, jokerman, jrosenta, mmccomas, wsun, wzheng, yapei
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: 2019-04-10 19:40:38 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 Ryan Howe 2018-12-07 16:26:59 UTC
Document URL: 
https://docs.openshift.com/container-platform/3.11/install_config/registry/deploy_registry_existing_clusters.html

Describe the issue: 

For imagestreams to work properly REGISTRY_OPENSHIFT_SERVER_ADDR must be equal to what is set for internalRegistryHostname set in the master-config.yaml. 

imagePolicyConfig:
internalRegistryHostname: docker-registry.default.svc:5000


Information is deep in the configuration doc but should be moved to deploying existing registry doc.  
https://docs.openshift.com/container-platform/3.11/install_config/registry/extended_registry_configuration.html#setting-the-registry-hostname

# oc set env dc docker-registry REGISTRY_OPENSHIFT_SERVER_ADDR=docker-registry.default.svc:5000


Ansible installer sets this: 
https://github.com/openshift/openshift-ansible/blob/release-3.11/roles/openshift_hosted/tasks/registry.yml#L15-L22

Comment 1 Ashley Hardin 2019-03-26 21:24:10 UTC
Work in progress: https://github.com/openshift/openshift-docs/pull/14242

Ryan, please review. Thanks!

Comment 2 Wei Sun 2019-04-10 03:13:04 UTC
Hi Yadan,please check if it could be verified.

Comment 3 Wenjing Zheng 2019-04-10 09:32:44 UTC
Verified the doc pr.