search

elastic / apm-agent-go

https://www.elastic.co/guide/en/apm/agent/go/current/index.html
Apache License 2.0
422 stars 197 forks source link

feat: update CODEOWNERS #1624

Open kruskall opened 6 months ago

dmathieu commented 6 months ago

Should we then delete the @elastic/apm-agent-go team? Alternative: we could keep the owners file as-is, and change the members of the team?

dmathieu commented 6 months ago

cc @elastic/obs-ds-ebpf-go

rockdaboot commented 6 months ago

Should we then delete the @elastic/apm-agent-go team? Alternative: we could keep the owners file as-is, and change the members of the team?

The goal should be that people who are not directly involved in the maintenance of apm-agent-go or in the development of the lambda extension won't get GH notification emails ("reducing noise" for developers). And I don't think that every team member of @elastic/obs-ds-ebpf-go will be directly involved here. One way to achieve less noise is having more fine-grained GH groups.

Ideally, the decision on how to move forward is made by @AlexanderWert and @simitt.

dmathieu commented 6 months ago

If we don't want everyone from obs-ds-ebpf-go to get notifications, then keeping the team and changing its members seems like the best way.

simitt commented 6 months ago

Ideally, the decision on how to move forward is made by @AlexanderWert and @simitt.

This is team internal to how the work is split up within the obs-ds-ebpf-go team, so leaving this to @AlexanderWert; happy to remove the current members from https://github.com/orgs/elastic/teams/apm-agent-go and add new members to it, or completely sunset the apm-agent-go team.

kruskall commented 2 months ago

run docs-build