Skip to content

Commit 5336e40

Browse files
jmacdsbylica-splunk
authored andcommitted
(internal/otelarrow) Remove time-based test in e2e tests (open-telemetry#36034)
#### Description Remove a flaky portion of the internal/otelarrow/test e2e test. #### Link to tracking issue Fixes open-telemetry#34719. #### Testing There was a time-based test that has proven unreliable. #### Documentation n/a
1 parent 089ba2c commit 5336e40

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

internal/otelarrow/test/e2e_test.go

-7
Original file line numberDiff line numberDiff line change
@@ -93,13 +93,6 @@ func (tc *testConsumer) ConsumeTraces(ctx context.Context, td ptrace.Traces) err
9393
timeout := time.Until(dead)
9494

9595
require.Equal(tc.t, tc.expectDeadline, hasDeadline, "deadline set or not set: %v", timeout)
96-
if tc.expectDeadline {
97-
// expect allows 1/6 of the deadline to elapse in transit,
98-
// so 1m becomes 50s.
99-
expect := tc.expCfg.TimeoutSettings.Timeout * 5 / 6
100-
require.Less(tc.t, expect, timeout)
101-
require.Greater(tc.t, tc.expCfg.TimeoutSettings.Timeout, timeout)
102-
}
10396

10497
return tc.sink.ConsumeTraces(ctx, td)
10598
}

0 commit comments

Comments
 (0)