Request to create semconv-{container,k8s}-approvers

[ChrsMark]

I'd like to request creating 2 new approvers groups semconv-container-approvers and semconv-k8s-approvers (as per suggestion at #427 (comment)) that would be responsible for reviewing and maintaining semantic conventions related to the containers' and kubernetess areas respectively.

Folks proposed for that groups / interested:

semconv-container-approvers:

• @frzifus
• @dmitryax
• @ChrsMark
• @mx-psi
• @TylerHelmuth
• @jinja2
• @mlunadia
• TBA

semconv-k8s-approvers:

• @frzifus
• @dmitryax
• @ChrsMark
• @mx-psi
• @TylerHelmuth
• @jinja2
• @dashpole
• @mlunadia
• TBA

Related PRs and issues:

• Add oci.manifest.digest, container.image.repo_digests and make container.image.tag array #159
• Add container metric fields (from ECS) #282
• Add container labels field from ECS #125
• Container semantic conventions: follow Open Container Initiative spec #48
• Remove abbreviation of kubernetes in resource semantic conventions #58

[ChrsMark]

@dmitryax could you also ping people from the Collector's side that would be interested to join?

@mlunadia would you be interested into joining as well?

[TylerHelmuth]

I am interested in joining. I would also like to nominate @jinja2 who has been a tremendous source of Kubernetes knowledge for the Collector components.

[ChrsMark]

Awesome @TylerHelmuth , I have added you both :)!

[dmitryax]

Thank you, @ChrsMark!

[TylerHelmuth]

Additional related issue: #430

[jsuereth]

I think this is a little to broad of a scope. E.g. are you going to be owning every semconv for technology coming from CNCF, including otel itself?

Let's alter the name to denote some limits, e.g.

• semconv-container-approvers: All things container related (not just docker)
• semconv-k8s-approvers: All things kubernetes related (but not nomad, e.g.)

I think it makes sense for us to keep the two lists separate even if it's the same set of folks initially.

[ChrsMark]

Thank's @jsuereth , that makes sense! I can split the group into 2 different groups then.

[dashpole]

I'd like to nominate myself for the k8s-approvers group.

[mlunadia]

@ChrsMark yes, please add me in

[arminru]

I created @open-telemetry/semconv-container-approvers and added the following members:

• @frzifus
• @dmitryax
• @ChrsMark
• @mx-psi
• @TylerHelmuth
• @jinja2

@mlunadia I was not able to add you as you are not a member of the OpenTelemetry organization yet.
Please review the requirements and application steps described here: https://github.com/open-telemetry/community/blob/main/community-membership.md#member

[arminru]

I created @open-telemetry/semconv-k8s-approvers and added the following members:

• @frzifus
• @dmitryax
• @ChrsMark
• @mx-psi
• @TylerHelmuth
• @jinja2
• @dashpole

@mlunadia please see my comment above.

[arminru]

@open-telemetry/semconv-container-approvers and @open-telemetry/semconv-k8s-approvers you can go ahead and update https://github.com/open-telemetry/semantic-conventions/blob/main/.github/CODEOWNERS to add OWNERS for the relevant subdirectories.

[ChrsMark]

Thank's @arminru. I have filed #510 to add the CODEOWNERS.

[mlunadia]

@arminru I have now my membership and have requested to join. Thanks!

[mlunadia]

@arminru just checking if you can add me to the approvers for both of these groups.
TIA

[arminru]

Hi @mlunadia!
After the initial bootstrapping of a newly founded group, approvers are expected to satisfy the approver requirements laid out here: https://github.com/open-telemetry/community/blob/main/community-membership.md#approver
Please open an issue if all of these apply so we can add you to the group.
Thanks!