From fbe2ace6b7d777e3fe2b5a7423f67d470a7b36ee Mon Sep 17 00:00:00 2001 From: Mauri de Souza Meneguzzo Date: Thu, 20 Nov 2025 11:58:02 -0300 Subject: [PATCH] otel(tests): skip flaky assertion in TestContainerCMDAgentMonitoringRuntimeExperimental (#11231) (cherry picked from commit 259dd3e9664b24f97c4131f98233fe2dd2c2b8c6) --- testing/integration/ess/container_cmd_test.go | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/testing/integration/ess/container_cmd_test.go b/testing/integration/ess/container_cmd_test.go index c90347d71fc..6b83db23568 100644 --- a/testing/integration/ess/container_cmd_test.go +++ b/testing/integration/ess/container_cmd_test.go @@ -550,9 +550,12 @@ func TestContainerCMDAgentMonitoringRuntimeExperimental(t *testing.T) { } t.Logf("Component ID: %s, version info: %s, runtime: %s", comp.ID, comp.VersionInfo.Name, compRuntime) switch comp.ID { - case "beat/metrics-monitoring", "filestream-monitoring", "http/metrics-monitoring", "prometheus/metrics-monitoring": + case "beat/metrics-monitoring", "filestream-monitoring", "prometheus/metrics-monitoring": // Monitoring components should use the expected runtime assert.Equalf(t, tc.expectedRuntimeName, compRuntime, "expected correct runtime name for monitoring component %s with id %s", comp.Name, comp.ID) + case "http/metrics-monitoring": + // The comp.VersionInfo.Name for this component is empty at times. + // See https://github.com/elastic/elastic-agent/issues/11162. default: // Non-monitoring components should use the default runtime assert.Equalf(t, string(component.DefaultRuntimeManager), compRuntime, "expected default runtime for non-monitoring component %s with id %s", comp.Name, comp.ID) @@ -671,9 +674,12 @@ func TestContainerCMDAgentMonitoringRuntimeExperimentalPolicy(t *testing.T) { } t.Logf("Component ID: %s, version info: %s, runtime: %s", comp.ID, comp.VersionInfo.Name, compRuntime) switch comp.ID { - case "beat/metrics-monitoring", "filestream-monitoring", "http/metrics-monitoring", "prometheus/metrics-monitoring": + case "beat/metrics-monitoring", "filestream-monitoring", "prometheus/metrics-monitoring": // Monitoring components should use the expected runtime assert.Equalf(t, tc.expectedRuntimeName, compRuntime, "unexpected runtime name for monitoring component %s with id %s", comp.Name, comp.ID) + case "http/metrics-monitoring": + // The comp.VersionInfo.Name for this component is empty at times. + // See https://github.com/elastic/elastic-agent/issues/11162. default: // Non-monitoring components should use the default runtime assert.Equalf(t, string(component.DefaultRuntimeManager), compRuntime, "expected default runtime for non-monitoring component %s with id %s", comp.Name, comp.ID)