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

Bug 1337428

Summary: [DOCS] Need to update documentation for docker log option in ansible installer
Product: OpenShift Container Platform Reporter: Kenjiro Nakayama <knakayam>
Component: DocumentationAssignee: Ashley Hardin <ahardin>
Status: CLOSED CURRENTRELEASE QA Contact: Kenjiro Nakayama <knakayam>
Severity: high Docs Contact: Vikram Goyal <vigoyal>
Priority: high    
Version: 3.2.0CC: aos-bugs, dgoodwin, jokerman, 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: 2016-05-31 14:15:29 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 Kenjiro Nakayama 2016-05-19 08:06:26 UTC
Description of problem:

- Since this bz#1319663 has been released, the doc should be updated.
- Please refer to this comment https://bugzilla.redhat.com/show_bug.cgi?id=1319663#c10 

Version-Release number of selected component (if applicable):

- v3.2 or later

Document URL: 

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

Section Number and Name: 

- explanation of docker_log_options

Describe the issue: 

Suggestions for improvement: 

- update as https://bugzilla.redhat.com/show_bug.cgi?id=1319663#c10 

Additional information: 

- doc author : https://github.com/openshift/openshift-docs/pull/1495

Comment 1 Kenjiro Nakayama 2016-05-19 09:34:41 UTC
There are two points to update in [1]:

1. Change the option name "docker_log_options" to "openshift_docker_log_options".
2. Add example settings (since the style is unique):
  (e.g)
  openshift_docker_log_options=["max-size=1M", "max-file=3"]

[1] https://docs.openshift.com/enterprise/3.2/install_config/install/advanced_install.html#configuring-host-variables

Comment 2 Devan Goodwin 2016-05-19 12:17:49 UTC
I submitted a PR for this: https://github.com/openshift/openshift-docs/pull/2129

Comment 3 Kenjiro Nakayama 2016-05-24 08:20:43 UTC
Ashley, we are waiting for the documentation sync now? 

I can't see the update yet, even though it has been merged. https://docs.openshift.com/enterprise/3.2/install_config/install/advanced_install.html#configuring-host-variables

Comment 4 Ashley Hardin 2016-05-24 13:18:41 UTC
Hi Kenjiro, 
Yes, our docs team is working on the weekly release. Once the change is cherry picked into the documentation, the changes will be visible to users.

Thanks!
Ashley