open-telemetry / opamp-spec

OpAMP Specification
Apache License 2.0
108 stars 34 forks source link

Add ability to indicate Capabilities in the protocol #33

Closed tigrannajaryan closed 2 years ago

tigrannajaryan commented 2 years ago

Resolves https://github.com/open-telemetry/opamp-spec/issues/7 Resolves https://github.com/open-telemetry/opamp-spec/issues/25

This change clarifies how the protocols should work when a particular capability is not supported. It also defines how future extensions for the protocol capabilities will work.

tigrannajaryan commented 2 years ago

@pmm-sumo I think you raised this question. Please see if this PR answers it.

tigrannajaryan commented 2 years ago

Let's merge this. We may need to tweak the capabilities when we do the actual implementation. The prototyping will help clarify the edge cases when the information about capabilities is necessary.