log-level-config: add option to configure log levels #474

alexmohr · 2023-04-25T08:48:12Z

The default log level can now be overwritten with
a different level stored in a configuration file.
Levels can be set for all ctid's of an apid
or only selected ctids

The program was tested solely for our own use cases, which might differ from yours.
Licensed under Mozilla Public License Version 2.0

^{Alexander Mohr, [email protected], Mercedes-Benz Tech Innovation GmbH, imprint}

lti9hc · 2023-05-15T04:20:17Z

Hello @alexmohr,
Currently, dlt-daemon was failed with unknown option, because you forgot to add definition DLT_LOG_LEVEL_APP_CONFIG in CMakeList file

You should add it below codes in dlt-daemon/CMakeLists.txt
if (WITH_LOG_LEVEL_APP_CONFIG)
add_definitions(-DDLT_LOG_LEVEL_APP_CONFIG)
endif()

src/daemon/dlt-daemon.c

alexmohr · 2023-05-16T06:59:47Z

Hello @alexmohr, Currently, dlt-daemon was failed with unknown option, because you forgot to add definition DLT_LOG_LEVEL_APP_CONFIG in CMakeList file

You should add it below codes in dlt-daemon/CMakeLists.txt if (WITH_LOG_LEVEL_APP_CONFIG) add_definitions(-DDLT_LOG_LEVEL_APP_CONFIG) endif()

fixed 👍

CMakeLists.txt

lti9hc · 2023-05-22T09:14:43Z

Hello @alexmohr,
Thanks for your contribution an interesting PR.
I have tested this PR on my local environment. The result was good.

@michael-methner or @lvklevankhanh could merge this PR
Thank you!

doc/dlt_build_options.md

src/daemon/dlt-daemon.c

src/daemon/dlt-daemon.h

src/daemon/dlt_daemon_common.h

src/daemon/dlt_daemon_common.c

src/daemon/dlt-daemon.c

src/daemon/dlt_daemon_common.c

src/daemon/dlt-log-levels.conf

src/daemon/dlt_daemon_common.c

The default log level can now be overwritten with a different level stored in a configuration file. Levels can be set for all ctid's of an apid or only selected ctids Signed-off-by: Alexander Mohr <[email protected]>

michael-methner · 2023-05-24T09:09:37Z

Hello @alexmohr ,
thank you very much for the rework. PR is merged 👍

lti9hc reviewed May 16, 2023

View reviewed changes

src/daemon/dlt-daemon.c Show resolved Hide resolved

lti9hc reviewed May 16, 2023

View reviewed changes

src/daemon/dlt-daemon.c Show resolved Hide resolved

alexmohr force-pushed the log-level-config branch from 0625559 to 88d63a4 Compare May 16, 2023 06:59

lti9hc reviewed May 17, 2023

View reviewed changes

CMakeLists.txt Outdated Show resolved Hide resolved

lti9hc reviewed May 17, 2023

View reviewed changes

CMakeLists.txt Show resolved Hide resolved

lti9hc reviewed May 17, 2023

View reviewed changes

CMakeLists.txt Outdated Show resolved Hide resolved

alexmohr force-pushed the log-level-config branch 3 times, most recently from 5bc73b7 to 5f1aef6 Compare May 17, 2023 14:45

lvklevankhanh reviewed May 22, 2023

View reviewed changes

doc/dlt_build_options.md Outdated Show resolved Hide resolved

michael-methner reviewed May 22, 2023

View reviewed changes

src/daemon/dlt-daemon.c Outdated Show resolved Hide resolved