From 63ad09f96e12f7db149c73db5ce06492d317d460 Mon Sep 17 00:00:00 2001 From: zz_y Date: Thu, 11 Jun 2026 13:40:57 -0600 Subject: [PATCH] =?UTF-8?q?test(monitor):=20add=20rate=20field=20to=20moni?= =?UTF-8?q?tor=5Fgrpc=20report()=20=E2=80=94=20fix=20integration-test=20co?= =?UTF-8?q?mpile?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The dynamic sampling-coupling change (#368) added MonitorReport.rate but didn't update the tests/monitor_grpc.rs report() helper, leaving the integration test uncompilable (E0063: missing field `rate`). The --lib suite was green so it slipped through (integration tests build separately). Set rate: 0.0 (unknown — preserves the existing slack-only behavior the test asserts). The live-gRPC alert test now compiles + passes: fires_alert_once_past_tau (alert fires once at estimate≥(1-ε)τ over real gRPC) + quiet_below_tau. Co-Authored-By: Claude Opus 4.8 (1M context) --- data_plane/tests/monitor_grpc.rs | 1 + 1 file changed, 1 insertion(+) diff --git a/data_plane/tests/monitor_grpc.rs b/data_plane/tests/monitor_grpc.rs index a0dd0f85..aa9e212f 100644 --- a/data_plane/tests/monitor_grpc.rs +++ b/data_plane/tests/monitor_grpc.rs @@ -56,6 +56,7 @@ fn report(value: f64, seq: u64) -> EdgeToCoord { local_value: value, round: 0, seq, + rate: 0.0, })), } }