Infrastructure/ansible
1
0
Fork 0
Code Issues Pull requests 6 Projects Releases Packages Wiki Activity Actions

Technically we only need os.walk() on forks

Browse source 
The other namespace won't have sub-folders

Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
This commit is contained in:
Pierre-Yves Chibon 2018-04-17 15:46:40 +02:00
parent cab27352d2
commit 7b6407df81
1 changed files with 24 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

32
roles/git/checks/files/distgit_check_hook.py
View file

@ -57,7 +57,7 @@ def is_valid_hook(hook, target_link):
return output return output
def process_namespace(namespace, check): def process_namespace(namespace, check, walk=False):
""" Process all the git repo in a specified namespace. """ """ Process all the git repo in a specified namespace. """
target_link = _target_link target_link = _target_link
if namespace == 'forks': if namespace == 'forks':
@ -68,12 +68,22 @@ def process_namespace(namespace, check):
if not os.path.isdir(path): if not os.path.isdir(path):
return return
for dirpath, dirnames, filenames in os.walk(path): if walk:
# Don't go down the .git repos for dirpath, dirnames, filenames in os.walk(path):
if '.git' in dirpath: # Don't go down the .git repos
continue if '.git' in dirpath:
continue
for repo in dirnames: for repo in dirnames:
repo_path = os.path.join(dirpath, repo)
if not repo_path.endswith('.git'):
continue
hook = os.path.join(repo_path, 'hooks', 'post-receive')
if not is_valid_hook(hook, target_link) and not check:
fix_link(hook, target_link)
else:
for repo in os.listdir(path):
repo_path = os.path.join(dirpath, repo) repo_path = os.path.join(dirpath, repo)
if not repo_path.endswith('.git'): if not repo_path.endswith('.git'):
continue continue
@ -111,11 +121,17 @@ def main():
fix_link(hook, target_link) fix_link(hook, target_link)
elif args.namespace: elif args.namespace:
process_namespace(args.namespace, args.check) walk = False
if namespace == 'forks':
walk = true
process_namespace(args.namespace, args.check, walk=walk)
else: else:
# Check all repos # Check all repos
for namespace in namespaces: for namespace in namespaces:
process_namespace(namespace, args.check) walk = False
if namespace == 'forks':
walk = true
process_namespace(namespace, args.check, walk=walk)
if __name__ == '__main__': if __name__ == '__main__':