Hey, good ?

That package adds an attribute to an specific metric, right ?

But what I mean is the display on traces/logs, it is always v{version:apiVersion}/route-blabla. I think it should be the version.

In my MVC controllers I always use [Route("v{version:apiVersion}/asset-classes")], and it is expected to replace that pattern to the selected version.

It emits a metrics giving the latency of each API with API version as a dimension. By attaching a TraceID, you can figure out the API version of the traces.

I agree, but this behaviour is expected ? Should not the display be v1/asset-classes on traces titles ?

That I don't know. I have only used API version as a query parameter and not in the URL. I plan to play around with Aspire soon.

Both Aspire and VS 17. 9 is in preview so I will wait for the release version.

This looks like it might be for a Minimal API. Is it? It also looks like you want the value of http.target to show up for Activity.DisplayName and http.route. These appear to be showing you your defined route templates. I don't know there is a solution for http.route, but there may be a way to specify an alternate Activity.DisplayName. I would expect the same behavior for any route template that contains parameters; otherwise, something special is happening. That doesn't make sense because the routing system has to match a request to the template. There is no in-between. When you match, you have a template against the request URL, when you don't, you only have the request URL.

@commonsensesoftware, this example is a MVC Controller:

 [ApiController]
 [Route("v{version:apiVersion}/[controller]")]
 public class WeatherForecastController(ISampleApiClient sampleApiClient) : ControllerBase
 {
 }

Full trace:

I think this implementation is the source:

https://github.com/open-telemetry/opentelemetry-dotnet/blob/7dff32480d8b2fe8d41e0958257c00e3602e3b3a/src/OpenTelemetry.Instrumentation.AspNetCore/Implementation/HttpInListener.cs#L455

It uses httpRoute.

I agree as well, I will open an issue there. Thank you for your attention. For now I will remove the pattern from route and explicitly set the version.