Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[extension/opamp][cmd/opampsupervisor] Support reporting of available components via OpAMP extension and supervisor #37248

Closed

Conversation

mrsillydog
Copy link
Contributor

@mrsillydog mrsillydog commented Jan 15, 2025

Description

This feature implements support for the new AvailableComponents message in opamp-go:
open-telemetry/opamp-go#340

Since the AvailableComponents message has not yet been released, this PR contains replace directives for testing purposes.

This PR is only for reference of the larger picture, and will be broken into two smaller PRs for reviewability. These PRs will not be merged until opamp-go has been released with the new AvailableComponents message implementation, at which point the replace directives will be removed.

#37249

Link to tracking issue

Fixes
#37246
#37247

Testing

Unit testing
Manual end-to-end testing against an external OpAMP server.

Documentation

Added documentation of new message implementations to corresponding READMEs.

@mrsillydog mrsillydog changed the title [extension/opamp] Support reporting of available components via OpAMP extension and supervisor [extension/opamp][cmd/opampsupervisor] Support reporting of available components via OpAMP extension and supervisor Jan 15, 2025
@mrsillydog mrsillydog closed this Feb 24, 2025
@mrsillydog mrsillydog deleted the feat/available-components branch February 24, 2025 15:59
evan-bradley pushed a commit that referenced this pull request Feb 28, 2025
…#37250)

#### Description
This feature implements support for the new AvailableComponents message
in opamp-go for the OpAMP supervisor:
open-telemetry/opamp-go#340

~~Since the AvailableComponents message has not yet been released, this
PR contains replace directives for testing purposes. This PR will not be
merged until opamp-go has been released with the new AvailableComponents
message implementation, at which point the replace directives will be
removed.~~ opamp-go v0.19.0 has been released.

This PR is one of two that make up
#37248

#### Link to tracking issue
Fixes
#37247

#### Testing
Updated unit tests
Manual end-to-end testing using the OpAMP extension and an external
OpAMP server.

#### Documentation
Updated README

---------

Co-authored-by: Daniel Jaglowski <jaglows3@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant