Skip to content

[v1] Filter out auto-configured local exports when drains are configured#182

Merged
dvoytenko merged 2 commits intov1.xfrom
dima/filter-out-auto-when-drain-v1
Oct 17, 2025
Merged

[v1] Filter out auto-configured local exports when drains are configured#182
dvoytenko merged 2 commits intov1.xfrom
dima/filter-out-auto-when-drain-v1

Conversation

@dvoytenko
Copy link
Member

This approach allows us to avoid double-sending of traces: one via a drain, and the other via local exporter. Over time this can be improved for per-destination filtering.

The original 2.x: #181

@vercel
Copy link

vercel bot commented Oct 17, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
otel-sample Ready Ready Preview Comment Oct 17, 2025 6:38pm
otel-site Ready Ready Preview Comment Oct 17, 2025 6:38pm

@dvoytenko dvoytenko merged commit e44a0d8 into v1.x Oct 17, 2025
8 checks passed
@dvoytenko dvoytenko deleted the dima/filter-out-auto-when-drain-v1 branch October 17, 2025 20:09
@github-actions github-actions bot mentioned this pull request Oct 17, 2025
kakadiadarpan added a commit that referenced this pull request Oct 20, 2025
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to v1.x, this PR will
be updated.


# Releases
## @vercel/otel@1.14.0

### Minor Changes

- [#182](#182)
[`e44a0d8`](e44a0d8)
Thanks [@dvoytenko](https://github.com/dvoytenko)! - Ignore
auto-configuration based on the OTEL_EXPORTER_OTLP_ env vars when trace
drains are used. This avoids duplicate trace export.

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Darpan Kakadia <kakadiadarpan@gmail.com>
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.

2 participants