open-telemetry / opamp-spec

OpAMP Specification
Apache License 2.0
102 stars 33 forks source link

Split connection settings by types #87

Closed tigrannajaryan closed 2 years ago

tigrannajaryan commented 2 years ago

This changes splits connection settings by the type of the connection so that each type has a corresponding message that records the appropriate data.

See proposal here https://github.com/open-telemetry/opamp-spec/issues/80#issuecomment-1125193943

I also deleted proxy-related settings since it wasn't clear when and how when can be used. These can be added later when the use case is clear.

Contributes to https://github.com/open-telemetry/opamp-spec/issues/80

pmm-sumo commented 2 years ago

I think the change looks good and it clarifies the spec