Ensuring Consistency between service.instance.id and instanceId in opampextension

[mike9421]

Component(s)

extension/opamp

Is your feature request related to a problem? Please describe.

While using opampextension, I found that when I wanted to query self-monitoring metrics and only knew the instanceId but not the service.instance.id, due to the inconsistency in their values, I was unable to query using instanceId.

If it were possible for instanceId and service.instance.id to be kept consistent as they are in opampsupervisor, then it would be convenient for me to query using the instanceId.

Describe the solution you'd like

"Considering the OpAMP and OTEL specifications, where instanceId must be a ULID and service.instance.id is recommended to use UUID, I suggest the following:

1. service.instance.id uses ULID by default
2. opampextension directly reuses service.instance.id without setting the instanceId configuration item.

Describe alternatives you've considered

No response

Additional context

No response

[github-actions]

Pinging code owners:

• extension/opamp: @portertech @evan-bradley @tigrannajaryan

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[evan-bradley]

Thanks for the detailed write-up. I will have to think over the solution more, but I think this is valid.

[tigrannajaryan]

FYI, we may move from ULID to UUID in opamp too: open-telemetry/opamp-spec#186

[github-actions]

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

• extension/opamp: @portertech @evan-bradley @tigrannajaryan

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[mike9421]

Information can be synchronized at open-telemetry/opamp-spec#186

[github-actions]

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

• extension/opamp: @portertech @evan-bradley @tigrannajaryan @BinaryFissionGames

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[mike9421]

FYI, we may move from ULID to UUID in opamp too: open-telemetry/opamp-spec#186

Hi @tigrannajaryan It seems that OpAMP already uses UUID, this issue can be closed.