Skip to content

Conversation

@hmstepanek
Copy link
Contributor

@hmstepanek hmstepanek commented Dec 29, 2025

Overview

Add sampler supportability metrics.
Remove conversion to Comma Separated Value now that pipeline supports lists.

@hmstepanek hmstepanek requested a review from a team as a code owner December 29, 2025 23:52
@hmstepanek hmstepanek changed the base branch from main to develop-hybrid-core-tracing December 29, 2025 23:52
@github-actions
Copy link

github-actions bot commented Dec 29, 2025

MegaLinter analysis: Error

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ ACTION actionlint 7 0 0 0.92s
✅ MARKDOWN markdownlint 7 0 0 0 1.16s
❌ PYTHON ruff 994 23 1 0 1.11s
✅ PYTHON ruff-format 994 27 0 0 0.4s
✅ YAML prettier 15 0 0 0 1.6s
✅ YAML v8r 15 0 0 4.5s
✅ YAML yamllint 15 0 0 0.7s

Detailed Issues

❌ PYTHON / ruff - 1 error
::error title=Ruff (RUF005),file=tests/hybridagent_fastapi/test_otel_application.py,line=61,col=24,endLine=61,endColumn=96::tests/hybridagent_fastapi/test_otel_application.py:61:24: RUF005 Consider iterable unpacking instead of concatenation

See detailed reports in MegaLinter artifacts

MegaLinter is graciously provided by OX Security

@hmstepanek hmstepanek changed the title Add sampler supportability metrics Add sampler supportability metrics & conversion to CSV Dec 30, 2025
@mergify mergify bot added the tests-failing Tests failing in CI. label Dec 30, 2025
@codecov-commenter
Copy link

codecov-commenter commented Dec 30, 2025

Codecov Report

❌ Patch coverage is 33.33333% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.14%. Comparing base (d71dad1) to head (f94001d).
⚠️ Report is 18 commits behind head on develop-hybrid-core-tracing.

Files with missing lines Patch % Lines
newrelic/core/application.py 33.33% 3 Missing and 3 partials ⚠️
Additional details and impacted files
@@                       Coverage Diff                       @@
##           develop-hybrid-core-tracing    #1618      +/-   ##
===============================================================
+ Coverage                        79.93%   80.14%   +0.21%     
===============================================================
  Files                              213      213              
  Lines                            25168    25213      +45     
  Branches                          4001     4017      +16     
===============================================================
+ Hits                             20117    20207      +90     
+ Misses                            3622     3579      -43     
+ Partials                          1429     1427       -2     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hmstepanek hmstepanek force-pushed the add-supportability-metrics branch from 1c56ee4 to af3a85c Compare January 2, 2026 23:13
@hmstepanek hmstepanek merged commit 9dc8bf6 into develop-hybrid-core-tracing Jan 5, 2026
53 of 61 checks passed
@hmstepanek hmstepanek deleted the add-supportability-metrics branch January 5, 2026 17:24
@mergify mergify bot removed the tests-failing Tests failing in CI. label Jan 5, 2026
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