search

open-telemetry / opamp-spec

OpAMP Specification
Apache License 2.0
102 stars 33 forks source link

Declare stability per capability or for all spec? #146

Closed tigrannajaryan closed 1 year ago

tigrannajaryan commented 1 year ago

We discussed that we want to wait for some OpAMP implementations to complete before we declare the spec stable, so that there is time to learn from the implementations and adjust the spec if needed. However the implementations that are in progress don't utilize all OpAMP capabilities, so we can't expect to learn much about these areas.

Should we declare stability of the spec for the capabilities that we know are relatively well-tested and move the other capabilities to a separate section and mark as "Beta" addition to the spec for now?

The spec sections and capabilities I am less confident about and which I think are not used in any implementations so far are:

We can probably mark these sections as [Beta] and the rest of the sections can be declared Stable.

@open-telemetry/opamp-spec-approvers what do you think?

andykellr commented 1 year ago

I think that makes sense. BindPlane does use Packages and PackageStatuses for agent upgrade and it was worked well. It does not currently use ConnectionSettings.

tigrannajaryan commented 1 year ago

I think that makes sense. BindPlane does use Packages and PackageStatuses for agent upgrade and it was worked well. It does not currently use ConnectionSettings.

@andykellr are there any other capabilities that BindPlane does not use? Do you utilize everything except Connection Settings?

andykellr commented 1 year ago

I think that makes sense. BindPlane does use Packages and PackageStatuses for agent upgrade and it was worked well. It does not currently use ConnectionSettings.

@andykellr are there any other capabilities that BindPlane does not use? Do you utilize everything except Connection Settings?

ConnectionSettings and AgentIdentification are not currently used by BindPlane.

tigrannajaryan commented 1 year ago

Discussed in workgroup meeting today and decided that it is reasonable to declare only portions of the spec stable and keep less-tested parts as beta. I will work on figuring out the exact labeling mechanism.

tigrannajaryan commented 1 year ago

Done in https://github.com/open-telemetry/opamp-spec/pull/147