Removed rx_partial_writes

- metric is not used

Signed-off-by: Milan Dhaduk <[email protected]>
Signed-off-by: LDagnachew <[email protected]>

Author: LDagnachew

src/vmm/src/devices/virtio/net/metrics.rs

@@ -161,8 +161,6 @@ pub struct NetDeviceMetrics {
     pub rx_queue_event_count: SharedIncMetric,
     /// Number of events associated with the rate limiter installed on the receiving path.
     pub rx_event_rate_limiter_count: SharedIncMetric,
-    /// Number of RX partial writes to guest.
-    pub rx_partial_writes: SharedIncMetric,
     /// Number of RX rate limiter throttling events.
     pub rx_rate_limiter_throttled: SharedIncMetric,
     /// Number of events received on the associated tap.
@@ -233,8 +231,6 @@ impl NetDeviceMetrics {
             .add(other.rx_queue_event_count.fetch_diff());
         self.rx_event_rate_limiter_count
             .add(other.rx_event_rate_limiter_count.fetch_diff());
-        self.rx_partial_writes
-            .add(other.rx_partial_writes.fetch_diff());
         self.rx_rate_limiter_throttled
             .add(other.rx_rate_limiter_throttled.fetch_diff());
         self.rx_tap_event_count

tests/host_tools/fcmetrics.py

@@ -104,7 +104,6 @@ def validate_fc_metrics(metrics):
         "event_fails",
         "rx_queue_event_count",
         "rx_event_rate_limiter_count",
-        "rx_partial_writes",
         "rx_rate_limiter_throttled",
         "rx_tap_event_count",
         "rx_bytes_count",

tests/integration_tests/functional/test_firecracker_instrumenting.py

@@ -0,0 +1,37 @@
+import time
+import subprocess
+from pathlib import Path
+import re
+import pytest
+
+
+def test_clippy_tracing_instrumentation(uvm_plain):
+    """
+    Test Firecracker with Clippy linting and tracing instrumentation enabled.
+    """
+
+    vm = uvm_plain
+    vm.spawn(log_file=None)
+    vm.basic_config(boot_args="log-level=Trace")
+
+    log_path = Path(vm.path) / "log"
+    log_path.touch()
+    vm.api.logger.put(
+        log_path=vm.create_jailed_resource(log_path),
+        level="Trace",
+        show_level=True,
+        show_log_origin=True,
+    )
+    vm.log_file = log_path
+    vm.time_api_requests = False
+
+    vm.start()
+    time.sleep(10) 
+
+    logs = log_path.read_text()
+    print("Tracing Logs:")
+    print(logs)
+
+    assert ">>" in logs, "Expected tracing markers not found in logs"
+
+    vm.kill()

tests/integration_tests/functional/test_pause_resume.py

@@ -13,7 +13,6 @@ def verify_net_emulation_paused(metrics):
     """Verify net emulation is paused based on provided metrics."""
     net_metrics = metrics["net"]
     assert net_metrics["rx_queue_event_count"] == 0
-    assert net_metrics["rx_partial_writes"] == 0
     assert net_metrics["rx_tap_event_count"] == 0
     assert net_metrics["rx_bytes_count"] == 0
     assert net_metrics["rx_packets_count"] == 0