fix: tracing

Replaces custom logger with `tracing_subscriber`.

Signed-off-by: Jonathan Woollett-Light <jcawl@amazon.co.uk>

Author: Jonathan Woollett-Light

Cargo.lock

@@ -21,6 +21,8 @@ mmds = { path = "../mmds" }
 seccompiler = { path = "../seccompiler" }
 utils = { path = "../utils" }
 vmm = { path = "../vmm" }
+tracing = { version = "0.1.37", features = ["attributes"] }
 
 [dev-dependencies]
 libc = "0.2.117"
+log = "0.4.19"

src/api_server/Cargo.toml

@@ -14,15 +14,14 @@ use std::fmt::Debug;
 use std::path::PathBuf;
 use std::sync::mpsc;
 
-use logger::{
-    debug, error, info, update_metric_with_elapsed_time, warn, ProcessTimeReporter, METRICS,
-};
+use logger::{update_metric_with_elapsed_time, ProcessTimeReporter, METRICS};
 pub use micro_http::{
     Body, HttpServer, Method, Request, RequestError, Response, ServerError, ServerRequest,
     ServerResponse, StatusCode, Version,
 };
 use seccompiler::BpfProgramRef;
 use serde_json::json;
+use tracing::{debug, error, info, warn};
 use utils::eventfd::EventFd;
 use vmm::rpc_interface::{VmmAction, VmmActionError, VmmData};
 use vmm::vmm_config::snapshot::SnapshotType;

src/api_server/src/lib.rs

@@ -3,10 +3,10 @@
 
 use std::fmt::Debug;
 
-use logger::{error, info, log_enabled, Level};
 use micro_http::{Body, Method, Request, Response, StatusCode, Version};
 use serde::ser::Serialize;
 use serde_json::Value;
+use tracing::{error, info};
 use vmm::rpc_interface::{VmmAction, VmmActionError};
 
 use super::VmmData;
@@ -229,19 +229,7 @@ fn log_received_api_request(api_description: String) {
 fn describe(method: Method, path: &str, body: Option<&Body>) -> String {
     match (path, body) {
         ("/mmds", Some(_)) | (_, None) => format!("{:?} request on {:?}", method, path),
-        ("/cpu-config", Some(payload_value)) => {
-            // If the log level is at Debug or higher, include the CPU template in
-            // the log line.
-            if log_enabled!(Level::Debug) {
-                describe_with_body(method, path, payload_value)
-            } else {
-                format!(
-                    "{:?} request on {:?}. To view the CPU template received by the API, \
-                     configure log-level to DEBUG",
-                    method, path
-                )
-            }
-        }
+        ("/cpu-config", Some(payload_value)) => describe_with_body(method, path, payload_value),
         (_, Some(payload_value)) => describe_with_body(method, path, payload_value),
     }
 }
@@ -763,7 +751,7 @@ pub mod tests {
     fn test_try_from_put_logger() {
         let (mut sender, receiver) = UnixStream::pair().unwrap();
         let mut connection = HttpConnection::new(receiver);
-        let body = "{ \"log_path\": \"string\", \"level\": \"Warning\", \"show_level\": false, \
+        let body = "{ \"log_path\": \"string\", \"level\": \"Warn\", \"show_level\": false, \
                     \"show_log_origin\": false }";
         sender
             .write_all(http_request("PUT", "/logger", Some(body)).as_bytes())