Helm-charts: Give chart maintainer collaborators correct GitHub repo access level

Created on 19 Aug 2020  路  10Comments  路  Source: prometheus-community/helm-charts

After #11

See related #12 and #15

Context

Until #11 this repo is under my individual user account, therefore there is only one "collaborator" access level. But after this transfer, we will have options. One suggestion https://github.com/scottrigby/prometheus-helm-charts/issues/12#issuecomment-676499306 was that all chart maintainers be given repo "Maintain" access level.

Example options list:
Screen Shot 2020-08-19 at 11 44 35 AM

Open questions

  1. GitHub docs for Maintain access says it allows members to Push to protected branches. Does this mean bypassing the rules in CODEOWNERS? I don't have a current repo set up for testing this

Most helpful comment

Update, now that #11 is complete, there is a new team set up for this 鈥撀燼ll current chart maintainers will be invited soon. Until then, you are all still "collaborators" on this repo.

All 10 comments

Imho you can disable push access for admins too. I hope I can try later. I'm in the phone most of the time as I'm on vacation.

Update, now that #11 is complete, there is a new team set up for this 鈥撀燼ll current chart maintainers will be invited soon. Until then, you are all still "collaborators" on this repo.

For now the prometheus-community helm-charts-maintainers github team access is Write for this git repo, until we check out the above on a test repo.

All existing chart maintainers have been invite to the @prometheus-community/helm-charts-maintainers team 馃憤 I'll remove your old collaborator status. You will all need to accept the invite to maintain Write access to your charts.

@scottrigby aah, that makes sense. All of a sudden git started complaining about the write access 馃ぃ . I will accept that now.

@gkarthiks ok great. Please lmk if you run into any trouble- feel free to ping me on slack 馃檪

This is done

I don't think I've been invited, as I don't see any mail nor invitation in the github notifications pages, could you double check?

Thanks @desaintmartin you should have an invite now

Done, thanks!

Was this page helpful?
0 / 5 - 0 ratings