-
Notifications
You must be signed in to change notification settings - Fork 29
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
CA-296532: mpathalert log spam #49
Conversation
http-svr/xmlrpc_client.ml
Outdated
@@ -21,6 +21,8 @@ open D | |||
|
|||
module E = Debug.Make(struct let name = "mscgen" end) | |||
|
|||
let initialize_logging = Debug.disable ~level:Syslog.Debug "mscgen" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you forget the ()
parameter?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated 👍
http-svr/xmlrpc_client.ml
Outdated
@@ -371,6 +373,7 @@ module Protocol = functor(F: FORMAT) -> struct | |||
(* Caution: req can contain sensitive information such as passwords in its parameters, | |||
* so we should not log the parameters or a string representation of the whole thing. | |||
* The name should be safe though, e.g. req.Rpc.name when F is XMLRPC. *) | |||
initialize_logging; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
initialize_logging
needs to be a function if you want to call it like this. Currently this is just a unit value.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated, thanks.
3061a0f
to
da8bf21
Compare
http-svr/xmlrpc_client.ml
Outdated
@@ -21,6 +21,8 @@ open D | |||
|
|||
module E = Debug.Make(struct let name = "mscgen" end) | |||
|
|||
let initialize_logging () = Debug.disable ~level:Syslog.Debug "mscgen" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does it work if we do it once here:
let () = Debug.disable ~level:Syslog.Debug "mscgen"
That would be more efficient than calling this in every rpc call.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, good point. I will change and test. thanks.
4881643
to
9474829
Compare
Disable Debug Level for brand 'mscgen' Signed-off-by: Min Li <min.li1@citrix.com>
9474829
to
2447c33
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After the last update I think this looks good.
Test passed, help to merge please. Thanks. |
Disable debug log level for brand 'mscgen'
Signed-off-by: Min Li min.li1@citrix.com