commit b7a9ef7a60b26c3b51fdc69a3ae99662fb841dc4
author Shuhei Takahashi <nya@chromium.org> Thu Jun 29 12:40:59 2017 +0900
committer chrome-bot <chrome-bot@chromium.org> Thu Jun 29 01:16:18 2017 -0700
tree 5e03146bd7d66ccb1d1b277c1a55c9950c5c3b65
parent 37663f4f282bf573e725d3afa00bfd78c92162bf

reporting: Skip sending mails if there is no recipient.

This will resolve the issue that a suite job crashes when it
attempts to send a mail to no recipient.

BUG=chromium:737836
TEST=run_pylint.py

Change-Id: I38d7cfc155adb4546da3ea447026a6d153c1d197
Reviewed-on: https://chromium-review.googlesource.com/554398
Commit-Ready: Shuhei Takahashi <nya@chromium.org>
Tested-by: Shuhei Takahashi <nya@chromium.org>
Reviewed-by: Dan Shi <dshi@google.com>

server/cros/dynamic_suite/reporting.py

diff --git a/server/cros/dynamic_suite/reporting.py b/server/cros/dynamic_suite/reporting.py
index 004f21f..7680530 100644
--- a/server/cros/dynamic_suite/reporting.py
+++ b/server/cros/dynamic_suite/reporting.py
@@ -987,6 +987,9 @@
     if bug_template.get('owner'):
         to_set.add(bug_template.get('owner'))
     recipients = ', '.join(to_set)
+    if not recipients:
+        logging.warning('No owner/cc found. Will skip sending a mail.')
+        return
     success = False
     try:
         gmail_lib.send_email(