-
Notifications
You must be signed in to change notification settings - Fork 896
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
Agree on the scope for HTTP semantic conventions v1.0 #2499
Comments
@tigrannajaryan as per your suggestion given at
I'm going to (on behalf of Instrumentation SIG) reach out to maintainers of the following SDKs:
The goal is to get an explicit answer to the question: "are you comfortable with the HTTP semantic conventions specification being marked stable, within the scope captured in #2499"? Once confirmations from maintainers of these 3 SDKs are received, I will reach out to maintainers of the rest of SDKs via maintainers meeting (Monday 9am PT) asking to express any disagreements. Obtaining these confirmations and absence of disagreements means that everyone (including Instrumentation SIG, TC and SDK maintainers) is ready to approve a PR marking HTTP semantic conventions specification stable. Is my understanding correct? If yes, can you please also suggest how best to record confirmations from the maintainers of those 3 SDKs? |
There are 2 possible ways forward:
Option 1 may be more preferable this time, since you already made significant progress and going back to OTEP stage can slow down things. For the future I would highly advise to start and go though a proper OTEP process (again, not just a roadmap OTEP), which is designed exactly to facilitate complicated initiatives like this.
You can ask a maintainer from each language to review and approve the PR or to comment on the issue. |
@tigrannajaryan can we clarify that 3 SIGs would be enough of a maintainer review? Reaching out to every single language is more review overhead then we normally go with. |
Yes, I think it is fine. I suggested it primarily because I thought maintainers may have suggestions, not because I thought they may want to veto it. |
closing this, since HTTP conventions are stable |
Several months ago the OTEP that captures the scope and roadmap for HTTP semantic conventions v1.0 (along with the scope for vNext) was reviewed and merged (open-telemetry/oteps#174). It was actively discussed at Instrumentation SIG meetings (Tue 4PM PST) and it was an agreement within the group, but it feels that there is no shared understanding with TC on the path of bringing HTTP semantic conventions to v1.0 (stable). Submitting this issue to fill this gap.
As a part of the OTEP, 4 main items were identified as required to be resolved before v1.0:
The group also maintains the Project board that reflects the current status for this work.
That said, the only item left before HTTP semantic conventions v1.0 is #2469 (which also resolves #2114, #2056 and #2028).
It would be beneficial to discuss this scope once again at the upcoming Spec meeting to make sure the Instrumentation SIG and TC are on the same page.
@open-telemetry/technical-committee, @open-telemetry/specs-trace-approvers
The text was updated successfully, but these errors were encountered: