Make device_status_check.py email clank-infra if devices go offline.

build/android/device_status_check.py

 #!/usr/bin/env python
 #
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """A class to keep track of devices across builds and report state."""
 
 import optparse
 import os
+import smtplib
+import string
 import sys
 
 from pylib import buildbot_report
 from pylib.android_commands import GetAttachedDevices
 from pylib.cmd_helper import GetCmdOutput
 
 
 def DeviceInfo(serial):
   """Gathers info on a device via various adb calls.
 
@@ skipping 42 matching lines @@
   last_devices = []
   try:
     with open(last_devices_path) as f:
       last_devices = f.read().splitlines()
   except IOError:
     # Ignore error, file might not exist
     pass
 
   missing_devs = list(set(last_devices) - set(adb_online_devs))
   if missing_devs:
+    from_address = 'buildbot@chromium.org'
+    to_address = 'clank-infrastructure-team@google.com'

[Isaac (away), 2012/09/29 03:17:04]

Suggest we add a keyword to this to make it easier to filter, such as
clank-infrastructure-team+devicealert@

[Isaac (away), 2012/09/29 03:33:08]

You can ignore this earlier comment.  But we can't use we can't use this
listserv here.  Use 
chromium-android-device-alerts [@] google.com  

[navabi1, 2012/10/01 21:52:15]

Done.

+    script_path_list = os.getcwd().split('/')
+    # Assumes name of bot is after .../build/slave/<BOT NAME>/...

[Isaac (away), 2012/09/29 03:17:04]

I think cleaner to either use environment variable BUILDBOT_BUILDERNAME or parse
factory properties.

[navabi1, 2012/10/01 21:52:15]

Done.

+    slave_index = script_path_list.index('slave')
+    bot_name = script_path_list[slave_index + 1]
+    subject = 'Device not detected on %s' % bot_name

[Isaac (away), 2012/09/29 03:17:04]

Several of the testers have the same name (they're on different waterfalls), so
maybe we can include the waterfall name in the email too.  This means passing in
$FACTORY_PROPERTIES to the build script (see bb_zip_build) and then parsing it.

We're going to get a lot of messages.  Maybe we can put enough info in the
subject so that we can decide how important is. Something like: name, waterfall,
how many devices left and how many total?

[navabi1, 2012/10/01 21:52:15]

Rather than pass FACTORY_PROPERTIES, I'll use the SLAVE_NAME which is part of
the environment.  That will uniquely identify which bot has missing devices.

I think we can leave the number of devices that have went offline in the message
text.

[Isaac (away), 2012/10/01 22:35:45]

sg

+
     buildbot_report.PrintWarning()
-    buildbot_report.PrintSummaryText(
-        '%d devices not detected.' % len(missing_devs))
-    print 'Current online devices: %s' % adb_online_devs
-    print '%s are no longer visible. Were they removed?\n' % missing_devs
-    print 'SHERIFF: See go/chrome_device_monitor'
-    print 'Cache file: %s\n\n' % last_devices_path
-    print 'adb devices'
-    print GetCmdOutput(['adb', 'devices'])
+    devices_missing_msg = '%d devices not detected.' % len(missing_devs)
+    buildbot_report.PrintSummaryText(devices_missing_msg)
+
+    body = string.join((

[Isaac (away), 2012/09/29 03:17:04]

nit: python will implicitly join strings that are next to each other.  You can
do multiple lines by surrounding with parens.  Something like 

body = ( 
    'yada yada'
    'yada yada'
    'yada yada' % (
    var1, var2, var3))

might be cleaner

[navabi1, 2012/10/01 21:52:15]

I prefer to leave it like this.  While it is cleaner the way you suggested, I
think it is less clear, because it is less obvious which string variable
corresponds to the format place holders.

string.join is deprecated. changed to '\r\n'.join([...])

[Isaac (away), 2012/10/01 22:35:45]

sg, was confused by the string.join

+        'Current online devices: %s' % adb_online_devs,
+        '%s are no longer visible. Were they removed?\n' % missing_devs,
+        'SHERIFF: See go/clank/engineering/buildbots/troubleshooting',
+        'Cache file: %s\n\n' % last_devices_path,
+        'adb devices', GetCmdOutput(['adb', 'devices'])), "\r\n")
+
+    msg_body = string.join((
+        "From: %s" % from_address,
+        "To: %s" % to_address,
+        "Subject: %s" % subject,
+        "",
+        body), "\r\n")
+
+    print body
+
+    # Send email from to clank infra
+    server = smtplib.SMTP('localhost')
+    server.sendmail(from_address, [to_address], msg_body)
+    server.quit()
   else:
     new_devs = set(adb_online_devs) - set(last_devices)
     if new_devs and os.path.exists(last_devices_path):
       buildbot_report.PrintWarning()
       buildbot_report.PrintSummaryText(
           '%d new devices detected' % len(new_devs))
       print ('New devices detected %s. And now back to your '
              'regularly scheduled program.' % list(new_devs))
 
   if not os.path.exists(out_dir):
@@ skipping 22 matching lines @@
   unique_types = list(set(types))
   unique_builds = list(set(builds))
 
   buildbot_report.PrintMsg('Online devices: %d. Device types %s, builds %s'
                            % (len(devices), unique_types, unique_builds))
   print '\n'.join(reports)
   CheckForMissingDevices(options, devices)
 
 if __name__ == '__main__':
   sys.exit(main())