Bug 1471668 - "Aggregated logging is only supported using the journald driver in Docker" is wrong in documentations.
Summary: "Aggregated logging is only supported using the journald driver in Docker" is...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Documentation
Version: 3.5.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: ---
Assignee: Brandi Munilla
QA Contact: Vikram Goyal
Vikram Goyal
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-07-17 08:24 UTC by Junqi Zhao
Modified: 2021-06-10 12:37 UTC (History)
9 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-05-02 19:34:04 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Junqi Zhao 2017-07-17 08:24:19 UTC
Logging not only support journald log driver, but also support json-file log driver now,  in the following documentations, there are description: 
"Aggregated logging is only supported using the journald driver in Docker", this is wrong.
I searched "Aggregated logging is only supported using the journald driver in Docker" at 
https://docs.openshift.com/container-platform/3.5/welcome/index.html
https://docs.openshift.org/latest/welcome/index.html

There are  6 pages is wrong, maybe not limited to these pages.

Document URL: 
https://docs.openshift.com/container-platform/3.5/install_config/aggregate_logging.html
https://docs.openshift.com/container-platform/3.5/install_config/install/host_preparation.html
https://docs.openshift.com/container-platform/3.5/install_config/revhistory_install_config.html
https://docs.openshift.com/container-platform/3.5/welcome/revhistory_full.html

https://docs.openshift.org/latest/install_config/install/host_preparation.html
https://docs.openshift.org/latest/install_config/aggregate_logging.html

Section Number and Name: 
Search "Aggregated logging is only supported using the journald driver in Docker" in the above pages.


Describe the issue: 
"Aggregated logging is only supported using the journald driver in Docker", Logging also supports json-file log driver.

Suggestions for improvement: 
Change "Aggregated logging is only supported using the journald driver in Docker" to
"Aggregated logging uses journald as default log driver in Docker".

Additional information:

Comment 1 Junqi Zhao 2017-07-17 08:26:16 UTC
@Jeff,

Do you have comments about "Suggestions for improvement" part, is that OK for the improvement?

Comment 2 Jeff Cantrill 2017-08-08 16:04:37 UTC
It is correct that we can support both drivers but we recommend using journald, the reasons that escape me now. We need to confirm if the statement should be something like:

* We recommend journal
* We support json as well.

Rich can you comment?

Comment 3 Rich Megginson 2017-08-08 16:16:58 UTC
We recommend journal because there is a lot more metadata available, but json-file is supported.

Comment 4 Peter Portante 2017-08-08 17:37:28 UTC
(In reply to Jeff Cantrill from comment #2)
> It is correct that we can support both drivers but we recommend using
> journald, the reasons that escape me now. We need to confirm if the
> statement should be something like:
> 
> * We recommend journal
> * We support json as well.

I think we have to support json-file until journald fully supports isolated quotas for amount of logs (curation) and rate of logging per-namespace.  Once it does, we'll want to drop json-file since there are performance implications with using it (file-system buffer pollution from all the writes, number of open file handles required to scale up per-node pod counts, and probably others).


Note You need to log in before you can comment on or make changes to this bug.