Skip to content

Revisit peer.* namespace to match service.* #2945

@mmanciop

Description

@mmanciop

Area(s)

area:peer

What's missing?

The peer.service attribute does not match the way we define the identity of services in the service.* namespace via service.name and service.namespace.

Part of this issue was also brought up as #2338

Describe the solution you'd like

Add service.peer.{name|namespace} to the service.* namespace for spans.

Other options considered, and discarded, were:

  • introduce peer.service.name and peer.service.namespace with the intention of supporting match the usage of service.namespace + service.name as identity for services.
  • "special-casing" this to avoid breaking OpenTracing users by allowing peer.service to have a special syntax to represent combinations of service.namespace + service.name. However, I know from experience that some organizations already use / in their peer.service to represent the operation used on the invoked service.

Tip

React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding +1 or me too, to help us triage it. Learn more here.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    Status

    Need triage

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions