Add option to disable syslog
output #675
|
@ -154,14 +154,14 @@ def make_parsers():
|
|||
type=int,
|
||||
choices=range(-2, 3),
|
||||
default=0,
|
||||
help='Display verbose progress to the console (from disabled, errors to very verbose: -2, -1, 0, 1, or 2)',
|
||||
help='Display verbose progress to the console (from disabled, errors only, default, some, or lots: -2, -1, 0, 1, or 2)',
|
||||
Soumik_Dutta marked this conversation as resolved
Outdated
|
||||
)
|
||||
global_group.add_argument(
|
||||
'--syslog-verbosity',
|
||||
type=int,
|
||||
choices=range(-2, 3),
|
||||
Soumik_Dutta marked this conversation as resolved
Outdated
witten
commented
For consistency, it would be good to add For consistency, it would be good to add `-2` support to the other verbosity flags as well. (Let me know if you disagree.)
Soumik_Dutta
commented
I was not sure initially about the scope of adding another global verbosity level so I tried to minimize the affected code. Though I agree that it will be good for consistency and I am working on it. Otherwise we will have to document around this. I was not sure initially about the scope of adding another global verbosity level so I tried to minimize the affected code.
Though I agree that it will be good for consistency and I am working on it. Otherwise we will have to document around this.
witten
commented
I'll admit I haven't thought through the scope implications, so if it grows too large, please let me know! I'll admit I haven't thought through the scope implications, so if it grows too large, please let me know!
|
||||
default=0,
|
||||
help='Log verbose progress to syslog (from disabled, errors to very verbose: -2, -1, 0, 1, or 2). Ignored when console is interactive or --log-file is given',
|
||||
help='Log verbose progress to syslog (from disabled, errors only, default, some, or lots: -2, -1, 0, 1, or 2). Ignored when console is interactive or --log-file is given',
|
||||
)
|
||||
global_group.add_argument(
|
||||
'--log-file-verbosity',
|
||||
|
@ -175,7 +175,7 @@ def make_parsers():
|
|||
type=int,
|
||||
choices=range(-2, 3),
|
||||
default=0,
|
||||
help='Log verbose progress to monitoring integrations that support logging (from disabled, errors to very verbose: -2, -1, 0, 1, or 2)',
|
||||
help='Log verbose progress to monitoring integrations that support logging (from disabled, errors only, default, some, or lots: -2, -1, 0, 1, or 2)',
|
||||
)
|
||||
global_group.add_argument(
|
||||
'--log-file',
|
||||
|
|
Loading…
Reference in New Issue
At this point, it might be clearer to change this help to something like:
Display verbose progress to the console (disabled, errors only, default, some, or lots: -2, -1, 0, 1, or 2)
Or even something like:
Display verbose progress to the console at the requested level: -2 (disabled), -1 (errors only), 0 (default), 1 (some), or 2 (lots)
Maybe remove "from" here and in the other helps, as it's no longer "from"/"to".