Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

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

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mrsillydog
Copy link
Contributor

Description

This is a clone of #37248, rebased against main.

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.

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

Successfully merging this pull request may close these issues.

1 participant