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

feat: Instrumentation for Amazon Simple Queuing Service (AWSSDK.SQS) #2620

Merged
merged 22 commits into from
Jul 12, 2024

Conversation

tippmar-nr
Copy link
Member

Adds instrumentation for Amazon SQS, including distributed tracing support (Cross-Application Tracing (CAT) is not supported).

@tippmar-nr tippmar-nr requested a review from a team as a code owner July 11, 2024 19:52
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 68.91892% with 23 lines in your changes missing coverage. Please review.

Project coverage is 80.81%. Comparing base (b24d2c9) to head (98c2135).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2620      +/-   ##
==========================================
- Coverage   80.82%   80.81%   -0.01%     
==========================================
  Files         458      459       +1     
  Lines       28990    29064      +74     
  Branches     3185     3191       +6     
==========================================
+ Hits        23430    23488      +58     
- Misses       4769     4787      +18     
+ Partials      791      789       -2     
Flag Coverage Δ
Agent 81.72% <68.91%> (-0.01%) ⬇️
Profiler 72.59% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
src/Agent/NewRelic/Agent/Core/Agent.cs 74.30% <100.00%> (+0.96%) ⬆️
...ions/NewRelic.Agent.Extensions/AwsSdk/SqsHelper.cs 62.29% <62.29%> (ø)

... and 2 files with indirect coverage changes

Copy link
Member

@chynesNR chynesNR left a comment

Choose a reason for hiding this comment

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

Nice work!

@tippmar-nr tippmar-nr merged commit ac738ba into main Jul 12, 2024
100 checks passed
@tippmar-nr tippmar-nr deleted the feature/sqs-instrumentation branch July 12, 2024 19:36
kmudduluru pushed a commit that referenced this pull request Jul 16, 2024
🤖 I have created a release *beep* *boop*
---


##
[10.27.0](v10.26.0...v10.27.0)
(2024-07-15)


### New features

* Add support for disabling LLM monitoring at the account level.
([#2592](#2592))
([c31451a](c31451a))
* Instrumentation for Amazon Simple Queuing Service (AWSSDK.SQS)
([#2620](#2620))
([ac738ba](ac738ba))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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.

4 participants