Nir Soffer has posted comments on this change.

Change subject: docs: add README about logging configuration
......................................................................


Patch Set 6:

(1 comment)

https://gerrit.ovirt.org/#/c/60533/6/README.logging
File README.logging:

Line 23: 
Line 24: Using vdsClient, you can tune the log level of any logger Vdsm uses.
Line 25: The basic syntax is:
Line 26: 
Line 27: # vdsClient [OPTIONS] <server> setLogLevel LEVEL [logger]
Do we have other options?

Also, we should be consistent about values case.

How about:

    vdsClient [-s] SERVER setLogLevel LEVEL [LOGGER]
Line 28: 
Line 29: `LEVEL` is one of DEBUG, INFO, WARNING
Line 30: `logger` is any logger Vdsm uses. Please check the configuration file
Line 31: to learn about some predefined loggers.


-- 
To view, visit https://gerrit.ovirt.org/60533
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2653c2a68fcf42b1e085ae1ee703fb119a0cf1fc
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Milan Zamazal <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: gerrit-hooks <[email protected]>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/admin/lists/[email protected]

Reply via email to