diff options
author | Trevor Gamblin <tgamblin@baylibre.com> | 2024-02-15 15:52:05 -0500 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2024-02-16 15:14:26 +0000 |
commit | 122401ce482fe8db9d83ccbe74e7beb9aa3f850a (patch) | |
tree | bee217ffcf2b2638db68020625bec4611377ec9f /scripts | |
parent | 6d6a9247186432214de507e46a284cd4ebc12932 (diff) | |
download | poky-122401ce482fe8db9d83ccbe74e7beb9aa3f850a.tar.gz |
patchtest-send-results: add --debug option
Add a -d/--debug option to patchtest-send-results so that the contents
of the constructed raw email can be checked without actually sending
anything to the list or patch author.
(From OE-Core rev: e50827d23fd3789e7a13df6918a06b198bbf4f95)
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'scripts')
-rwxr-xr-x | scripts/patchtest-send-results | 17 |
1 files changed, 11 insertions, 6 deletions
diff --git a/scripts/patchtest-send-results b/scripts/patchtest-send-results index f65d862074..8a3dadbd11 100755 --- a/scripts/patchtest-send-results +++ b/scripts/patchtest-send-results | |||
@@ -38,6 +38,7 @@ def has_a_failed_test(raw_results): | |||
38 | 38 | ||
39 | parser = argparse.ArgumentParser(description="Send patchtest results to a submitter for a given patch") | 39 | parser = argparse.ArgumentParser(description="Send patchtest results to a submitter for a given patch") |
40 | parser.add_argument("-p", "--patch", dest="patch", required=True, help="The patch file to summarize") | 40 | parser.add_argument("-p", "--patch", dest="patch", required=True, help="The patch file to summarize") |
41 | parser.add_argument("-d", "--debug", dest="debug", required=False, action='store_true', help="Print raw email headers and content, but don't actually send it") | ||
41 | args = parser.parse_args() | 42 | args = parser.parse_args() |
42 | 43 | ||
43 | if not os.path.exists(args.patch): | 44 | if not os.path.exists(args.patch): |
@@ -94,12 +95,16 @@ if has_a_failed_test(testresult): | |||
94 | '\nMIME-Version: 1.0" + \ | 95 | '\nMIME-Version: 1.0" + \ |
95 | "\nContent-type: Multipart/Mixed;boundary="NextPart"\n\n--NextPart\nContent-Type: text/plain\n\n' + \ | 96 | "\nContent-type: Multipart/Mixed;boundary="NextPart"\n\n--NextPart\nContent-Type: text/plain\n\n' + \ |
96 | reply_contents + '\n\n--NextPart' | 97 | reply_contents + '\n\n--NextPart' |
97 | response = ses_client.send_raw_email( | 98 | |
98 | Source="patchtest@automation.yoctoproject.org", | 99 | if args.debug: |
99 | RawMessage={ | 100 | print(f"RawMessage: \n\n{raw_data}") |
100 | "Data": raw_data, | 101 | else: |
101 | }, | 102 | response = ses_client.send_raw_email( |
102 | ) | 103 | Source="patchtest@automation.yoctoproject.org", |
104 | RawMessage={ | ||
105 | "Data": raw_data, | ||
106 | }, | ||
107 | ) | ||
103 | 108 | ||
104 | else: | 109 | else: |
105 | print(f"No failures identified for {args.patch}.") | 110 | print(f"No failures identified for {args.patch}.") |