<div dir="ltr">Sounds good to me. I merged the PR and re-deployed the api server.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 9, 2018 at 5:57 AM, Alexandre Viau <span dir="ltr"><<a href="mailto:aviau@debian.org" target="_blank">aviau@debian.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<br>
Currently, *-guest users in <a href="http://salsa.debian.org/go-team" rel="noreferrer" target="_blank">salsa.debian.org/go-team</a> have "Master"<br>
permissions. I would like to change that to "developer".<br>
<br>
The only group-level permission difference between "Master" and<br>
"Developer" on gitlab is:<br>
 - "Create project in group" (this is originally why I gave them Master<br>
access)<br>
<br>
Project-level differences include:<br>
 - pushing to protected branches (they need this!)<br>
 - Edit project (and all project settings)<br>
<br>
Remarks:<br>
 - *-guests users should use dh-make-golang to create projects. They<br>
shouldn't require the "create project in group" permission.<br>
<br>
 - *-guest users should be able to push to protected branches<br>
<br>
We can allow developers to push to protected branches. This is<br>
configurable. When you protect a branch, you may allow "developers" to<br>
push to it.<br>
<br>
I reconfigured all of our protected branches to allow pushes from<br>
developers.<br>
<br>
I sent a merge request to go-team/ci to allow developers to push on<br>
protected branches:<br>
 - <a href="https://salsa.debian.org/go-team/ci/merge_requests/1" rel="noreferrer" target="_blank">https://salsa.debian.org/go-<wbr>team/ci/merge_requests/1</a><br>
<br>
This will have the following benefits:<br>
 - *-guest users will no longer be able to **force push** to important<br>
branches. This will protect our repositories from mass destruction by a<br>
guest, but they still can work on repositories as usual.<br>
<br>
 - *-guest users will now be forced to use dh-make-golang to create<br>
repositories. This is nice because some of them currently create<br>
repositories manually without the team-wide config, as they don't know<br>
our tools very well yet.<br>
<br>
When/if my merge request on go-team/ci is merged, I will change all<br>
guest users permissions to "developer".<br>
<br>
What do you think?<br>
<br>
cc-ing Michael because he will need to rebuild and deploy the server<br>
component of dh-make-golang if this goes trough.<br>
<br>
Cheers,<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Alexandre Viau<br>
<a href="mailto:aviau@debian.org">aviau@debian.org</a><br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Best regards,<br>Michael</div>
</div>