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

When sending alternative arch excluding emails, include the full change

Browse source 
Fixes https://pagure.io/fedora-infrastructure/issue/5923
This commit is contained in:
Pierre-Yves Chibon 2017-04-18 14:09:26 +02:00
parent d5fd760c55
commit 678e30e837
1 changed files with 9 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

10
roles/git/hooks/files/post-receive-alternativearch
View file

@ -35,6 +35,10 @@ Change:
%(change)s %(change)s
Thanks. Thanks.
Full change:
%(full_change)s
""" """
def read_output(cmd, abspath, input=None, keepends=False, **kw): def read_output(cmd, abspath, input=None, keepends=False, **kw):
@ -158,11 +162,14 @@ def run_as_post_receive_hook():
if DEBUG: if DEBUG:
print('List of commits:', new_commits_list) print('List of commits:', new_commits_list)
full_change = ''
exclude_arch = {} exclude_arch = {}
for commit in new_commits_list: for commit in new_commits_list:
if DEBUG: if DEBUG:
print('Diff of commit:', commit) print('Diff of commit:', commit)
full_change += '\nCommit: %s\n' % commit
for line in read_git_lines(['show', commit], abspath): for line in read_git_lines(['show', commit], abspath):
full_change += '%s\n' % line
if DEBUG: if DEBUG:
print(line) print(line)
if line.strip().startswith(PATTERNS): if line.strip().startswith(PATTERNS):
@ -185,7 +192,8 @@ def run_as_post_receive_hook():
send_email( send_email(
text=TEXT % dict( text=TEXT % dict(
pkg=package, url='\n'.join(links), change="\n".join(diffs) pkg=package, url='\n'.join(links), change="\n".join(diffs),
full_change=full_change
), ),
subject='Architecture specific change in %s' % package, subject='Architecture specific change in %s' % package,
to_mail=TO_MAIL to_mail=TO_MAIL