Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set log level at plugin init #75

Open
FlorentP42 opened this issue Dec 14, 2023 · 1 comment · Fixed by #79
Open

Set log level at plugin init #75

FlorentP42 opened this issue Dec 14, 2023 · 1 comment · Fixed by #79
Assignees
Labels
enhancement New feature or request

Comments

@FlorentP42
Copy link

Is your feature request related to a problem? Please describe.
During testing or debugging, some logs that happen very early in the launch process could be missed if the lowest log level (debug) is not enabled from the start.

Describe the solution you'd like
Add a compile-time variable START_LOG_LEVEL that, if defined, will become the log level of this plugin at startup.
Example of usage when compiling for tests:

cmake -DCMAKE_BUILD_TYPE=Release -DFLEDGE_INCLUDE=/usr/local/fledge/include/ -DFLEDGE_LIB=/usr/local/fledge/lib/ -DCMAKE_CXX_FLAGS=-DSTART_LOG_LEVEL=debug ..

Describe alternatives you've considered
N/A

Additional context
Example of implementation in: fledge-power/fledge-south-hnz#77

@aklira aklira self-assigned this Jan 29, 2024
@aklira aklira added the enhancement New feature or request label Jan 29, 2024
@aklira aklira linked a pull request Feb 1, 2024 that will close this issue
@aklira aklira closed this as completed in #79 Feb 1, 2024
@FlorentP42
Copy link
Author

FlorentP42 commented Feb 1, 2024

Actually this was a different topic than the erroneous debug level fixed by #79, reopening.

@FlorentP42 FlorentP42 reopened this Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants