feat(otelcol.receiver): add awss3 receiver#4928
Merged
Conversation
x1unix
force-pushed
the
x1unix/feat/s3-receiver
branch
from
8a81d02 to
6cf4c82
Compare
Contributor
🔍 Dependency ReviewNo dependency upgrades detected in go.mod. This PR only adds net-new indirect dependencies, so there are no upgrade paths to review and no code changes required to adopt newer versions. NotesThe following net-new indirect dependencies were added (informational; not reviewed per rules):
|
Contributor
|
💻 Deploy preview deleted (feat(otelcol.receiver): add awss3 receiver). |
kalleep
reviewed
Nov 25, 2025
Contributor
kalleep
left a comment
kalleep
left a comment
There was a problem hiding this comment.
Nice work, left a few comments
3 tasks
docs/sources/reference/components/otelcol/otelcol.receiver.awss3.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/otelcol/otelcol.receiver.awss3.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/otelcol/otelcol.receiver.awss3.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/otelcol/otelcol.receiver.awss3.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/otelcol/otelcol.receiver.awss3.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/otelcol/otelcol.receiver.awss3.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/otelcol/otelcol.receiver.awss3.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/otelcol/otelcol.receiver.awss3.md
Outdated
Show resolved
Hide resolved
clayton-cornell
added
the
type/docs
Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
x1unix
force-pushed
the
x1unix/feat/s3-receiver
branch
from
85ebfe3 to
0ccb8f9
Compare
Contributor
clayton-cornell
left a comment
clayton-cornell
left a comment
There was a problem hiding this comment.
Caught two more things in the second review.
Looks good otherwise and it can go over to the devs for code review
docs/sources/reference/components/otelcol/otelcol.receiver.awss3.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
dehaansa
reviewed
Nov 25, 2025
docs/sources/reference/components/otelcol/otelcol.receiver.awss3.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/otelcol/otelcol.receiver.awss3.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/otelcol/otelcol.receiver.awss3.md
Outdated
Show resolved
Hide resolved
dehaansa
approved these changes
Nov 25, 2025
clayton-cornell
approved these changes
Nov 25, 2025
dehaansa
pushed a commit
to madhub/alloy
that referenced
this pull request
Dec 10, 2025
* feat: add awss3receiver component template * fix: imports * feat: map config * feat: remove currently not supported Encodings and Notification blocks * fix: filename * feat: add arguments test * feat: register component * feat: add awss3 otelconverter * chore: update changelog * fix: state type * feat: add component docs * chore: make generate-gh-issue-templates * fix: docs link typo * fix: otelconvert fixtures * fix: typo * fix: doc * chore: go generate ./internal/tools/docs_generator/ * feat: map awss3receiver defaults to SetToDefault * feat: add config defaults break change test * fix: apply doc suggestions from code review Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com> * fix: tests * fix: Apply suggestions from code review Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com> * fix: mention that awss3 supports any kind of telemetry * fix: codegen --------- Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Description
This PR adds
otelcol.receiver.awss3component, which is a wrapper for awss3receiver opentelemetry collector component.CC @clayton-cornell @dehaansa
Which issue(s) this PR fixes
Partially fixes #4800
Notes to the Reviewer
Note
The wrapper doesn't have support for custom encoding extensions.
Support for custom encoding extensions (and extensions themselves) will be done in next PRs.
See the tracking issue: #4934
PR Checklist