From ef14e6eaa710930dbfa1a41257bf6454b2d32a37 Mon Sep 17 00:00:00 2001 From: Joshua MacDonald Date: Tue, 29 Oct 2024 02:43:05 -0700 Subject: [PATCH] (internal/otelarrow) Remove time-based test in e2e tests (#36034) #### Description Remove a flaky portion of the internal/otelarrow/test e2e test. #### Link to tracking issue Fixes #34719. #### Testing There was a time-based test that has proven unreliable. #### Documentation n/a --- internal/otelarrow/test/e2e_test.go | 7 ------- 1 file changed, 7 deletions(-) diff --git a/internal/otelarrow/test/e2e_test.go b/internal/otelarrow/test/e2e_test.go index 852a0ff0308f..d02ec733917f 100644 --- a/internal/otelarrow/test/e2e_test.go +++ b/internal/otelarrow/test/e2e_test.go @@ -93,13 +93,6 @@ func (tc *testConsumer) ConsumeTraces(ctx context.Context, td ptrace.Traces) err timeout := time.Until(dead) require.Equal(tc.t, tc.expectDeadline, hasDeadline, "deadline set or not set: %v", timeout) - if tc.expectDeadline { - // expect allows 1/6 of the deadline to elapse in transit, - // so 1m becomes 50s. - expect := tc.expCfg.TimeoutSettings.Timeout * 5 / 6 - require.Less(tc.t, expect, timeout) - require.Greater(tc.t, tc.expCfg.TimeoutSettings.Timeout, timeout) - } return tc.sink.ConsumeTraces(ctx, td) }