-
Notifications
You must be signed in to change notification settings - Fork 1.8k
feat(opentelemetry source): support OTLP logs, metrics & traces #23524
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
Conversation
4d95185
to
3f9e76d
Compare
3f9e76d
to
e6dab61
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/ci-run-e2e-opentelemetry-logs
Co-authored-by: Ursula Chen <58821586+urseberry@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the clarification to my feedback on the release note. I didn't understand the language. I assume "emits treats" is a domain-specific phrase that I'm not familiar with. I'll approve the PR.
This was a typo, my bad. Fixed in 0980161 |
* wip * WIP2 * wip3 * ran cargo fmt * refactoring otel source * wip * ran cargo fmt * commit the otel proto desc * refactoring * fix re-build mess * unrelated web-playground build fix * avoid duplication in tests * dbg tests * dbg tests * ran cargo fmt * chore(dev): cargo vdev build licenses * wip - cleanup * ran cargo fmt * chore(deps): cargo update -p vrl * chore(dev): cargo vdev build licenses * e2e - wip * cleanup * ran cargo fmt * set default * wip * vdev cruft * ran cargo fmt * cleanup * fix new config * use JSON names - wip * ran cargo fmt * chore(dev): cargo vdev build licenses * update VRL * ran cargo fmt * chore(dev): cargo vdev build licenses * handle metrics too * ran cargo fmt * vrl update fixes * ran cargo fmt * wip * chore(dev): cargo vdev build licenses * fix typo and add first version of the changelog * ran cargo fmt * fix typo and add first version of the changelog * add author * Apply suggestions from code review Co-authored-by: Ursula Chen <58821586+urseberry@users.noreply.github.com> * tweaks to changelog * gen comp docs * fix changelog typo --------- Co-authored-by: Ursula Chen <58821586+urseberry@users.noreply.github.com>
Summary
The
opentelemetry
source now supports a new decoding mode which can be enabled by settinguse_otlp_decoding
totrue
. In this mode,all events will preserve the OTLP format. These events can be forwarded directly to
the
opentelemetry
sink without modifications.A caveat here is that OTLP metrics and Vector metric format differ and thus we treat as logs as they come out the source. These events
cannot be used with existing metrics transforms. However, these can be ingested by the OTEL collectors as metrics.
Vector configuration
Vector Config:
How did you test this PR?
Added a new E2E environment that exercises the new code for logs.
Metrics and traces were tested manually. I have these documented and will probably follow up with a docs improvement PR.
Change Type
Is this a breaking change?
Does this PR include user facing changes?
no-changelog
label to this PR.References
Notes
@vectordotdev/vector
to reach out to us regarding this PR.pre-push
hook, please see this template.cargo fmt --all
cargo clippy --workspace --all-targets -- -D warnings
cargo nextest run --workspace
(alternatively, you can runcargo test --all
)git merge origin master
andgit push
.Cargo.lock
), pleaserun
cargo vdev build licenses
to regenerate the license inventory and commit the changes (if any). More details here.