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 API #347

Merged
merged 3 commits into from
Jun 29, 2023
Merged

Set Log Level API #347

merged 3 commits into from
Jun 29, 2023

Conversation

ArtiomDivak
Copy link
Contributor

@ArtiomDivak ArtiomDivak commented Jun 11, 2023

Api feature for setting the log level for the nodes and the manager.
for more details please read #189 (comment)

Copy link
Member

@mwperina mwperina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please mention in the commit subject that this patch add log level API to the manager only. Also please fit the typo in the commit message

@ArtiomDivak ArtiomDivak force-pushed the issue-189 branch 9 times, most recently from d222483 to 67b96b4 Compare June 14, 2023 07:47
@ArtiomDivak ArtiomDivak force-pushed the issue-189 branch 2 times, most recently from 9258008 to b6c21fe Compare June 14, 2023 08:00
@ArtiomDivak ArtiomDivak force-pushed the issue-189 branch 4 times, most recently from ea2da31 to 193af4a Compare June 20, 2023 08:43
@ArtiomDivak ArtiomDivak force-pushed the issue-189 branch 2 times, most recently from 0951a59 to 961c3f5 Compare June 27, 2023 07:58
I added public API to the Manager.
Now a user  can use the with sdbus to set new log leve for the manger

Signed-off-by: Artiom Divak <[email protected]>
The APIs of the node and the agent are connected.
The API of Agent is internal. So if the user wants
to change the log level for the agent he needs to
use the public API of the node

Signed-off-by: Artiom Divak <[email protected]>
@ArtiomDivak ArtiomDivak force-pushed the issue-189 branch 3 times, most recently from f176674 to 60e3e7b Compare June 29, 2023 06:42
hirtectl will support set-loglevel command. This command is to set log
level on specific node or on the manager. For more info and usage
please go to
eclipse-bluechi#189 (comment).

Signed-off-by: Artiom Divak <[email protected]>
Copy link
Member

@engelmi engelmi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@engelmi engelmi merged commit b6d8165 into eclipse-bluechi:main Jun 29, 2023
@engelmi engelmi linked an issue Jun 29, 2023 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Set LogLevel via D-Bus API
4 participants