From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lj1-f194.google.com (mail-lj1-f194.google.com [209.85.208.194]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id 3D83945C320 for ; Tue, 2 Jun 2020 01:27:06 +0300 (MSK) Received: by mail-lj1-f194.google.com with SMTP id z6so10111475ljm.13 for ; Mon, 01 Jun 2020 15:27:06 -0700 (PDT) From: Cyrill Gorcunov Date: Tue, 2 Jun 2020 01:25:06 +0300 Message-Id: <20200601222507.560415-11-gorcunov@gmail.com> In-Reply-To: <20200601222507.560415-1-gorcunov@gmail.com> References: <20200601222507.560415-1-gorcunov@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [Tarantool-patches] [PATCH v7 10/11] log/lua: allow to specify logging level as a string List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: tml Allow to specify logging level as a string in log.cfg() call. @TarantoolBot document Title: Module log The `log.cfg({level='string'})` accepts not only number but symbolic repreentation of loglevels such as: `fatal`, `syserror`, `error`, `crit`, `warn`, `info`, `verbose` and `debug`. Part-of #689 Suggested-by: Oleg Babin Signed-off-by: Cyrill Gorcunov --- src/lua/log.lua | 36 ++++++++++++++++++++++++++++++++++-- test/app-tap/logger.test.lua | 29 ++++++++++++++++++++++++++++- 2 files changed, 62 insertions(+), 3 deletions(-) diff --git a/src/lua/log.lua b/src/lua/log.lua index dfdc326fe..3376932ff 100644 --- a/src/lua/log.lua +++ b/src/lua/log.lua @@ -140,6 +140,26 @@ local function update_box_cfg(k) end end +-- Logging levels symbolic representation. +local log_level_keys = { + ['fatal'] = ffi.C.S_FATAL, + ['syserror'] = ffi.C.S_SYSERROR, + ['error'] = ffi.C.S_ERROR, + ['crit'] = ffi.C.S_CRIT, + ['warn'] = ffi.C.S_WARN, + ['info'] = ffi.C.S_INFO, + ['verbose'] = ffi.C.S_VERBOSE, + ['debug'] = ffi.C.S_DEBUG, +} + +local function log_level_list() + local keyset = {} + for k in pairs(log_level_keys) do + keyset[#keyset + 1] = k + end + return table.concat(keyset, ',') +end + local function say(level, fmt, ...) if ffi.C.log_level < level then -- don't waste cycles on debug.getinfo() @@ -279,8 +299,20 @@ local function load_cfg(oldcfg, cfg) end if cfg.level ~= nil then - if type(cfg.level) ~= 'number' then - error("log: 'level' option must be a number") + if type(cfg.level) == 'number' then + if cfg.level < ffi.C.S_FATAL or + cfg.level > ffi.C.S_DEBUG then + local m = "log: 'level' must be in range [%d;%d]" + error(m:format(ffi.C.S_FATAL, ffi.C.S_DEBUG)) + end + elseif type(cfg.level) == 'string' then + if log_level_keys[cfg.level] == nil then + local m = "log: 'level' must be one of [%s]" + error(m:format(log_level_list())) + end + cfg.level = log_level_keys[cfg.level] + else + error("log: 'level' must be a number or string") end end diff --git a/test/app-tap/logger.test.lua b/test/app-tap/logger.test.lua index c6f207908..557c31f1a 100755 --- a/test/app-tap/logger.test.lua +++ b/test/app-tap/logger.test.lua @@ -1,7 +1,7 @@ #!/usr/bin/env tarantool local test = require('tap').test('log') -test:plan(34) +test:plan(44) -- gh-3946: Assertion failure when using log_format() before box.cfg() local log = require('log') @@ -60,6 +60,33 @@ log.cfg({format='plain', level=5}) test:is(box.cfg.log_format, 'plain', 'box sees plain format') test:is(box.cfg.log_level, 5, 'box sees level change') +-- Test symbolic names for loglevels +log.cfg({level='fatal'}) +test:ok(log.cfg.level == 0 and box.cfg.log_level == 0, 'both sees fatal') +log.cfg({level='syserror'}) +test:ok(log.cfg.level == 1 and box.cfg.log_level == 1, 'both sees fatal') +log.cfg({level='error'}) +test:ok(log.cfg.level == 2 and box.cfg.log_level == 2, 'both sees fatal') +log.cfg({level='crit'}) +test:ok(log.cfg.level == 3 and box.cfg.log_level == 3, 'both sees fatal') +log.cfg({level='warn'}) +test:ok(log.cfg.level == 4 and box.cfg.log_level == 4, 'both sees fatal') +log.cfg({level='info'}) +test:ok(log.cfg.level == 5 and box.cfg.log_level == 5, 'both sees fatal') +log.cfg({level='verbose'}) +test:ok(log.cfg.level == 6 and box.cfg.log_level == 6, 'both sees fatal') +log.cfg({level='debug'}) +test:ok(log.cfg.level == 7 and box.cfg.log_level == 7, 'both sees fatal') + +_,err = pcall(log, {level = -1}) +test:ok(err ~= nil, 'level = -1 error') + +_,err = pcall(log, {level = 'unknown'}) +test:ok(err ~= nil, "level = 'unknown' error") + +-- restore to info for next tests +log.cfg({level='info'}) + -- -- Check that Tarantool creates ADMIN session for #! script -- -- 2.26.2