Skip to content

Commit

Permalink
[chore] Add "Code Owners" field to new component template (open-telem…
Browse files Browse the repository at this point in the history
…etry#23780)

open-telemetry#20868 represents a push to ensure that all component code owners are
members of the OpenTelemetry organization.

This has direct implications for our notion of "vendor-specific
components", which are those that interact specifically with a
particular vendor, and are contributed and maintained by a
representative of a vendor. We require that the vendor representative
must maintain the component, which essentially means they must be a code
owner. Therefore, I am proposing that the representative must be a
member of the org before the component may be accepted.
  • Loading branch information
djaglowski authored Jul 26, 2023
1 parent 62c7b53 commit 635ec0b
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions .github/ISSUE_TEMPLATE/new_component.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,21 @@ body:
description: A vendor-specific component directly interfaces with a vendor-specific API and is expected to be maintained by a representative of the same vendor.
options:
- label: This is a vendor-specific component
- label: If this is a vendor-specific component, I am proposing to contribute this as a representative of the vendor.
- label: If this is a vendor-specific component, I am proposing to contribute and support it as a representative of the vendor.
- type: input
attributes:
label: Code Owner(s)
description: A code owner is responsible for supporting the component, including triaging issues, reviewing PRs, and submitting bug fixes.
Please list one or more members or aspiring members of the OpenTelemetry project who will serve as code owners.
For vendor-specific components, the code owner is required and must be a representative of the vendor.
For non-vendor components, having a code owner is strongly recommended. However, you may use the issue to try to find a code owner for your component.
- type: input
attributes:
label: Sponsor (optional)
description: "A sponsor is an approver who will be in charge of being the official reviewer of the code. For vendor-specific components, it's good to have a volunteer sponsor. If you can't find one, we'll assign one in a round-robin fashion. For non-vendor components, having a sponsor means that your use-case has been validated. If there are no sponsors yet for the component, it's fine: use the issue as a means to try to find a sponsor for your component."
description: "A sponsor is an approver who will be in charge of being the official reviewer of the code.
For vendor-specific components, it's good to have a volunteer sponsor. If you can't find one, we'll assign one in a round-robin fashion.
For non-vendor components, having a sponsor means that your use-case has been validated.
If there are no sponsors yet for the component, it's fine: use the issue as a means to try to find a sponsor for your component."
- type: textarea
attributes:
label: Additional context
Expand Down

0 comments on commit 635ec0b

Please sign in to comment.