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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxx>
> 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@xxxxxxxxx>
> ---
>  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
_______________________________________________
infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux