Conform attributes and span name to v1.27 semantics for OpentelemetryFinch

[ShapovalovaIrina]

Conform attributes and span name to v1.27 semantics for OpentelemetryFinch.

Follows the guidelines set out in https://github.com/open-telemetry/semantic-conventions/blob/v1.27.0/docs/http/http-spans.md.

Changed

Span name

• Change default span name HTTP {method} -> {method}

Required attributes

• Set error.type attribute when span status is error
• Rename attribute http.method -> http.request.method
• Rename attribute http.url -> url.full
• Rename attribute http.status_code -> http.response.status_code
• Rename attribute net.host.name -> server.address

Opt-in attributes

• Rename attribute http.scheme -> url.scheme

Added

Span name

• Add :span_name option for custom value
• Add :url_template option for {method} {url_template} span name

Required attributes

• Add attribute server.port

Opt-in attributes

• Add attribute http.request.body.size
• Add attribute http.response.body.size
• Add attribute network.transport
• Add attribute url.template
• Add attribute user.agent.original
• Add attribute http.request.header.<key>
• Add attribute http.request.header.<key>

The presence of opt-in attributes is configured using options:

• :opt_in_attrs
• :request_header_attrs
• :response_header_attrs

Other

• Set span status to error, when HTTP status code is in 4xx or 5xx

Removed

• Remove attribute http.target

[ShapovalovaIrina]

@bryannaegele Hello! Sorry for bothering you, I couldn't find any information about who is opentelemetry_finch codeowner, so I'm contacting you. Can you please tag the right person or leave some feedback on PR?

[bryannaegele]

Hey @ShapovalovaIrina , this is great!

Two requests:

• list out the aliases as fully qualified (makes it easier to find references during updates)
• use the semantic conventions for matches in tests so we don't have to know the values of the keys - example

[ShapovalovaIrina]

Two requests:

• list out the aliases as fully qualified (makes it easier to find references during updates)
• use the semantic conventions for matches in tests so we don't have to know the values of the keys - example

@bryannaegele made the changes that were requested :)

Similar changes have also been made for OpentelemetryTesla PR #469.

[ShapovalovaIrina]

Hi @bryannaegele, do you have an estimate for when you might be able to take a look at this? No rush, just wanted to check 🙂 Thanks!