[exporter/loki] Do not retry permanent errors

[wiardvanrij]

Component(s)

exporter/loki

What happened?

Description

Legacy exporter has: https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/exporter/lokiexporter/legacy_exporter.go#L164

New exporter hasn't: https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/exporter/lokiexporter/next_exporter.go#L105

This is pretty painful on 400 requests, like has timestamp too old = http 400. Which causes a ton of retries.

Collector version

latest

Environment information

No response

OpenTelemetry Collector configuration

No response

Log output

No response

Additional context

No response

[github-actions]

Pinging code owners:

• exporter/loki: @gramidt @gouthamve @jpkrohling @kovrus @mar4uk

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[jpkrohling]

@wiardvanrij would you like to send a PR for this one?

[wiardvanrij]

Sure, I can take a look

[wiardvanrij]

@jpkrohling thx! I was wondering what the release schedule is? We really would appreciate using this change asap as it now can potentially cause a lot of issues.

[jpkrohling]

We release every two weeks, the next one on Feb 6th: https://github.com/open-telemetry/opentelemetry-collector/blob/main/docs/release.md#release-schedule

[jpkrohling]

closed by #18083