Skip to content

logging: document dual-logging cache options#12696

Merged
usha-mandya merged 1 commit intodocker:masterfrom
thaJeztah:dual_logging_options
Apr 23, 2021
Merged

logging: document dual-logging cache options#12696
usha-mandya merged 1 commit intodocker:masterfrom
thaJeztah:dual_logging_options

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

relates to moby/moby#40543 (comment)

While dual-logging is enabled automatically, it is possible to disable
this feature, or to configure the default options. This patch adds
documentation for these options.

@thaJeztah thaJeztah added the area/engine Issue affects Docker engine/daemon label Apr 17, 2021
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 17, 2021

Deploy preview for docsdocker ready!

Built with commit c16602d

https://deploy-preview-12696--docsdocker.netlify.app

@usha-mandya
Copy link
Copy Markdown
Member

@thaJeztah The PR looks good. Added a minor observation about a cross-reference.

While dual-logging is enabled automatically, it is possible to disable
this feature, or to configure the default options. This patch adds
documentation for these options.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the dual_logging_options branch from c5ba94f to c16602d Compare April 23, 2021 10:55
Copy link
Copy Markdown
Member

@usha-mandya usha-mandya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@usha-mandya usha-mandya merged commit cffa291 into docker:master Apr 23, 2021
@thaJeztah thaJeztah deleted the dual_logging_options branch April 23, 2021 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/engine Issue affects Docker engine/daemon

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants