Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 1 | #!/usr/bin/python -u |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 2 | # -*- coding: utf-8 -*- |
| 3 | # |
Jon Salz | 37eccbd | 2012-05-25 16:06:52 +0800 | [diff] [blame] | 4 | # Copyright (c) 2012 The Chromium OS Authors. All rights reserved. |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 5 | # Use of this source code is governed by a BSD-style license that can be |
| 6 | # found in the LICENSE file. |
| 7 | |
| 8 | ''' |
| 9 | The main factory flow that runs the factory test and finalizes a device. |
| 10 | ''' |
| 11 | |
Jon Salz | e12c2b3 | 2013-06-25 16:24:34 +0800 | [diff] [blame] | 12 | import glob |
Jon Salz | 0405ab5 | 2012-03-16 15:26:52 +0800 | [diff] [blame] | 13 | import logging |
| 14 | import os |
Jon Salz | 73e0fd0 | 2012-04-04 11:46:38 +0800 | [diff] [blame] | 15 | import Queue |
Jon Salz | e12c2b3 | 2013-06-25 16:24:34 +0800 | [diff] [blame] | 16 | import shutil |
Jon Salz | 77c151e | 2012-08-28 07:20:37 +0800 | [diff] [blame] | 17 | import signal |
Jon Salz | 0405ab5 | 2012-03-16 15:26:52 +0800 | [diff] [blame] | 18 | import sys |
Jon Salz | eff9418 | 2013-06-19 15:06:28 +0800 | [diff] [blame] | 19 | import syslog |
Jon Salz | 0405ab5 | 2012-03-16 15:26:52 +0800 | [diff] [blame] | 20 | import threading |
| 21 | import time |
| 22 | import traceback |
Jon Salz | 258a40c | 2012-04-19 12:34:01 +0800 | [diff] [blame] | 23 | import uuid |
Jon Salz | b10cf51 | 2012-08-09 17:29:21 +0800 | [diff] [blame] | 24 | from xmlrpclib import Binary |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 25 | from collections import deque |
| 26 | from optparse import OptionParser |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 27 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 28 | import factory_common # pylint: disable=W0611 |
jcliang | cd68818 | 2012-08-20 21:01:26 +0800 | [diff] [blame] | 29 | from cros.factory import event_log |
| 30 | from cros.factory import system |
Cheng-Yi Chiang | d040652 | 2013-04-01 15:40:18 +0800 | [diff] [blame] | 31 | from cros.factory.event_log import EventLog, FloatDigit |
Tom Wai-Hong Tam | d33723e | 2013-04-10 21:14:37 +0800 | [diff] [blame] | 32 | from cros.factory.event_log_watcher import EventLogWatcher |
jcliang | cd68818 | 2012-08-20 21:01:26 +0800 | [diff] [blame] | 33 | from cros.factory.goofy import test_environment |
| 34 | from cros.factory.goofy import time_sanitizer |
Jon Salz | 8359178 | 2012-06-26 11:09:58 +0800 | [diff] [blame] | 35 | from cros.factory.goofy import updater |
jcliang | cd68818 | 2012-08-20 21:01:26 +0800 | [diff] [blame] | 36 | from cros.factory.goofy.goofy_rpc import GoofyRPC |
| 37 | from cros.factory.goofy.invocation import TestInvocation |
| 38 | from cros.factory.goofy.prespawner import Prespawner |
Cheng-Yi Chiang | 344b10f | 2013-05-03 16:44:03 +0800 | [diff] [blame] | 39 | from cros.factory.goofy.system_log_manager import SystemLogManager |
jcliang | cd68818 | 2012-08-20 21:01:26 +0800 | [diff] [blame] | 40 | from cros.factory.goofy.web_socket_manager import WebSocketManager |
Cheng-Yi Chiang | d818695 | 2013-04-04 23:41:14 +0800 | [diff] [blame] | 41 | from cros.factory.system.board import Board, BoardException |
jcliang | cd68818 | 2012-08-20 21:01:26 +0800 | [diff] [blame] | 42 | from cros.factory.system.charge_manager import ChargeManager |
Cheng-Yi Chiang | cdfa418 | 2013-05-05 03:20:19 +0800 | [diff] [blame] | 43 | from cros.factory.system.core_dump_manager import CoreDumpManager |
Jon Salz | ce6a7f8 | 2013-06-10 18:22:54 +0800 | [diff] [blame] | 44 | from cros.factory.system.cpufreq_manager import CpufreqManager |
Jon Salz | b92c511 | 2012-09-21 15:40:11 +0800 | [diff] [blame] | 45 | from cros.factory.system import disk_space |
jcliang | cd68818 | 2012-08-20 21:01:26 +0800 | [diff] [blame] | 46 | from cros.factory.test import factory |
| 47 | from cros.factory.test import state |
Jon Salz | 51528e1 | 2012-07-02 18:54:45 +0800 | [diff] [blame] | 48 | from cros.factory.test import shopfloor |
Jon Salz | 8359178 | 2012-06-26 11:09:58 +0800 | [diff] [blame] | 49 | from cros.factory.test import utils |
| 50 | from cros.factory.test.event import Event |
| 51 | from cros.factory.test.event import EventClient |
| 52 | from cros.factory.test.event import EventServer |
jcliang | cd68818 | 2012-08-20 21:01:26 +0800 | [diff] [blame] | 53 | from cros.factory.test.factory import TestState |
Dean Liao | 592e4d5 | 2013-01-10 20:06:39 +0800 | [diff] [blame] | 54 | from cros.factory.tools.key_filter import KeyFilter |
Jon Salz | 2af235d | 2013-06-24 14:47:21 +0800 | [diff] [blame] | 55 | from cros.factory.utils import file_utils |
Jon Salz | 78c3239 | 2012-07-25 14:18:29 +0800 | [diff] [blame] | 56 | from cros.factory.utils.process_utils import Spawn |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 57 | |
| 58 | |
Jon Salz | 2f757d4 | 2012-06-27 17:06:42 +0800 | [diff] [blame] | 59 | CUSTOM_DIR = os.path.join(factory.FACTORY_PATH, 'custom') |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 60 | HWID_CFG_PATH = '/usr/local/share/chromeos-hwid/cfg' |
Chun-ta Lin | 279e7e9 | 2013-02-19 17:40:39 +0800 | [diff] [blame] | 61 | CACHES_DIR = os.path.join(factory.get_state_root(), "caches") |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 62 | |
Jon Salz | 8796e36 | 2012-05-24 11:39:09 +0800 | [diff] [blame] | 63 | # File that suppresses reboot if present (e.g., for development). |
| 64 | NO_REBOOT_FILE = '/var/log/factory.noreboot' |
| 65 | |
Jon Salz | 5c344f6 | 2012-07-13 14:31:16 +0800 | [diff] [blame] | 66 | # Value for tests_after_shutdown that forces auto-run (e.g., after |
| 67 | # a factory update, when the available set of tests might change). |
| 68 | FORCE_AUTO_RUN = 'force_auto_run' |
| 69 | |
cychiang | 2188674 | 2012-07-05 15:16:32 +0800 | [diff] [blame] | 70 | RUN_QUEUE_TIMEOUT_SECS = 10 |
| 71 | |
Justin Chuang | 8381398 | 2013-05-13 01:26:32 +0800 | [diff] [blame] | 72 | # Sync disks when battery level is higher than this value. |
| 73 | # Otherwise, power loss during disk sync operation may incur even worse outcome. |
| 74 | MIN_BATTERY_LEVEL_FOR_DISK_SYNC = 1.0 |
| 75 | |
Jon Salz | e12c2b3 | 2013-06-25 16:24:34 +0800 | [diff] [blame] | 76 | MAX_CRASH_FILE_SIZE = 64*1024 |
| 77 | |
Jon Salz | 758e6cc | 2012-04-03 15:47:07 +0800 | [diff] [blame] | 78 | GOOFY_IN_CHROOT_WARNING = '\n' + ('*' * 70) + ''' |
| 79 | You are running Goofy inside the chroot. Autotests are not supported. |
| 80 | |
| 81 | To use Goofy in the chroot, first install an Xvnc server: |
| 82 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 83 | sudo apt-get install tightvncserver |
Jon Salz | 758e6cc | 2012-04-03 15:47:07 +0800 | [diff] [blame] | 84 | |
| 85 | ...and then start a VNC X server outside the chroot: |
| 86 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 87 | vncserver :10 & |
| 88 | vncviewer :10 |
Jon Salz | 758e6cc | 2012-04-03 15:47:07 +0800 | [diff] [blame] | 89 | |
| 90 | ...and run Goofy as follows: |
| 91 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 92 | env --unset=XAUTHORITY DISPLAY=localhost:10 python goofy.py |
Jon Salz | 758e6cc | 2012-04-03 15:47:07 +0800 | [diff] [blame] | 93 | ''' + ('*' * 70) |
Jon Salz | 73e0fd0 | 2012-04-04 11:46:38 +0800 | [diff] [blame] | 94 | suppress_chroot_warning = False |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 95 | |
| 96 | def get_hwid_cfg(): |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 97 | ''' |
| 98 | Returns the HWID config tag, or an empty string if none can be found. |
| 99 | ''' |
| 100 | if 'CROS_HWID' in os.environ: |
| 101 | return os.environ['CROS_HWID'] |
| 102 | if os.path.exists(HWID_CFG_PATH): |
| 103 | with open(HWID_CFG_PATH, 'rt') as hwid_cfg_handle: |
| 104 | return hwid_cfg_handle.read().strip() |
| 105 | return '' |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 106 | |
| 107 | |
| 108 | def find_test_list(): |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 109 | ''' |
| 110 | Returns the path to the active test list, based on the HWID config tag. |
Jon Salz | fb61589 | 2013-02-01 18:04:35 +0800 | [diff] [blame] | 111 | |
| 112 | The algorithm is: |
| 113 | |
| 114 | - Try $FACTORY/test_lists/active (the symlink reflecting the option chosen |
| 115 | in the UI). |
| 116 | - For each of $FACTORY/custom, $FACTORY/test_lists (and |
| 117 | autotest/site_tests/suite_Factory for backward compatibility): |
| 118 | - Try test_list_${hwid_cfg} (if hwid_cfg is set) |
| 119 | - Try test_list |
| 120 | - Try test_list.generic |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 121 | ''' |
Jon Salz | fb61589 | 2013-02-01 18:04:35 +0800 | [diff] [blame] | 122 | # If the 'active' symlink is present, that trumps everything else. |
| 123 | if os.path.lexists(factory.ACTIVE_TEST_LIST_SYMLINK): |
| 124 | return os.path.realpath(factory.ACTIVE_TEST_LIST_SYMLINK) |
| 125 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 126 | hwid_cfg = get_hwid_cfg() |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 127 | |
Jon Salz | fb61589 | 2013-02-01 18:04:35 +0800 | [diff] [blame] | 128 | search_dirs = [CUSTOM_DIR, factory.TEST_LISTS_PATH] |
Jon Salz | 4be56b0 | 2012-12-22 07:30:46 +0800 | [diff] [blame] | 129 | if not utils.in_chroot(): |
| 130 | # Also look in suite_Factory. For backward compatibility only; |
| 131 | # new boards should just put the test list in the "test_lists" |
| 132 | # directory. |
| 133 | search_dirs.insert(0, os.path.join( |
| 134 | os.path.dirname(factory.FACTORY_PATH), |
| 135 | 'autotest', 'site_tests', 'suite_Factory')) |
Jon Salz | 2f757d4 | 2012-06-27 17:06:42 +0800 | [diff] [blame] | 136 | |
Jon Salz | fb61589 | 2013-02-01 18:04:35 +0800 | [diff] [blame] | 137 | |
| 138 | search_files = [] |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 139 | if hwid_cfg: |
Jon Salz | fb61589 | 2013-02-01 18:04:35 +0800 | [diff] [blame] | 140 | search_files += [hwid_cfg] |
| 141 | search_files += ['test_list', 'test_list.generic'] |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 142 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 143 | for d in search_dirs: |
| 144 | for f in search_files: |
| 145 | test_list = os.path.join(d, f) |
| 146 | if os.path.exists(test_list): |
| 147 | return test_list |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 148 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 149 | logging.warn('Cannot find test lists named any of %s in any of %s', |
| 150 | search_files, search_dirs) |
| 151 | return None |
Jon Salz | 73e0fd0 | 2012-04-04 11:46:38 +0800 | [diff] [blame] | 152 | |
Jon Salz | fb61589 | 2013-02-01 18:04:35 +0800 | [diff] [blame] | 153 | |
Jon Salz | 73e0fd0 | 2012-04-04 11:46:38 +0800 | [diff] [blame] | 154 | _inited_logging = False |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 155 | |
| 156 | class Goofy(object): |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 157 | ''' |
| 158 | The main factory flow. |
| 159 | |
| 160 | Note that all methods in this class must be invoked from the main |
| 161 | (event) thread. Other threads, such as callbacks and TestInvocation |
| 162 | methods, should instead post events on the run queue. |
| 163 | |
| 164 | TODO: Unit tests. (chrome-os-partner:7409) |
| 165 | |
| 166 | Properties: |
| 167 | uuid: A unique UUID for this invocation of Goofy. |
| 168 | state_instance: An instance of FactoryState. |
| 169 | state_server: The FactoryState XML/RPC server. |
| 170 | state_server_thread: A thread running state_server. |
| 171 | event_server: The EventServer socket server. |
| 172 | event_server_thread: A thread running event_server. |
| 173 | event_client: A client to the event server. |
| 174 | connection_manager: The connection_manager object. |
Cheng-Yi Chiang | 835f268 | 2013-05-06 22:15:48 +0800 | [diff] [blame] | 175 | system_log_manager: The SystemLogManager object. |
| 176 | core_dump_manager: The CoreDumpManager object. |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 177 | ui_process: The factory ui process object. |
| 178 | run_queue: A queue of callbacks to invoke from the main thread. |
| 179 | invocations: A map from FactoryTest objects to the corresponding |
| 180 | TestInvocations objects representing active tests. |
| 181 | tests_to_run: A deque of tests that should be run when the current |
| 182 | test(s) complete. |
| 183 | options: Command-line options. |
| 184 | args: Command-line args. |
| 185 | test_list: The test list. |
| 186 | event_handlers: Map of Event.Type to the method used to handle that |
| 187 | event. If the method has an 'event' argument, the event is passed |
| 188 | to the handler. |
| 189 | exceptions: Exceptions encountered in invocation threads. |
Jon Salz | 3c493bb | 2013-02-07 17:24:58 +0800 | [diff] [blame] | 190 | last_log_disk_space_message: The last message we logged about disk space |
| 191 | (to avoid duplication). |
Cheng-Yi Chiang | cdfa418 | 2013-05-05 03:20:19 +0800 | [diff] [blame] | 192 | last_kick_sync_time: The last time to kick system_log_manager to sync |
| 193 | because of core dump files (to avoid kicking too soon then abort the |
| 194 | sync.) |
Jon Salz | 416f9cc | 2013-05-10 18:32:50 +0800 | [diff] [blame] | 195 | hooks: A Hooks object containing hooks for various Goofy actions. |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 196 | ''' |
| 197 | def __init__(self): |
| 198 | self.uuid = str(uuid.uuid4()) |
| 199 | self.state_instance = None |
| 200 | self.state_server = None |
| 201 | self.state_server_thread = None |
Jon Salz | 16d1054 | 2012-07-23 12:18:45 +0800 | [diff] [blame] | 202 | self.goofy_rpc = None |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 203 | self.event_server = None |
| 204 | self.event_server_thread = None |
| 205 | self.event_client = None |
| 206 | self.connection_manager = None |
Vic Yang | 4953fc1 | 2012-07-26 16:19:53 +0800 | [diff] [blame] | 207 | self.charge_manager = None |
Jon Salz | 8fa8e83 | 2012-07-13 19:04:09 +0800 | [diff] [blame] | 208 | self.time_sanitizer = None |
| 209 | self.time_synced = False |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 210 | self.log_watcher = None |
Cheng-Yi Chiang | 344b10f | 2013-05-03 16:44:03 +0800 | [diff] [blame] | 211 | self.system_log_manager = None |
Cheng-Yi Chiang | 835f268 | 2013-05-06 22:15:48 +0800 | [diff] [blame] | 212 | self.core_dump_manager = None |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 213 | self.event_log = None |
| 214 | self.prespawner = None |
| 215 | self.ui_process = None |
Jon Salz | c79a998 | 2012-08-30 04:42:01 +0800 | [diff] [blame] | 216 | self.dummy_shopfloor = None |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 217 | self.run_queue = Queue.Queue() |
| 218 | self.invocations = {} |
| 219 | self.tests_to_run = deque() |
| 220 | self.visible_test = None |
| 221 | self.chrome = None |
Jon Salz | 416f9cc | 2013-05-10 18:32:50 +0800 | [diff] [blame] | 222 | self.hooks = None |
Vic Yang | d8990da | 2013-06-27 16:57:43 +0800 | [diff] [blame^] | 223 | self.cpu_usage_watcher = None |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 224 | |
| 225 | self.options = None |
| 226 | self.args = None |
| 227 | self.test_list = None |
| 228 | self.on_ui_startup = [] |
| 229 | self.env = None |
Jon Salz | b22d117 | 2012-08-06 10:38:57 +0800 | [diff] [blame] | 230 | self.last_idle = None |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 231 | self.last_shutdown_time = None |
cychiang | 2188674 | 2012-07-05 15:16:32 +0800 | [diff] [blame] | 232 | self.last_update_check = None |
Jon Salz | 8fa8e83 | 2012-07-13 19:04:09 +0800 | [diff] [blame] | 233 | self.last_sync_time = None |
Jon Salz | b92c511 | 2012-09-21 15:40:11 +0800 | [diff] [blame] | 234 | self.last_log_disk_space_time = None |
Jon Salz | 3c493bb | 2013-02-07 17:24:58 +0800 | [diff] [blame] | 235 | self.last_log_disk_space_message = None |
Justin Chuang | 8381398 | 2013-05-13 01:26:32 +0800 | [diff] [blame] | 236 | self.last_check_battery_time = None |
| 237 | self.last_check_battery_message = None |
Cheng-Yi Chiang | cdfa418 | 2013-05-05 03:20:19 +0800 | [diff] [blame] | 238 | self.last_kick_sync_time = None |
Vic Yang | 311ddb8 | 2012-09-26 12:08:28 +0800 | [diff] [blame] | 239 | self.exclusive_items = set() |
Jon Salz | 0f99660 | 2012-10-03 15:26:48 +0800 | [diff] [blame] | 240 | self.event_log = None |
Dean Liao | 592e4d5 | 2013-01-10 20:06:39 +0800 | [diff] [blame] | 241 | self.key_filter = None |
Jon Salz | ce6a7f8 | 2013-06-10 18:22:54 +0800 | [diff] [blame] | 242 | self.cpufreq_manager = None |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 243 | |
Jon Salz | 85a3988 | 2012-07-05 16:45:04 +0800 | [diff] [blame] | 244 | def test_or_root(event, parent_or_group=True): |
| 245 | '''Returns the test affected by a particular event. |
| 246 | |
| 247 | Args: |
| 248 | event: The event containing an optional 'path' attribute. |
| 249 | parent_on_group: If True, returns the top-level parent for a test (the |
| 250 | root node of the tests that need to be run together if the given test |
| 251 | path is to be run). |
| 252 | ''' |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 253 | try: |
| 254 | path = event.path |
| 255 | except AttributeError: |
| 256 | path = None |
| 257 | |
| 258 | if path: |
Jon Salz | 85a3988 | 2012-07-05 16:45:04 +0800 | [diff] [blame] | 259 | test = self.test_list.lookup_path(path) |
| 260 | if parent_or_group: |
| 261 | test = test.get_top_level_parent_or_group() |
| 262 | return test |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 263 | else: |
| 264 | return self.test_list |
| 265 | |
| 266 | self.event_handlers = { |
| 267 | Event.Type.SWITCH_TEST: self.handle_switch_test, |
| 268 | Event.Type.SHOW_NEXT_ACTIVE_TEST: |
| 269 | lambda event: self.show_next_active_test(), |
| 270 | Event.Type.RESTART_TESTS: |
| 271 | lambda event: self.restart_tests(root=test_or_root(event)), |
| 272 | Event.Type.AUTO_RUN: |
| 273 | lambda event: self.auto_run(root=test_or_root(event)), |
| 274 | Event.Type.RE_RUN_FAILED: |
| 275 | lambda event: self.re_run_failed(root=test_or_root(event)), |
| 276 | Event.Type.RUN_TESTS_WITH_STATUS: |
| 277 | lambda event: self.run_tests_with_status( |
| 278 | event.status, |
| 279 | root=test_or_root(event)), |
| 280 | Event.Type.REVIEW: |
| 281 | lambda event: self.show_review_information(), |
| 282 | Event.Type.UPDATE_SYSTEM_INFO: |
| 283 | lambda event: self.update_system_info(), |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 284 | Event.Type.STOP: |
Jon Salz | 85a3988 | 2012-07-05 16:45:04 +0800 | [diff] [blame] | 285 | lambda event: self.stop(root=test_or_root(event, False), |
Jon Salz | 6dc031d | 2013-06-19 13:06:23 +0800 | [diff] [blame] | 286 | fail=getattr(event, 'fail', False), |
| 287 | reason=getattr(event, 'reason', None)), |
Jon Salz | 36fbbb5 | 2012-07-05 13:45:06 +0800 | [diff] [blame] | 288 | Event.Type.SET_VISIBLE_TEST: |
| 289 | lambda event: self.set_visible_test( |
| 290 | self.test_list.lookup_path(event.path)), |
Jon Salz | 4712ac7 | 2013-02-07 17:12:05 +0800 | [diff] [blame] | 291 | Event.Type.CLEAR_STATE: |
| 292 | lambda event: self.clear_state(self.test_list.lookup_path(event.path)), |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 293 | } |
| 294 | |
| 295 | self.exceptions = [] |
| 296 | self.web_socket_manager = None |
| 297 | |
| 298 | def destroy(self): |
| 299 | if self.chrome: |
| 300 | self.chrome.kill() |
| 301 | self.chrome = None |
Jon Salz | c79a998 | 2012-08-30 04:42:01 +0800 | [diff] [blame] | 302 | if self.dummy_shopfloor: |
| 303 | self.dummy_shopfloor.kill() |
| 304 | self.dummy_shopfloor = None |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 305 | if self.ui_process: |
| 306 | utils.kill_process_tree(self.ui_process, 'ui') |
| 307 | self.ui_process = None |
| 308 | if self.web_socket_manager: |
| 309 | logging.info('Stopping web sockets') |
| 310 | self.web_socket_manager.close() |
| 311 | self.web_socket_manager = None |
| 312 | if self.state_server_thread: |
| 313 | logging.info('Stopping state server') |
| 314 | self.state_server.shutdown() |
| 315 | self.state_server_thread.join() |
| 316 | self.state_server.server_close() |
| 317 | self.state_server_thread = None |
| 318 | if self.state_instance: |
| 319 | self.state_instance.close() |
| 320 | if self.event_server_thread: |
| 321 | logging.info('Stopping event server') |
| 322 | self.event_server.shutdown() # pylint: disable=E1101 |
| 323 | self.event_server_thread.join() |
| 324 | self.event_server.server_close() |
| 325 | self.event_server_thread = None |
| 326 | if self.log_watcher: |
| 327 | if self.log_watcher.IsThreadStarted(): |
| 328 | self.log_watcher.StopWatchThread() |
| 329 | self.log_watcher = None |
Cheng-Yi Chiang | 344b10f | 2013-05-03 16:44:03 +0800 | [diff] [blame] | 330 | if self.system_log_manager: |
| 331 | if self.system_log_manager.IsThreadRunning(): |
| 332 | self.system_log_manager.StopSyncThread() |
| 333 | self.system_log_manager = None |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 334 | if self.prespawner: |
| 335 | logging.info('Stopping prespawner') |
| 336 | self.prespawner.stop() |
| 337 | self.prespawner = None |
| 338 | if self.event_client: |
| 339 | logging.info('Closing event client') |
| 340 | self.event_client.close() |
| 341 | self.event_client = None |
Jon Salz | ddf0d05 | 2013-06-18 12:52:44 +0800 | [diff] [blame] | 342 | if self.cpufreq_manager: |
| 343 | self.cpufreq_manager.Stop() |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 344 | if self.event_log: |
| 345 | self.event_log.Close() |
| 346 | self.event_log = None |
Dean Liao | 592e4d5 | 2013-01-10 20:06:39 +0800 | [diff] [blame] | 347 | if self.key_filter: |
| 348 | self.key_filter.Stop() |
Vic Yang | d8990da | 2013-06-27 16:57:43 +0800 | [diff] [blame^] | 349 | if self.cpu_usage_watcher: |
| 350 | self.cpu_usage_watcher.terminate() |
Dean Liao | 592e4d5 | 2013-01-10 20:06:39 +0800 | [diff] [blame] | 351 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 352 | self.check_exceptions() |
| 353 | logging.info('Done destroying Goofy') |
| 354 | |
| 355 | def start_state_server(self): |
Jon Salz | 2af235d | 2013-06-24 14:47:21 +0800 | [diff] [blame] | 356 | # Before starting state server, remount stateful partitions with |
| 357 | # no commit flag. The default commit time (commit=600) makes corruption |
| 358 | # too likely. |
| 359 | file_utils.ResetCommitTime() |
| 360 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 361 | self.state_instance, self.state_server = ( |
| 362 | state.create_server(bind_address='0.0.0.0')) |
Jon Salz | 16d1054 | 2012-07-23 12:18:45 +0800 | [diff] [blame] | 363 | self.goofy_rpc = GoofyRPC(self) |
| 364 | self.goofy_rpc.RegisterMethods(self.state_instance) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 365 | logging.info('Starting state server') |
| 366 | self.state_server_thread = threading.Thread( |
| 367 | target=self.state_server.serve_forever, |
| 368 | name='StateServer') |
| 369 | self.state_server_thread.start() |
| 370 | |
| 371 | def start_event_server(self): |
| 372 | self.event_server = EventServer() |
| 373 | logging.info('Starting factory event server') |
| 374 | self.event_server_thread = threading.Thread( |
| 375 | target=self.event_server.serve_forever, |
| 376 | name='EventServer') # pylint: disable=E1101 |
| 377 | self.event_server_thread.start() |
| 378 | |
| 379 | self.event_client = EventClient( |
| 380 | callback=self.handle_event, event_loop=self.run_queue) |
| 381 | |
| 382 | self.web_socket_manager = WebSocketManager(self.uuid) |
| 383 | self.state_server.add_handler("/event", |
| 384 | self.web_socket_manager.handle_web_socket) |
| 385 | |
| 386 | def start_ui(self): |
| 387 | ui_proc_args = [ |
| 388 | os.path.join(factory.FACTORY_PACKAGE_PATH, 'test', 'ui.py'), |
| 389 | self.options.test_list] |
| 390 | if self.options.verbose: |
| 391 | ui_proc_args.append('-v') |
| 392 | logging.info('Starting ui %s', ui_proc_args) |
Jon Salz | 78c3239 | 2012-07-25 14:18:29 +0800 | [diff] [blame] | 393 | self.ui_process = Spawn(ui_proc_args) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 394 | logging.info('Waiting for UI to come up...') |
| 395 | self.event_client.wait( |
| 396 | lambda event: event.type == Event.Type.UI_READY) |
| 397 | logging.info('UI has started') |
| 398 | |
| 399 | def set_visible_test(self, test): |
| 400 | if self.visible_test == test: |
| 401 | return |
Jon Salz | 2f2d42c | 2012-07-30 12:30:34 +0800 | [diff] [blame] | 402 | if test and not test.has_ui: |
| 403 | return |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 404 | |
| 405 | if test: |
| 406 | test.update_state(visible=True) |
| 407 | if self.visible_test: |
| 408 | self.visible_test.update_state(visible=False) |
| 409 | self.visible_test = test |
| 410 | |
Jon Salz | d4306c8 | 2012-11-30 15:16:36 +0800 | [diff] [blame] | 411 | def _log_startup_messages(self): |
| 412 | '''Logs the tail of var/log/messages and mosys and EC console logs.''' |
| 413 | # TODO(jsalz): This is mostly a copy-and-paste of code in init_states, |
| 414 | # for factory-3004.B only. Consolidate and merge back to ToT. |
| 415 | if utils.in_chroot(): |
| 416 | return |
| 417 | |
| 418 | try: |
| 419 | var_log_messages = ( |
| 420 | utils.var_log_messages_before_reboot()) |
| 421 | logging.info( |
| 422 | 'Tail of /var/log/messages before last reboot:\n' |
| 423 | '%s', ('\n'.join( |
| 424 | ' ' + x for x in var_log_messages))) |
| 425 | except: # pylint: disable=W0702 |
| 426 | logging.exception('Unable to grok /var/log/messages') |
| 427 | |
| 428 | try: |
| 429 | mosys_log = utils.Spawn( |
| 430 | ['mosys', 'eventlog', 'list'], |
| 431 | read_stdout=True, log_stderr_on_error=True).stdout_data |
| 432 | logging.info('System eventlog from mosys:\n%s\n', mosys_log) |
| 433 | except: # pylint: disable=W0702 |
| 434 | logging.exception('Unable to read mosys eventlog') |
| 435 | |
| 436 | try: |
Vic Yang | 8341dde | 2013-01-29 16:48:52 +0800 | [diff] [blame] | 437 | board = system.GetBoard() |
| 438 | ec_console_log = board.GetECConsoleLog() |
Jon Salz | d4306c8 | 2012-11-30 15:16:36 +0800 | [diff] [blame] | 439 | logging.info('EC console log after reboot:\n%s\n', ec_console_log) |
| 440 | except: # pylint: disable=W0702 |
| 441 | logging.exception('Error retrieving EC console log') |
| 442 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 443 | def handle_shutdown_complete(self, test, test_state): |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 444 | ''' |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 445 | Handles the case where a shutdown was detected during a shutdown step. |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 446 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 447 | @param test: The ShutdownStep. |
| 448 | @param test_state: The test state. |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 449 | ''' |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 450 | test_state = test.update_state(increment_shutdown_count=1) |
| 451 | logging.info('Detected shutdown (%d of %d)', |
| 452 | test_state.shutdown_count, test.iterations) |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 453 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 454 | def log_and_update_state(status, error_msg, **kw): |
| 455 | self.event_log.Log('rebooted', |
| 456 | status=status, error_msg=error_msg, **kw) |
Jon Salz | d4306c8 | 2012-11-30 15:16:36 +0800 | [diff] [blame] | 457 | logging.info('Rebooted: status=%s, %s', status, |
| 458 | (('error_msg=%s' % error_msg) if error_msg else None)) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 459 | test.update_state(status=status, error_msg=error_msg) |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 460 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 461 | if not self.last_shutdown_time: |
| 462 | log_and_update_state(status=TestState.FAILED, |
| 463 | error_msg='Unable to read shutdown_time') |
| 464 | return |
Jon Salz | 258a40c | 2012-04-19 12:34:01 +0800 | [diff] [blame] | 465 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 466 | now = time.time() |
| 467 | logging.info('%.03f s passed since reboot', |
| 468 | now - self.last_shutdown_time) |
Jon Salz | 258a40c | 2012-04-19 12:34:01 +0800 | [diff] [blame] | 469 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 470 | if self.last_shutdown_time > now: |
| 471 | test.update_state(status=TestState.FAILED, |
| 472 | error_msg='Time moved backward during reboot') |
| 473 | elif (isinstance(test, factory.RebootStep) and |
| 474 | self.test_list.options.max_reboot_time_secs and |
| 475 | (now - self.last_shutdown_time > |
| 476 | self.test_list.options.max_reboot_time_secs)): |
| 477 | # A reboot took too long; fail. (We don't check this for |
| 478 | # HaltSteps, because the machine could be halted for a |
| 479 | # very long time, and even unplugged with battery backup, |
| 480 | # thus hosing the clock.) |
| 481 | log_and_update_state( |
| 482 | status=TestState.FAILED, |
| 483 | error_msg=('More than %d s elapsed during reboot ' |
| 484 | '(%.03f s, from %s to %s)' % ( |
| 485 | self.test_list.options.max_reboot_time_secs, |
| 486 | now - self.last_shutdown_time, |
| 487 | utils.TimeString(self.last_shutdown_time), |
| 488 | utils.TimeString(now))), |
| 489 | duration=(now-self.last_shutdown_time)) |
Jon Salz | d4306c8 | 2012-11-30 15:16:36 +0800 | [diff] [blame] | 490 | self._log_startup_messages() |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 491 | elif test_state.shutdown_count == test.iterations: |
| 492 | # Good! |
| 493 | log_and_update_state(status=TestState.PASSED, |
| 494 | duration=(now - self.last_shutdown_time), |
| 495 | error_msg='') |
| 496 | elif test_state.shutdown_count > test.iterations: |
| 497 | # Shut down too many times |
| 498 | log_and_update_state(status=TestState.FAILED, |
| 499 | error_msg='Too many shutdowns') |
Jon Salz | d4306c8 | 2012-11-30 15:16:36 +0800 | [diff] [blame] | 500 | self._log_startup_messages() |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 501 | elif utils.are_shift_keys_depressed(): |
| 502 | logging.info('Shift keys are depressed; cancelling restarts') |
| 503 | # Abort shutdown |
| 504 | log_and_update_state( |
| 505 | status=TestState.FAILED, |
| 506 | error_msg='Shutdown aborted with double shift keys') |
Jon Salz | a6711d7 | 2012-07-18 14:33:03 +0800 | [diff] [blame] | 507 | self.cancel_pending_tests() |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 508 | else: |
| 509 | def handler(): |
| 510 | if self._prompt_cancel_shutdown( |
| 511 | test, test_state.shutdown_count + 1): |
Jon Salz | a6711d7 | 2012-07-18 14:33:03 +0800 | [diff] [blame] | 512 | factory.console.info('Shutdown aborted by operator') |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 513 | log_and_update_state( |
| 514 | status=TestState.FAILED, |
| 515 | error_msg='Shutdown aborted by operator') |
Jon Salz | a6711d7 | 2012-07-18 14:33:03 +0800 | [diff] [blame] | 516 | self.cancel_pending_tests() |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 517 | return |
Jon Salz | 0405ab5 | 2012-03-16 15:26:52 +0800 | [diff] [blame] | 518 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 519 | # Time to shutdown again |
| 520 | log_and_update_state( |
| 521 | status=TestState.ACTIVE, |
| 522 | error_msg='', |
| 523 | iteration=test_state.shutdown_count) |
Jon Salz | 73e0fd0 | 2012-04-04 11:46:38 +0800 | [diff] [blame] | 524 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 525 | self.event_log.Log('shutdown', operation='reboot') |
| 526 | self.state_instance.set_shared_data('shutdown_time', |
| 527 | time.time()) |
| 528 | self.env.shutdown('reboot') |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 529 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 530 | self.on_ui_startup.append(handler) |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 531 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 532 | def _prompt_cancel_shutdown(self, test, iteration): |
| 533 | if self.options.ui != 'chrome': |
| 534 | return False |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 535 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 536 | pending_shutdown_data = { |
| 537 | 'delay_secs': test.delay_secs, |
| 538 | 'time': time.time() + test.delay_secs, |
| 539 | 'operation': test.operation, |
| 540 | 'iteration': iteration, |
| 541 | 'iterations': test.iterations, |
| 542 | } |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 543 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 544 | # Create a new (threaded) event client since we |
| 545 | # don't want to use the event loop for this. |
| 546 | with EventClient() as event_client: |
| 547 | event_client.post_event(Event(Event.Type.PENDING_SHUTDOWN, |
| 548 | **pending_shutdown_data)) |
| 549 | aborted = event_client.wait( |
| 550 | lambda event: event.type == Event.Type.CANCEL_SHUTDOWN, |
| 551 | timeout=test.delay_secs) is not None |
| 552 | if aborted: |
| 553 | event_client.post_event(Event(Event.Type.PENDING_SHUTDOWN)) |
| 554 | return aborted |
Jon Salz | 258a40c | 2012-04-19 12:34:01 +0800 | [diff] [blame] | 555 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 556 | def init_states(self): |
| 557 | ''' |
| 558 | Initializes all states on startup. |
| 559 | ''' |
| 560 | for test in self.test_list.get_all_tests(): |
| 561 | # Make sure the state server knows about all the tests, |
| 562 | # defaulting to an untested state. |
| 563 | test.update_state(update_parent=False, visible=False) |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 564 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 565 | var_log_messages = None |
Vic Yang | a9c3221 | 2012-08-16 20:07:54 +0800 | [diff] [blame] | 566 | mosys_log = None |
Vic Yang | e4c275d | 2012-08-28 01:50:20 +0800 | [diff] [blame] | 567 | ec_console_log = None |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 568 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 569 | # Any 'active' tests should be marked as failed now. |
| 570 | for test in self.test_list.walk(): |
Jon Salz | a6711d7 | 2012-07-18 14:33:03 +0800 | [diff] [blame] | 571 | if not test.is_leaf(): |
| 572 | # Don't bother with parents; they will be updated when their |
| 573 | # children are updated. |
| 574 | continue |
| 575 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 576 | test_state = test.get_state() |
| 577 | if test_state.status != TestState.ACTIVE: |
| 578 | continue |
| 579 | if isinstance(test, factory.ShutdownStep): |
| 580 | # Shutdown while the test was active - that's good. |
| 581 | self.handle_shutdown_complete(test, test_state) |
| 582 | else: |
| 583 | # Unexpected shutdown. Grab /var/log/messages for context. |
| 584 | if var_log_messages is None: |
| 585 | try: |
| 586 | var_log_messages = ( |
| 587 | utils.var_log_messages_before_reboot()) |
| 588 | # Write it to the log, to make it easier to |
| 589 | # correlate with /var/log/messages. |
| 590 | logging.info( |
| 591 | 'Unexpected shutdown. ' |
| 592 | 'Tail of /var/log/messages before last reboot:\n' |
| 593 | '%s', ('\n'.join( |
| 594 | ' ' + x for x in var_log_messages))) |
| 595 | except: # pylint: disable=W0702 |
| 596 | logging.exception('Unable to grok /var/log/messages') |
| 597 | var_log_messages = [] |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 598 | |
Jon Salz | 008f4ea | 2012-08-28 05:39:45 +0800 | [diff] [blame] | 599 | if mosys_log is None and not utils.in_chroot(): |
| 600 | try: |
| 601 | mosys_log = utils.Spawn( |
| 602 | ['mosys', 'eventlog', 'list'], |
| 603 | read_stdout=True, log_stderr_on_error=True).stdout_data |
| 604 | # Write it to the log also. |
| 605 | logging.info('System eventlog from mosys:\n%s\n', mosys_log) |
| 606 | except: # pylint: disable=W0702 |
| 607 | logging.exception('Unable to read mosys eventlog') |
Vic Yang | a9c3221 | 2012-08-16 20:07:54 +0800 | [diff] [blame] | 608 | |
Vic Yang | e4c275d | 2012-08-28 01:50:20 +0800 | [diff] [blame] | 609 | if ec_console_log is None: |
| 610 | try: |
Vic Yang | 8341dde | 2013-01-29 16:48:52 +0800 | [diff] [blame] | 611 | board = system.GetBoard() |
| 612 | ec_console_log = board.GetECConsoleLog() |
Vic Yang | e4c275d | 2012-08-28 01:50:20 +0800 | [diff] [blame] | 613 | logging.info('EC console log after reboot:\n%s\n', ec_console_log) |
Jon Salz | fe1f665 | 2012-09-07 05:40:14 +0800 | [diff] [blame] | 614 | except: # pylint: disable=W0702 |
Vic Yang | e4c275d | 2012-08-28 01:50:20 +0800 | [diff] [blame] | 615 | logging.exception('Error retrieving EC console log') |
| 616 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 617 | error_msg = 'Unexpected shutdown while test was running' |
| 618 | self.event_log.Log('end_test', |
| 619 | path=test.path, |
| 620 | status=TestState.FAILED, |
| 621 | invocation=test.get_state().invocation, |
| 622 | error_msg=error_msg, |
Vic Yang | a9c3221 | 2012-08-16 20:07:54 +0800 | [diff] [blame] | 623 | var_log_messages='\n'.join(var_log_messages), |
| 624 | mosys_log=mosys_log) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 625 | test.update_state( |
| 626 | status=TestState.FAILED, |
| 627 | error_msg=error_msg) |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 628 | |
Jon Salz | 50efe94 | 2012-07-26 11:54:10 +0800 | [diff] [blame] | 629 | if not test.never_fails: |
| 630 | # For "never_fails" tests (such as "Start"), don't cancel |
| 631 | # pending tests, since reboot is expected. |
| 632 | factory.console.info('Unexpected shutdown while test %s ' |
| 633 | 'running; cancelling any pending tests', |
| 634 | test.path) |
| 635 | self.state_instance.set_shared_data('tests_after_shutdown', []) |
Jon Salz | 69806bb | 2012-07-20 18:05:02 +0800 | [diff] [blame] | 636 | |
Jon Salz | 008f4ea | 2012-08-28 05:39:45 +0800 | [diff] [blame] | 637 | self.update_skipped_tests() |
| 638 | |
| 639 | def update_skipped_tests(self): |
| 640 | ''' |
| 641 | Updates skipped states based on run_if. |
| 642 | ''' |
| 643 | for t in self.test_list.walk(): |
| 644 | if t.is_leaf() and t.run_if_table_name: |
| 645 | skip = False |
| 646 | try: |
| 647 | aux = shopfloor.get_selected_aux_data(t.run_if_table_name) |
| 648 | value = aux.get(t.run_if_col) |
| 649 | if value is not None: |
| 650 | skip = (not value) ^ t.run_if_not |
| 651 | except ValueError: |
| 652 | # Not available; assume it shouldn't be skipped |
| 653 | pass |
| 654 | |
| 655 | test_state = t.get_state() |
| 656 | if ((not skip) and |
| 657 | (test_state.status == TestState.PASSED) and |
| 658 | (test_state.error_msg == TestState.SKIPPED_MSG)): |
| 659 | # It was marked as skipped before, but now we need to run it. |
| 660 | # Mark as untested. |
| 661 | t.update_state(skip=skip, status=TestState.UNTESTED, error_msg='') |
| 662 | else: |
| 663 | t.update_state(skip=skip) |
| 664 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 665 | def show_next_active_test(self): |
| 666 | ''' |
| 667 | Rotates to the next visible active test. |
| 668 | ''' |
| 669 | self.reap_completed_tests() |
| 670 | active_tests = [ |
| 671 | t for t in self.test_list.walk() |
| 672 | if t.is_leaf() and t.get_state().status == TestState.ACTIVE] |
| 673 | if not active_tests: |
| 674 | return |
Jon Salz | 4f6c717 | 2012-06-11 20:45:36 +0800 | [diff] [blame] | 675 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 676 | try: |
| 677 | next_test = active_tests[ |
| 678 | (active_tests.index(self.visible_test) + 1) % len(active_tests)] |
| 679 | except ValueError: # visible_test not present in active_tests |
| 680 | next_test = active_tests[0] |
Jon Salz | 4f6c717 | 2012-06-11 20:45:36 +0800 | [diff] [blame] | 681 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 682 | self.set_visible_test(next_test) |
Jon Salz | 4f6c717 | 2012-06-11 20:45:36 +0800 | [diff] [blame] | 683 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 684 | def handle_event(self, event): |
| 685 | ''' |
| 686 | Handles an event from the event server. |
| 687 | ''' |
| 688 | handler = self.event_handlers.get(event.type) |
| 689 | if handler: |
| 690 | handler(event) |
| 691 | else: |
| 692 | # We don't register handlers for all event types - just ignore |
| 693 | # this event. |
| 694 | logging.debug('Unbound event type %s', event.type) |
Jon Salz | 4f6c717 | 2012-06-11 20:45:36 +0800 | [diff] [blame] | 695 | |
Vic Yang | aabf9fd | 2013-04-09 18:56:13 +0800 | [diff] [blame] | 696 | def check_critical_factory_note(self): |
| 697 | ''' |
| 698 | Returns True if the last factory note is critical. |
| 699 | ''' |
| 700 | notes = self.state_instance.get_shared_data('factory_note', True) |
| 701 | return notes and notes[-1]['level'] == 'CRITICAL' |
| 702 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 703 | def run_next_test(self): |
| 704 | ''' |
| 705 | Runs the next eligible test (or tests) in self.tests_to_run. |
| 706 | ''' |
| 707 | self.reap_completed_tests() |
Vic Yang | aabf9fd | 2013-04-09 18:56:13 +0800 | [diff] [blame] | 708 | if self.tests_to_run and self.check_critical_factory_note(): |
| 709 | self.tests_to_run.clear() |
| 710 | return |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 711 | while self.tests_to_run: |
| 712 | logging.debug('Tests to run: %s', |
| 713 | [x.path for x in self.tests_to_run]) |
Jon Salz | 94eb56f | 2012-06-12 18:01:12 +0800 | [diff] [blame] | 714 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 715 | test = self.tests_to_run[0] |
Jon Salz | 94eb56f | 2012-06-12 18:01:12 +0800 | [diff] [blame] | 716 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 717 | if test in self.invocations: |
| 718 | logging.info('Next test %s is already running', test.path) |
| 719 | self.tests_to_run.popleft() |
| 720 | return |
Jon Salz | 94eb56f | 2012-06-12 18:01:12 +0800 | [diff] [blame] | 721 | |
Jon Salz | a141292 | 2012-07-23 16:04:17 +0800 | [diff] [blame] | 722 | for requirement in test.require_run: |
| 723 | for i in requirement.test.walk(): |
| 724 | if i.get_state().status == TestState.ACTIVE: |
Jon Salz | 304a75d | 2012-07-06 11:14:15 +0800 | [diff] [blame] | 725 | logging.info('Waiting for active test %s to complete ' |
Jon Salz | a141292 | 2012-07-23 16:04:17 +0800 | [diff] [blame] | 726 | 'before running %s', i.path, test.path) |
Jon Salz | 304a75d | 2012-07-06 11:14:15 +0800 | [diff] [blame] | 727 | return |
| 728 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 729 | if self.invocations and not (test.backgroundable and all( |
| 730 | [x.backgroundable for x in self.invocations])): |
| 731 | logging.debug('Waiting for non-backgroundable tests to ' |
| 732 | 'complete before running %s', test.path) |
| 733 | return |
Jon Salz | 94eb56f | 2012-06-12 18:01:12 +0800 | [diff] [blame] | 734 | |
Jon Salz | 3e6f520 | 2012-10-15 15:08:29 +0800 | [diff] [blame] | 735 | if test.get_state().skip: |
| 736 | factory.console.info('Skipping test %s', test.path) |
| 737 | test.update_state(status=TestState.PASSED, |
| 738 | error_msg=TestState.SKIPPED_MSG) |
| 739 | self.tests_to_run.popleft() |
| 740 | continue |
| 741 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 742 | self.tests_to_run.popleft() |
Jon Salz | 94eb56f | 2012-06-12 18:01:12 +0800 | [diff] [blame] | 743 | |
Jon Salz | 304a75d | 2012-07-06 11:14:15 +0800 | [diff] [blame] | 744 | untested = set() |
Jon Salz | a141292 | 2012-07-23 16:04:17 +0800 | [diff] [blame] | 745 | for requirement in test.require_run: |
| 746 | for i in requirement.test.walk(): |
| 747 | if i == test: |
Jon Salz | 304a75d | 2012-07-06 11:14:15 +0800 | [diff] [blame] | 748 | # We've hit this test itself; stop checking |
| 749 | break |
Jon Salz | a141292 | 2012-07-23 16:04:17 +0800 | [diff] [blame] | 750 | if ((i.get_state().status == TestState.UNTESTED) or |
| 751 | (requirement.passed and i.get_state().status != |
| 752 | TestState.PASSED)): |
Jon Salz | 304a75d | 2012-07-06 11:14:15 +0800 | [diff] [blame] | 753 | # Found an untested test; move on to the next |
| 754 | # element in require_run. |
Jon Salz | a141292 | 2012-07-23 16:04:17 +0800 | [diff] [blame] | 755 | untested.add(i) |
Jon Salz | 304a75d | 2012-07-06 11:14:15 +0800 | [diff] [blame] | 756 | break |
| 757 | |
| 758 | if untested: |
| 759 | untested_paths = ', '.join(sorted([x.path for x in untested])) |
| 760 | if self.state_instance.get_shared_data('engineering_mode', |
| 761 | optional=True): |
| 762 | # In engineering mode, we'll let it go. |
| 763 | factory.console.warn('In engineering mode; running ' |
| 764 | '%s even though required tests ' |
| 765 | '[%s] have not completed', |
| 766 | test.path, untested_paths) |
| 767 | else: |
| 768 | # Not in engineering mode; mark it failed. |
| 769 | error_msg = ('Required tests [%s] have not been run yet' |
| 770 | % untested_paths) |
| 771 | factory.console.error('Not running %s: %s', |
| 772 | test.path, error_msg) |
| 773 | test.update_state(status=TestState.FAILED, |
| 774 | error_msg=error_msg) |
| 775 | continue |
| 776 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 777 | if isinstance(test, factory.ShutdownStep): |
| 778 | if os.path.exists(NO_REBOOT_FILE): |
| 779 | test.update_state( |
| 780 | status=TestState.FAILED, increment_count=1, |
| 781 | error_msg=('Skipped shutdown since %s is present' % |
Jon Salz | 304a75d | 2012-07-06 11:14:15 +0800 | [diff] [blame] | 782 | NO_REBOOT_FILE)) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 783 | continue |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 784 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 785 | test.update_state(status=TestState.ACTIVE, increment_count=1, |
| 786 | error_msg='', shutdown_count=0) |
| 787 | if self._prompt_cancel_shutdown(test, 1): |
| 788 | self.event_log.Log('reboot_cancelled') |
| 789 | test.update_state( |
| 790 | status=TestState.FAILED, increment_count=1, |
| 791 | error_msg='Shutdown aborted by operator', |
| 792 | shutdown_count=0) |
chungyi | afe8f77 | 2012-08-15 19:36:29 +0800 | [diff] [blame] | 793 | continue |
Jon Salz | 2f757d4 | 2012-06-27 17:06:42 +0800 | [diff] [blame] | 794 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 795 | # Save pending test list in the state server |
Jon Salz | dbf398f | 2012-06-14 17:30:01 +0800 | [diff] [blame] | 796 | self.state_instance.set_shared_data( |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 797 | 'tests_after_shutdown', |
| 798 | [t.path for t in self.tests_to_run]) |
| 799 | # Save shutdown time |
| 800 | self.state_instance.set_shared_data('shutdown_time', |
| 801 | time.time()) |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 802 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 803 | with self.env.lock: |
| 804 | self.event_log.Log('shutdown', operation=test.operation) |
| 805 | shutdown_result = self.env.shutdown(test.operation) |
| 806 | if shutdown_result: |
| 807 | # That's all, folks! |
| 808 | self.run_queue.put(None) |
| 809 | return |
| 810 | else: |
| 811 | # Just pass (e.g., in the chroot). |
| 812 | test.update_state(status=TestState.PASSED) |
| 813 | self.state_instance.set_shared_data( |
| 814 | 'tests_after_shutdown', None) |
| 815 | # Send event with no fields to indicate that there is no |
| 816 | # longer a pending shutdown. |
| 817 | self.event_client.post_event(Event( |
| 818 | Event.Type.PENDING_SHUTDOWN)) |
| 819 | continue |
Jon Salz | 258a40c | 2012-04-19 12:34:01 +0800 | [diff] [blame] | 820 | |
Cheng-Yi Chiang | ce05c00 | 2013-04-04 02:13:17 +0800 | [diff] [blame] | 821 | self._run_test(test, test.iterations, test.retries) |
Jon Salz | 1acc874 | 2012-07-17 17:45:55 +0800 | [diff] [blame] | 822 | |
Cheng-Yi Chiang | ce05c00 | 2013-04-04 02:13:17 +0800 | [diff] [blame] | 823 | def _run_test(self, test, iterations_left=None, retries_left=None): |
Jon Salz | 1acc874 | 2012-07-17 17:45:55 +0800 | [diff] [blame] | 824 | invoc = TestInvocation(self, test, on_completion=self.run_next_test) |
| 825 | new_state = test.update_state( |
| 826 | status=TestState.ACTIVE, increment_count=1, error_msg='', |
Jon Salz | bd42ce1 | 2012-09-18 08:03:59 +0800 | [diff] [blame] | 827 | invocation=invoc.uuid, iterations_left=iterations_left, |
Cheng-Yi Chiang | ce05c00 | 2013-04-04 02:13:17 +0800 | [diff] [blame] | 828 | retries_left=retries_left, |
Jon Salz | bd42ce1 | 2012-09-18 08:03:59 +0800 | [diff] [blame] | 829 | visible=(self.visible_test == test)) |
Jon Salz | 1acc874 | 2012-07-17 17:45:55 +0800 | [diff] [blame] | 830 | invoc.count = new_state.count |
| 831 | |
| 832 | self.invocations[test] = invoc |
| 833 | if self.visible_test is None and test.has_ui: |
| 834 | self.set_visible_test(test) |
Vic Yang | 311ddb8 | 2012-09-26 12:08:28 +0800 | [diff] [blame] | 835 | self.check_exclusive() |
Jon Salz | 1acc874 | 2012-07-17 17:45:55 +0800 | [diff] [blame] | 836 | invoc.start() |
Jon Salz | 5f2a067 | 2012-05-22 17:14:06 +0800 | [diff] [blame] | 837 | |
Vic Yang | 311ddb8 | 2012-09-26 12:08:28 +0800 | [diff] [blame] | 838 | def check_exclusive(self): |
Jon Salz | ce6a7f8 | 2013-06-10 18:22:54 +0800 | [diff] [blame] | 839 | # alias since this is really long |
| 840 | EXCL_OPT = factory.FactoryTest.EXCLUSIVE_OPTIONS |
| 841 | |
Vic Yang | 311ddb8 | 2012-09-26 12:08:28 +0800 | [diff] [blame] | 842 | current_exclusive_items = set([ |
Jon Salz | ce6a7f8 | 2013-06-10 18:22:54 +0800 | [diff] [blame] | 843 | item for item in EXCL_OPT |
Vic Yang | 311ddb8 | 2012-09-26 12:08:28 +0800 | [diff] [blame] | 844 | if any([test.is_exclusive(item) for test in self.invocations])]) |
| 845 | |
| 846 | new_exclusive_items = current_exclusive_items - self.exclusive_items |
Jon Salz | ce6a7f8 | 2013-06-10 18:22:54 +0800 | [diff] [blame] | 847 | if EXCL_OPT.NETWORKING in new_exclusive_items: |
Vic Yang | 311ddb8 | 2012-09-26 12:08:28 +0800 | [diff] [blame] | 848 | logging.info('Disabling network') |
| 849 | self.connection_manager.DisableNetworking() |
Jon Salz | ce6a7f8 | 2013-06-10 18:22:54 +0800 | [diff] [blame] | 850 | if EXCL_OPT.CHARGER in new_exclusive_items: |
Vic Yang | 311ddb8 | 2012-09-26 12:08:28 +0800 | [diff] [blame] | 851 | logging.info('Stop controlling charger') |
| 852 | |
| 853 | new_non_exclusive_items = self.exclusive_items - current_exclusive_items |
Jon Salz | ce6a7f8 | 2013-06-10 18:22:54 +0800 | [diff] [blame] | 854 | if EXCL_OPT.NETWORKING in new_non_exclusive_items: |
Vic Yang | 311ddb8 | 2012-09-26 12:08:28 +0800 | [diff] [blame] | 855 | logging.info('Re-enabling network') |
| 856 | self.connection_manager.EnableNetworking() |
Jon Salz | ce6a7f8 | 2013-06-10 18:22:54 +0800 | [diff] [blame] | 857 | if EXCL_OPT.CHARGER in new_non_exclusive_items: |
Vic Yang | 311ddb8 | 2012-09-26 12:08:28 +0800 | [diff] [blame] | 858 | logging.info('Start controlling charger') |
| 859 | |
Jon Salz | ce6a7f8 | 2013-06-10 18:22:54 +0800 | [diff] [blame] | 860 | if self.cpufreq_manager: |
| 861 | enabled = EXCL_OPT.CPUFREQ not in current_exclusive_items |
| 862 | try: |
| 863 | self.cpufreq_manager.SetEnabled(enabled) |
| 864 | except: # pylint: disable=W0702 |
| 865 | logging.exception('Unable to %s cpufreq services', |
| 866 | 'enable' if enabled else 'disable') |
| 867 | |
Vic Yang | 311ddb8 | 2012-09-26 12:08:28 +0800 | [diff] [blame] | 868 | # Only adjust charge state if not excluded |
Jon Salz | ce6a7f8 | 2013-06-10 18:22:54 +0800 | [diff] [blame] | 869 | if (EXCL_OPT.CHARGER not in current_exclusive_items and |
| 870 | not utils.in_chroot()): |
Cheng-Yi Chiang | d818695 | 2013-04-04 23:41:14 +0800 | [diff] [blame] | 871 | if self.charge_manager: |
| 872 | self.charge_manager.AdjustChargeState() |
| 873 | else: |
| 874 | try: |
| 875 | system.GetBoard().SetChargeState(Board.ChargeState.CHARGE) |
| 876 | except BoardException: |
| 877 | logging.exception('Unable to set charge state on this board') |
Vic Yang | 311ddb8 | 2012-09-26 12:08:28 +0800 | [diff] [blame] | 878 | |
| 879 | self.exclusive_items = current_exclusive_items |
Jon Salz | 5da61e6 | 2012-05-31 13:06:22 +0800 | [diff] [blame] | 880 | |
cychiang | 2188674 | 2012-07-05 15:16:32 +0800 | [diff] [blame] | 881 | def check_for_updates(self): |
| 882 | ''' |
| 883 | Schedules an asynchronous check for updates if necessary. |
| 884 | ''' |
| 885 | if not self.test_list.options.update_period_secs: |
| 886 | # Not enabled. |
| 887 | return |
| 888 | |
| 889 | now = time.time() |
| 890 | if self.last_update_check and ( |
| 891 | now - self.last_update_check < |
| 892 | self.test_list.options.update_period_secs): |
| 893 | # Not yet time for another check. |
| 894 | return |
| 895 | |
| 896 | self.last_update_check = now |
| 897 | |
| 898 | def handle_check_for_update(reached_shopfloor, md5sum, needs_update): |
| 899 | if reached_shopfloor: |
| 900 | new_update_md5sum = md5sum if needs_update else None |
| 901 | if system.SystemInfo.update_md5sum != new_update_md5sum: |
| 902 | logging.info('Received new update MD5SUM: %s', new_update_md5sum) |
| 903 | system.SystemInfo.update_md5sum = new_update_md5sum |
| 904 | self.run_queue.put(self.update_system_info) |
| 905 | |
| 906 | updater.CheckForUpdateAsync( |
| 907 | handle_check_for_update, |
| 908 | self.test_list.options.shopfloor_timeout_secs) |
| 909 | |
Jon Salz | a6711d7 | 2012-07-18 14:33:03 +0800 | [diff] [blame] | 910 | def cancel_pending_tests(self): |
| 911 | '''Cancels any tests in the run queue.''' |
| 912 | self.run_tests([]) |
| 913 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 914 | def run_tests(self, subtrees, untested_only=False): |
| 915 | ''' |
| 916 | Runs tests under subtree. |
Jon Salz | 258a40c | 2012-04-19 12:34:01 +0800 | [diff] [blame] | 917 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 918 | The tests are run in order unless one fails (then stops). |
| 919 | Backgroundable tests are run simultaneously; when a foreground test is |
| 920 | encountered, we wait for all active tests to finish before continuing. |
Jon Salz | b1b3909 | 2012-05-03 02:05:09 +0800 | [diff] [blame] | 921 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 922 | @param subtrees: Node or nodes containing tests to run (may either be |
| 923 | a single test or a list). Duplicates will be ignored. |
| 924 | ''' |
| 925 | if type(subtrees) != list: |
| 926 | subtrees = [subtrees] |
Jon Salz | 258a40c | 2012-04-19 12:34:01 +0800 | [diff] [blame] | 927 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 928 | # Nodes we've seen so far, to avoid duplicates. |
| 929 | seen = set() |
Jon Salz | 94eb56f | 2012-06-12 18:01:12 +0800 | [diff] [blame] | 930 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 931 | self.tests_to_run = deque() |
| 932 | for subtree in subtrees: |
| 933 | for test in subtree.walk(): |
| 934 | if test in seen: |
| 935 | continue |
| 936 | seen.add(test) |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 937 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 938 | if not test.is_leaf(): |
| 939 | continue |
| 940 | if (untested_only and |
| 941 | test.get_state().status != TestState.UNTESTED): |
| 942 | continue |
| 943 | self.tests_to_run.append(test) |
| 944 | self.run_next_test() |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 945 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 946 | def reap_completed_tests(self): |
| 947 | ''' |
| 948 | Removes completed tests from the set of active tests. |
| 949 | |
| 950 | Also updates the visible test if it was reaped. |
| 951 | ''' |
Cheng-Yi Chiang | 5ac22ca | 2013-04-12 17:45:26 +0800 | [diff] [blame] | 952 | test_completed = False |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 953 | for t, v in dict(self.invocations).iteritems(): |
| 954 | if v.is_completed(): |
Cheng-Yi Chiang | 5ac22ca | 2013-04-12 17:45:26 +0800 | [diff] [blame] | 955 | test_completed = True |
Jon Salz | 1acc874 | 2012-07-17 17:45:55 +0800 | [diff] [blame] | 956 | new_state = t.update_state(**v.update_state_on_completion) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 957 | del self.invocations[t] |
| 958 | |
Chun-Ta Lin | 54e17e4 | 2012-09-06 22:05:13 +0800 | [diff] [blame] | 959 | # Stop on failure if flag is true. |
| 960 | if (self.test_list.options.stop_on_failure and |
| 961 | new_state.status == TestState.FAILED): |
| 962 | # Clean all the tests to cause goofy to stop. |
| 963 | self.tests_to_run = [] |
| 964 | factory.console.info("Stop on failure triggered. Empty the queue.") |
| 965 | |
Jon Salz | 1acc874 | 2012-07-17 17:45:55 +0800 | [diff] [blame] | 966 | if new_state.iterations_left and new_state.status == TestState.PASSED: |
| 967 | # Play it again, Sam! |
| 968 | self._run_test(t) |
Cheng-Yi Chiang | ce05c00 | 2013-04-04 02:13:17 +0800 | [diff] [blame] | 969 | # new_state.retries_left is obtained after update. |
| 970 | # For retries_left == 0, test can still be run for the last time. |
| 971 | elif (new_state.retries_left >= 0 and |
| 972 | new_state.status == TestState.FAILED): |
| 973 | # Still have to retry, Sam! |
| 974 | self._run_test(t) |
Jon Salz | 1acc874 | 2012-07-17 17:45:55 +0800 | [diff] [blame] | 975 | |
Cheng-Yi Chiang | 5ac22ca | 2013-04-12 17:45:26 +0800 | [diff] [blame] | 976 | if test_completed: |
Vic Yang | f01c59f | 2013-04-19 17:37:56 +0800 | [diff] [blame] | 977 | self.log_watcher.KickWatchThread() |
Cheng-Yi Chiang | 5ac22ca | 2013-04-12 17:45:26 +0800 | [diff] [blame] | 978 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 979 | if (self.visible_test is None or |
Jon Salz | 85a3988 | 2012-07-05 16:45:04 +0800 | [diff] [blame] | 980 | self.visible_test not in self.invocations): |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 981 | self.set_visible_test(None) |
| 982 | # Make the first running test, if any, the visible test |
| 983 | for t in self.test_list.walk(): |
| 984 | if t in self.invocations: |
| 985 | self.set_visible_test(t) |
| 986 | break |
| 987 | |
Jon Salz | 6dc031d | 2013-06-19 13:06:23 +0800 | [diff] [blame] | 988 | def kill_active_tests(self, abort, root=None, reason=None): |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 989 | ''' |
| 990 | Kills and waits for all active tests. |
| 991 | |
Jon Salz | 85a3988 | 2012-07-05 16:45:04 +0800 | [diff] [blame] | 992 | Args: |
| 993 | abort: True to change state of killed tests to FAILED, False for |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 994 | UNTESTED. |
Jon Salz | 85a3988 | 2012-07-05 16:45:04 +0800 | [diff] [blame] | 995 | root: If set, only kills tests with root as an ancestor. |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 996 | ''' |
| 997 | self.reap_completed_tests() |
| 998 | for test, invoc in self.invocations.items(): |
Jon Salz | 85a3988 | 2012-07-05 16:45:04 +0800 | [diff] [blame] | 999 | if root and not test.has_ancestor(root): |
| 1000 | continue |
| 1001 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1002 | factory.console.info('Killing active test %s...' % test.path) |
Jon Salz | 6dc031d | 2013-06-19 13:06:23 +0800 | [diff] [blame] | 1003 | invoc.abort_and_join(reason) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1004 | factory.console.info('Killed %s' % test.path) |
Jon Salz | 1acc874 | 2012-07-17 17:45:55 +0800 | [diff] [blame] | 1005 | test.update_state(**invoc.update_state_on_completion) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1006 | del self.invocations[test] |
Jon Salz | 1acc874 | 2012-07-17 17:45:55 +0800 | [diff] [blame] | 1007 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1008 | if not abort: |
| 1009 | test.update_state(status=TestState.UNTESTED) |
| 1010 | self.reap_completed_tests() |
| 1011 | |
Jon Salz | 6dc031d | 2013-06-19 13:06:23 +0800 | [diff] [blame] | 1012 | def stop(self, root=None, fail=False, reason=None): |
| 1013 | self.kill_active_tests(fail, root, reason) |
Jon Salz | 85a3988 | 2012-07-05 16:45:04 +0800 | [diff] [blame] | 1014 | # Remove any tests in the run queue under the root. |
| 1015 | self.tests_to_run = deque([x for x in self.tests_to_run |
| 1016 | if root and not x.has_ancestor(root)]) |
| 1017 | self.run_next_test() |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1018 | |
Jon Salz | 4712ac7 | 2013-02-07 17:12:05 +0800 | [diff] [blame] | 1019 | def clear_state(self, root=None): |
Jon Salz | 6dc031d | 2013-06-19 13:06:23 +0800 | [diff] [blame] | 1020 | self.stop(root, reason='Clearing test state') |
Jon Salz | 4712ac7 | 2013-02-07 17:12:05 +0800 | [diff] [blame] | 1021 | for f in root.walk(): |
| 1022 | if f.is_leaf(): |
| 1023 | f.update_state(status=TestState.UNTESTED) |
| 1024 | |
Jon Salz | 6dc031d | 2013-06-19 13:06:23 +0800 | [diff] [blame] | 1025 | def abort_active_tests(self, reason=None): |
| 1026 | self.kill_active_tests(True, reason=reason) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1027 | |
| 1028 | def main(self): |
Jon Salz | eff9418 | 2013-06-19 15:06:28 +0800 | [diff] [blame] | 1029 | syslog.openlog('goofy') |
| 1030 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1031 | try: |
| 1032 | self.init() |
| 1033 | self.event_log.Log('goofy_init', |
| 1034 | success=True) |
| 1035 | except: |
| 1036 | if self.event_log: |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 1037 | try: |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1038 | self.event_log.Log('goofy_init', |
| 1039 | success=False, |
| 1040 | trace=traceback.format_exc()) |
| 1041 | except: # pylint: disable=W0702 |
| 1042 | pass |
| 1043 | raise |
| 1044 | |
Jon Salz | eff9418 | 2013-06-19 15:06:28 +0800 | [diff] [blame] | 1045 | syslog.syslog('Goofy (factory test harness) starting') |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1046 | self.run() |
| 1047 | |
| 1048 | def update_system_info(self): |
| 1049 | '''Updates system info.''' |
| 1050 | system_info = system.SystemInfo() |
| 1051 | self.state_instance.set_shared_data('system_info', system_info.__dict__) |
| 1052 | self.event_client.post_event(Event(Event.Type.SYSTEM_INFO, |
| 1053 | system_info=system_info.__dict__)) |
| 1054 | logging.info('System info: %r', system_info.__dict__) |
| 1055 | |
Jon Salz | eb42f0d | 2012-07-27 19:14:04 +0800 | [diff] [blame] | 1056 | def update_factory(self, auto_run_on_restart=False, post_update_hook=None): |
| 1057 | '''Commences updating factory software. |
| 1058 | |
| 1059 | Args: |
| 1060 | auto_run_on_restart: Auto-run when the machine comes back up. |
| 1061 | post_update_hook: Code to call after update but immediately before |
| 1062 | restart. |
| 1063 | |
| 1064 | Returns: |
| 1065 | Never if the update was successful (we just reboot). |
| 1066 | False if the update was unnecessary (no update available). |
| 1067 | ''' |
Jon Salz | 6dc031d | 2013-06-19 13:06:23 +0800 | [diff] [blame] | 1068 | self.kill_active_tests(False, reason='Factory software update') |
Jon Salz | a6711d7 | 2012-07-18 14:33:03 +0800 | [diff] [blame] | 1069 | self.cancel_pending_tests() |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1070 | |
Jon Salz | 5c344f6 | 2012-07-13 14:31:16 +0800 | [diff] [blame] | 1071 | def pre_update_hook(): |
| 1072 | if auto_run_on_restart: |
| 1073 | self.state_instance.set_shared_data('tests_after_shutdown', |
| 1074 | FORCE_AUTO_RUN) |
| 1075 | self.state_instance.close() |
| 1076 | |
Jon Salz | eb42f0d | 2012-07-27 19:14:04 +0800 | [diff] [blame] | 1077 | if updater.TryUpdate(pre_update_hook=pre_update_hook): |
| 1078 | if post_update_hook: |
| 1079 | post_update_hook() |
| 1080 | self.env.shutdown('reboot') |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1081 | |
Jon Salz | cef132a | 2012-08-30 04:58:08 +0800 | [diff] [blame] | 1082 | def handle_sigint(self, dummy_signum, dummy_frame): |
Jon Salz | 77c151e | 2012-08-28 07:20:37 +0800 | [diff] [blame] | 1083 | logging.error('Received SIGINT') |
| 1084 | self.run_queue.put(None) |
| 1085 | raise KeyboardInterrupt() |
| 1086 | |
Jon Salz | e12c2b3 | 2013-06-25 16:24:34 +0800 | [diff] [blame] | 1087 | def find_kcrashes(self): |
| 1088 | """Finds kcrash files, logs them, and marks them as seen.""" |
| 1089 | seen_crashes = set( |
| 1090 | self.state_instance.get_shared_data('seen_crashes', optional=True) |
| 1091 | or []) |
| 1092 | |
| 1093 | for path in glob.glob('/var/spool/crash/*'): |
| 1094 | if not os.path.isfile(path): |
| 1095 | continue |
| 1096 | if path in seen_crashes: |
| 1097 | continue |
| 1098 | try: |
| 1099 | stat = os.stat(path) |
| 1100 | mtime = utils.TimeString(stat.st_mtime) |
| 1101 | logging.info( |
| 1102 | 'Found new crash file %s (%d bytes at %s)', |
| 1103 | path, stat.st_size, mtime) |
| 1104 | extra_log_args = {} |
| 1105 | |
| 1106 | try: |
| 1107 | _, ext = os.path.splitext(path) |
| 1108 | if ext in ['.kcrash', '.meta']: |
| 1109 | ext = ext.replace('.', '') |
| 1110 | with open(path) as f: |
| 1111 | data = f.read(MAX_CRASH_FILE_SIZE) |
| 1112 | tell = f.tell() |
| 1113 | logging.info( |
| 1114 | 'Contents of %s%s:%s', |
| 1115 | path, |
| 1116 | ('' if tell == stat.st_size |
| 1117 | else '(truncated to %d bytes)' % MAX_CRASH_FILE_SIZE), |
| 1118 | ('\n' + data).replace('\n', '\n ' + ext + '> ')) |
| 1119 | extra_log_args['data'] = data |
| 1120 | |
| 1121 | # Copy to /var/factory/kcrash for posterity |
| 1122 | kcrash_dir = factory.get_factory_root('kcrash') |
| 1123 | utils.TryMakeDirs(kcrash_dir) |
| 1124 | shutil.copy(path, kcrash_dir) |
| 1125 | logging.info('Copied to %s', |
| 1126 | os.path.join(kcrash_dir, os.path.basename(path))) |
| 1127 | finally: |
| 1128 | # Even if something goes wrong with the above, still try to |
| 1129 | # log to event log |
| 1130 | self.event_log.Log('crash_file', |
| 1131 | path=path, size=stat.st_size, mtime=mtime, |
| 1132 | **extra_log_args) |
| 1133 | except: # pylint: disable=W0702 |
| 1134 | logging.exception('Unable to handle crash files %s', path) |
| 1135 | seen_crashes.add(path) |
| 1136 | |
| 1137 | self.state_instance.set_shared_data('seen_crashes', list(seen_crashes)) |
| 1138 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1139 | def init(self, args=None, env=None): |
| 1140 | '''Initializes Goofy. |
| 1141 | |
| 1142 | Args: |
| 1143 | args: A list of command-line arguments. Uses sys.argv if |
| 1144 | args is None. |
| 1145 | env: An Environment instance to use (or None to choose |
| 1146 | FakeChrootEnvironment or DUTEnvironment as appropriate). |
| 1147 | ''' |
Jon Salz | 77c151e | 2012-08-28 07:20:37 +0800 | [diff] [blame] | 1148 | signal.signal(signal.SIGINT, self.handle_sigint) |
| 1149 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1150 | parser = OptionParser() |
| 1151 | parser.add_option('-v', '--verbose', dest='verbose', |
Jon Salz | 8fa8e83 | 2012-07-13 19:04:09 +0800 | [diff] [blame] | 1152 | action='store_true', |
| 1153 | help='Enable debug logging') |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1154 | parser.add_option('--print_test_list', dest='print_test_list', |
Jon Salz | 8fa8e83 | 2012-07-13 19:04:09 +0800 | [diff] [blame] | 1155 | metavar='FILE', |
| 1156 | help='Read and print test list FILE, and exit') |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1157 | parser.add_option('--restart', dest='restart', |
Jon Salz | 8fa8e83 | 2012-07-13 19:04:09 +0800 | [diff] [blame] | 1158 | action='store_true', |
| 1159 | help='Clear all test state') |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1160 | parser.add_option('--ui', dest='ui', type='choice', |
Jon Salz | 8fa8e83 | 2012-07-13 19:04:09 +0800 | [diff] [blame] | 1161 | choices=['none', 'gtk', 'chrome'], |
Jon Salz | 2f881df | 2013-02-01 17:00:35 +0800 | [diff] [blame] | 1162 | default='chrome', |
Jon Salz | 8fa8e83 | 2012-07-13 19:04:09 +0800 | [diff] [blame] | 1163 | help='UI to use') |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1164 | parser.add_option('--ui_scale_factor', dest='ui_scale_factor', |
Jon Salz | 8fa8e83 | 2012-07-13 19:04:09 +0800 | [diff] [blame] | 1165 | type='int', default=1, |
| 1166 | help=('Factor by which to scale UI ' |
| 1167 | '(Chrome UI only)')) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1168 | parser.add_option('--test_list', dest='test_list', |
Jon Salz | 8fa8e83 | 2012-07-13 19:04:09 +0800 | [diff] [blame] | 1169 | metavar='FILE', |
| 1170 | help='Use FILE as test list') |
Jon Salz | c79a998 | 2012-08-30 04:42:01 +0800 | [diff] [blame] | 1171 | parser.add_option('--dummy_shopfloor', action='store_true', |
| 1172 | help='Use a dummy shopfloor server') |
chungyi | afe8f77 | 2012-08-15 19:36:29 +0800 | [diff] [blame] | 1173 | parser.add_option('--automation', dest='automation', |
| 1174 | action='store_true', |
| 1175 | help='Enable automation on running factory test') |
Ricky Liang | 09216dc | 2013-02-22 17:26:45 +0800 | [diff] [blame] | 1176 | parser.add_option('--one_pixel_less', dest='one_pixel_less', |
| 1177 | action='store_true', |
| 1178 | help=('Start Chrome one pixel less than the full screen.' |
| 1179 | 'Needed by Exynos platform to run GTK.')) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1180 | (self.options, self.args) = parser.parse_args(args) |
| 1181 | |
Jon Salz | 46b8956 | 2012-07-05 11:49:22 +0800 | [diff] [blame] | 1182 | # Make sure factory directories exist. |
| 1183 | factory.get_log_root() |
| 1184 | factory.get_state_root() |
| 1185 | factory.get_test_data_root() |
| 1186 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1187 | global _inited_logging # pylint: disable=W0603 |
| 1188 | if not _inited_logging: |
| 1189 | factory.init_logging('goofy', verbose=self.options.verbose) |
| 1190 | _inited_logging = True |
Jon Salz | 8fa8e83 | 2012-07-13 19:04:09 +0800 | [diff] [blame] | 1191 | |
Jon Salz | 0f99660 | 2012-10-03 15:26:48 +0800 | [diff] [blame] | 1192 | if self.options.print_test_list: |
| 1193 | print factory.read_test_list( |
| 1194 | self.options.print_test_list).__repr__(recursive=True) |
| 1195 | sys.exit(0) |
| 1196 | |
Jon Salz | ee85d52 | 2012-07-17 14:34:46 +0800 | [diff] [blame] | 1197 | event_log.IncrementBootSequence() |
Jon Salz | d15bbcf | 2013-05-21 17:33:57 +0800 | [diff] [blame] | 1198 | # Don't defer logging the initial event, so we can make sure |
| 1199 | # that device_id, reimage_id, etc. are all set up. |
| 1200 | self.event_log = EventLog('goofy', defer=False) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1201 | |
| 1202 | if (not suppress_chroot_warning and |
| 1203 | factory.in_chroot() and |
| 1204 | self.options.ui == 'gtk' and |
| 1205 | os.environ.get('DISPLAY') in [None, '', ':0', ':0.0']): |
| 1206 | # That's not going to work! Tell the user how to run |
| 1207 | # this way. |
| 1208 | logging.warn(GOOFY_IN_CHROOT_WARNING) |
| 1209 | time.sleep(1) |
| 1210 | |
| 1211 | if env: |
| 1212 | self.env = env |
| 1213 | elif factory.in_chroot(): |
| 1214 | self.env = test_environment.FakeChrootEnvironment() |
| 1215 | logging.warn( |
| 1216 | 'Using chroot environment: will not actually run autotests') |
| 1217 | else: |
| 1218 | self.env = test_environment.DUTEnvironment() |
| 1219 | self.env.goofy = self |
| 1220 | |
| 1221 | if self.options.restart: |
| 1222 | state.clear_state() |
| 1223 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1224 | if self.options.ui_scale_factor != 1 and utils.in_qemu(): |
| 1225 | logging.warn( |
| 1226 | 'In QEMU; ignoring ui_scale_factor argument') |
| 1227 | self.options.ui_scale_factor = 1 |
| 1228 | |
| 1229 | logging.info('Started') |
| 1230 | |
| 1231 | self.start_state_server() |
| 1232 | self.state_instance.set_shared_data('hwid_cfg', get_hwid_cfg()) |
| 1233 | self.state_instance.set_shared_data('ui_scale_factor', |
Ricky Liang | 09216dc | 2013-02-22 17:26:45 +0800 | [diff] [blame] | 1234 | self.options.ui_scale_factor) |
| 1235 | self.state_instance.set_shared_data('one_pixel_less', |
| 1236 | self.options.one_pixel_less) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1237 | self.last_shutdown_time = ( |
| 1238 | self.state_instance.get_shared_data('shutdown_time', optional=True)) |
| 1239 | self.state_instance.del_shared_data('shutdown_time', optional=True) |
| 1240 | |
Jon Salz | b19ea07 | 2013-02-07 16:35:00 +0800 | [diff] [blame] | 1241 | self.state_instance.del_shared_data('startup_error', optional=True) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1242 | if not self.options.test_list: |
| 1243 | self.options.test_list = find_test_list() |
Jon Salz | b19ea07 | 2013-02-07 16:35:00 +0800 | [diff] [blame] | 1244 | if self.options.test_list: |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1245 | logging.info('Using test list %s', self.options.test_list) |
Jon Salz | b19ea07 | 2013-02-07 16:35:00 +0800 | [diff] [blame] | 1246 | try: |
| 1247 | self.test_list = factory.read_test_list( |
| 1248 | self.options.test_list, |
| 1249 | self.state_instance) |
| 1250 | except: # pylint: disable=W0702 |
| 1251 | logging.exception('Unable to read test list %r', self.options.test_list) |
| 1252 | self.state_instance.set_shared_data('startup_error', |
| 1253 | 'Unable to read test list %s\n%s' % ( |
| 1254 | self.options.test_list, |
| 1255 | traceback.format_exc())) |
| 1256 | else: |
| 1257 | logging.error('No test list found.') |
| 1258 | self.state_instance.set_shared_data('startup_error', |
| 1259 | 'No test list found.') |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1260 | |
Jon Salz | b19ea07 | 2013-02-07 16:35:00 +0800 | [diff] [blame] | 1261 | if not self.test_list: |
| 1262 | if self.options.ui == 'chrome': |
| 1263 | # Create an empty test list with default options so that the rest of |
| 1264 | # startup can proceed. |
| 1265 | self.test_list = factory.FactoryTestList( |
| 1266 | [], self.state_instance, factory.Options()) |
| 1267 | else: |
| 1268 | # Bail with an error; no point in starting up. |
| 1269 | sys.exit('No valid test list; exiting.') |
| 1270 | |
Jon Salz | 822838b | 2013-03-25 17:32:33 +0800 | [diff] [blame] | 1271 | if self.test_list.options.clear_state_on_start: |
| 1272 | self.state_instance.clear_test_state() |
| 1273 | |
Vic Yang | 3e1cf5d | 2013-06-05 18:50:24 +0800 | [diff] [blame] | 1274 | if system.SystemInfo().firmware_version is None and not utils.in_chroot(): |
Vic Yang | 9bd4f77 | 2013-06-04 17:34:00 +0800 | [diff] [blame] | 1275 | self.state_instance.set_shared_data('startup_error', |
| 1276 | 'Netboot firmware detected\n' |
| 1277 | 'Connect Ethernet and reboot to re-image.\n' |
| 1278 | u'侦测到网路开机固件\n' |
| 1279 | u'请连接乙太网并重启') |
| 1280 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1281 | if not self.state_instance.has_shared_data('ui_lang'): |
| 1282 | self.state_instance.set_shared_data('ui_lang', |
| 1283 | self.test_list.options.ui_lang) |
| 1284 | self.state_instance.set_shared_data( |
| 1285 | 'test_list_options', |
| 1286 | self.test_list.options.__dict__) |
| 1287 | self.state_instance.test_list = self.test_list |
| 1288 | |
Jon Salz | 83ef34b | 2012-11-01 19:46:35 +0800 | [diff] [blame] | 1289 | if not utils.in_chroot() and self.test_list.options.disable_log_rotation: |
| 1290 | open('/var/lib/cleanup_logs_paused', 'w').close() |
| 1291 | |
Jon Salz | 2392642 | 2012-09-01 03:38:13 +0800 | [diff] [blame] | 1292 | if self.options.dummy_shopfloor: |
| 1293 | os.environ[shopfloor.SHOPFLOOR_SERVER_ENV_VAR_NAME] = ( |
| 1294 | 'http://localhost:%d/' % shopfloor.DEFAULT_SERVER_PORT) |
| 1295 | self.dummy_shopfloor = Spawn( |
| 1296 | [os.path.join(factory.FACTORY_PATH, 'bin', 'shopfloor_server'), |
| 1297 | '--dummy']) |
| 1298 | elif self.test_list.options.shopfloor_server_url: |
| 1299 | shopfloor.set_server_url(self.test_list.options.shopfloor_server_url) |
Jon Salz | 2bf2f6b | 2013-03-28 18:49:26 +0800 | [diff] [blame] | 1300 | shopfloor.set_enabled(True) |
Jon Salz | 2392642 | 2012-09-01 03:38:13 +0800 | [diff] [blame] | 1301 | |
Jon Salz | 0f99660 | 2012-10-03 15:26:48 +0800 | [diff] [blame] | 1302 | if self.test_list.options.time_sanitizer and not utils.in_chroot(): |
Jon Salz | 8fa8e83 | 2012-07-13 19:04:09 +0800 | [diff] [blame] | 1303 | self.time_sanitizer = time_sanitizer.TimeSanitizer( |
| 1304 | base_time=time_sanitizer.GetBaseTimeFromFile( |
| 1305 | # lsb-factory is written by the factory install shim during |
| 1306 | # installation, so it should have a good time obtained from |
Jon Salz | 54882d0 | 2012-08-31 01:57:54 +0800 | [diff] [blame] | 1307 | # the mini-Omaha server. If it's not available, we'll use |
| 1308 | # /etc/lsb-factory (which will be much older, but reasonably |
| 1309 | # sane) and rely on a shopfloor sync to set a more accurate |
| 1310 | # time. |
| 1311 | '/usr/local/etc/lsb-factory', |
| 1312 | '/etc/lsb-release')) |
Jon Salz | 8fa8e83 | 2012-07-13 19:04:09 +0800 | [diff] [blame] | 1313 | self.time_sanitizer.RunOnce() |
| 1314 | |
Vic Yang | d8990da | 2013-06-27 16:57:43 +0800 | [diff] [blame^] | 1315 | if self.test_list.options.check_cpu_usage_period_secs: |
| 1316 | self.cpu_usage_watcher = Spawn(['py/tools/cpu_usage_monitor.py', |
| 1317 | '-p', str(self.test_list.options.check_cpu_usage_period_secs)], |
| 1318 | cwd=factory.FACTORY_PATH) |
| 1319 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1320 | self.init_states() |
| 1321 | self.start_event_server() |
| 1322 | self.connection_manager = self.env.create_connection_manager( |
Tai-Hsu Lin | 371351a | 2012-08-27 14:17:14 +0800 | [diff] [blame] | 1323 | self.test_list.options.wlans, |
| 1324 | self.test_list.options.scan_wifi_period_secs) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1325 | # Note that we create a log watcher even if |
| 1326 | # sync_event_log_period_secs isn't set (no background |
| 1327 | # syncing), since we may use it to flush event logs as well. |
| 1328 | self.log_watcher = EventLogWatcher( |
| 1329 | self.test_list.options.sync_event_log_period_secs, |
Jon Salz | d15bbcf | 2013-05-21 17:33:57 +0800 | [diff] [blame] | 1330 | event_log_db_file=None, |
Jon Salz | 16d1054 | 2012-07-23 12:18:45 +0800 | [diff] [blame] | 1331 | handle_event_logs_callback=self.handle_event_logs) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1332 | if self.test_list.options.sync_event_log_period_secs: |
| 1333 | self.log_watcher.StartWatchThread() |
| 1334 | |
Cheng-Yi Chiang | 344b10f | 2013-05-03 16:44:03 +0800 | [diff] [blame] | 1335 | # Note that we create a system log manager even if |
| 1336 | # sync_log_period_secs isn't set (no background |
| 1337 | # syncing), since we may kick it to sync logs in its |
| 1338 | # thread. |
| 1339 | self.system_log_manager = SystemLogManager( |
Cheng-Yi Chiang | 835f268 | 2013-05-06 22:15:48 +0800 | [diff] [blame] | 1340 | sync_log_paths=self.test_list.options.sync_log_paths, |
| 1341 | sync_period_sec=self.test_list.options.sync_log_period_secs, |
| 1342 | clear_log_paths=self.test_list.options.clear_log_paths) |
Cheng-Yi Chiang | 344b10f | 2013-05-03 16:44:03 +0800 | [diff] [blame] | 1343 | self.system_log_manager.StartSyncThread() |
| 1344 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1345 | self.update_system_info() |
| 1346 | |
Vic Yang | 4953fc1 | 2012-07-26 16:19:53 +0800 | [diff] [blame] | 1347 | assert ((self.test_list.options.min_charge_pct is None) == |
| 1348 | (self.test_list.options.max_charge_pct is None)) |
Vic Yang | e83d9a1 | 2013-04-19 20:00:20 +0800 | [diff] [blame] | 1349 | if utils.in_chroot(): |
| 1350 | logging.info('In chroot, ignoring charge manager and charge state') |
| 1351 | elif self.test_list.options.min_charge_pct is not None: |
Vic Yang | 4953fc1 | 2012-07-26 16:19:53 +0800 | [diff] [blame] | 1352 | self.charge_manager = ChargeManager(self.test_list.options.min_charge_pct, |
| 1353 | self.test_list.options.max_charge_pct) |
Jon Salz | ad7353b | 2012-10-15 16:22:46 +0800 | [diff] [blame] | 1354 | system.SystemStatus.charge_manager = self.charge_manager |
Cheng-Yi Chiang | d818695 | 2013-04-04 23:41:14 +0800 | [diff] [blame] | 1355 | else: |
| 1356 | # Goofy should set charger state to charge if charge_manager is disabled. |
| 1357 | try: |
| 1358 | system.GetBoard().SetChargeState(Board.ChargeState.CHARGE) |
| 1359 | except BoardException: |
| 1360 | logging.exception('Unable to set charge state on this board') |
Vic Yang | 4953fc1 | 2012-07-26 16:19:53 +0800 | [diff] [blame] | 1361 | |
Cheng-Yi Chiang | cdfa418 | 2013-05-05 03:20:19 +0800 | [diff] [blame] | 1362 | self.core_dump_manager = CoreDumpManager( |
| 1363 | self.test_list.options.core_dump_watchlist) |
| 1364 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1365 | os.environ['CROS_FACTORY'] = '1' |
| 1366 | os.environ['CROS_DISABLE_SITE_SYSINFO'] = '1' |
| 1367 | |
| 1368 | # Set CROS_UI since some behaviors in ui.py depend on the |
| 1369 | # particular UI in use. TODO(jsalz): Remove this (and all |
| 1370 | # places it is used) when the GTK UI is removed. |
| 1371 | os.environ['CROS_UI'] = self.options.ui |
| 1372 | |
Jon Salz | 416f9cc | 2013-05-10 18:32:50 +0800 | [diff] [blame] | 1373 | # Initialize hooks. |
| 1374 | module, cls = self.test_list.options.hooks_class.rsplit('.', 1) |
| 1375 | self.hooks = getattr(__import__(module, fromlist=[cls]), cls)() |
| 1376 | assert isinstance(self.hooks, factory.Hooks), ( |
| 1377 | "hooks should be of type Hooks but is %r" % type(self.hooks)) |
| 1378 | self.hooks.test_list = self.test_list |
| 1379 | |
Jon Salz | ce6a7f8 | 2013-06-10 18:22:54 +0800 | [diff] [blame] | 1380 | if not utils.in_chroot(): |
Jon Salz | ddf0d05 | 2013-06-18 12:52:44 +0800 | [diff] [blame] | 1381 | self.cpufreq_manager = CpufreqManager(event_log=self.event_log) |
Jon Salz | ce6a7f8 | 2013-06-10 18:22:54 +0800 | [diff] [blame] | 1382 | |
Jon Salz | 416f9cc | 2013-05-10 18:32:50 +0800 | [diff] [blame] | 1383 | # Call startup hook. |
| 1384 | self.hooks.OnStartup() |
| 1385 | |
Jon Salz | e12c2b3 | 2013-06-25 16:24:34 +0800 | [diff] [blame] | 1386 | self.find_kcrashes() |
| 1387 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1388 | if self.options.ui == 'chrome': |
| 1389 | self.env.launch_chrome() |
| 1390 | logging.info('Waiting for a web socket connection') |
Cheng-Yi Chiang | fd8ed39 | 2013-03-08 21:37:31 +0800 | [diff] [blame] | 1391 | self.web_socket_manager.wait() |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1392 | |
| 1393 | # Wait for the test widget size to be set; this is done in |
| 1394 | # an asynchronous RPC so there is a small chance that the |
| 1395 | # web socket might be opened first. |
| 1396 | for _ in range(100): # 10 s |
| 1397 | try: |
| 1398 | if self.state_instance.get_shared_data('test_widget_size'): |
| 1399 | break |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 1400 | except KeyError: |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1401 | pass # Retry |
| 1402 | time.sleep(0.1) # 100 ms |
| 1403 | else: |
| 1404 | logging.warn('Never received test_widget_size from UI') |
Jon Salz | 4529728 | 2013-05-18 14:31:47 +0800 | [diff] [blame] | 1405 | |
| 1406 | # Send Chrome a Tab to get focus to the factory UI |
| 1407 | # (http://crosbug.com/p/19444). TODO(jsalz): remove this hack |
| 1408 | # and figure out the right way to get the focus to Chrome. |
| 1409 | if not utils.in_chroot(): |
| 1410 | Spawn( |
| 1411 | [os.path.join(factory.FACTORY_PATH, 'bin', 'send_key'), 'Tab'], |
| 1412 | check_call=True, log=True) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1413 | elif self.options.ui == 'gtk': |
| 1414 | self.start_ui() |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 1415 | |
Ricky Liang | 650f6bf | 2012-09-28 13:22:54 +0800 | [diff] [blame] | 1416 | # Create download path for autotest beforehand or autotests run at |
| 1417 | # the same time might fail due to race condition. |
| 1418 | if not factory.in_chroot(): |
| 1419 | utils.TryMakeDirs(os.path.join('/usr/local/autotest', 'tests', |
| 1420 | 'download')) |
| 1421 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1422 | def state_change_callback(test, test_state): |
| 1423 | self.event_client.post_event( |
| 1424 | Event(Event.Type.STATE_CHANGE, |
| 1425 | path=test.path, state=test_state)) |
| 1426 | self.test_list.state_change_callback = state_change_callback |
Jon Salz | 73e0fd0 | 2012-04-04 11:46:38 +0800 | [diff] [blame] | 1427 | |
Jon Salz | a6711d7 | 2012-07-18 14:33:03 +0800 | [diff] [blame] | 1428 | for handler in self.on_ui_startup: |
| 1429 | handler() |
| 1430 | |
| 1431 | self.prespawner = Prespawner() |
| 1432 | self.prespawner.start() |
| 1433 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1434 | try: |
| 1435 | tests_after_shutdown = self.state_instance.get_shared_data( |
| 1436 | 'tests_after_shutdown') |
| 1437 | except KeyError: |
| 1438 | tests_after_shutdown = None |
Jon Salz | 57717ca | 2012-04-04 16:47:25 +0800 | [diff] [blame] | 1439 | |
Jon Salz | 5c344f6 | 2012-07-13 14:31:16 +0800 | [diff] [blame] | 1440 | force_auto_run = (tests_after_shutdown == FORCE_AUTO_RUN) |
| 1441 | if not force_auto_run and tests_after_shutdown is not None: |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1442 | logging.info('Resuming tests after shutdown: %s', |
| 1443 | tests_after_shutdown) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1444 | self.tests_to_run.extend( |
| 1445 | self.test_list.lookup_path(t) for t in tests_after_shutdown) |
| 1446 | self.run_queue.put(self.run_next_test) |
| 1447 | else: |
Jon Salz | 5c344f6 | 2012-07-13 14:31:16 +0800 | [diff] [blame] | 1448 | if force_auto_run or self.test_list.options.auto_run_on_start: |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1449 | self.run_queue.put( |
| 1450 | lambda: self.run_tests(self.test_list, untested_only=True)) |
Jon Salz | 5c344f6 | 2012-07-13 14:31:16 +0800 | [diff] [blame] | 1451 | self.state_instance.set_shared_data('tests_after_shutdown', None) |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 1452 | |
Dean Liao | 592e4d5 | 2013-01-10 20:06:39 +0800 | [diff] [blame] | 1453 | self.may_disable_cros_shortcut_keys() |
| 1454 | |
| 1455 | def may_disable_cros_shortcut_keys(self): |
| 1456 | test_options = self.test_list.options |
| 1457 | if test_options.disable_cros_shortcut_keys: |
| 1458 | logging.info('Filter ChromeOS shortcut keys.') |
| 1459 | self.key_filter = KeyFilter( |
| 1460 | unmap_caps_lock=test_options.disable_caps_lock, |
| 1461 | caps_lock_keycode=test_options.caps_lock_keycode) |
| 1462 | self.key_filter.Start() |
| 1463 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1464 | def run(self): |
| 1465 | '''Runs Goofy.''' |
| 1466 | # Process events forever. |
| 1467 | while self.run_once(True): |
| 1468 | pass |
Jon Salz | 73e0fd0 | 2012-04-04 11:46:38 +0800 | [diff] [blame] | 1469 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1470 | def run_once(self, block=False): |
| 1471 | '''Runs all items pending in the event loop. |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 1472 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1473 | Args: |
| 1474 | block: If true, block until at least one event is processed. |
Jon Salz | 7c15e8b | 2012-06-19 17:10:37 +0800 | [diff] [blame] | 1475 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1476 | Returns: |
| 1477 | True to keep going or False to shut down. |
| 1478 | ''' |
| 1479 | events = utils.DrainQueue(self.run_queue) |
cychiang | 2188674 | 2012-07-05 15:16:32 +0800 | [diff] [blame] | 1480 | while not events: |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1481 | # Nothing on the run queue. |
| 1482 | self._run_queue_idle() |
| 1483 | if block: |
| 1484 | # Block for at least one event... |
cychiang | 2188674 | 2012-07-05 15:16:32 +0800 | [diff] [blame] | 1485 | try: |
| 1486 | events.append(self.run_queue.get(timeout=RUN_QUEUE_TIMEOUT_SECS)) |
| 1487 | except Queue.Empty: |
| 1488 | # Keep going (calling _run_queue_idle() again at the top of |
| 1489 | # the loop) |
| 1490 | continue |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1491 | # ...and grab anything else that showed up at the same |
| 1492 | # time. |
| 1493 | events.extend(utils.DrainQueue(self.run_queue)) |
cychiang | 2188674 | 2012-07-05 15:16:32 +0800 | [diff] [blame] | 1494 | else: |
| 1495 | break |
Jon Salz | 51528e1 | 2012-07-02 18:54:45 +0800 | [diff] [blame] | 1496 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1497 | for event in events: |
| 1498 | if not event: |
| 1499 | # Shutdown request. |
| 1500 | self.run_queue.task_done() |
| 1501 | return False |
Jon Salz | 51528e1 | 2012-07-02 18:54:45 +0800 | [diff] [blame] | 1502 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1503 | try: |
| 1504 | event() |
Jon Salz | 85a3988 | 2012-07-05 16:45:04 +0800 | [diff] [blame] | 1505 | except: # pylint: disable=W0702 |
| 1506 | logging.exception('Error in event loop') |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1507 | self.record_exception(traceback.format_exception_only( |
| 1508 | *sys.exc_info()[:2])) |
| 1509 | # But keep going |
| 1510 | finally: |
| 1511 | self.run_queue.task_done() |
| 1512 | return True |
Jon Salz | 0405ab5 | 2012-03-16 15:26:52 +0800 | [diff] [blame] | 1513 | |
Jon Salz | 0e6532d | 2012-10-25 16:30:11 +0800 | [diff] [blame] | 1514 | def _should_sync_time(self, foreground=False): |
| 1515 | '''Returns True if we should attempt syncing time with shopfloor. |
| 1516 | |
| 1517 | Args: |
| 1518 | foreground: If True, synchronizes even if background syncing |
| 1519 | is disabled (e.g., in explicit sync requests from the |
| 1520 | SyncShopfloor test). |
| 1521 | ''' |
| 1522 | return ((foreground or |
| 1523 | self.test_list.options.sync_time_period_secs) and |
Jon Salz | 54882d0 | 2012-08-31 01:57:54 +0800 | [diff] [blame] | 1524 | self.time_sanitizer and |
| 1525 | (not self.time_synced) and |
| 1526 | (not factory.in_chroot())) |
| 1527 | |
Jon Salz | 0e6532d | 2012-10-25 16:30:11 +0800 | [diff] [blame] | 1528 | def sync_time_with_shopfloor_server(self, foreground=False): |
Jon Salz | 54882d0 | 2012-08-31 01:57:54 +0800 | [diff] [blame] | 1529 | '''Syncs time with shopfloor server, if not yet synced. |
| 1530 | |
Jon Salz | 0e6532d | 2012-10-25 16:30:11 +0800 | [diff] [blame] | 1531 | Args: |
| 1532 | foreground: If True, synchronizes even if background syncing |
| 1533 | is disabled (e.g., in explicit sync requests from the |
| 1534 | SyncShopfloor test). |
| 1535 | |
Jon Salz | 54882d0 | 2012-08-31 01:57:54 +0800 | [diff] [blame] | 1536 | Returns: |
| 1537 | False if no time sanitizer is available, or True if this sync (or a |
| 1538 | previous sync) succeeded. |
| 1539 | |
| 1540 | Raises: |
| 1541 | Exception if unable to contact the shopfloor server. |
| 1542 | ''' |
Jon Salz | 0e6532d | 2012-10-25 16:30:11 +0800 | [diff] [blame] | 1543 | if self._should_sync_time(foreground): |
Jon Salz | 54882d0 | 2012-08-31 01:57:54 +0800 | [diff] [blame] | 1544 | self.time_sanitizer.SyncWithShopfloor() |
| 1545 | self.time_synced = True |
| 1546 | return self.time_synced |
| 1547 | |
Jon Salz | b92c511 | 2012-09-21 15:40:11 +0800 | [diff] [blame] | 1548 | def log_disk_space_stats(self): |
Jon Salz | 18e0e02 | 2013-06-11 17:13:39 +0800 | [diff] [blame] | 1549 | if (utils.in_chroot() or |
| 1550 | not self.test_list.options.log_disk_space_period_secs): |
Jon Salz | b92c511 | 2012-09-21 15:40:11 +0800 | [diff] [blame] | 1551 | return |
| 1552 | |
| 1553 | now = time.time() |
| 1554 | if (self.last_log_disk_space_time and |
| 1555 | now - self.last_log_disk_space_time < |
| 1556 | self.test_list.options.log_disk_space_period_secs): |
| 1557 | return |
| 1558 | self.last_log_disk_space_time = now |
| 1559 | |
Cheng-Yi Chiang | d040652 | 2013-04-01 15:40:18 +0800 | [diff] [blame] | 1560 | # Upload event if stateful partition usage is above threshold. |
| 1561 | # Stateful partition is mounted on /usr/local, while |
| 1562 | # encrypted stateful partition is mounted on /var. |
| 1563 | # If there are too much logs in the factory process, |
| 1564 | # these two partitions might get full. |
Jon Salz | b92c511 | 2012-09-21 15:40:11 +0800 | [diff] [blame] | 1565 | try: |
Cheng-Yi Chiang | d040652 | 2013-04-01 15:40:18 +0800 | [diff] [blame] | 1566 | vfs_infos = disk_space.GetAllVFSInfo() |
| 1567 | stateful_info, encrypted_info = None, None |
| 1568 | for vfs_info in vfs_infos.values(): |
| 1569 | if '/usr/local' in vfs_info.mount_points: |
| 1570 | stateful_info = vfs_info |
| 1571 | if '/var' in vfs_info.mount_points: |
| 1572 | encrypted_info = vfs_info |
| 1573 | |
| 1574 | stateful = disk_space.GetPartitionUsage(stateful_info) |
| 1575 | encrypted = disk_space.GetPartitionUsage(encrypted_info) |
| 1576 | |
| 1577 | above_threshold = ( |
| 1578 | self.test_list.options.stateful_usage_threshold and |
| 1579 | max(stateful.bytes_used_pct, |
| 1580 | stateful.inodes_used_pct, |
| 1581 | encrypted.bytes_used_pct, |
| 1582 | encrypted.inodes_used_pct) > |
| 1583 | self.test_list.options.stateful_usage_threshold) |
| 1584 | |
| 1585 | if above_threshold: |
| 1586 | self.event_log.Log('stateful_partition_usage', |
| 1587 | partitions={ |
| 1588 | 'stateful': { |
| 1589 | 'bytes_used_pct': FloatDigit(stateful.bytes_used_pct, 2), |
| 1590 | 'inodes_used_pct': FloatDigit(stateful.inodes_used_pct, 2)}, |
| 1591 | 'encrypted_stateful': { |
| 1592 | 'bytes_used_pct': FloatDigit(encrypted.bytes_used_pct, 2), |
| 1593 | 'inodes_used_pct': FloatDigit(encrypted.inodes_used_pct, 2)} |
| 1594 | }) |
| 1595 | self.log_watcher.ScanEventLogs() |
Cheng-Yi Chiang | 00798e7 | 2013-06-20 18:16:39 +0800 | [diff] [blame] | 1596 | if (not utils.in_chroot() and |
| 1597 | self.test_list.options.stateful_usage_above_threshold_action): |
| 1598 | Spawn(self.test_list.options.stateful_usage_above_threshold_action, |
| 1599 | call=True) |
Cheng-Yi Chiang | d040652 | 2013-04-01 15:40:18 +0800 | [diff] [blame] | 1600 | |
| 1601 | message = disk_space.FormatSpaceUsedAll(vfs_infos) |
Jon Salz | 3c493bb | 2013-02-07 17:24:58 +0800 | [diff] [blame] | 1602 | if message != self.last_log_disk_space_message: |
Cheng-Yi Chiang | d040652 | 2013-04-01 15:40:18 +0800 | [diff] [blame] | 1603 | if above_threshold: |
| 1604 | logging.warning(message) |
| 1605 | else: |
| 1606 | logging.info(message) |
Jon Salz | 3c493bb | 2013-02-07 17:24:58 +0800 | [diff] [blame] | 1607 | self.last_log_disk_space_message = message |
Jon Salz | b92c511 | 2012-09-21 15:40:11 +0800 | [diff] [blame] | 1608 | except: # pylint: disable=W0702 |
| 1609 | logging.exception('Unable to get disk space used') |
| 1610 | |
Justin Chuang | 8381398 | 2013-05-13 01:26:32 +0800 | [diff] [blame] | 1611 | def check_battery(self): |
| 1612 | '''Checks the current battery status. |
| 1613 | |
| 1614 | Logs current battery charging level and status to log. If the battery level |
| 1615 | is lower below warning_low_battery_pct, send warning event to shopfloor. |
| 1616 | If the battery level is lower below critical_low_battery_pct, flush disks. |
| 1617 | ''' |
| 1618 | if not self.test_list.options.check_battery_period_secs: |
| 1619 | return |
| 1620 | |
| 1621 | now = time.time() |
| 1622 | if (self.last_check_battery_time and |
| 1623 | now - self.last_check_battery_time < |
| 1624 | self.test_list.options.check_battery_period_secs): |
| 1625 | return |
| 1626 | self.last_check_battery_time = now |
| 1627 | |
| 1628 | message = '' |
| 1629 | log_level = logging.INFO |
| 1630 | try: |
| 1631 | power = system.GetBoard().power |
| 1632 | if not power.CheckBatteryPresent(): |
| 1633 | message = 'Battery is not present' |
| 1634 | else: |
| 1635 | ac_present = power.CheckACPresent() |
| 1636 | charge_pct = power.GetChargePct(get_float=True) |
| 1637 | message = ('Current battery level %.1f%%, AC charger is %s' % |
| 1638 | (charge_pct, 'connected' if ac_present else 'disconnected')) |
| 1639 | |
| 1640 | if charge_pct > self.test_list.options.critical_low_battery_pct: |
| 1641 | critical_low_battery = False |
| 1642 | else: |
| 1643 | critical_low_battery = True |
| 1644 | # Only sync disks when battery level is still above minimum |
| 1645 | # value. This can be used for offline analysis when shopfloor cannot |
| 1646 | # be connected. |
| 1647 | if charge_pct > MIN_BATTERY_LEVEL_FOR_DISK_SYNC: |
| 1648 | logging.warning('disk syncing for critical low battery situation') |
| 1649 | os.system('sync; sync; sync') |
| 1650 | else: |
| 1651 | logging.warning('disk syncing is cancelled ' |
| 1652 | 'because battery level is lower than %.1f', |
| 1653 | MIN_BATTERY_LEVEL_FOR_DISK_SYNC) |
| 1654 | |
| 1655 | # Notify shopfloor server |
| 1656 | if (critical_low_battery or |
| 1657 | (not ac_present and |
| 1658 | charge_pct <= self.test_list.options.warning_low_battery_pct)): |
| 1659 | log_level = logging.WARNING |
| 1660 | |
| 1661 | self.event_log.Log('low_battery', |
| 1662 | battery_level=charge_pct, |
| 1663 | charger_connected=ac_present, |
| 1664 | critical=critical_low_battery) |
| 1665 | self.log_watcher.KickWatchThread() |
| 1666 | self.system_log_manager.KickSyncThread() |
| 1667 | except: # pylint: disable=W0702 |
| 1668 | logging.exception('Unable to check battery or notify shopfloor') |
| 1669 | finally: |
| 1670 | if message != self.last_check_battery_message: |
| 1671 | logging.log(log_level, message) |
| 1672 | self.last_check_battery_message = message |
| 1673 | |
Cheng-Yi Chiang | cdfa418 | 2013-05-05 03:20:19 +0800 | [diff] [blame] | 1674 | def check_core_dump(self): |
| 1675 | '''Checks if there is any core dumped file. |
| 1676 | |
| 1677 | Removes unwanted core dump files immediately. |
| 1678 | Syncs those files matching watch list to server with a delay between |
| 1679 | each sync. After the files have been synced to server, deletes the files. |
| 1680 | ''' |
| 1681 | core_dump_files = self.core_dump_manager.ScanFiles() |
| 1682 | if core_dump_files: |
| 1683 | now = time.time() |
| 1684 | if (self.last_kick_sync_time and now - self.last_kick_sync_time < |
| 1685 | self.test_list.options.kick_sync_min_interval_secs): |
| 1686 | return |
| 1687 | self.last_kick_sync_time = now |
| 1688 | |
| 1689 | # Sends event to server |
| 1690 | self.event_log.Log('core_dumped', files=core_dump_files) |
| 1691 | self.log_watcher.KickWatchThread() |
| 1692 | |
| 1693 | # Syncs files to server |
| 1694 | self.system_log_manager.KickSyncThread( |
| 1695 | core_dump_files, self.core_dump_manager.ClearFiles) |
| 1696 | |
Jon Salz | 8fa8e83 | 2012-07-13 19:04:09 +0800 | [diff] [blame] | 1697 | def sync_time_in_background(self): |
Jon Salz | b22d117 | 2012-08-06 10:38:57 +0800 | [diff] [blame] | 1698 | '''Writes out current time and tries to sync with shopfloor server.''' |
| 1699 | if not self.time_sanitizer: |
| 1700 | return |
| 1701 | |
| 1702 | # Write out the current time. |
| 1703 | self.time_sanitizer.SaveTime() |
| 1704 | |
Jon Salz | 54882d0 | 2012-08-31 01:57:54 +0800 | [diff] [blame] | 1705 | if not self._should_sync_time(): |
Jon Salz | 8fa8e83 | 2012-07-13 19:04:09 +0800 | [diff] [blame] | 1706 | return |
| 1707 | |
| 1708 | now = time.time() |
| 1709 | if self.last_sync_time and ( |
| 1710 | now - self.last_sync_time < |
| 1711 | self.test_list.options.sync_time_period_secs): |
| 1712 | # Not yet time for another check. |
| 1713 | return |
| 1714 | self.last_sync_time = now |
| 1715 | |
| 1716 | def target(): |
| 1717 | try: |
Jon Salz | 54882d0 | 2012-08-31 01:57:54 +0800 | [diff] [blame] | 1718 | self.sync_time_with_shopfloor_server() |
Jon Salz | 8fa8e83 | 2012-07-13 19:04:09 +0800 | [diff] [blame] | 1719 | except: # pylint: disable=W0702 |
| 1720 | # Oh well. Log an error (but no trace) |
| 1721 | logging.info( |
| 1722 | 'Unable to get time from shopfloor server: %s', |
| 1723 | utils.FormatExceptionOnly()) |
| 1724 | |
| 1725 | thread = threading.Thread(target=target) |
| 1726 | thread.daemon = True |
| 1727 | thread.start() |
| 1728 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1729 | def _run_queue_idle(self): |
Vic Yang | 4953fc1 | 2012-07-26 16:19:53 +0800 | [diff] [blame] | 1730 | '''Invoked when the run queue has no events. |
| 1731 | |
| 1732 | This method must not raise exception. |
| 1733 | ''' |
Jon Salz | b22d117 | 2012-08-06 10:38:57 +0800 | [diff] [blame] | 1734 | now = time.time() |
| 1735 | if (self.last_idle and |
| 1736 | now < (self.last_idle + RUN_QUEUE_TIMEOUT_SECS - 1)): |
| 1737 | # Don't run more often than once every (RUN_QUEUE_TIMEOUT_SECS - |
| 1738 | # 1) seconds. |
| 1739 | return |
| 1740 | |
| 1741 | self.last_idle = now |
| 1742 | |
Vic Yang | 311ddb8 | 2012-09-26 12:08:28 +0800 | [diff] [blame] | 1743 | self.check_exclusive() |
cychiang | 2188674 | 2012-07-05 15:16:32 +0800 | [diff] [blame] | 1744 | self.check_for_updates() |
Jon Salz | 8fa8e83 | 2012-07-13 19:04:09 +0800 | [diff] [blame] | 1745 | self.sync_time_in_background() |
Jon Salz | b92c511 | 2012-09-21 15:40:11 +0800 | [diff] [blame] | 1746 | self.log_disk_space_stats() |
Justin Chuang | 8381398 | 2013-05-13 01:26:32 +0800 | [diff] [blame] | 1747 | self.check_battery() |
Cheng-Yi Chiang | cdfa418 | 2013-05-05 03:20:19 +0800 | [diff] [blame] | 1748 | self.check_core_dump() |
Jon Salz | 57717ca | 2012-04-04 16:47:25 +0800 | [diff] [blame] | 1749 | |
Jon Salz | d15bbcf | 2013-05-21 17:33:57 +0800 | [diff] [blame] | 1750 | def handle_event_logs(self, chunks): |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1751 | '''Callback for event watcher. |
Jon Salz | 258a40c | 2012-04-19 12:34:01 +0800 | [diff] [blame] | 1752 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1753 | Attempts to upload the event logs to the shopfloor server. |
Vic Yang | 9302761 | 2013-05-06 02:42:49 +0800 | [diff] [blame] | 1754 | |
| 1755 | Args: |
Jon Salz | d15bbcf | 2013-05-21 17:33:57 +0800 | [diff] [blame] | 1756 | chunks: A list of Chunk objects. |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1757 | ''' |
Vic Yang | 9302761 | 2013-05-06 02:42:49 +0800 | [diff] [blame] | 1758 | first_exception = None |
| 1759 | exception_count = 0 |
| 1760 | |
Jon Salz | d15bbcf | 2013-05-21 17:33:57 +0800 | [diff] [blame] | 1761 | for chunk in chunks: |
Vic Yang | 9302761 | 2013-05-06 02:42:49 +0800 | [diff] [blame] | 1762 | try: |
Jon Salz | cddb640 | 2013-05-23 12:56:42 +0800 | [diff] [blame] | 1763 | description = 'event logs (%s)' % str(chunk) |
Vic Yang | 9302761 | 2013-05-06 02:42:49 +0800 | [diff] [blame] | 1764 | start_time = time.time() |
| 1765 | shopfloor_client = shopfloor.get_instance( |
| 1766 | detect=True, |
| 1767 | timeout=self.test_list.options.shopfloor_timeout_secs) |
Jon Salz | d15bbcf | 2013-05-21 17:33:57 +0800 | [diff] [blame] | 1768 | shopfloor_client.UploadEvent(chunk.log_name + "." + |
| 1769 | event_log.GetReimageId(), |
| 1770 | Binary(chunk.chunk)) |
Vic Yang | 9302761 | 2013-05-06 02:42:49 +0800 | [diff] [blame] | 1771 | logging.info( |
| 1772 | 'Successfully synced %s in %.03f s', |
| 1773 | description, time.time() - start_time) |
| 1774 | except: # pylint: disable=W0702 |
Jon Salz | d15bbcf | 2013-05-21 17:33:57 +0800 | [diff] [blame] | 1775 | first_exception = (first_exception or (chunk.log_name + ': ' + |
Vic Yang | 9302761 | 2013-05-06 02:42:49 +0800 | [diff] [blame] | 1776 | utils.FormatExceptionOnly())) |
| 1777 | exception_count += 1 |
| 1778 | |
| 1779 | if exception_count: |
| 1780 | if exception_count == 1: |
| 1781 | msg = 'Log upload failed: %s' % first_exception |
| 1782 | else: |
| 1783 | msg = '%d log upload failed; first is: %s' % ( |
| 1784 | exception_count, first_exception) |
| 1785 | raise Exception(msg) |
| 1786 | |
Jon Salz | 57717ca | 2012-04-04 16:47:25 +0800 | [diff] [blame] | 1787 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1788 | def run_tests_with_status(self, statuses_to_run, starting_at=None, |
| 1789 | root=None): |
| 1790 | '''Runs all top-level tests with a particular status. |
Jon Salz | 0405ab5 | 2012-03-16 15:26:52 +0800 | [diff] [blame] | 1791 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1792 | All active tests, plus any tests to re-run, are reset. |
Jon Salz | 57717ca | 2012-04-04 16:47:25 +0800 | [diff] [blame] | 1793 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1794 | Args: |
| 1795 | starting_at: If provided, only auto-runs tests beginning with |
| 1796 | this test. |
| 1797 | ''' |
| 1798 | root = root or self.test_list |
Jon Salz | 57717ca | 2012-04-04 16:47:25 +0800 | [diff] [blame] | 1799 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1800 | if starting_at: |
| 1801 | # Make sure they passed a test, not a string. |
| 1802 | assert isinstance(starting_at, factory.FactoryTest) |
Jon Salz | 0405ab5 | 2012-03-16 15:26:52 +0800 | [diff] [blame] | 1803 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1804 | tests_to_reset = [] |
| 1805 | tests_to_run = [] |
Jon Salz | 0405ab5 | 2012-03-16 15:26:52 +0800 | [diff] [blame] | 1806 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1807 | found_starting_at = False |
Jon Salz | 0405ab5 | 2012-03-16 15:26:52 +0800 | [diff] [blame] | 1808 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1809 | for test in root.get_top_level_tests(): |
| 1810 | if starting_at: |
| 1811 | if test == starting_at: |
| 1812 | # We've found starting_at; do auto-run on all |
| 1813 | # subsequent tests. |
| 1814 | found_starting_at = True |
| 1815 | if not found_starting_at: |
| 1816 | # Don't start this guy yet |
| 1817 | continue |
Jon Salz | 0405ab5 | 2012-03-16 15:26:52 +0800 | [diff] [blame] | 1818 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1819 | status = test.get_state().status |
| 1820 | if status == TestState.ACTIVE or status in statuses_to_run: |
| 1821 | # Reset the test (later; we will need to abort |
| 1822 | # all active tests first). |
| 1823 | tests_to_reset.append(test) |
| 1824 | if status in statuses_to_run: |
| 1825 | tests_to_run.append(test) |
Jon Salz | 0405ab5 | 2012-03-16 15:26:52 +0800 | [diff] [blame] | 1826 | |
Jon Salz | 6dc031d | 2013-06-19 13:06:23 +0800 | [diff] [blame] | 1827 | self.abort_active_tests('Operator requested run/re-run of certain tests') |
Jon Salz | 258a40c | 2012-04-19 12:34:01 +0800 | [diff] [blame] | 1828 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1829 | # Reset all statuses of the tests to run (in case any tests were active; |
| 1830 | # we want them to be run again). |
| 1831 | for test_to_reset in tests_to_reset: |
| 1832 | for test in test_to_reset.walk(): |
| 1833 | test.update_state(status=TestState.UNTESTED) |
Jon Salz | 57717ca | 2012-04-04 16:47:25 +0800 | [diff] [blame] | 1834 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1835 | self.run_tests(tests_to_run, untested_only=True) |
Jon Salz | 0405ab5 | 2012-03-16 15:26:52 +0800 | [diff] [blame] | 1836 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1837 | def restart_tests(self, root=None): |
| 1838 | '''Restarts all tests.''' |
| 1839 | root = root or self.test_list |
Jon Salz | 0405ab5 | 2012-03-16 15:26:52 +0800 | [diff] [blame] | 1840 | |
Jon Salz | 6dc031d | 2013-06-19 13:06:23 +0800 | [diff] [blame] | 1841 | self.abort_active_tests('Operator requested restart of certain tests') |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1842 | for test in root.walk(): |
| 1843 | test.update_state(status=TestState.UNTESTED) |
| 1844 | self.run_tests(root) |
Hung-Te Lin | 9663236 | 2012-03-20 21:14:18 +0800 | [diff] [blame] | 1845 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1846 | def auto_run(self, starting_at=None, root=None): |
| 1847 | '''"Auto-runs" tests that have not been run yet. |
Hung-Te Lin | 9663236 | 2012-03-20 21:14:18 +0800 | [diff] [blame] | 1848 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1849 | Args: |
| 1850 | starting_at: If provide, only auto-runs tests beginning with |
| 1851 | this test. |
| 1852 | ''' |
| 1853 | root = root or self.test_list |
| 1854 | self.run_tests_with_status([TestState.UNTESTED, TestState.ACTIVE], |
| 1855 | starting_at=starting_at, |
| 1856 | root=root) |
Jon Salz | 968e90b | 2012-03-18 16:12:43 +0800 | [diff] [blame] | 1857 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1858 | def re_run_failed(self, root=None): |
| 1859 | '''Re-runs failed tests.''' |
| 1860 | root = root or self.test_list |
| 1861 | self.run_tests_with_status([TestState.FAILED], root=root) |
Jon Salz | 57717ca | 2012-04-04 16:47:25 +0800 | [diff] [blame] | 1862 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1863 | def show_review_information(self): |
| 1864 | '''Event handler for showing review information screen. |
Jon Salz | 57717ca | 2012-04-04 16:47:25 +0800 | [diff] [blame] | 1865 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1866 | The information screene is rendered by main UI program (ui.py), so in |
| 1867 | goofy we only need to kill all active tests, set them as untested, and |
| 1868 | clear remaining tests. |
| 1869 | ''' |
| 1870 | self.kill_active_tests(False) |
Jon Salz | a6711d7 | 2012-07-18 14:33:03 +0800 | [diff] [blame] | 1871 | self.cancel_pending_tests() |
Jon Salz | 57717ca | 2012-04-04 16:47:25 +0800 | [diff] [blame] | 1872 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1873 | def handle_switch_test(self, event): |
| 1874 | '''Switches to a particular test. |
Jon Salz | 0405ab5 | 2012-03-16 15:26:52 +0800 | [diff] [blame] | 1875 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1876 | @param event: The SWITCH_TEST event. |
| 1877 | ''' |
| 1878 | test = self.test_list.lookup_path(event.path) |
| 1879 | if not test: |
| 1880 | logging.error('Unknown test %r', event.key) |
| 1881 | return |
Jon Salz | 73e0fd0 | 2012-04-04 11:46:38 +0800 | [diff] [blame] | 1882 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1883 | invoc = self.invocations.get(test) |
| 1884 | if invoc and test.backgroundable: |
| 1885 | # Already running: just bring to the front if it |
| 1886 | # has a UI. |
| 1887 | logging.info('Setting visible test to %s', test.path) |
Jon Salz | 36fbbb5 | 2012-07-05 13:45:06 +0800 | [diff] [blame] | 1888 | self.set_visible_test(test) |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1889 | return |
Jon Salz | 73e0fd0 | 2012-04-04 11:46:38 +0800 | [diff] [blame] | 1890 | |
Jon Salz | 6dc031d | 2013-06-19 13:06:23 +0800 | [diff] [blame] | 1891 | self.abort_active_tests('Operator requested abort (switch_test)') |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1892 | for t in test.walk(): |
| 1893 | t.update_state(status=TestState.UNTESTED) |
Jon Salz | 73e0fd0 | 2012-04-04 11:46:38 +0800 | [diff] [blame] | 1894 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1895 | if self.test_list.options.auto_run_on_keypress: |
| 1896 | self.auto_run(starting_at=test) |
| 1897 | else: |
| 1898 | self.run_tests(test) |
Jon Salz | 73e0fd0 | 2012-04-04 11:46:38 +0800 | [diff] [blame] | 1899 | |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1900 | def wait(self): |
| 1901 | '''Waits for all pending invocations. |
| 1902 | |
| 1903 | Useful for testing. |
| 1904 | ''' |
Jon Salz | 1acc874 | 2012-07-17 17:45:55 +0800 | [diff] [blame] | 1905 | while self.invocations: |
| 1906 | for k, v in self.invocations.iteritems(): |
| 1907 | logging.info('Waiting for %s to complete...', k) |
| 1908 | v.thread.join() |
| 1909 | self.reap_completed_tests() |
Jon Salz | 0697cbf | 2012-07-04 15:14:04 +0800 | [diff] [blame] | 1910 | |
| 1911 | def check_exceptions(self): |
| 1912 | '''Raises an error if any exceptions have occurred in |
| 1913 | invocation threads.''' |
| 1914 | if self.exceptions: |
| 1915 | raise RuntimeError('Exception in invocation thread: %r' % |
| 1916 | self.exceptions) |
| 1917 | |
| 1918 | def record_exception(self, msg): |
| 1919 | '''Records an exception in an invocation thread. |
| 1920 | |
| 1921 | An exception with the given message will be rethrown when |
| 1922 | Goofy is destroyed.''' |
| 1923 | self.exceptions.append(msg) |
Jon Salz | 73e0fd0 | 2012-04-04 11:46:38 +0800 | [diff] [blame] | 1924 | |
Hung-Te Lin | f2f78f7 | 2012-02-08 19:27:11 +0800 | [diff] [blame] | 1925 | |
| 1926 | if __name__ == '__main__': |
Jon Salz | 77c151e | 2012-08-28 07:20:37 +0800 | [diff] [blame] | 1927 | goofy = Goofy() |
| 1928 | try: |
| 1929 | goofy.main() |
Jon Salz | 0f99660 | 2012-10-03 15:26:48 +0800 | [diff] [blame] | 1930 | except SystemExit: |
| 1931 | # Propagate SystemExit without logging. |
| 1932 | raise |
Jon Salz | 31373eb | 2012-09-21 16:19:49 +0800 | [diff] [blame] | 1933 | except: |
Jon Salz | 0f99660 | 2012-10-03 15:26:48 +0800 | [diff] [blame] | 1934 | # Log the error before trying to shut down (unless it's a graceful |
| 1935 | # exit). |
Jon Salz | 31373eb | 2012-09-21 16:19:49 +0800 | [diff] [blame] | 1936 | logging.exception('Error in main loop') |
| 1937 | raise |
Jon Salz | 77c151e | 2012-08-28 07:20:37 +0800 | [diff] [blame] | 1938 | finally: |
| 1939 | goofy.destroy() |