Skip to content

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

rucciva
Copy link

@rucciva rucciva commented Jul 1, 2025

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

  • CHANGELOG.md updated
  • Documentation added
  • Tests updated
  • Config converters updated

@rucciva rucciva requested review from clayton-cornell and a team as code owners July 1, 2025 08:08
@CLAassistant
Copy link

CLAassistant commented Jul 1, 2025

CLA assistant check
All committers have signed the CLA.

@ptodev ptodev self-assigned this Jul 4, 2025
Copy link
Contributor

@ptodev ptodev left a 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>
Copy link
Contributor

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?

Copy link
Author

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

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Copy link
Contributor

@ptodev ptodev left a 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)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- 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)

@dehaansa
Copy link
Contributor

dehaansa commented Jul 7, 2025

I believe tests are failing as you need to run go generate ./internal/tools/docs_generator/

@rucciva
Copy link
Author

rucciva commented Jul 8, 2025

hey @dehaansa , addressed your comment.
thanks

@dehaansa dehaansa enabled auto-merge (squash) July 8, 2025 18:44
@rucciva rucciva requested a review from ptodev July 8, 2025 23:36
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.

Fluentforward component support
4 participants