Skip to content

Add utilization for Azure App Service - #1838

Merged
lrafeei merged 23 commits into
mainfrom
instrument-azure-app-service
Sep 30, 2026
Merged

lrafeei merged 23 commits into
mainfrom
instrument-azure-app-service

Conversation

@lrafeei

@lrafeei lrafeei commented Aug 27, 2026 •

Copy link
Copy Markdown
Contributor

Add utilization for Azure App Service to enable entity relationship mapping.

This also adds a new setting, utilization.detect_azureappservice

@github-actions

github-actions Bot commented Aug 27, 2026 •

Copy link
Copy Markdown
Contributor

✅MegaLinter analysis: Success

Descriptor Linter Files Fixed Errors Max errors Warnings Elapsed time
✅ ACTION actionlint 11 0 0 1.65s
✅ MARKDOWN markdownlint 8 0 0 0 1.6s
✅ PYTHON ruff 1127 0 0 0 1.58s
✅ PYTHON ruff-format 1127 0 0 0 0.28s
✅ YAML prettier 25 0 0 0 1.82s
✅ YAML v8r 25 0 0 8.07s
✅ YAML yamllint 25 0 0 1.27s

See detailed reports in MegaLinter artifacts

MegaLinter is provided by OX Security
Show us your support by starring ⭐ the repository

@mergify mergify Bot added the tests-failing Tests failing in CI. label Aug 27, 2026
@codecov-commenter

codecov-commenter commented Aug 27, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 76.92308% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.95%. Comparing base (b50aca5) to head (54891de).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
newrelic/common/utilization.py 74.28% 7 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1838      +/-   ##
==========================================
+ Coverage   81.88%   81.95%   +0.07%     
==========================================
  Files         221      221              
  Lines       27568    27605      +37     
  Branches     4369     4374       +5     
==========================================
+ Hits        22575    22625      +50     
+ Misses       3474     3466       -8     
+ Partials     1519     1514       -5     

☔ View full report in Codecov by Harness.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@lrafeei
lrafeei marked this pull request as ready for review September 2, 2026 18:48
@lrafeei
lrafeei requested a review from a team as a code owner September 2, 2026 18:48
@mergify mergify Bot removed the tests-failing Tests failing in CI. label Sep 14, 2026
@mergify mergify Bot added the tests-failing Tests failing in CI. label Sep 15, 2026
@TimPansino
TimPansino self-requested a review September 17, 2026 17:55
@TimPansino TimPansino self-assigned this Sep 17, 2026
@mergify mergify Bot removed the tests-failing Tests failing in CI. label Sep 17, 2026
@mergify mergify Bot added the tests-failing Tests failing in CI. label Sep 21, 2026
@lrafeei
lrafeei enabled auto-merge (squash) September 23, 2026 17:59
Comment on lines +44 to +48

# -------------------------
# The following tests are actually for Azure App Services:
# -------------------------

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This comment could probably be deleted considering the tests are named as such.

@lrafeei
lrafeei merged commit 6c9fa59 into main Sep 30, 2026
54 of 63 checks passed
@lrafeei
lrafeei deleted the instrument-azure-app-service branch September 30, 2026 18:12
@mergify mergify Bot removed the tests-failing Tests failing in CI. label Sep 30, 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.

3 participants