chore: Reduce cpu-config API log verbosity

PUT /cpu-config API will include the request payload which
should contain the CPU config template only for DEBUG log levels
or higher.

Signed-off-by: Matthew Schlebusch <[email protected]>

Author: mattschlebusch

src/api_server/src/parsed_request.rs

@@ -1,7 +1,7 @@
 // Copyright 2019 Amazon.com, Inc. or its affiliates. All Rights Reserved.
 // SPDX-License-Identifier: Apache-2.0
 
-use logger::{error, info};
+use logger::{error, info, log_enabled, Level};
 use micro_http::{Body, Method, Request, Response, StatusCode, Version};
 use serde::ser::Serialize;
 use serde_json::Value;
@@ -232,17 +232,34 @@ 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),
-        (_, Some(value)) => format!(
-            "{:?} request on {:?} with body {:?}",
-            method,
-            path,
-            std::str::from_utf8(value.body.as_slice())
-                .unwrap_or("inconvertible to UTF-8")
-                .to_string()
-        ),
+        ("/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
+                )
+            }
+        }
+        (_, Some(payload_value)) => describe_with_body(method, path, payload_value),
     }
 }
 
+fn describe_with_body(method: Method, path: &str, payload_value: &Body) -> String {
+    format!(
+        "{:?} request on {:?} with body {:?}",
+        method,
+        path,
+        std::str::from_utf8(payload_value.body.as_slice())
+            .unwrap_or("inconvertible to UTF-8")
+            .to_string()
+    )
+}
+
 /// Generates a `GenericError` for each request method.
 pub(crate) fn method_to_error(method: Method) -> Result<ParsedRequest, Error> {
     match method {

tests/integration_tests/build/test_coverage.py

@@ -25,7 +25,7 @@ def is_on_skylake():
 # Checkout the cpuid crate. In the future other
 # differences may appear.
 if utils.is_io_uring_supported():
-    COVERAGE_DICT = {"Intel": 83.76, "AMD": 83.34, "ARM": 83.12}
+    COVERAGE_DICT = {"Intel": 83.76, "AMD": 83.34, "ARM": 83.18}
 else:
     COVERAGE_DICT = {"Intel": 81.02, "AMD": 80.55, "ARM": 80.19}