-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[receiver:awsfirehosereceiver] added support for OTLP v1.0.0 format #34982
Open
vihangvk
wants to merge
1
commit into
open-telemetry:main
Choose a base branch
from
vihangvk:feature/awsfirehosereceiver_otelformat_support
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[receiver:awsfirehosereceiver] added support for OTLP v1.0.0 format #34982
vihangvk
wants to merge
1
commit into
open-telemetry:main
from
vihangvk:feature/awsfirehosereceiver_otelformat_support
+235
−2
Conversation
This file contains 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
|
vihangvk
force-pushed
the
feature/awsfirehosereceiver_otelformat_support
branch
7 times, most recently
from
September 6, 2024 11:25
1cfd813
to
ec9b6fc
Compare
Aneurysm9
reviewed
Sep 6, 2024
receiver/awsfirehosereceiver/internal/unmarshaler/otelmetricstream/unmarshaler.go
Outdated
Show resolved
Hide resolved
receiver/awsfirehosereceiver/internal/unmarshaler/otelmetricstream/unmarshaler.go
Outdated
Show resolved
Hide resolved
receiver/awsfirehosereceiver/internal/unmarshaler/otelmetricstream/unmarshaler.go
Outdated
Show resolved
Hide resolved
receiver/awsfirehosereceiver/internal/unmarshaler/otelmetricstream/unmarshaler.go
Outdated
Show resolved
Hide resolved
receiver/awsfirehosereceiver/internal/unmarshaler/otelmetricstream/unmarshaler.go
Outdated
Show resolved
Hide resolved
receiver/awsfirehosereceiver/internal/unmarshaler/otelmetricstream/unmarshaler.go
Outdated
Show resolved
Hide resolved
vihangvk
force-pushed
the
feature/awsfirehosereceiver_otelformat_support
branch
2 times, most recently
from
September 10, 2024 11:36
842dbe7
to
c910564
Compare
vihangvk
changed the title
[receiver:awsfirehosereceiver] added support for Otel format
[receiver:awsfirehosereceiver] added support for OTLP v1.0.0 format
Sep 10, 2024
vihangvk
force-pushed
the
feature/awsfirehosereceiver_otelformat_support
branch
7 times, most recently
from
September 13, 2024 12:41
de3e1b3
to
12aedd1
Compare
Aneurysm9
reviewed
Sep 13, 2024
receiver/awsfirehosereceiver/internal/unmarshaler/otlpmetricstream/unmarshaler.go
Outdated
Show resolved
Hide resolved
vihangvk
force-pushed
the
feature/awsfirehosereceiver_otelformat_support
branch
from
October 1, 2024 10:29
12aedd1
to
a197824
Compare
vihangvk
force-pushed
the
feature/awsfirehosereceiver_otelformat_support
branch
2 times, most recently
from
October 1, 2024 11:14
e8a6e7a
to
9fcb7aa
Compare
vihangvk
force-pushed
the
feature/awsfirehosereceiver_otelformat_support
branch
4 times, most recently
from
October 7, 2024 09:29
fa0ec51
to
a5ddedd
Compare
vihangvk
force-pushed
the
feature/awsfirehosereceiver_otelformat_support
branch
from
October 9, 2024 07:20
a5ddedd
to
bb3fcd1
Compare
Aneurysm9
approved these changes
Oct 9, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description: Added support for OpenTelemetry 1.0.0 format
Current implementation for
awsfirehosereceiver
supports only JSON format.Added support to handle OpenTelemetry 1.0.0 format.
Standard OpenTelemetry receiver does not work as AWS Cloudwatch metrics are streamed via AWS Firehose stream. So the actual OpenTelemetry data is embedded in AWS Firehose records.
This can be enabled in configuration by setting
record_type
tootlp_v1
. (Added suffix v1 as AWS CloudWatch metrics streaming also supports OpenTelemetry 0.7.0 format).Unmarshaler
under new packageotlpmetricstream
.defaultMetricsUnmarshalers
infactory.go
to create instance of new Unmarshaler.availableRecordTypes
to support the OpenTelemetry 1.0.0 format.Testing: Added test cases to test new Unmarshler to test record unmarshaling.
Documentation: Updated readme to document new
record_type
.