Skip to content
New issue

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

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

Already on GitHub? # to your account

[exporter/prometheusremotewrite] Fix: Validate context is canceled during retries #30308

Merged

Conversation

rapphil
Copy link
Contributor

@rapphil rapphil commented Jan 5, 2024

Description: Validates that the context is canceled in order to avoid unnecessary retries in the prometheus remote write exporter

Link to tracking Issue: No tracking issue.

Testing: Unit tests were added.

…otewriteexporter

Validates that the context is canceled in order to avoid unnecessary retries in the prometheus remote write exporter

Signed-off-by: Raphael Silva <rapphil@gmail.com>
Signed-off-by: Raphael Silva <rapphil@gmail.com>
@rapphil rapphil force-pushed the rapphil-fix-prw-context-retries branch from 38640c2 to 87d4b49 Compare January 5, 2024 19:50
Signed-off-by: Raphael Silva <rapphil@gmail.com>
@rapphil rapphil requested a review from bryan-aguilar January 6, 2024 01:15
Signed-off-by: Raphael Silva <rapphil@gmail.com>
@rapphil rapphil changed the title Fix: Validate context is canceled during retries in prw [exporter/prometheusremotewrite] Fix: Validate context is canceled during retries Jan 7, 2024
@Aneurysm9 Aneurysm9 added the ready to merge Code review completed; ready to merge by maintainers label Jan 8, 2024
@bogdandrutu
Copy link
Member

This needs a rebase before being able to merge

@bryan-aguilar bryan-aguilar removed the ready to merge Code review completed; ready to merge by maintainers label Jan 10, 2024
@bryan-aguilar bryan-aguilar added the ready to merge Code review completed; ready to merge by maintainers label Jan 10, 2024
@bogdandrutu bogdandrutu merged commit 187c393 into open-telemetry:main Jan 10, 2024
90 checks passed
@github-actions github-actions bot added this to the next release milestone Jan 10, 2024
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Jan 10, 2024
…ring retries (open-telemetry#30308)

**Description:** Validates that the context is canceled in order to
avoid unnecessary retries in the prometheus remote write exporter


**Link to tracking Issue:** No tracking issue.

**Testing:** Unit tests were added.

---------

Signed-off-by: Raphael Silva <rapphil@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
exporter/prometheusremotewrite ready to merge Code review completed; ready to merge by maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants