Skip to content
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

Define specification sponsor and seed with spec approvers #2070

Merged
merged 16 commits into from
Jun 10, 2024
Merged
Show file tree
Hide file tree
Changes from 9 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .cspell.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -135,6 +135,7 @@ words:
- passcodes
- poncelow
- proto
- protos
- pytest
- raesene
- reiley
Expand Down
20 changes: 8 additions & 12 deletions community-members.md
Original file line number Diff line number Diff line change
Expand Up @@ -45,27 +45,23 @@ Members of the Technical Committee are the maintainers of
[Project specs](https://github.com/open-telemetry/opentelemetry-specification)
and [Proto definitions](https://github.com/open-telemetry/opentelemetry-proto).

Trace Approvers:
[Specification sponsors](./community-membership.md#specification-sponsor) are
trusted collaborators of the Technical Committee on Project specs and related
repositories:

- [Christian Beedgen](https://github.com/kumoroku), Sumo Logic
- [Christian Neumüller](https://github.com/Oberon00), Dynatrace
- [Nikita Salnikov-Tarnovski](https://github.com/iNikem), Splunk
- [Ted Young](https://github.com/tedsuo), Lightstep

Metrics Approvers:

- [Cijo Thomas](https://github.com/cijothomas), Microsoft
- [Leighton Chen](https://github.com/lzchen), Microsoft
- [Tyler Yahn](https://github.com/MrAlias), Splunk

Logs Approvers:

- [Christian Beedgen](https://github.com/kumoroku), Sumo Logic
- [Daniel Jaglowski](https://github.com/djaglowski), observIQ
trask marked this conversation as resolved.
Show resolved Hide resolved
- [Robert Pająk](https://github.com/pellared), Splunk

Logs Approvers Emeritus:
jack-berg marked this conversation as resolved.
Show resolved Hide resolved

- [David Poncelow](https://github.com/zenmoto), Splunk
trask marked this conversation as resolved.
Show resolved Hide resolved
- [Leighton Chen](https://github.com/lzchen), Microsoft
- [Nikita Salnikov-Tarnovski](https://github.com/iNikem), Splunk
jack-berg marked this conversation as resolved.
Show resolved Hide resolved
- [Ted Young](https://github.com/tedsuo), Lightstep
- [Tyler Yahn](https://github.com/MrAlias), Splunk

## Semantic Conventions

Expand Down
84 changes: 77 additions & 7 deletions community-membership.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,14 @@ Group). Responsibilities for most roles are scoped to these subprojects (repos).
The OpenTelemetry [Governance Committee][] owns this document and process until
delegated. They can be reached via e-mail <cncf-opentelemetry-governance@lists.cncf.io>.

| **Role** | **Responsibilities** | **Requirements** | **Defined by** |
| ---------- | ----------------------------------------------------- | ------------------------------------------------------------ | ------------------------------------------------------------ |
| member | active contributor in the community. reviewer of PRs | sponsored by 2 approvers or maintainers. multiple contributions to the project. | OpenTelemetry GitHub org member. |
| triager | assist with project management and backlog organization. | nominated by a maintainer. help with issues for one month. | CONTRIBUTING, CODEOWNERS, or the README. |
| approver | approve incoming contributions | highly experienced and active reviewer + contributor to a subproject | [CODEOWNERS](https://help.github.com/en/articles/about-code-owners) in GitHub |
| maintainer | set direction and priorities for a subproject | demonstrated responsibility and excellent technical judgement for the subproject | [CODEOWNERS](https://help.github.com/en/articles/about-code-owners), GitHub Team and repo ownership in GitHub |
| emeritus | position of honor for former maintainers, approvers, and triagers | must have previously held a community role and not have been removed from that role for a [Code of Conduct](code-of-conduct.md) violation. | Listed as an emeritus maintainer/approver/triager in CONTRIBUTING, CODEOWNERS, or README |
| **Role** | **Responsibilities** | **Requirements** | **Defined by** |
|-----------------------|-------------------------------------------------------------------|--------------------------------------------------------------------------------------------------------------------------------------------|---------------------------------------------------------------------------------------------------------------|
| member | active contributor in the community. reviewer of PRs | sponsored by 2 approvers or maintainers. multiple contributions to the project. | OpenTelemetry GitHub org member. |
| triager | assist with project management and backlog organization. | nominated by a maintainer. help with issues for one month. | CONTRIBUTING, CODEOWNERS, or the README. |
| approver | approve incoming contributions | highly experienced and active reviewer + contributor to a subproject | [CODEOWNERS](https://help.github.com/en/articles/about-code-owners) in GitHub |
| maintainer | set direction and priorities for a subproject | demonstrated responsibility and excellent technical judgement for the subproject | [CODEOWNERS](https://help.github.com/en/articles/about-code-owners), GitHub Team and repo ownership in GitHub |
| emeritus | position of honor for former maintainers, approvers, and triagers | must have previously held a community role and not have been removed from that role for a [Code of Conduct](code-of-conduct.md) violation. | Listed as an emeritus maintainer/approver/triager in CONTRIBUTING, CODEOWNERS, or README |
| specification sponsor | trusted collaborators for the specification | nominated by [Technical Committee][] | Listed in [Community Members -> Specification and Protos](./community-members.md#specifications-and-proto) |

## New contributors

Expand Down Expand Up @@ -297,5 +298,74 @@ with the project. When a contributor returns to being more active, they may be
promoted back to their previous position at the discretion of the current
maintainers following the process defined above.

## Specification Sponsor

Specification sponsors are trusted collaborators of the technical committee, and
work to review, approve, and sponsor [opentelemetry-specification][] issues and
PRs.

Members are defined
in [Community Members -> Specification and Protos](./community-members.md#specifications-and-proto)
and are members of the `spec-sponsors` team (TODO: add link to github team when
available). Specification sponsors SHOULD list
their [areas of interest](./areas-of-interest.md).

### Requirements

Specification sponsors are trusted to drive specification work which can be
complex and highly impactful to downstream language SIGs and to the
OpenTelemetry project as a whole. Approvals from specification sponsors count
towards the approvals required to merge specification PRs. Therefore, there is a
high bar to becoming a specification sponsor.

- Contributor to OpenTelemetry project for at least 1 year
jack-berg marked this conversation as resolved.
Show resolved Hide resolved
- Has been the reviewer or author (through the sponsorship of another
specification sponsor) of at least 10 substantial PRs to specification or
related repositories
- Demonstrated sound technical judgement, and a deep understanding of the
specification and the OpenTelemetry project
- Nominated by technical committee member (
see [becoming a specification sponsor](#becoming-a-specification-sponsor))

### Responsibilities and privileges
jack-berg marked this conversation as resolved.
Show resolved Hide resolved
jack-berg marked this conversation as resolved.
Show resolved Hide resolved

- Review, respond to, and approve issues / PRs when tagged or when within areas
of interest.
- Sponsor issues, taking responsibility to be point on requirement gathering,
jack-berg marked this conversation as resolved.
Show resolved Hide resolved
developing solutions and prototypes, writing PRs, and responding to /
incorporating PR feedback. This can be done by the sponsor themselves, or by
another collaborator who is sponsored. If sponsoring another collaborator, the
sponsor should work with them to ensure the work is consistent with existing
specification.
- Demonstrate sound technical judgement.
- Frequently attend the specification SIG meeting where issues and PRs are
synchronously discussed.

jack-berg marked this conversation as resolved.
Show resolved Hide resolved
Sponsors who are unable to fulfill their responsibilities (e.g. no longer have
time available to commit) are expected to be moved
to [emeritus](#emeritus-maintainerapprovertriager) sponsors. The TC may also
vote to move a sponsor to emeritus.

### Becoming a Specification Sponsor

Technical committee members nominate specification sponsors by opening a PR to
the community repo to add the nominee to the specification sponsor list. The
vote is officially started when a pull request is opened, and ends when the pull
request is merged. The pull request may be merged when the following conditions
are met:

- The person being nominated has accepted the nomination by approving the pull
request.
- All TC members have approved the pull request or a majority of TC members have
jack-berg marked this conversation as resolved.
Show resolved Hide resolved
approved the pull request and no other TC member has objected by requesting
changes on the pull request. In the case that all TC members have not given
approval, the pull request should stay open for a minimum of 5 days before
merging.

The nominee is considered a specification sponsor after the pull request is
merged. The merger should update the `spec-sponsors` team with the new member.

[Governance Committee]: ./governance-charter.md
[Technical Committee]: ./tech-committee-charter.md
[opentelemetry-specification]: https://github.com/open-telemetry/opentelemetry-specification
[Members of Standing]: ./governance-charter.md#members-of-standing