[Tarantool-patches] [PATCH v7 10/11] log/lua: allow to specify logging level as a string

Cyrill Gorcunov gorcunov at gmail.com
Tue Jun 2 11:12:41 MSK 2020


On Tue, Jun 02, 2020 at 11:05:57AM +0300, Oleg Babin wrote:
> Also. I've found that it's not allowed to set log level as string via box
> API:
> 
> ```
> 
> tarantool> box.cfg{log_level = 'info'}
> ---
> - error: 'Incorrect value for option ''log_level'': should be of type
> number'
> ...
> 
> ```
> 
> But it could be easily fixed here: https://github.com/tarantool/tarantool/blob/master/src/box/lua/load_cfg.lua#L129

Yeah, I left it on intent. Didn't want to change box.cfg api.
Currently the keywords are parsed in log module solely so I'll
have to provide some way to make similar inside load_cfg code,
and I tried to escape it.

You know, the base idea is to configure logging via log module
solely and maybe deprecate it from box.cfg.

Anyway even if I fix it I'll do it on top of the series.


More information about the Tarantool-patches mailing list