[Tarantool-patches] [PATCH v3 0/3] log: allow json formatter in boottime logger
Yaroslav Dynnikov
yaroslav.dynnikov at tarantool.org
Thu Jul 2 13:46:14 MSK 2020
Hi!
Thank you for the patchset, and thank Oleg for the detailed review!
Whole patch set LGTM.
Best regards
Yaroslav Dynnikov
On Thu, 2 Jul 2020 at 12:50, Cyrill Gorcunov <gorcunov at gmail.com> wrote:
> For some reason we've disabled usage of json formatter in early
> logging since the commit 098324556. Lets allow it back.
>
> v2:
> - add early verification of options for cases like
> box.cfg{log="syslog:", log_format="json"}
> - extend test
>
> v3:
> - pass the whole cfg table into verify_option helper
> - use string.startswith helper in verify_format
>
> branch gorcunov/gh-5121-logger-boot-json-3
> issue https://github.com/tarantool/tarantool/issues/5121
>
> Cyrill Gorcunov (3):
> core/say: allow to use json in boot logger
> lua/log: allow to use json formatter early
> test: app-tap/logger -- test json in boottime logger
>
> src/box/lua/load_cfg.lua | 6 +++---
> src/lib/core/say.c | 7 ++----
> src/lua/log.lua | 42 +++++++++++++++++++++++++++---------
> test/app-tap/logger.test.lua | 19 ++++++++++++----
> 4 files changed, 52 insertions(+), 22 deletions(-)
>
> --
> 2.26.2
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.tarantool.org/pipermail/tarantool-patches/attachments/20200702/6ba4b033/attachment.html>
More information about the Tarantool-patches
mailing list