chore(common): remove unused code and suppress warnings

- Prefixed unused guest server function with underscore to prevent dead code warning
- Cleaned up commented code and clarified comments
- No functional changes, only warning suppression and code hygiene

Author: LGSI-KarumuriHari

src/agent/nodeagent/Cargo.toml

@@ -4,6 +4,9 @@ version = "0.1.0"
 edition = "2021"
 license = "Apache-2.0"
 
+[features]
+tarpaulin_include = []
+
 [dependencies]
 common = { workspace = true }
 tonic = "0.12.3"

src/agent/nodeagent/src/grpc/sender.rs

@@ -5,7 +5,7 @@
 
 use common::apiserver::api_server_connection_client::ApiServerConnectionClient;
 use common::monitoringserver::{
-    ContainerList, NodeInfo, SendContainerListResponse, SendNodeInfoResponse,
+    ContainerList, SendContainerListResponse,
 };
 use common::nodeagent::{
     HeartbeatRequest, HeartbeatResponse, NodeRegistrationRequest, NodeRegistrationResponse,
@@ -37,10 +37,10 @@ impl NodeAgentSender {
             }
             Err(e) => {
                 // Handle connection error
-                return Err(Status::unknown(format!(
+                Err(Status::unknown(format!(
                     "Failed to connect statemanager: {}",
                     e
-                )));
+                )))
             }
         }
     }

src/agent/nodeagent/src/main.rs

@@ -117,7 +117,8 @@ async fn initialize(tx_grpc: Sender<HandleYamlRequest>, hostname: String, config
 
     // Use hostname from config if available
     let config_hostname = config.get_hostname();
-    let hostname_to_check = if config_hostname.is_empty() || config_hostname == "$(hostname)" {
+    //adding _ to prevent warnings as it was never used anywhere to prevent warnings
+    let _hostname_to_check = if config_hostname.is_empty() || config_hostname == "$(hostname)" {
         hostname.clone()
     } else {
         config_hostname
@@ -151,7 +152,7 @@ struct Args {
     config: PathBuf,
 }
 
-#[cfg(not(tarpaulin_include))]
+#[cfg(not(feature = "tarpaulin_include"))]
 #[tokio::main]
 async fn main() {
     // Parse command line arguments

src/common/build.rs

@@ -7,7 +7,7 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
     tonic_build::configure()
         .type_attribute(".", "#[derive(serde::Serialize, serde::Deserialize)]")
         .protoc_arg("--experimental_allow_proto3_optional")
-        .compile(
+        .compile_protos(
             &[
                 "proto/apiserver.proto",
                 "proto/actioncontroller.proto",

src/common/src/lib.rs

@@ -24,7 +24,8 @@ fn connect_server(port: u16) -> String {
 }
 
 // guest 서버 연결 함수 수정: 이제 항상 호스트 서버 주소 반환
-fn connect_guest_server(port: u16) -> String {
+//using rust build in _ to below code , as it was never used anywhere to prevent warnings
+fn _connect_guest_server(port: u16) -> String {
     // 항상 호스트 서버 주소 반환
     connect_server(port)
 }

src/common/src/spec/k8s/pod.rs

@@ -98,7 +98,7 @@ impl PodSpec {
     /// If no containers are present, returns `None`.
     pub fn get_image(&self) -> Option<&str> {
         self.containers
-            .get(0)
+            .first()
             .map(|container| container.image.as_str())
     }