Bug 1543869

Summary:	openshift_docker_log_driver doesn't work
Product:	OpenShift Container Platform	Reporter:	Gan Huang <ghuang>
Component:	Installer	Assignee:	Michael Gugino <mgugino>
Status:	CLOSED ERRATA	QA Contact:	Gan Huang <ghuang>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	3.9.0	CC:	aos-bugs, jokerman, lmeyer, mmccomas, rteague
Target Milestone:	---
Target Release:	3.9.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:	2018-03-28 14:27:28 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 Gan Huang 2018-02-09 12:54:53 UTC

Description of problem:
Set openshift_docker_log_driver to journald in inventory file, but it still resulted in that the docker log driver defaulted to json-file.

Version-Release number of the following components:
openshift-ansible-3.9.0-0.38.0.git.0.57e1184.el7.noarch

How reproducible:
always

Steps to Reproduce:
1. Set openshift_docker_log_driver=journald in inventory file
2. trigger intallation
3.

Actual results:
Docker log driver was not set to /etc/sysconfig/docker. The docker log driver still defaulted to json-file

Expected results:
The docker log driver set journald as expected.

Additional info:
Please attach logs from ansible-playbook with the -vvv flag

Comment 1 Gan Huang 2018-02-09 12:55:43 UTC

Proposed PR: https://github.com/openshift/openshift-ansible/pull/7085

Comment 2 Russell Teague 2018-02-13 14:57:42 UTC

Assigning to mgugino since he is already involved in the PR above.

Comment 5 Gan Huang 2018-02-23 07:05:52 UTC

Verified in openshift-ansible-3.9.0-0.50.0.git.0.bb78b91.el7.noarch.rpm

Comment 8 errata-xmlrpc 2018-03-28 14:27:28 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2018:0489

Comment 9 Luke Meyer 2018-03-28 19:19:42 UTC

It's worth noting that this option is undocumented, and prior to 3.9, it would be overridden by the default openshift_docker_options which included "--log-driver=journald".