Why not use "--version" to get the version of OTEL?

[mike9421]

Component(s)

cmd/opampsupervisor

Describe the issue you're reporting

The way the version number is obtained in function getBootstrapInfo is by opampextension connecting to opampsupervisor, then opampsupervisor traverses the data.

Why not use "--version" to let otel return the version number? I think this method will be more convenient.

[github-actions]

Pinging code owners:

• cmd/opampsupervisor: @evan-bradley @atoulme @tigrannajaryan

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

[evan-bradley]

Hi @mike9421, thanks for raising this. In short, using the OpAMP extension instead of the --version flag on the Collector binary allows us to Collect additional information about the Collector outside of the information baked into the binary that the --version flag has access to. You can see a few attributes here, but more may be reported to the Supervisor in the future.

[mike9421]

Hi @mike9421, thanks for raising this. In short, using the OpAMP extension instead of the --version flag on the Collector binary allows us to Collect additional information about the Collector outside of the information baked into the binary that the --version flag has access to. You can see a few attributes here, but more may be reported to the Supervisor in the future.

Do you mean that more information about the agent will be retrieved in getBootstrapInfo() in the future,

opentelemetry-collector-contrib/cmd/opampsupervisor/supervisor/supervisor.go

Line 237 in 5c765a3

func (s *Supervisor) getBootstrapInfo() (err error) {

but for now, only the version number has been obtained, right?

If that's the case, then I have no issues. Initially, I thought if we are just retrieving the version number, we could simply use --version instead of establishing an OpAMP connection and then disconnecting it.

It seems that from version v0.93.0 onwards, the supervisor no longer retrieves the version number in getBootstrapInfo().
v0.93.0

opentelemetry-collector-contrib/cmd/opampsupervisor/supervisor/supervisor.go

Line 247 in c06bed3

if attr.Key == semconv.AttributeServiceInstanceID {

v0.92.0

opentelemetry-collector-contrib/cmd/opampsupervisor/supervisor/supervisor.go

Line 172 in e833bc5

s.agentVersion = "1.0.0"

@evan-bradley Thank you for your reply, I am about to close this issue.