Skip to content

[extension/opamp] avoid host ID dependency in OS description#47742

Open
jeevan6996 wants to merge 1 commit intoopen-telemetry:mainfrom
jeevan6996:fix-opamp-windows-host-platform-info
Open

[extension/opamp] avoid host ID dependency in OS description#47742
jeevan6996 wants to merge 1 commit intoopen-telemetry:mainfrom
jeevan6996:fix-opamp-windows-host-platform-info

Conversation

@jeevan6996
Copy link
Copy Markdown

Summary

  • switch OS description lookup from host.Info() to host.PlatformInformation()
  • avoid relying on host ID retrieval while composing the OpAMP agent OS description
  • keep fallback behavior to runtime GOOS if platform lookup fails
  • add tests that verify fallback on lookup failure and expected description formatting

Fixes #46965.

Testing

  • cd extension/opampextension && go test ./...

@linux-foundation-easycla
Copy link
Copy Markdown

CLA Not Signed

@github-actions github-actions bot added the first-time contributor PRs made by new contributors label Apr 18, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Welcome, contributor! Thank you for your contribution to opentelemetry-collector-contrib.

Important reminders:

A maintainer will review your pull request soon. Thank you for helping make OpenTelemetry better!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

extension/opamp first-time contributor PRs made by new contributors

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[extension/opampextension] Fails on Windows containers: "getting host ID: The system cannot find the file specified"

2 participants