Bug 1374890

Summary: [docs] documentation should mention about "oreg_url" and "openshift_examples_modify_imagestreams" parmeters
Product: OpenShift Container Platform Reporter: Kenjiro Nakayama <knakayam>
Component: DocumentationAssignee: brice <bfallonf>
Status: CLOSED CURRENTRELEASE QA Contact: Johnny Liu <jialiu>
Severity: medium Docs Contact: Vikram Goyal <vigoyal>
Priority: low    
Version: 3.2.1CC: aos-bugs, jokerman, knakayam, 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: 2017-02-09 01:22:34 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:

Description Kenjiro Nakayama 2016-09-10 03:42:31 UTC 
Document URL:

  https://docs.openshift.com/enterprise/3.2/install_config/install/advanced_install.html#configuring-cluster-variables

Section Number and Name:

  Configuring Cluster Variables

Describe the issue:

  To change imageConfig.format values in /etc/origin/master/master-config.yaml and /etc/origin/node/node-config.yaml, "oreg_url" should be explained in the openshift docs.


Suggestions for improvement:

  Variable: oreg_url
  Example: oreg_url=example.com/openshift3/ose-${component}:${version}
  Purpose: <Please refer to the comment in [3]>

  Variable: openshift_examples_modify_imagestreams
  Example:  openshift_examples_modify_imagestreams=True
  Purpose: <Please refer to the comment in [3]>

Additional information:
 
  These values are necessary for users who are not using "registry.access.redhat.com". They need to configure these values during installation.

[1] /etc/origin/master/master-config.yaml
imageConfig:
  format: openshift3/ose-${component}:${version}

[2] /etc/origin/node/node-config.yaml
imageConfig:
  format: openshift3/ose-${component}:${version}
[3] https://github.com/openshift/openshift-ansible/blob/enterprise-3.2/inventory/byo/hosts.aep.example#L100-L106
Comment 1 brice 2017-01-10 04:26:26 UTC 
Kenjiro,

I've created a PR for this:

https://github.com/openshift/openshift-docs/pull/3471

Can I ask if you think this fulfills this issue?

Thanks!
Comment 2 Kenjiro Nakayama 2017-01-11 11:58:04 UTC 
Thank you, Brice! Give me some time, I will review the PR soon.
Comment 3 brice 2017-01-16 05:30:32 UTC 
Kenjiro,

Bumping this. This PR will have to merge this week, so please let me know what you think soon. I hope that it fulfills this BZ.

Thank you in advance.
Comment 4 Kenjiro Nakayama 2017-01-16 05:55:04 UTC 
Sorry for my delay. I commented on the PR.
Comment 5 openshift-github-bot 2017-01-20 00:15:03 UTC 
Commit pushed to master at https://github.com/openshift/openshift-docs

https://github.com/openshift/openshift-docs/commit/91cf5caee14016d9b8c21bd150c2aae88ff51744
Merge pull request #3471 from bfallonf/variables_1374890

Bug 1374890 added Configuring a Registry Location section
Comment 6 brice 2017-02-09 01:22:34 UTC 
Link to released docs:

https://access.redhat.com/documentation/en/openshift-container-platform/3.4/single/installation-and-configuration/#configuring-ansible