PR#49: Add ACL for mass rebuilding - pagure-dist-git

pagure-dist-git

#49 Add ACL for mass rebuilding

Merged 7 years ago by pingou. Opened 7 years ago by puiterwijk.

puiterwijk/pagure-dist-git rebuilders into master

Add ACL for mass rebuilding

Patrick Uiterwijk • 7 years ago

15a3cf3

dist_git_auth.py

file modified

+8 -1

		`@@ -98,12 +98,18 @@`

		`for repos in ['repos', 'requests/']:`
		`if repos == 'repos':`
		`+ config.append('repo %s' % project.fullname)`
		`+`
		`+ # Add ACL for mass rebuilds, even in PR-only`
		`+ config.append(' RW = releng')`
		`+`
		`# Do not grant access to project enforcing the PR model`
		`if pr_only or (global_pr_only and not project.is_fork):`
		`continue`
		`repos = ''`
		`+ else:`
		`+ config.append('repo %s%s' % (repos, project.fullname))`

		`- config.append('repo %s%s' % (repos, project.fullname))`
		`if repos not in ['tickets/', 'requests/']:`
		`config.append(' R = @all')`

		`@@ -165,6 +171,7 @@`
		`(access,`
		`werkzeug.secure_filename(project.fullname),`
		`deploykey.id))`
		`+`
		`config.append('')`

		`return config`

dist_git_auth_tests.py

file modified

+15

		`@@ -16,6 +16,7 @@`

		`expected = """`
		`repo test`
		`+ RW = releng`
		`R = @all`
		`RWC master = pingou`
		`RWC master = @provenpackager`
		`@@ -33,6 +34,7 @@`
		`RWC = pingou`

		`repo test2`
		`+ RW = releng`
		`R = @all`
		`RWC master = pingou`
		`RWC master = @provenpackager`
		`@@ -50,6 +52,7 @@`
		`RWC = pingou`

		`repo somenamespace/test3`
		`+ RW = releng`
		`R = @all`
		`RWC master = pingou`
		`RWC master = @provenpackager`
		`@@ -171,6 +174,7 @@`
		`contents = f.read()`

		`expected = '''repo test2`
		`+ RW = releng`
		`R = @all`
		`RWC master = pingou`
		`RWC master = @provenpackager`
		`@@ -188,6 +192,7 @@`
		`RWC = pingou`

		`repo somenamespace/test3`
		`+ RW = releng`
		`R = @all`
		`RWC master = pingou`
		`RWC master = @provenpackager`
		`@@ -205,6 +210,7 @@`
		`RWC = pingou`

		`repo test`
		`+ RW = releng`
		`R = @all`
		`RWC master = foo pingou`
		`RWC master = @provenpackager`
		`@@ -283,6 +289,7 @@`
		`# end of groups`

		`repo test2`
		`+ RW = releng`
		`R = @all`
		`RWC master = pingou`
		`RWC master = @provenpackager`
		`@@ -300,6 +307,7 @@`
		`RWC = pingou`

		`repo somenamespace/test3`
		`+ RW = releng`
		`R = @all`
		`RWC master = pingou`
		`RWC master = @provenpackager`
		`@@ -317,6 +325,7 @@`
		`RWC = pingou`

		`repo test`
		`+ RW = releng`
		`R = @all`
		`RWC master = foo`
		`RWC master = @test_grp @provenpackager`
		`@@ -371,6 +380,7 @@`
		`contents = f.read()`

		`expected = '''repo test`
		`+ RW = releng`
		`R = @all`
		`RWC master = pingou`
		`RWC master = @provenpackager`
		`@@ -388,6 +398,7 @@`
		`RWC = pingou`

		`repo test2`
		`+ RW = releng`
		`R = @all`
		`RWC master = pingou`
		`RWC master = @provenpackager`
		`@@ -405,6 +416,7 @@`
		`RWC = pingou`

		`repo somenamespace/test3`
		`+ RW = releng`
		`R = @all`
		`RWC master = pingou`
		`RWC master = @provenpackager`
		`@@ -422,6 +434,7 @@`
		`RWC = pingou`

		`repo forks/pingou/test`
		`+ RW = releng`
		`R = @all`
		`RW+C = pingou`

		`@@ -460,6 +473,7 @@`
		`with open(self.configfile, 'r') as f:`
		`contents = f.read()`
		`expected = """repo rpms/firefox`
		`+ RW = releng`
		`R = @all`
		`RWC master = pingou`
		`RWC f9000 = pingou`
		`@@ -505,6 +519,7 @@`
		`with open(self.configfile, 'r') as f:`
		`contents = f.read()`
		`expected = """repo firefox`
		`+ RW = releng`
		`R = @all`
		`RWC master = pingou`
		`RWC master = @provenpackager`