Removed sync_vmm_send_timeout_count + cleaned up

LDagnachew · LDagnachew · commit 637edc0c5074 · 2025-05-05T07:39:08.000Z
Signed-off-by: Milan Dhaduk &lt;mdhaduk7@gmail.com&gt;

Signed-off-by: LDagnachew &lt;leulmdagnachew@gmail.com&gt;
diff --git a/src/vmm/src/logger/metrics.rs b/src/vmm/src/logger/metrics.rs
@@ -354,17 +354,13 @@ pub struct ApiServerMetrics {
     pub process_startup_time_us: SharedStoreMetric,
     /// Measures the cpu's startup time in microseconds.
     pub process_startup_time_cpu_us: SharedStoreMetric,
-    /// Number of timeouts during communication with the VMM.
-    pub sync_vmm_send_timeout_count: SharedIncMetric,
 }
 impl ApiServerMetrics {
     /// Const default construction.
     pub const fn new() -> Self {
         Self {
             process_startup_time_us: SharedStoreMetric::new(),
             process_startup_time_cpu_us: SharedStoreMetric::new(),
-            sync_response_fails: SharedIncMetric::new(),
-            sync_vmm_send_timeout_count: SharedIncMetric::new(),
         }
     }
 }
diff --git a/tests/host_tools/fcmetrics.py b/tests/host_tools/fcmetrics.py
@@ -131,8 +131,6 @@ def validate_fc_metrics(metrics):
         "api_server": [
             "process_startup_time_us",
             "process_startup_time_cpu_us",
-            "sync_response_fails",
-            "sync_vmm_send_timeout_count",
         ],
         "balloon": [
             "activate_fails",

Original file line number	Diff line number	Diff line change
`@@ -354,17 +354,13 @@ pub struct ApiServerMetrics {`
`354`	`354`	`pub process_startup_time_us: SharedStoreMetric,`
`355`	`355`	`/// Measures the cpu's startup time in microseconds.`
`356`	`356`	`pub process_startup_time_cpu_us: SharedStoreMetric,`
`357`		`- /// Number of timeouts during communication with the VMM.`
`358`		`- pub sync_vmm_send_timeout_count: SharedIncMetric,`
`359`	`357`	`}`
`360`	`358`	`impl ApiServerMetrics {`
`361`	`359`	`/// Const default construction.`
`362`	`360`	`pub const fn new() -> Self {`
`363`	`361`	`Self {`
`364`	`362`	`process_startup_time_us: SharedStoreMetric::new(),`
`365`	`363`	`process_startup_time_cpu_us: SharedStoreMetric::new(),`
`366`		`- sync_response_fails: SharedIncMetric::new(),`
`367`		`- sync_vmm_send_timeout_count: SharedIncMetric::new(),`
`368`	`364`	`}`
`369`	`365`	`}`
`370`	`366`	`}`