Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Define transient error in OTLP exporter specification #3653

Merged

Conversation

alanwest
Copy link
Member

@alanwest alanwest commented Aug 10, 2023

Fixes #3652

This PR attempts to clearly define transient error in the OTLP exporter specification.

It also fixes a bug in that the set of retryable status codes in the OTLP exporter spec vs. the protocol spec are not aligned.

@alanwest alanwest requested review from a team August 10, 2023 00:55
@arminru arminru added the spec:protocol Related to the specification/protocol directory label Aug 10, 2023
specification/protocol/exporter.md Outdated Show resolved Hide resolved
specification/protocol/exporter.md Outdated Show resolved Hide resolved
@github-actions
Copy link

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Aug 29, 2023
@carlosalberto
Copy link
Contributor

Ping @tigrannajaryan

@carlosalberto
Copy link
Contributor

@alanwest Please fix lint/links ;)

@carlosalberto carlosalberto merged commit e5b3109 into open-telemetry:main Sep 7, 2023
6 checks passed
@alanwest alanwest deleted the alanwest/define-transient-error branch September 7, 2023 17:22
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
…#3653)

Fixes open-telemetry#3652

This PR attempts to clearly define `transient error` in the OTLP
exporter specification.

It also fixes a bug in that the set of retryable status codes in the
OTLP exporter spec vs. the protocol spec are not aligned.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec:protocol Related to the specification/protocol directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[OTLP exporter retry] Fix and further clarify the definition of "transient error"
6 participants