Skip to content
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

Add AWS Firehose SDK Instrumentation #2149

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

obenkenobi
Copy link
Contributor

Overview

Adds instrumentation for firehose. Every operation with firehose will include the following attributes:

  • cloud.platform: which will contain the string aws_kinesis_delivery_streams
  • cloud.resource_id, which will contain an ARN in the format arn:aws:firehose:${region}:${accountId}:deliverystream/${streamName}. If it is not possible to get the full ARN, the property will not be set.

All firehose spans/traces are named in the formats:

  • Firehose/{operationName} (e.g. Firehose/listDeliveryStreams)
  • Firehose/{operationName}/{deliveryStreamName} (e.g. Firehose/putRecord/someStream)

Related Github Issue

#2070

@obenkenobi obenkenobi changed the title Add AWS Firehose Instrumentation Add AWS Firehose SDK Instrumentation Nov 20, 2024
@codecov-commenter
Copy link

codecov-commenter commented Nov 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.68%. Comparing base (1e56ef1) to head (4d5b9e8).
Report is 38 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2149      +/-   ##
============================================
- Coverage     70.70%   70.68%   -0.03%     
  Complexity     9987     9987              
============================================
  Files           841      841              
  Lines         40290    40312      +22     
  Branches       6107     6109       +2     
============================================
+ Hits          28489    28495       +6     
- Misses         9061     9070       +9     
- Partials       2740     2747       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

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.

2 participants