-
Notifications
You must be signed in to change notification settings - Fork 367
Add the fluentforward receiver #3925
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
base: main
Are you sure you want to change the base?
Conversation
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! I'm happy to add this in. It's quite simple. Also, in the latest OTel user survey 14 out of 187 respondents said they use this component.
CHANGELOG.md
Outdated
- [`otelcol.exporter.otlp`] Fixed a bug where the receiver's http response was not properly translating grpc error codes to http status codes. | ||
https://github.com/open-telemetry/opentelemetry-collector/pull/10574 | ||
<https://github.com/open-telemetry/opentelemetry-collector/pull/10574> |
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.
These look like unintended changes?
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.
I Think its auto format, ill change it
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.
done
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.
Just needs a fix to the changelog
CHANGELOG.md
Outdated
@@ -14,7 +14,7 @@ Main (unreleased) | |||
|
|||
- (_Experimental_) Add an `array.group_by` stdlib function to group items in an array by a key. (@wildum) | |||
- Add the `otelcol.exporter.faro` exporter to export traces and logs to Faro endpoint. (@mar4uk) | |||
|
|||
- Add the `otelcol.receiver.fluentforward` receiver to recive logs via Fluent Forward Protocol. (@rucciva) |
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.
- Add the `otelcol.receiver.fluentforward` receiver to recive logs via Fluent Forward Protocol. (@rucciva) | |
- Add the `otelcol.receiver.fluentforward` receiver to receive logs via Fluent Forward Protocol. (@rucciva) |
I believe tests are failing as you need to run |
hey @dehaansa , addressed your comment. |
PR Description
This PR adds support for the otelcol fluentforward receiver
Which issue(s) this PR fixes
Fixes #3529
Notes to the Reviewer
PR Checklist