Skip to content

feat(apim): Add implementation for azure apim#11743

Open
TophrC-dd wants to merge 5 commits into
masterfrom
TopherC-dd/azure-apim-proxy-imp
Open

feat(apim): Add implementation for azure apim#11743
TophrC-dd wants to merge 5 commits into
masterfrom
TopherC-dd/azure-apim-proxy-imp

Conversation

@TophrC-dd

@TophrC-dd TophrC-dd commented Jun 25, 2026

Copy link
Copy Markdown

What Does This Do

This PR introduces the api-management inferred proxy spans to Azure. It leverages the existing inferred proxy span component. This works by looking for a magic value in one of the received headers.
APMSVLS-539
image

In the headers we also add metadata for Http Method, Path, Region and origin domain name.

Motivation

My motivation for this is have the java-tracer have feature parity with Node, Python and .Net that already support this inferred span.

Additional Notes

For testing I have setup a multi service distributed trace with another service upstream from the Api Management span.

image

@TophrC-dd TophrC-dd marked this pull request as ready for review June 25, 2026 15:46
@TophrC-dd TophrC-dd requested review from a team as code owners June 25, 2026 15:46
@dd-octo-sts

dd-octo-sts Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Hi! 👋 Thanks for your pull request! 🎉

To help us review it, please make sure to:

  • Add at least one type, and one component or instrumentation label to the pull request

If you need help, please check our contributing guidelines.

@TophrC-dd TophrC-dd changed the title [APMSVLS-539] adding implementation for azure apim [Java] adding implementation for azure apim Jun 25, 2026
@TophrC-dd TophrC-dd changed the title [Java] adding implementation for azure apim Adding implementation for azure apim Jun 25, 2026
@TophrC-dd TophrC-dd added type: enhancement Enhancements and improvements inst: azure-functions Azure Functions instrumentation labels Jun 25, 2026
@TophrC-dd TophrC-dd changed the title Adding implementation for azure apim feat(apim): Addi implementation for azure apim Jun 25, 2026
@dd-octo-sts

dd-octo-sts Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

🟢 Java Benchmark SLOs — All performance SLOs passed

Suite Status
Startup 🟢 pass

SLO thresholds are defined here based on automatically generated metrics. A warning is raised when results are within 5% of the threshold.

PR vs. master results
Scenario Candidate master Δ (95% CI of mean)
startup:insecure-bank:iast:Agent 13.99 s 13.92 s [-0.3%; +1.3%] (no difference)
startup:insecure-bank:tracing:Agent 12.88 s 12.94 s [-1.2%; +0.2%] (no difference)
startup:petclinic:appsec:Agent 16.96 s 16.66 s [+0.9%; +2.7%] (maybe worse)
startup:petclinic:iast:Agent 16.95 s 16.38 s [-0.9%; +7.9%] (no difference)
startup:petclinic:profiling:Agent 16.79 s 16.86 s [-1.4%; +0.7%] (no difference)
startup:petclinic:sca:Agent 16.84 s 16.75 s [-0.5%; +1.6%] (no difference)
startup:petclinic:tracing:Agent 15.65 s 16.04 s [-6.7%; +1.8%] (no difference)

Commit: 27db883b · CI Pipeline · Benchmarking Platform UI


Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion.

@TophrC-dd TophrC-dd changed the title feat(apim): Addi implementation for azure apim feat(apim): Add ]implementation for azure apim Jun 25, 2026
@TophrC-dd TophrC-dd changed the title feat(apim): Add ]implementation for azure apim feat(apim): Add implementation for azure apim Jun 25, 2026

@jcstorms1 jcstorms1 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

inst: azure-functions Azure Functions instrumentation type: enhancement Enhancements and improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants