Re: [FBR] Make cvsadmin the admin group for dist-git pagure

[Pierre-Yves Chibon]

On Sun, Sep 24, 2017 at 10:42:37PM +0200, Till Maas wrote:
>  On Sun, Sep 24, 2017 at 11:04:51AM +0200, Pierre-Yves Chibon wrote:
>  
>  > I would make both group admins, unless you don't want help from us :)
>  
>  Help is always good. I did not want to accidently break something by
>  adding a second group instead of just changing the group. Here is my
>  proposal for a FBR to add the cvsadmin group:
>  
>  From bc61c3e99fff2c30cce9300a1d70761ec5a42dfc Mon Sep 17 00:00:00 2001
>  From: Till Maas <opensource(a)till.name&gt;
>  Date: Fri, 22 Sep 2017 21:46:47 +0200
>  Subject: [PATCH] Make cvsadmin group admin for dist-git pagure
>  
>  Releng manages dist-git pagure and in the past cvsadmin was used for
>  adminstrative permissions for dist-git. Therefore use it in dist-git
>  pagure as well.
>  
>  Reference: https://pagure.io/releng/issue/7061
>  
>  Signed-off-by: Till Maas <opensource(a)till.name&gt;
>  ---
>   roles/distgit/pagure/templates/pagure.cfg | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>  
>  diff --git a/roles/distgit/pagure/templates/pagure.cfg
> b/roles/distgit/pagure/templates/pagure.cfg
>  index 833c57dc8..53417c364 100644
>  --- a/roles/distgit/pagure/templates/pagure.cfg
>  +++ b/roles/distgit/pagure/templates/pagure.cfg
>  @@ -28,8 +28,8 @@ PREFERRED_URL_SCHEME='https'
>   #DB_URL=postgres://user:pass@host/db_name
>   DB_URL = 'postgresql://{{ distgit_pagure_db_user }}:{{ distgit_pagure_db_pass }}@{{
> distgit_pagure_db_host }}/{{ distgit_pagure_db_name }}'
>   
>  -### The FAS group in which the admin of pagure are
>  -ADMIN_GROUP = ['sysadmin-main']
>  +### FAS groups of pagure admins
>  +ADMIN_GROUP = ['cvsadmin', 'sysadmin-main'] 
+1 for me as well.


Pierre