refactor: improve the log information

eth-cscs · May 23, 2024 · fb366e7 · fb366e7
1 parent b14a4ac
commit fb366e7
Show file tree

Hide file tree

Showing 5 changed files with 21 additions and 2 deletions.
diff --git a/src/bos/session/shasta/http_client.rs b/src/bos/session/shasta/http_client.rs
@@ -202,6 +202,8 @@ pub mod v2 {
         shasta_root_cert: &[u8],
         id_opt: Option<&str>,
     ) -> Result<Vec<BosSession>, Error> {
+        log::info!("Get BOS sessions '{}'", id_opt.unwrap_or("all available"));
+
         let client;
 
         let client_builder = reqwest::Client::builder()

diff --git a/src/bos/template/shasta/http_client.rs b/src/bos/template/shasta/http_client.rs
@@ -8,7 +8,10 @@ pub mod v1 {
         shasta_root_cert: &[u8],
         bos_session_template_id_opt: Option<&String>,
     ) -> Result<Vec<BosSessionTemplate>, reqwest::Error> {
-        log::info!("Get BOS sessiontemplte {:?}", bos_session_template_id_opt);
+        log::info!(
+            "Get BOS sessiontemplates '{}'",
+            bos_session_template_id_opt.unwrap_or(&"all available".to_string())
+        );
 
         let client_builder = reqwest::Client::builder()
             .add_root_certificate(reqwest::Certificate::from_pem(shasta_root_cert)?);

diff --git a/src/cfs/session.rs b/src/cfs/session.rs
@@ -20,6 +20,11 @@ pub mod shasta {
                 session_name_opt: Option<&String>,
                 is_succeded_opt: Option<bool>,
             ) -> Result<Vec<CfsSessionGetResponse>, Error> {
+                log::info!(
+                    "Get CFS sessions '{}'",
+                    session_name_opt.unwrap_or(&"all available".to_string())
+                );
+
                 let client_builder = reqwest::Client::builder()
                     .add_root_certificate(reqwest::Certificate::from_pem(shasta_root_cert)?);
 

diff --git a/src/ims/image/mesa/http_client.rs b/src/ims/image/mesa/http_client.rs
@@ -6,6 +6,11 @@ pub async fn get(
     shasta_root_cert: &[u8],
     image_id_opt: Option<&str>,
 ) -> Result<Vec<Image>, reqwest::Error> {
+    log::info!(
+        "Get IMS images '{}'",
+        image_id_opt.unwrap_or("all available")
+    );
+
     let response_rslt = crate::ims::image::shasta::http_client::get_raw(
         shasta_token,
         shasta_base_url,

diff --git a/src/ims/image/shasta/http_client.rs b/src/ims/image/shasta/http_client.rs
@@ -6,7 +6,11 @@ pub async fn get_raw(
     shasta_root_cert: &[u8],
     image_id_opt: Option<&str>,
 ) -> Result<reqwest::Response, reqwest::Error> {
-    log::info!("Fetching images - id: {:?}", image_id_opt);
+    log::info!(
+        "Get IMS images '{}'",
+        image_id_opt.unwrap_or("all available")
+    );
+
     let client_builder = reqwest::Client::builder()
         .add_root_certificate(reqwest::Certificate::from_pem(shasta_root_cert)?);