From 4fb07da7dab880b9f6e4acd8a06292869ac863bb Mon Sep 17 00:00:00 2001 From: Charles William Dick Date: Tue, 2 Aug 2022 16:15:01 +0900 Subject: [PATCH] don't log Ok in client We don't need to log that a request succeeded. BUG=b:240470388 TEST=tast run dut multivm.LifecycleShifting.arc_host verify those lines are gone in messages Change-Id: I375ca717328323ba4ebaebe87e90673f1e4776e8 Reviewed-on: https://chromium-review.googlesource.com/c/crosvm/crosvm/+/3805091 Commit-Queue: Charles William Dick Tested-by: Charles William Dick Auto-Submit: Charles William Dick Reviewed-by: Kameron Lutes Reviewed-by: Junichi Uekawa --- vm_control/src/client.rs | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/vm_control/src/client.rs b/vm_control/src/client.rs index e48842c870..d715d7edc5 100644 --- a/vm_control/src/client.rs +++ b/vm_control/src/client.rs @@ -6,7 +6,6 @@ use std::fs::OpenOptions; use std::path::Path; use std::path::PathBuf; -use base::info; use base::open_file; use remain::sorted; use thiserror::Error; @@ -42,8 +41,7 @@ pub fn vms_request + std::fmt::Debug>( request: &VmRequest, socket_path: T, ) -> VmsRequestResult { - let response = handle_request(request, socket_path)?; - info!("request response was {}", response); + handle_request(request, socket_path)?; Ok(()) }