opentelemetry-specification icon indicating copy to clipboard operation
opentelemetry-specification copied to clipboard

Stabilize part of SDK spec for synchronous instrument Enabled

Open pellared opened this issue 3 weeks ago • 4 comments

Per https://github.com/open-telemetry/opentelemetry-specification/pull/4746#pullrequestreview-3505966069

Side note. I also made sure that this wording used for all signals (and in the same place)

It is not necessary for implementations to ensure that changes to any of these parameters are immediately visible to callers of Enabled. However, the changes MUST be eventually visible.

so that the spec is similar for all signals

pellared avatar Dec 08 '25 09:12 pellared

Is this something we should wait to include in the January's release?

carlosalberto avatar Dec 08 '25 21:12 carlosalberto

Is this something we should wait to include in the January's release?

I think this one should be high confidence and low risk, I suggest that we merge.

reyang avatar Dec 08 '25 21:12 reyang

@reyang You mean for this month's release (happening later today)?

carlosalberto avatar Dec 10 '25 15:12 carlosalberto

Let's play it on the defensive side and include this in the next release (January). IMHO no need to rush and include in today's (December) release.

carlosalberto avatar Dec 12 '25 17:12 carlosalberto