Update span name recommendation for HTTP Server

[gouthamve]

This is to bring it inline with: https://github.com/open-telemetry/semantic-conventions/blob/main/docs/http/http-spans.md#http-server-semantic-conventions

Merge requirement checklist

• CONTRIBUTING.md guidelines followed.
• CHANGELOG.md updated for non-trivial changes.
• schema-next.yaml updated with changes to existing conventions.

[gouthamve]

I am not sure if this is correct, and might conflict with #270

Should I just remove this line from here?

[pyohannes]

@gouthamve Please rebase to make the checks pass.

[AlexanderWert]

@gouthamve

I don't fully get the change here, I guess I'm missing some context here.
Why would we stop recommending to use the route in HTTP server span names?

[pyohannes]

@AlexanderWert

I don't fully get the change here, I guess I'm missing some context here.
Why would we stop recommending to use the route in HTTP server span names?

It's not about not using the route anymore. How to set the span name is clearly defined in a dedicated section of this document:

HTTP server span names SHOULD be {method} {http.route} if there is a (low-cardinality) http.route available (see below for the exact definition of the [{method}](https://github.com/open-telemetry/semantic-conventions/blob/main/docs/http/http spans.md#method-placeholder) placeholder).

If there is no (low-cardinality) http.route available, HTTP server span names SHOULD be {method}.

The paragraph to remove in this PR suggested that the span name should be set just to {http.route}, without a trailing {method}. That's incorrect.

[AlexanderWert]

@AlexanderWert

I don't fully get the change here, I guess I'm missing some context here.
Why would we stop recommending to use the route in HTTP server span names?

It's not about not using the route anymore. How to set the span name is clearly defined in a dedicated section of this document:

HTTP server span names SHOULD be {method} {http.route} if there is a (low-cardinality) http.route available (see below for the exact definition of the [{method}](https://github.com/open-telemetry/semantic-conventions/blob/main/docs/http/http spans.md#method-placeholder) placeholder).
If there is no (low-cardinality) http.route available, HTTP server span names SHOULD be {method}.

The paragraph to remove in this PR suggested that the span name should be set just to {http.route}, without a trailing {method}. That's incorrect.

@pyohannes Thanks! Now I get it!

[trask]

thx @gouthamve!

and thx @pyohannes for the explainer!

[trask]

@open-telemetry/specs-semconv-maintainers I think this is good to merge