summaryrefslogtreecommitdiffstats
path: root/scripts/patchtest-send-results
diff options
context:
space:
mode:
Diffstat (limited to 'scripts/patchtest-send-results')
-rwxr-xr-xscripts/patchtest-send-results19
1 files changed, 13 insertions, 6 deletions
diff --git a/scripts/patchtest-send-results b/scripts/patchtest-send-results
index 078651ec38..01b071159b 100755
--- a/scripts/patchtest-send-results
+++ b/scripts/patchtest-send-results
@@ -17,6 +17,7 @@ import boto3
17import configparser 17import configparser
18import mailbox 18import mailbox
19import os 19import os
20import re
20import sys 21import sys
21 22
22greeting = """Thank you for your submission. Patchtest identified one 23greeting = """Thank you for your submission. Patchtest identified one
@@ -25,10 +26,12 @@ more information:\n\n---\n"""
25 26
26suggestions = """\n---\n\nPlease address the issues identified and 27suggestions = """\n---\n\nPlease address the issues identified and
27submit a new revision of the patch, or alternatively, reply to this 28submit a new revision of the patch, or alternatively, reply to this
28email with an explanation of why the patch format should be accepted. If 29email with an explanation of why the patch should be accepted. If you
29you believe these results are due to an error in patchtest, please 30believe these results are due to an error in patchtest, please submit a
30submit a bug at https://bugzilla.yoctoproject.org/ (use the 'Patchtest' 31bug at https://bugzilla.yoctoproject.org/ (use the 'Patchtest' category
31category under 'Yocto Project Subprojects'). Thank you!""" 32under 'Yocto Project Subprojects'). For more information on specific
33failures, see: https://wiki.yoctoproject.org/wiki/Patchtest. Thank
34you!"""
32 35
33parser = argparse.ArgumentParser(description="Send patchtest results to a submitter for a given patch") 36parser = argparse.ArgumentParser(description="Send patchtest results to a submitter for a given patch")
34parser.add_argument("-p", "--patch", dest="patch", required=True, help="The patch file to summarize") 37parser.add_argument("-p", "--patch", dest="patch", required=True, help="The patch file to summarize")
@@ -54,6 +57,10 @@ mbox = mailbox.mbox(args.patch)
54mbox_subject = mbox[0]['subject'] 57mbox_subject = mbox[0]['subject']
55subject_line = f"Patchtest results for {mbox_subject}" 58subject_line = f"Patchtest results for {mbox_subject}"
56 59
60# extract the submitter email address and use it as the reply address
61# for the results
62reply_address = re.findall("<(.*)>", mbox[0]['from'])
63
57if "FAIL" in testresult: 64if "FAIL" in testresult:
58 reply_contents = None 65 reply_contents = None
59 if len(max(open(result_file, 'r'), key=len)) > 220: 66 if len(max(open(result_file, 'r'), key=len)) > 220:
@@ -66,9 +73,9 @@ if "FAIL" in testresult:
66 response = ses_client.send_email( 73 response = ses_client.send_email(
67 Source='patchtest@automation.yoctoproject.org', 74 Source='patchtest@automation.yoctoproject.org',
68 Destination={ 75 Destination={
69 'ToAddresses': ['test-list@lists.yoctoproject.org'], 76 'CcAddresses': ['openembedded-core@lists.openembedded.org'],
70 }, 77 },
71 ReplyToAddresses=['test-list@lists.yoctoproject.org'], 78 ReplyToAddresses=reply_address,
72 Message={ 79 Message={
73 'Subject': { 80 'Subject': {
74 'Data': subject_line, 81 'Data': subject_line,