open-telemetry / opentelemetry-collector-contrib

Contrib repository for the OpenTelemetry Collector
https://opentelemetry.io
Apache License 2.0
3.1k stars 2.38k forks source link

[sumologicexporter]: validate maximum timeout #33151

Closed sumo-drosiek closed 5 months ago

sumo-drosiek commented 6 months ago

Component(s)

exporter/sumologic

Is your feature request related to a problem? Please describe.

We claim that the maximum timeout value is 55s, but as far as I see we do not validate it

Describe the solution you'd like

We should add validation and unit tests for the timeout value

Describe alternatives you've considered

No response

Additional context

Outcome of the following: https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/33021#discussion_r1608489822

github-actions[bot] commented 6 months ago

Pinging code owners for exporter/sumologic: @aboguszewski-sumo @kkujawa-sumo @mat-rumian @rnishtala-sumo @sumo-drosiek @swiatekm-sumo. See Adding Labels via Comments if you do not have permissions to add labels yourself.