report error from container cpu usage monitor

Signed-off-by: nix1n <nikhil.murari@hotstar.com>
envoyproxy · Jan 8, 2025 · 114f444 · 114f444
1 parent ca96425
commit 114f444
Showing 1 changed file with 19 additions and 0 deletions.
diff --git a/test/extensions/resource_monitors/cpu_utilization/cpu_utilization_monitor_test.cc b/test/extensions/resource_monitors/cpu_utilization/cpu_utilization_monitor_test.cc
@@ -163,6 +163,25 @@ TEST(ContainerCpuUsageMonitorTest, GetsErroneousStatsNumerator) {
   ASSERT_TRUE(resource.hasError());
 }
 
+TEST(ContainerCpuUtilizationMonitorTest, ReportsError) {
+  envoy::extensions::resource_monitors::cpu_utilization::v3::CpuUtilizationConfig config;
+  config.set_mode(
+      envoy::extensions::resource_monitors::cpu_utilization::v3::CpuUtilizationConfig::CONTAINER);
+  auto stats_reader = std::make_unique<MockContainerStatsReader>();
+  EXPECT_CALL(*stats_reader, getCgroupStats())
+      .WillOnce(Return(CpuTimes{false, 0, 0}))
+      .WillOnce(Return(CpuTimes{false, 0, 0}))
+      .WillOnce(Return(CpuTimes{false, 0, 200}));
+  auto monitor = std::make_unique<CpuUtilizationMonitor>(config, std::move(stats_reader));
+
+  ResourcePressure resource;
+  monitor->updateResourceUsage(resource);
+  ASSERT_TRUE(resource.hasError());
+
+  monitor->updateResourceUsage(resource);
+  ASSERT_TRUE(resource.hasError());
+}
+
 } // namespace
 } // namespace CpuUtilizationMonitor
 } // namespace ResourceMonitors