New teams docs-pt-approvers and docs-pt-maintainers

[svrnm]

Affected Repository

https://github.com/open-telemetry/opentelemetry.io

Requested changes

• Create a new group docs-pt-approvers with and @emdneto, @joaopgrassi, @jpkrohling, @theletterf, @cartermp, @austinlparker, @chalin and @svrnm as members (all as "maintainers" for the group)
• Create a new sub-group to docs-pt-approvers named docs-pt-maintainers with @jpkrohling, @theletterf, @cartermp, @austinlparker, @chalin and @svrnm as members (all as "maintainers" for the group)

Note: We have individual contributors (like @edsoncelio, @EzzioMoreira), who helped making this happen. We are looking forward to have them go through the process of becoming established members and approvers eventually, following the membership guidelines

Purpose

Team to manage Portuguese translations for opentelemetry.io

Expected Duration

permanently

Repository Maintainers

@open-telemetry/docs-maintainers

[joaopgrassi]

Feel free to also add me as an approver as a native pt-BR speaker :)

[emdneto]

I'm more than glad to also help with as a native pt-BR speaker as well 🚀

[svrnm]

Any updates? @open-telemetry/technical-committee @austinlparker @trask can someone take a look and create these teams for us? Thanks

[lmolkova]

done
/cc @open-telemetry/docs-pt-approvers and @open-telemetry/docs-pt-maintainers

[trask]

• Create a new group docs-pt-approvers with and @emdneto, @joaopgrassi, @jpkrohling, @theletterf, @cartermp, @austinlparker, @chalin and @svrnm as members (all as "maintainers" for the group)

typically only members of docs-pt-maintainers would be team maintainers for docs-pt-approvers

[lmolkova]

• Create a new group docs-pt-approvers with and @emdneto, @joaopgrassi, @jpkrohling, @theletterf, @cartermp, @austinlparker, @chalin and @svrnm as members (all as "maintainers" for the group)

typically only members of docs-pt-maintainers would be team maintainers for docs-pt-approvers

👍
@svrnm was there a specific reason to have everyone in approvers be a maintainer?

[svrnm]

• Create a new group docs-pt-approvers with and @emdneto, @joaopgrassi, @jpkrohling, @theletterf, @cartermp, @austinlparker, @chalin and @svrnm as members (all as "maintainers" for the group)

typically only members of docs-pt-maintainers would be team maintainers for docs-pt-approvers

👍 @svrnm was there a specific reason to have everyone in approvers be a maintainer?

No, probably a copy and paste error on my end.

[lmolkova]

fixed!

[chalin]

@lmolkova et al., any idea why we have this issue:

• [infra] Can't request review from docs-pt-approvers / maintainers opentelemetry.io#5090

[trask]

hey @chalin I think you need to give @open-telemetry/docs-pt-approvers write access to the repo first

[lmolkova]

hey @chalin I think you need to give @open-telemetry/docs-pt-approvers write access to the repo first

Thanks! Can confirm - I just did it and it works. I assume you also want to do it for @open-telemetry/docs-es-approvers

[chalin]

Thanks all!

Thanks! Can confirm - I just did it and it works.

Thanks all!

Btw, I thought that we were trying to avoid granting members or groups write access to repos, but if this is the way to go, so be it. /cc @svrnm

I assume you also want to do it for @open-telemetry/docs-es-approvers

Yes, for all localisation approvers, thanks.

[svrnm]

Btw, I thought that we were trying to avoid granting members or groups write access to repos, but if this is the way to go, so be it. /cc @svrnm

That's why we have component-owners today, we do not need repo write permissions anymore. Maybe teams still need triage or read permissions for that?

[chalin]

Ok, so is there anything to be adjusted here -- e.g., write permission revoked for @open-telemetry/docs-XX-approvers etc.

[svrnm]

Ok, so is there anything to be adjusted here -- e.g., write permission revoked for @open-telemetry/docs-XX-approvers etc.

Eventually, yes, it looks like we can lower permissions to "triage" for most teams that way