Skip to content

[exporter/sumologic] Modify retry settings to prevent exporter dropping data on transient backend unavailability.#47503

Merged
andrzej-stencel merged 7 commits into
open-telemetry:mainfrom
jagan2221:main
Apr 13, 2026
Merged

[exporter/sumologic] Modify retry settings to prevent exporter dropping data on transient backend unavailability.#47503
andrzej-stencel merged 7 commits into
open-telemetry:mainfrom
jagan2221:main

Conversation

@jagan2221

@jagan2221 jagan2221 commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Description

Modifying retry settings for sumologic exporter to be more reliable to transient backend unavailability.
Currently data is dropped after 5 mins of retry, but we need a higher timeout limit.
Increased total retry timeout from 5mins to 1hr.
Added exponential backoff increasing from 5s to 5min.

Testing

  1. Unit tests passing
  2. collector startup and exponential retry settings added verified with a dummy backend url which is unreachable. Sumologic exporter retry timeout started from 5s and grew exponentially.

testsumoExporter.log

@jagan2221 jagan2221 requested a review from a team as a code owner April 9, 2026 14:23
@jagan2221 jagan2221 requested a review from braydonk April 9, 2026 14:23
Comment thread exporter/sumologicexporter/factory_test.go
Comment thread .chloggen/47498.yaml Outdated
@github-actions github-actions Bot requested a review from rnishtala-sumo April 12, 2026 18:38
Comment thread .chloggen/47498.yaml Outdated
Co-authored-by: Curtis Robert <crobert@splunk.com>
Co-authored-by: Andrzej Stencel <andrzej.stencel@elastic.co>
@andrzej-stencel andrzej-stencel added the ready to merge Code review completed; ready to merge by maintainers label Apr 13, 2026
@andrzej-stencel andrzej-stencel merged commit bc82cfd into open-telemetry:main Apr 13, 2026
207 checks passed
AndrewCharlesHay pushed a commit to AndrewCharlesHay/opentelemetry-collector-contrib that referenced this pull request Apr 23, 2026
…ng data on transient backend unavailability. (open-telemetry#47503)

#### Description
Modifying retry settings for sumologic exporter to be more reliable to
transient backend unavailability.
Currently data is dropped after 5 mins of retry, but we need a higher
timeout limit.
Increased total retry timeout from 5mins to 1hr.
Added exponential backoff increasing from 5s to 5min. 

#### Testing
1. Unit tests passing 
2. collector startup and exponential retry settings added verified with
a dummy backend url which is unreachable. Sumologic exporter retry
timeout started from 5s and grew exponentially.


[testsumoExporter.log](https://github.com/user-attachments/files/26605794/testsumoExporter.log)

---------

Co-authored-by: Andrzej Stencel <andrzej.stencel@elastic.co>
Co-authored-by: Curtis Robert <crobert@splunk.com>
gracewehner pushed a commit to gracewehner/opentelemetry-collector-contrib that referenced this pull request Apr 29, 2026
…ng data on transient backend unavailability. (open-telemetry#47503)

#### Description
Modifying retry settings for sumologic exporter to be more reliable to
transient backend unavailability.
Currently data is dropped after 5 mins of retry, but we need a higher
timeout limit.
Increased total retry timeout from 5mins to 1hr.
Added exponential backoff increasing from 5s to 5min. 

#### Testing
1. Unit tests passing 
2. collector startup and exponential retry settings added verified with
a dummy backend url which is unreachable. Sumologic exporter retry
timeout started from 5s and grew exponentially.


[testsumoExporter.log](https://github.com/user-attachments/files/26605794/testsumoExporter.log)

---------

Co-authored-by: Andrzej Stencel <andrzej.stencel@elastic.co>
Co-authored-by: Curtis Robert <crobert@splunk.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

exporter/sumologic ready to merge Code review completed; ready to merge by maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants