Skip to content

Commit

Permalink
remove timeout for admission test
Browse files Browse the repository at this point in the history
  • Loading branch information
jmacd committed Oct 30, 2024
1 parent eafee7a commit b268442
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions internal/otelarrow/test/e2e_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -680,6 +680,7 @@ func TestIntegrationAdmissionLimited(t *testing.T) {
requestWhileTrue: func(test *testConsumer) bool {
return test.sink.SpanCount() < 10000
},
missingDeadline: true,
}

var ending func(*testing.T, testParams, *testConsumer, [][]ptrace.Traces) (_, _ map[string]int)
Expand Down Expand Up @@ -712,12 +713,11 @@ func TestIntegrationAdmissionLimited(t *testing.T) {

ecfg.Arrow.NumStreams = 10

// Shorten timeouts for this test, because we intend
// for it to fail and don't want to wait for retries.
ecfg.TimeoutSettings.Timeout = 5 * time.Second
ecfg.RetryConfig.InitialInterval = 1 * time.Second
ecfg.RetryConfig.MaxInterval = 2 * time.Second
ecfg.RetryConfig.MaxElapsedTime = 10 * time.Second
// Disable export timeout and retry,
// because we expect progress to be
// slow for this test.
ecfg.TimeoutSettings.Timeout = 0
ecfg.RetryConfig.Enabled = false
ecfg.Arrow.MaxStreamLifetime = 5 * time.Second
}, nearLimitGenFunc(), consumerFailure, ending)
})
Expand Down

0 comments on commit b268442

Please sign in to comment.