feat: tidy verbose telemetry service name

[jamesstocktonj1]

Feature or Problem

Tidy up the open telemetry attributes:

• Replace provider executable name in favour of the Provider Key for service.name
  • Provider Key is unique to each provider instance whereas the executable name is not
• Add Host ID to host.id
• Add Instance ID to container.id
• Move provider binary name to container.image.name

Related Issues

Release Information

Consumer Impact

Testing

Tested using some simple modifications to the http-server example which can be found in this Gist. The tracer collecter is Jaeger.

Before

After

[joonas]

Thanks for the PR, these are solid improvements!

I just added a couple of comments in terms of adjusting the keys that we use for these to better align the semantics with the values.

@lxfontes may have further thoughts to add, I'll happily approve from my end once you have a chance the address the feedback.

[joonas]

Looks good on my part!

[joonas]

I'm leaving this open pending any feedback from @lxfontes, otherwise it looks good to merge on my end.

Lucas, feel free to merge this if you don't have anything to add. I’ll check back tomorrow.

[brooksmtownsend]

LGTM to me as well, @lxfontes feel free to merge when approved!