Re: Review Request 51258: Add documentation for the ExternalContainerLogger module.

2017-02-13 Thread Will Rouesnel
/51258/diff/ Testing --- Markdown is valid and parses correctly. Thanks, Will Rouesnel

Re: Review Request 51257: Add external process container logger.

2017-02-13 Thread Will Rouesnel
. The other basic checks are duplicated from the Logrotate container logger, from where this change inherits a lot of its code. Thanks, Will Rouesnel

Re: Review Request 51258: Add documentation for the ExternalContainerLogger module.

2016-10-11 Thread Will Rouesnel
/lib_externallogger.cpp PRE-CREATION src/tests/container_logger_external.sh PRE-CREATION src/tests/module.cpp 5b83fd6358ddea4c9d849b8992e1a6040ef74505 Diff: https://reviews.apache.org/r/51258/diff/ Testing --- Markdown is valid and parses correctly. Thanks, Will Rouesnel

Re: Review Request 51257: Add external process container logger.

2016-10-11 Thread Will Rouesnel
. Thanks, Will Rouesnel

Re: Review Request 51258: Add documentation for the ExternalContainerLogger module.

2016-09-25 Thread Will Rouesnel
, Will Rouesnel

Re: Review Request 51257: Add external process container logger.

2016-09-25 Thread Will Rouesnel
a lot of its code. Thanks, Will Rouesnel

Re: Review Request 51258: Add documentation for the ExternalContainerLogger module.

2016-09-24 Thread Will Rouesnel
, Will Rouesnel

Re: Review Request 51257: Add external process container logger.

2016-09-24 Thread Will Rouesnel
gs.add`. > > > > --- > > > > Also, add a newline after this block. This doesn't actually compile for me at the moment, unless something's changed on master I'm not rebased against? - Will -----

Re: Review Request 51257: Add external process container logger.

2016-09-24 Thread Will Rouesnel
. Thanks, Will Rouesnel

Re: Review Request 51258: Add documentation for the ExternalContainerLogger module.

2016-09-04 Thread Will Rouesnel
, Will Rouesnel

Re: Review Request 51257: Add external process container logger.

2016-09-02 Thread Will Rouesnel
code. Thanks, Will Rouesnel

Re: Review Request 51257: Add external process container logger.

2016-08-25 Thread Will Rouesnel
apache.org/r/51257/#review146379 --- On Aug. 20, 2016, 4:57 p.m., Will Rouesnel wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https:

Review Request 51258: Add documentation for the ExternalContainerLogger module.

2016-08-20 Thread Will Rouesnel
--- Add documentation for the ExternalContainerLogger module. Diffs - docs/logging.md 3b36870c22336440b0d7bf1359e1d0a97986a0f6 Diff: https://reviews.apache.org/r/51258/diff/ Testing --- Markdown is valid and parses correctly. Thanks, Will Rouesnel

Review Request 51257: Add external process container logger.

2016-08-20 Thread Will Rouesnel
/container_logger_external.sh which is setup to fail if any important output is unavailable to the logging process. The other basic checks are duplicated from the Logrotate container logger, from where this change inherits a lot of its code. Thanks, Will Rouesnel

Review Request 50898: Add new contributor (Will Rouesnel).

2016-08-08 Thread Will Rouesnel
contributor (Will Rouesnel). Diffs - docs/contributors.yaml fdb21d44a1c2289c05fa6cf898ca364de8c8c040 Diff: https://reviews.apache.org/r/50898/diff/ Testing --- Thanks, Will Rouesnel