Copies device_status_check to the buildbot directory

build/android/buildbot/bb_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 logging
 import optparse
 import os
 import smtplib
 import sys
 import re
 
+import bb_annotations

[frankf, 2013/07/15 17:43:02]

convention is to order the imports from most to least scope

[gkanwar1, 2013/07/15 17:54:09]

This is following the pattern of imports that occur in the other bb files. It
looks like this order was chosen because all imports after this smallest scope
require adding to the path, so those were moved to the end.

+
+sys.path.append(os.path.join(os.path.dirname(__file__), '..'))

[frankf, 2013/07/15 17:43:02]

Why do you need this?

[gkanwar1, 2013/07/15 17:54:09]

Since the file is being moved into buildbot, we need to add the parent directory
to make sure imports still work (bb_device_steps.py does the same thing, for
example).

 from pylib import android_commands
-from pylib import buildbot_report
 from pylib import constants
 from pylib.cmd_helper import GetCmdOutput
 
 
 def DeviceInfo(serial, options):
   """Gathers info on a device via various adb calls.
 
   Args:
     serial: The serial of the attached device to construct info about.
 
@@ skipping 109 matching lines @@
   all_known_devices = list(set(adb_online_devs) | set(last_devices))
   WriteDeviceList('.last_devices', all_known_devices)
   WriteDeviceList('.last_missing', missing_devs)
 
   if not all_known_devices:
     # This can happen if for some reason the .last_devices file is not
     # present or if it was empty.
     return ['No online devices. Have any devices been plugged in?']
   if missing_devs:
     devices_missing_msg = '%d devices not detected.' % len(missing_devs)
-    buildbot_report.PrintSummaryText(devices_missing_msg)
+    bb_annotations.PrintSummaryText(devices_missing_msg)
 
     # TODO(navabi): Debug by printing both output from GetCmdOutput and
     # GetAttachedDevices to compare results.
     return ['Current online devices: %s' % adb_online_devs,
             '%s are no longer visible. Were they removed?\n' % missing_devs,
             'SHERIFF: See go/chrome_device_monitor',
             'Cache file: %s\n\n' % last_devices_path,
             'adb devices: %s' % GetCmdOutput(['adb', 'devices']),
             'adb devices(GetAttachedDevices): %s' %
             android_commands.GetAttachedDevices()]
   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(
+      bb_annotations.PrintWarning()
+      bb_annotations.PrintSummaryText(
           '%d new devices detected' % len(new_devs))
       print ('New devices detected %s. And now back to your '
              'regularly scheduled program.' % list(new_devs))
 
 
 def SendDeviceStatusAlert(msg):
   from_address = 'buildbot@chromium.org'
   to_address = 'chromium-android-device-alerts@google.com'
   bot_name = os.environ.get('BUILDBOT_BUILDERNAME')
   slave_name = os.environ.get('BUILDBOT_SLAVENAME')
@@ skipping 25 matching lines @@
   fail_step_lst = []
   if devices:
     types, builds, reports, errors, fail_step_lst = (
         zip(*[DeviceInfo(dev, options) for dev in devices]))
 
   err_msg = CheckForMissingDevices(options, devices) or []
 
   unique_types = list(set(types))
   unique_builds = list(set(builds))
 
-  buildbot_report.PrintMsg('Online devices: %d. Device types %s, builds %s'
+  bb_annotations.PrintMsg('Online devices: %d. Device types %s, builds %s'
                            % (len(devices), unique_types, unique_builds))
   print '\n'.join(reports)
 
   for serial, dev_errors in zip(devices, errors):
     if dev_errors:
       err_msg += ['%s errors:' % serial]
       err_msg += ['    %s' % error for error in dev_errors]
 
   if err_msg:
-    buildbot_report.PrintWarning()
+    bb_annotations.PrintWarning()
     msg = '\n'.join(err_msg)
     print msg
     SendDeviceStatusAlert(msg)
 
   if False in fail_step_lst:
     # TODO(navabi): Build fails on device status check step if there exists any
     # devices with critically low battery or install speed. Remove those devices
     # from testing, allowing build to continue with good devices.
     return 1
 
   if not devices:
     return 1
 
 
 if __name__ == '__main__':
   sys.exit(main())