Skip to content

revert: restore previous OTLP exporter behavior - #635

Merged
alongubkin merged 1 commit into
mainfrom
alon/alien-850-revert-log-delivery
Sep 17, 2026
Merged

alongubkin merged 1 commit into
mainfrom
alon/alien-850-revert-log-delivery

Conversation

@alongubkin

Copy link
Copy Markdown
Member

Plain revert of #632. The exporter upgrade lost HTTPS support through a new HTTP-client dependency, and the 750ms timeout regressed otherwise healthy slower receivers. Restore the prior dependency versions, timeout, and shutdown behavior; any retry changes will be proposed separately.

Validation: the complete resulting Git tree exactly matches the parent of e930057. No replacement implementation or new tests are mixed into this rollback. Full CI is not being waited on for the requested admin squash merge.

ALIEN-850

Reverts e930057. Restore the pre-change tree without adding replacement behavior. Follow-up changes will be reviewed and validated independently. ALIEN-850.
@alongubkin

Copy link
Copy Markdown
Member Author

Admin squash merge requested by the repository owner to roll back this unreleased change quickly. Verified that this PR restores the exact complete tree immediately preceding e930057; no new behavior is included. Full CI has not completed and is not claimed green. Follow-up fixes will be reviewed separately; this merge is not a deployment.

@alongubkin
alongubkin merged commit 87dc10a into main Sep 17, 2026
20 checks passed
@alongubkin
alongubkin deleted the alon/alien-850-revert-log-delivery branch September 17, 2026 05:34
@greptile-apps

greptile-apps Bot commented Sep 17, 2026

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

The PR appears safe to merge as the requested plain rollback.

Summary

This PR cleanly reverts the OTLP exporter upgrade, restoring the prior OpenTelemetry dependency set, exporter defaults, and shutdown behavior.

  • Restores OpenTelemetry crates from 0.32 to 0.30 and removes the experimental retry feature.
  • Restores ExportConfig-based HTTP exporter construction and the prior shutdown call.
  • Restores HTTPS support through the previous reqwest dependency path and removes the 750ms timeout.
  • The resulting tree was verified to exactly match the parent of the reverted base commit.

Reviews (1) · Last reviewed commit: "revert: restore previous OTLP exporter b..."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant