LGTM.

On Mon, Sep 12, 2016 at 2:41 PM, 'Brian Foley' via ganeti-devel <
[email protected]> wrote:

> It's been useful information when debugging performance issues in
> 2.15/2.16, but not useful enough to leave turned on in non-debug builds
> as it clutters up commands.log, jobs.log, rapi-daemon.log, watcher.log
>
> Signed-off-by: Brian Foley <[email protected]>
> ---
>  lib/rpc/client.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/lib/rpc/client.py b/lib/rpc/client.py
> index 267353b..ce586bd 100644
> --- a/lib/rpc/client.py
> +++ b/lib/rpc/client.py
> @@ -146,8 +146,8 @@ def CallRPCMethod(transport_cb, method, args,
> version=None):
>    t3 = time.time() * 1000
>    (success, result, resp_version) = ParseResponse(response_msg)
>    t4 = time.time() * 1000
> -  logging.info("CallRPCMethod %s: format: %dms, sock: %dms, parse: %dms",
> -               method, int(t2 - t1), int(t3 - t2), int(t4 - t3))
> +  logging.debug("CallRPCMethod %s: format: %dms, sock: %dms, parse: %dms",
> +                method, int(t2 - t1), int(t3 - t2), int(t4 - t3))
>    # Verify version if there was one in the response
>    if resp_version is not None and resp_version != version:
>      raise LuxiError("RPC version mismatch, client %s, response %s" %
> --
> 2.8.0.rc3.226.g39d4020
>
>

Reply via email to