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

summary: AWS Bedrock instrumentation #2314

Merged
merged 43 commits into from
Apr 2, 2024
Merged

summary: AWS Bedrock instrumentation #2314

merged 43 commits into from
Apr 2, 2024

Conversation

tippmar-nr
Copy link
Member

@tippmar-nr tippmar-nr commented Mar 7, 2024

feat: Add auto-instrumentation for AWS Bedrock

feat: New configuration options are available specific to AI monitoring.

feat: A new AI monitoring related public API method has been added - SetLlmTokenCountingCallback

notice: New Relic AI monitoring is the industry’s first APM solution that provides end-to-end visibility for AI Large Language Model (LLM) applications. It enables end-to-end visibility into the key components of an AI LLM application. With AI monitoring, users can monitor, alert, and debug AI-powered applications for reliability, latency, performance, security and cost. AI monitoring also enables AI/LLM specific insights (metrics, events, logs and traces) which can easily integrate to build advanced guardrails for enterprise security, privacy and compliance.

notice: AI monitoring offers custom-built insights and tracing for the complete lifecycle of an LLM’s prompts and responses, from raw user input to repaired/polished responses. AI monitoring provides built-in integrations with popular LLMs and components of the AI development stack. This release provides instrumentation for AWS Bedrock.

notice: When AI monitoring is enabled, the agent will now capture AI LLM related data. This data will be visible under a new APM tab called AI Responses. See our AI Monitoring documentation for more details.

jaffinito and others added 30 commits February 8, 2024 14:48
* First pass at Bedrock integration tests
* Response payloads do not have a model ID
* Fixed issue with errors getting dropped; Fixed Titan Embedded response model
* Adding custom attributes test
* Test fixes and cleanup
* Added AwsBedrockConfiguration
* Fix issue with embedding event
* Fixing intermittent test failures
* Add new integration tests to workflows
* Reduce test timeout
@chynesNR chynesNR changed the title ** DRAFT** feat: AWS Bedrock instrumentation feat: AWS Bedrock instrumentation Mar 18, 2024
Copy link
Member

@nr-ahemsath nr-ahemsath left a comment

Choose a reason for hiding this comment

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

Just had this one question which I think Chris V. already asked as well.

tippmar-nr and others added 2 commits March 26, 2024 14:15
* Move Json deserialization from wrapper to core

* Also needed to move the models to core as well

* Add and tweak comments/readme
@chynesNR chynesNR changed the title feat: AWS Bedrock instrumentation summary: AWS Bedrock instrumentation Mar 27, 2024
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 76.82540% with 73 lines in your changes are missing coverage. Please review.

Project coverage is 80.47%. Comparing base (9f1a6af) to head (ee4cd31).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2314      +/-   ##
==========================================
- Coverage   80.53%   80.47%   -0.06%     
==========================================
  Files         441      450       +9     
  Lines       27631    27942     +311     
  Branches     2989     3023      +34     
==========================================
+ Hits        22252    22487     +235     
- Misses       4611     4682      +71     
- Partials      768      773       +5     
Flag Coverage Δ
Agent 81.45% <76.82%> (-0.08%) ⬇️
Profiler 71.93% <ø> (ø)

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

Files Coverage Δ
...gent/Core/Attributes/AttributeDefinitionService.cs 95.18% <100.00%> (+0.03%) ⬆️
.../Agent/Core/Configuration/ReportedConfiguration.cs 99.08% <100.00%> (+0.01%) ⬆️
...c/Agent/Core/Configuration/RunTimeConfiguration.cs 100.00% <100.00%> (ø)
...Core/Events/LlmTokenCountingCallbackUpdateEvent.cs 100.00% <100.00%> (ø)
...nt/Core/Metrics/ApiSupportabilityMetricCounters.cs 79.62% <ø> (ø)
...nt/NewRelic/Agent/Core/Transactions/Transaction.cs 80.18% <100.00%> (+0.07%) ⬆️
...lic/Agent/Core/Transactions/TransactionMetadata.cs 96.82% <100.00%> (+0.13%) ⬆️
...ransactionTransformer/TransactionAttributeMaker.cs 99.32% <100.00%> (+0.01%) ⬆️
...ewRelic.Agent.Extensions/Helpers/VersionHelpers.cs 100.00% <100.00%> (ø)
...sions/NewRelic.Agent.Extensions/Llm/EventHelper.cs 100.00% <100.00%> (ø)
... and 13 more

... and 1 file with indirect coverage changes

@chynesNR chynesNR merged commit a6f3364 into main Apr 2, 2024
85 checks passed
@chynesNR chynesNR deleted the feature/aws-bedrock branch April 2, 2024 18:06
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.

6 participants