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 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']
 
 ### The email address to which the flask.log will send the errors (tracebacks)
 EMAIL_ERROR = 'pingou@xxxxxxxxxxxx'
-- 
2.13.5
_______________________________________________
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