Chris Sosa | 5e4246b | 2012-05-22 18:05:22 -0700 | [diff] [blame] | 1 | # Copyright (c) 2012 The Chromium OS Authors. All rights reserved. |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 2 | # Use of this source code is governed by a BSD-style license that can be |
| 3 | # found in the LICENSE file. |
| 4 | |
Don Garrett | 56b1cc8 | 2013-12-06 17:49:20 -0800 | [diff] [blame] | 5 | import glob |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 6 | import httplib |
| 7 | import logging |
Chris Sosa | 77556d8 | 2012-04-05 15:23:14 -0700 | [diff] [blame] | 8 | import multiprocessing |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 9 | import os |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 10 | import re |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 11 | import urlparse |
Prashanth B | 32baa9b | 2014-03-13 13:23:01 -0700 | [diff] [blame] | 12 | import urllib2 |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 13 | |
Chris Sosa | 6542508 | 2013-10-16 13:26:22 -0700 | [diff] [blame] | 14 | from autotest_lib.client.bin import utils |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 15 | from autotest_lib.client.common_lib import error, global_config |
Prashanth B | 32baa9b | 2014-03-13 13:23:01 -0700 | [diff] [blame] | 16 | from autotest_lib.client.common_lib.cros import dev_server |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 17 | |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 18 | # Local stateful update path is relative to the CrOS source directory. |
| 19 | LOCAL_STATEFUL_UPDATE_PATH = 'src/platform/dev/stateful_update' |
Chris Sosa | a3ac215 | 2012-05-23 22:23:13 -0700 | [diff] [blame] | 20 | LOCAL_CHROOT_STATEFUL_UPDATE_PATH = '/usr/bin/stateful_update' |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 21 | REMOTE_STATEUL_UPDATE_PATH = '/usr/local/bin/stateful_update' |
| 22 | STATEFUL_UPDATE = '/tmp/stateful_update' |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 23 | UPDATER_BIN = '/usr/bin/update_engine_client' |
| 24 | UPDATER_IDLE = 'UPDATE_STATUS_IDLE' |
Sean O | c053dfe | 2010-08-23 18:22:26 +0200 | [diff] [blame] | 25 | UPDATER_NEED_REBOOT = 'UPDATE_STATUS_UPDATED_NEED_REBOOT' |
Darin Petkov | 7d57299 | 2010-09-23 10:11:05 -0700 | [diff] [blame] | 26 | UPDATED_MARKER = '/var/run/update_engine_autoupdate_completed' |
Gwendal Grignou | 8193956 | 2014-02-14 12:25:00 -0800 | [diff] [blame] | 27 | UPDATER_LOGS = ['/var/log/messages', '/var/log/update_engine'] |
beeps | 5e8c45a | 2013-12-17 22:05:11 -0800 | [diff] [blame] | 28 | # A list of update engine client states that occur after an update is triggered. |
| 29 | UPDATER_PROCESSING_UPDATE = ['UPDATE_STATUS_CHECKING_FORUPDATE', |
| 30 | 'UPDATE_STATUS_UPDATE_AVAILABLE', |
| 31 | 'UPDATE_STATUS_DOWNLOADING', |
| 32 | 'UPDATE_STATUS_FINALIZING'] |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 33 | |
| 34 | class ChromiumOSError(error.InstallError): |
| 35 | """Generic error for ChromiumOS-specific exceptions.""" |
| 36 | pass |
| 37 | |
| 38 | |
Chris Sosa | 77556d8 | 2012-04-05 15:23:14 -0700 | [diff] [blame] | 39 | class RootFSUpdateError(ChromiumOSError): |
| 40 | """Raised when the RootFS fails to update.""" |
| 41 | pass |
| 42 | |
| 43 | |
| 44 | class StatefulUpdateError(ChromiumOSError): |
| 45 | """Raised when the stateful partition fails to update.""" |
| 46 | pass |
| 47 | |
| 48 | |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 49 | def url_to_version(update_url): |
Dan Shi | 0f466e8 | 2013-02-22 15:44:58 -0800 | [diff] [blame] | 50 | """Return the version based on update_url. |
| 51 | |
| 52 | @param update_url: url to the image to update to. |
| 53 | |
| 54 | """ |
Dale Curtis | ddfdb94 | 2011-07-14 13:59:24 -0700 | [diff] [blame] | 55 | # The Chrome OS version is generally the last element in the URL. The only |
| 56 | # exception is delta update URLs, which are rooted under the version; e.g., |
| 57 | # http://.../update/.../0.14.755.0/au/0.14.754.0. In this case we want to |
| 58 | # strip off the au section of the path before reading the version. |
Dan Shi | 5002cfc | 2013-04-29 10:45:05 -0700 | [diff] [blame] | 59 | return re.sub('/au/.*', '', |
| 60 | urlparse.urlparse(update_url).path).split('/')[-1].strip() |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 61 | |
| 62 | |
Scott Zawalski | eadbf70 | 2013-03-14 09:23:06 -0400 | [diff] [blame] | 63 | def url_to_image_name(update_url): |
| 64 | """Return the image name based on update_url. |
| 65 | |
| 66 | From a URL like: |
| 67 | http://172.22.50.205:8082/update/lumpy-release/R27-3837.0.0 |
| 68 | return lumpy-release/R27-3837.0.0 |
| 69 | |
| 70 | @param update_url: url to the image to update to. |
| 71 | @returns a string representing the image name in the update_url. |
| 72 | |
| 73 | """ |
| 74 | return '/'.join(urlparse.urlparse(update_url).path.split('/')[-2:]) |
| 75 | |
| 76 | |
Prashanth B | 32baa9b | 2014-03-13 13:23:01 -0700 | [diff] [blame] | 77 | def _get_devserver_build_from_update_url(update_url): |
| 78 | """Get the devserver and build from the update url. |
| 79 | |
| 80 | @param update_url: The url for update. |
| 81 | Eg: http://devserver:port/update/build. |
| 82 | |
| 83 | @return: A tuple of (devserver url, build) or None if the update_url |
| 84 | doesn't match the expected pattern. |
| 85 | |
| 86 | @raises ValueError: If the update_url doesn't match the expected pattern. |
| 87 | @raises ValueError: If no global_config was found, or it doesn't contain an |
| 88 | image_url_pattern. |
| 89 | """ |
| 90 | pattern = global_config.global_config.get_config_value( |
| 91 | 'CROS', 'image_url_pattern', type=str, default='') |
| 92 | if not pattern: |
| 93 | raise ValueError('Cannot parse update_url, the global config needs ' |
| 94 | 'an image_url_pattern.') |
| 95 | re_pattern = pattern.replace('%s', '(\S+)') |
| 96 | parts = re.search(re_pattern, update_url) |
| 97 | if not parts or len(parts.groups()) < 2: |
| 98 | raise ValueError('%s is not an update url' % update_url) |
| 99 | return parts.groups() |
| 100 | |
| 101 | |
| 102 | def list_image_dir_contents(update_url): |
| 103 | """Lists the contents of the devserver for a given build/update_url. |
| 104 | |
| 105 | @param update_url: An update url. Eg: http://devserver:port/update/build. |
| 106 | """ |
| 107 | if not update_url: |
| 108 | logging.warning('Need update_url to list contents of the devserver.') |
| 109 | return |
| 110 | error_msg = 'Cannot check contents of devserver, update url %s' % update_url |
| 111 | try: |
| 112 | devserver_url, build = _get_devserver_build_from_update_url(update_url) |
| 113 | except ValueError as e: |
| 114 | logging.warning('%s: %s', error_msg, e) |
| 115 | return |
| 116 | devserver = dev_server.ImageServer(devserver_url) |
| 117 | try: |
| 118 | devserver.list_image_dir(build) |
| 119 | # The devserver will retry on URLError to avoid flaky connections, but will |
| 120 | # eventually raise the URLError if it persists. All HTTPErrors get |
| 121 | # converted to DevServerExceptions. |
| 122 | except (dev_server.DevServerException, urllib2.URLError) as e: |
| 123 | logging.warning('%s: %s', error_msg, e) |
| 124 | |
| 125 | |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 126 | class ChromiumOSUpdater(): |
Dan Shi | 0f466e8 | 2013-02-22 15:44:58 -0800 | [diff] [blame] | 127 | """Helper class used to update DUT with image of desired version.""" |
Dale Curtis | a94c19c | 2011-05-02 15:05:17 -0700 | [diff] [blame] | 128 | KERNEL_A = {'name': 'KERN-A', 'kernel': 2, 'root': 3} |
| 129 | KERNEL_B = {'name': 'KERN-B', 'kernel': 4, 'root': 5} |
Chris Sosa | 6542508 | 2013-10-16 13:26:22 -0700 | [diff] [blame] | 130 | # Time to wait for new kernel to be marked successful after |
| 131 | # auto update. |
| 132 | KERNEL_UPDATE_TIMEOUT = 120 |
Dale Curtis | a94c19c | 2011-05-02 15:05:17 -0700 | [diff] [blame] | 133 | |
| 134 | |
Chris Sosa | a3ac215 | 2012-05-23 22:23:13 -0700 | [diff] [blame] | 135 | def __init__(self, update_url, host=None, local_devserver=False): |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 136 | self.host = host |
| 137 | self.update_url = update_url |
Chris Sosa | 77556d8 | 2012-04-05 15:23:14 -0700 | [diff] [blame] | 138 | self._update_error_queue = multiprocessing.Queue(2) |
Chris Sosa | a3ac215 | 2012-05-23 22:23:13 -0700 | [diff] [blame] | 139 | self.local_devserver = local_devserver |
| 140 | if not local_devserver: |
| 141 | self.update_version = url_to_version(update_url) |
| 142 | else: |
| 143 | self.update_version = None |
Sean O | c053dfe | 2010-08-23 18:22:26 +0200 | [diff] [blame] | 144 | |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 145 | def check_update_status(self): |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 146 | """Return current status from update-engine.""" |
| 147 | update_status = self._run( |
| 148 | '%s -status 2>&1 | grep CURRENT_OP' % UPDATER_BIN) |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 149 | return update_status.stdout.strip().split('=')[-1] |
| 150 | |
Sean O | c053dfe | 2010-08-23 18:22:26 +0200 | [diff] [blame] | 151 | |
| 152 | def reset_update_engine(self): |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 153 | """Restarts the update-engine service.""" |
Darin Petkov | 7d57299 | 2010-09-23 10:11:05 -0700 | [diff] [blame] | 154 | self._run('rm -f %s' % UPDATED_MARKER) |
Sean O | 267c00b | 2010-08-31 15:54:55 +0200 | [diff] [blame] | 155 | try: |
| 156 | self._run('initctl stop update-engine') |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 157 | except error.AutoservRunError: |
Ilja H. Friedel | 04be2bd | 2014-05-07 21:29:59 -0700 | [diff] [blame] | 158 | logging.warning('Stopping update-engine service failed. Already dead?') |
Sean O | c053dfe | 2010-08-23 18:22:26 +0200 | [diff] [blame] | 159 | self._run('initctl start update-engine') |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 160 | |
Sean O | c053dfe | 2010-08-23 18:22:26 +0200 | [diff] [blame] | 161 | if self.check_update_status() != UPDATER_IDLE: |
| 162 | raise ChromiumOSError('%s is not in an installable state' % |
| 163 | self.host.hostname) |
| 164 | |
| 165 | |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 166 | def _run(self, cmd, *args, **kwargs): |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 167 | """Abbreviated form of self.host.run(...)""" |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 168 | return self.host.run(cmd, *args, **kwargs) |
| 169 | |
Sean O | c053dfe | 2010-08-23 18:22:26 +0200 | [diff] [blame] | 170 | |
Dale Curtis | a94c19c | 2011-05-02 15:05:17 -0700 | [diff] [blame] | 171 | def rootdev(self, options=''): |
Dan Shi | 0f466e8 | 2013-02-22 15:44:58 -0800 | [diff] [blame] | 172 | """Returns the stripped output of rootdev <options>. |
| 173 | |
| 174 | @param options: options to run rootdev. |
| 175 | |
| 176 | """ |
Dale Curtis | a94c19c | 2011-05-02 15:05:17 -0700 | [diff] [blame] | 177 | return self._run('rootdev %s' % options).stdout.strip() |
| 178 | |
| 179 | |
| 180 | def get_kernel_state(self): |
| 181 | """Returns the (<active>, <inactive>) kernel state as a pair.""" |
| 182 | active_root = int(re.findall('\d+\Z', self.rootdev('-s'))[0]) |
| 183 | if active_root == self.KERNEL_A['root']: |
| 184 | return self.KERNEL_A, self.KERNEL_B |
| 185 | elif active_root == self.KERNEL_B['root']: |
| 186 | return self.KERNEL_B, self.KERNEL_A |
| 187 | else: |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 188 | raise ChromiumOSError('Encountered unknown root partition: %s' % |
Dale Curtis | a94c19c | 2011-05-02 15:05:17 -0700 | [diff] [blame] | 189 | active_root) |
| 190 | |
| 191 | |
| 192 | def _cgpt(self, flag, kernel, dev='$(rootdev -s -d)'): |
| 193 | """Return numeric cgpt value for the specified flag, kernel, device. """ |
| 194 | return int(self._run('cgpt show -n -i %d %s %s' % ( |
| 195 | kernel['kernel'], flag, dev)).stdout.strip()) |
| 196 | |
| 197 | |
| 198 | def get_kernel_priority(self, kernel): |
Dan Shi | 0f466e8 | 2013-02-22 15:44:58 -0800 | [diff] [blame] | 199 | """Return numeric priority for the specified kernel. |
| 200 | |
| 201 | @param kernel: information of the given kernel, KERNEL_A or KERNEL_B. |
| 202 | |
| 203 | """ |
Dale Curtis | a94c19c | 2011-05-02 15:05:17 -0700 | [diff] [blame] | 204 | return self._cgpt('-P', kernel) |
| 205 | |
| 206 | |
| 207 | def get_kernel_success(self, kernel): |
Dan Shi | 0f466e8 | 2013-02-22 15:44:58 -0800 | [diff] [blame] | 208 | """Return boolean success flag for the specified kernel. |
| 209 | |
| 210 | @param kernel: information of the given kernel, KERNEL_A or KERNEL_B. |
| 211 | |
| 212 | """ |
Dale Curtis | a94c19c | 2011-05-02 15:05:17 -0700 | [diff] [blame] | 213 | return self._cgpt('-S', kernel) != 0 |
| 214 | |
| 215 | |
| 216 | def get_kernel_tries(self, kernel): |
Dan Shi | 0f466e8 | 2013-02-22 15:44:58 -0800 | [diff] [blame] | 217 | """Return tries count for the specified kernel. |
| 218 | |
| 219 | @param kernel: information of the given kernel, KERNEL_A or KERNEL_B. |
| 220 | |
| 221 | """ |
Dale Curtis | a94c19c | 2011-05-02 15:05:17 -0700 | [diff] [blame] | 222 | return self._cgpt('-T', kernel) |
Sean O | 267c00b | 2010-08-31 15:54:55 +0200 | [diff] [blame] | 223 | |
| 224 | |
Chris Sosa | 5e4246b | 2012-05-22 18:05:22 -0700 | [diff] [blame] | 225 | def get_stateful_update_script(self): |
| 226 | """Returns the path to the stateful update script on the target.""" |
Chris Sosa | a3ac215 | 2012-05-23 22:23:13 -0700 | [diff] [blame] | 227 | # We attempt to load the local stateful update path in 3 different |
| 228 | # ways. First we use the location specified in the autotest global |
| 229 | # config. If this doesn't exist, we attempt to use the Chromium OS |
| 230 | # Chroot path to the installed script. If all else fails, we use the |
| 231 | # stateful update script on the host. |
Chris Sosa | 5e4246b | 2012-05-22 18:05:22 -0700 | [diff] [blame] | 232 | stateful_update_path = os.path.join( |
| 233 | global_config.global_config.get_config_value( |
| 234 | 'CROS', 'source_tree', default=''), |
| 235 | LOCAL_STATEFUL_UPDATE_PATH) |
| 236 | |
Chris Sosa | a3ac215 | 2012-05-23 22:23:13 -0700 | [diff] [blame] | 237 | if not os.path.exists(stateful_update_path): |
Ilja H. Friedel | 04be2bd | 2014-05-07 21:29:59 -0700 | [diff] [blame] | 238 | logging.warning('Could not find Chrome OS source location for ' |
Chris Sosa | a3ac215 | 2012-05-23 22:23:13 -0700 | [diff] [blame] | 239 | 'stateful_update script at %s, falling back to chroot ' |
| 240 | 'copy.', stateful_update_path) |
| 241 | stateful_update_path = LOCAL_CHROOT_STATEFUL_UPDATE_PATH |
| 242 | |
| 243 | if not os.path.exists(stateful_update_path): |
Ilja H. Friedel | 04be2bd | 2014-05-07 21:29:59 -0700 | [diff] [blame] | 244 | logging.warning('Could not chroot stateful_update script, falling ' |
Chris Sosa | a3ac215 | 2012-05-23 22:23:13 -0700 | [diff] [blame] | 245 | 'back on client copy.') |
| 246 | statefuldev_script = REMOTE_STATEUL_UPDATE_PATH |
| 247 | else: |
Chris Sosa | 5e4246b | 2012-05-22 18:05:22 -0700 | [diff] [blame] | 248 | self.host.send_file( |
| 249 | stateful_update_path, STATEFUL_UPDATE, delete_dest=True) |
| 250 | statefuldev_script = STATEFUL_UPDATE |
Chris Sosa | 5e4246b | 2012-05-22 18:05:22 -0700 | [diff] [blame] | 251 | |
| 252 | return statefuldev_script |
| 253 | |
| 254 | |
| 255 | def reset_stateful_partition(self): |
Dan Shi | 0f466e8 | 2013-02-22 15:44:58 -0800 | [diff] [blame] | 256 | """Clear any pending stateful update request.""" |
Chris Sosa | 5e4246b | 2012-05-22 18:05:22 -0700 | [diff] [blame] | 257 | statefuldev_cmd = [self.get_stateful_update_script()] |
| 258 | statefuldev_cmd += ['--stateful_change=reset', '2>&1'] |
Chris Sosa | 66d7407 | 2013-09-19 11:21:29 -0700 | [diff] [blame] | 259 | self._run(' '.join(statefuldev_cmd)) |
Chris Sosa | 5e4246b | 2012-05-22 18:05:22 -0700 | [diff] [blame] | 260 | |
| 261 | |
Sean O | 267c00b | 2010-08-31 15:54:55 +0200 | [diff] [blame] | 262 | def revert_boot_partition(self): |
Dan Shi | 0f466e8 | 2013-02-22 15:44:58 -0800 | [diff] [blame] | 263 | """Revert the boot partition.""" |
Dale Curtis | d9b26b9 | 2011-10-24 13:34:46 -0700 | [diff] [blame] | 264 | part = self.rootdev('-s') |
Ilja H. Friedel | 04be2bd | 2014-05-07 21:29:59 -0700 | [diff] [blame] | 265 | logging.warning('Reverting update; Boot partition will be %s', part) |
Sean O | 267c00b | 2010-08-31 15:54:55 +0200 | [diff] [blame] | 266 | return self._run('/postinst %s 2>&1' % part) |
| 267 | |
| 268 | |
Gilad Arnold | 0ed760c | 2012-11-05 23:42:53 -0800 | [diff] [blame] | 269 | def trigger_update(self): |
| 270 | """Triggers a background update on a test image. |
| 271 | |
| 272 | @raise RootFSUpdateError if anything went wrong. |
| 273 | |
| 274 | """ |
| 275 | autoupdate_cmd = '%s --check_for_update --omaha_url=%s' % ( |
| 276 | UPDATER_BIN, self.update_url) |
Gilad Arnold | 0338ff3 | 2013-10-02 12:16:26 -0700 | [diff] [blame] | 277 | logging.info('Triggering update via: %s', autoupdate_cmd) |
Gilad Arnold | 0ed760c | 2012-11-05 23:42:53 -0800 | [diff] [blame] | 278 | try: |
Chris Sosa | 66d7407 | 2013-09-19 11:21:29 -0700 | [diff] [blame] | 279 | self._run(autoupdate_cmd) |
Don Garrett | df8aef7 | 2013-12-16 11:12:41 -0800 | [diff] [blame] | 280 | except (error.AutoservSshPermissionDeniedError, |
| 281 | error.AutoservSSHTimeout) as e: |
| 282 | raise RootFSUpdateError('SSH on %s is seeing %s' % |
| 283 | (self.host.hostname, type(e).__name__)) |
| 284 | except error.AutoservRunError as e: |
| 285 | |
| 286 | # Check if the exit code is 255, if so it's probably a generic |
| 287 | # SSH error. |
| 288 | result = e.args[1] |
| 289 | if result.exit_status == 255: |
| 290 | raise RootFSUpdateError('SSH on %s is seeing a generic error.' % |
| 291 | self.host.hostname) |
| 292 | |
| 293 | # We have ruled out all SSH cases, the error code is from |
| 294 | # update_engine_client, though we still don't know why. |
Prashanth B | 32baa9b | 2014-03-13 13:23:01 -0700 | [diff] [blame] | 295 | list_image_dir_contents(self.update_url) |
Don Garrett | df8aef7 | 2013-12-16 11:12:41 -0800 | [diff] [blame] | 296 | raise RootFSUpdateError( |
| 297 | 'devserver unreachable, payload unavailable, ' |
| 298 | 'or AU bug (unlikely) on %s: %s' % |
| 299 | (self.host.hostname, type(e).__name__)) |
| 300 | |
Gilad Arnold | 0ed760c | 2012-11-05 23:42:53 -0800 | [diff] [blame] | 301 | |
Chris Sosa | c193217 | 2013-10-16 13:28:53 -0700 | [diff] [blame] | 302 | def _verify_update_completed(self): |
| 303 | """Verifies that an update has completed. |
| 304 | |
| 305 | @raise RootFSUpdateError: if verification fails. |
| 306 | """ |
| 307 | status = self.check_update_status() |
| 308 | if status != UPDATER_NEED_REBOOT: |
| 309 | raise RootFSUpdateError('Update did not complete with correct ' |
| 310 | 'status. Expecting %s, actual %s' % |
| 311 | (UPDATER_NEED_REBOOT, status)) |
| 312 | |
| 313 | |
| 314 | def rollback_rootfs(self, powerwash): |
| 315 | """Triggers rollback and waits for it to complete. |
| 316 | |
| 317 | @param powerwash: If true, powerwash as part of rollback. |
| 318 | |
| 319 | @raise RootFSUpdateError if anything went wrong. |
| 320 | |
| 321 | """ |
Chris Sosa | c861752 | 2014-06-09 23:22:26 +0000 | [diff] [blame] | 322 | version = self.get_build_id() |
| 323 | # Introduced can_rollback in M36 (build 5772). # etc/lsb-release matches |
| 324 | # X.Y.Z. This version split just pulls the first part out. |
| 325 | try: |
| 326 | build_number = int(version.split('.')[0]) |
| 327 | except ValueError: |
| 328 | logging.error('Could not parse build number.') |
| 329 | build_number = 0 |
| 330 | |
| 331 | if build_number >= 5772: |
| 332 | can_rollback_cmd = '%s --can_rollback' % (UPDATER_BIN) |
| 333 | logging.info('Checking for rollback.') |
| 334 | try: |
| 335 | self._run(can_rollback_cmd) |
| 336 | except error.AutoservRunError as e: |
| 337 | raise RootFSUpdateError("Rollback isn't possible on %s: %s" % |
| 338 | (self.host.hostname, str(e))) |
| 339 | |
| 340 | rollback_cmd = '%s --rollback --follow' % (UPDATER_BIN) |
Chris Sosa | c193217 | 2013-10-16 13:28:53 -0700 | [diff] [blame] | 341 | if not powerwash: |
| 342 | rollback_cmd += ' --nopowerwash' |
| 343 | |
Chris Sosa | c861752 | 2014-06-09 23:22:26 +0000 | [diff] [blame] | 344 | logging.info('Performing rollback.') |
Chris Sosa | c193217 | 2013-10-16 13:28:53 -0700 | [diff] [blame] | 345 | try: |
| 346 | self._run(rollback_cmd) |
Chris Sosa | c193217 | 2013-10-16 13:28:53 -0700 | [diff] [blame] | 347 | except error.AutoservRunError as e: |
| 348 | raise RootFSUpdateError('Rollback failed on %s: %s' % |
| 349 | (self.host.hostname, str(e))) |
| 350 | |
| 351 | self._verify_update_completed() |
| 352 | |
Gilad Arnold | 0ed760c | 2012-11-05 23:42:53 -0800 | [diff] [blame] | 353 | |
Chris Sosa | 2f1ae9f | 2013-08-13 10:00:15 -0700 | [diff] [blame] | 354 | def update_rootfs(self): |
| 355 | """Updates the rootfs partition only.""" |
Chris Sosa | 77556d8 | 2012-04-05 15:23:14 -0700 | [diff] [blame] | 356 | logging.info('Updating root partition...') |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 357 | |
| 358 | # Run update_engine using the specified URL. |
| 359 | try: |
| 360 | autoupdate_cmd = '%s --update --omaha_url=%s 2>&1' % ( |
| 361 | UPDATER_BIN, self.update_url) |
| 362 | self._run(autoupdate_cmd, timeout=900) |
| 363 | except error.AutoservRunError: |
Prashanth B | 32baa9b | 2014-03-13 13:23:01 -0700 | [diff] [blame] | 364 | list_image_dir_contents(self.update_url) |
Chris Sosa | 77556d8 | 2012-04-05 15:23:14 -0700 | [diff] [blame] | 365 | update_error = RootFSUpdateError('update-engine failed on %s' % |
| 366 | self.host.hostname) |
| 367 | self._update_error_queue.put(update_error) |
| 368 | raise update_error |
Simran Basi | 3b858a2 | 2015-03-17 16:23:24 -0700 | [diff] [blame^] | 369 | except Exception as e: |
| 370 | # Don't allow other exceptions to not be caught. |
| 371 | self._update_error_queue.put(e) |
| 372 | raise e |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 373 | |
Chris Sosa | c193217 | 2013-10-16 13:28:53 -0700 | [diff] [blame] | 374 | try: |
| 375 | self._verify_update_completed() |
| 376 | except RootFSUpdateError as e: |
| 377 | self._update_error_queue.put(e) |
| 378 | raise |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 379 | |
| 380 | |
Chris Sosa | 7231260 | 2013-04-16 15:01:56 -0700 | [diff] [blame] | 381 | def update_stateful(self, clobber=True): |
| 382 | """Updates the stateful partition. |
| 383 | |
| 384 | @param clobber: If True, a clean stateful installation. |
| 385 | """ |
Chris Sosa | 77556d8 | 2012-04-05 15:23:14 -0700 | [diff] [blame] | 386 | logging.info('Updating stateful partition...') |
joychen | 03eaad9 | 2013-06-26 09:55:21 -0700 | [diff] [blame] | 387 | statefuldev_url = self.update_url.replace('update', |
| 388 | 'static') |
Chris Sosa | a3ac215 | 2012-05-23 22:23:13 -0700 | [diff] [blame] | 389 | |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 390 | # Attempt stateful partition update; this must succeed so that the newly |
| 391 | # installed host is testable after update. |
Chris Sosa | 7231260 | 2013-04-16 15:01:56 -0700 | [diff] [blame] | 392 | statefuldev_cmd = [self.get_stateful_update_script(), statefuldev_url] |
| 393 | if clobber: |
| 394 | statefuldev_cmd.append('--stateful_change=clean') |
| 395 | |
| 396 | statefuldev_cmd.append('2>&1') |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 397 | try: |
| 398 | self._run(' '.join(statefuldev_cmd), timeout=600) |
| 399 | except error.AutoservRunError: |
Chris Sosa | 77556d8 | 2012-04-05 15:23:14 -0700 | [diff] [blame] | 400 | update_error = StatefulUpdateError('stateful_update failed on %s' % |
| 401 | self.host.hostname) |
| 402 | self._update_error_queue.put(update_error) |
| 403 | raise update_error |
Simran Basi | 3b858a2 | 2015-03-17 16:23:24 -0700 | [diff] [blame^] | 404 | except Exception as e: |
| 405 | # Don't allow other exceptions to not be caught. |
| 406 | self._update_error_queue.put(e) |
| 407 | raise e |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 408 | |
| 409 | |
Dan Shi | 0f466e8 | 2013-02-22 15:44:58 -0800 | [diff] [blame] | 410 | def run_update(self, force_update, update_root=True): |
| 411 | """Update the DUT with image of specific version. |
Chris Sosa | a3ac215 | 2012-05-23 22:23:13 -0700 | [diff] [blame] | 412 | |
Dan Shi | 0f466e8 | 2013-02-22 15:44:58 -0800 | [diff] [blame] | 413 | @param force_update: True to update DUT even if it's running the same |
| 414 | version already. |
| 415 | @param update_root: True to force a kernel update. If it's False and |
| 416 | force_update is True, stateful update will be used to clean up |
| 417 | the DUT. |
| 418 | |
| 419 | """ |
| 420 | booted_version = self.get_build_id() |
| 421 | if (self.check_version() and not force_update): |
Dale Curtis | a94c19c | 2011-05-02 15:05:17 -0700 | [diff] [blame] | 422 | logging.info('System is already up to date. Skipping update.') |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 423 | return False |
| 424 | |
Chris Sosa | a3ac215 | 2012-05-23 22:23:13 -0700 | [diff] [blame] | 425 | if self.update_version: |
| 426 | logging.info('Updating from version %s to %s.', |
| 427 | booted_version, self.update_version) |
Dale Curtis | 53d5586 | 2011-05-16 12:17:59 -0700 | [diff] [blame] | 428 | |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 429 | # Check that Dev Server is accepting connections (from autoserv's host). |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 430 | # If we can't talk to it, the machine host probably can't either. |
| 431 | auserver_host = urlparse.urlparse(self.update_url)[1] |
| 432 | try: |
| 433 | httplib.HTTPConnection(auserver_host).connect() |
Dale Curtis | 5c32c72 | 2011-05-04 19:24:23 -0700 | [diff] [blame] | 434 | except IOError: |
| 435 | raise ChromiumOSError( |
| 436 | 'Update server at %s not available' % auserver_host) |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 437 | |
Chris Sosa | a3ac215 | 2012-05-23 22:23:13 -0700 | [diff] [blame] | 438 | logging.info('Installing from %s to %s', self.update_url, |
Chris Sosa | 77556d8 | 2012-04-05 15:23:14 -0700 | [diff] [blame] | 439 | self.host.hostname) |
| 440 | |
Chris Sosa | 5e4246b | 2012-05-22 18:05:22 -0700 | [diff] [blame] | 441 | # Reset update state. |
Chris Sosa | 77556d8 | 2012-04-05 15:23:14 -0700 | [diff] [blame] | 442 | self.reset_update_engine() |
Chris Sosa | 5e4246b | 2012-05-22 18:05:22 -0700 | [diff] [blame] | 443 | self.reset_stateful_partition() |
Sean O | c053dfe | 2010-08-23 18:22:26 +0200 | [diff] [blame] | 444 | |
Dale Curtis | 1e97318 | 2011-07-12 18:21:36 -0700 | [diff] [blame] | 445 | try: |
Chris Sosa | 77556d8 | 2012-04-05 15:23:14 -0700 | [diff] [blame] | 446 | updaters = [ |
Chris Sosa | 2f1ae9f | 2013-08-13 10:00:15 -0700 | [diff] [blame] | 447 | multiprocessing.process.Process(target=self.update_rootfs), |
Chris Sosa | 7231260 | 2013-04-16 15:01:56 -0700 | [diff] [blame] | 448 | multiprocessing.process.Process(target=self.update_stateful) |
Chris Sosa | 77556d8 | 2012-04-05 15:23:14 -0700 | [diff] [blame] | 449 | ] |
Dan Shi | 0f466e8 | 2013-02-22 15:44:58 -0800 | [diff] [blame] | 450 | if not update_root: |
| 451 | logging.info('Root update is skipped.') |
| 452 | updaters = updaters[1:] |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 453 | |
Chris Sosa | 77556d8 | 2012-04-05 15:23:14 -0700 | [diff] [blame] | 454 | # Run the updaters in parallel. |
| 455 | for updater in updaters: updater.start() |
| 456 | for updater in updaters: updater.join() |
| 457 | |
| 458 | # Re-raise the first error that occurred. |
| 459 | if not self._update_error_queue.empty(): |
| 460 | update_error = self._update_error_queue.get() |
| 461 | self.revert_boot_partition() |
Chris Sosa | 5e4246b | 2012-05-22 18:05:22 -0700 | [diff] [blame] | 462 | self.reset_stateful_partition() |
Chris Sosa | 77556d8 | 2012-04-05 15:23:14 -0700 | [diff] [blame] | 463 | raise update_error |
Sean O | c053dfe | 2010-08-23 18:22:26 +0200 | [diff] [blame] | 464 | |
Dale Curtis | 1e97318 | 2011-07-12 18:21:36 -0700 | [diff] [blame] | 465 | logging.info('Update complete.') |
| 466 | return True |
| 467 | except: |
| 468 | # Collect update engine logs in the event of failure. |
| 469 | if self.host.job: |
| 470 | logging.info('Collecting update engine logs...') |
| 471 | self.host.get_file( |
| 472 | UPDATER_LOGS, self.host.job.sysinfo.sysinfodir, |
| 473 | preserve_perm=False) |
Prashanth B | 32baa9b | 2014-03-13 13:23:01 -0700 | [diff] [blame] | 474 | list_image_dir_contents(self.update_url) |
Dale Curtis | 1e97318 | 2011-07-12 18:21:36 -0700 | [diff] [blame] | 475 | raise |
Dan Shi | 10e992b | 2013-08-30 11:02:59 -0700 | [diff] [blame] | 476 | finally: |
| 477 | self.host.show_update_engine_log() |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 478 | |
| 479 | |
Dale Curtis | a94c19c | 2011-05-02 15:05:17 -0700 | [diff] [blame] | 480 | def check_version(self): |
Dan Shi | 0f466e8 | 2013-02-22 15:44:58 -0800 | [diff] [blame] | 481 | """Check the image running in DUT has the desired version. |
| 482 | |
| 483 | @returns: True if the DUT's image version matches the version that |
| 484 | the autoupdater tries to update to. |
| 485 | |
| 486 | """ |
Dale Curtis | f57a25f | 2011-05-24 14:40:55 -0700 | [diff] [blame] | 487 | booted_version = self.get_build_id() |
Dan Shi | b95bb86 | 2013-03-22 16:29:28 -0700 | [diff] [blame] | 488 | return (self.update_version and |
| 489 | self.update_version.endswith(booted_version)) |
| 490 | |
| 491 | |
| 492 | def check_version_to_confirm_install(self): |
| 493 | """Check image running in DUT has the desired version to be installed. |
| 494 | |
| 495 | The method should not be used to check if DUT needs to have a full |
| 496 | reimage. Only use it to confirm a image is installed. |
| 497 | |
Dan Shi | 190c780 | 2013-04-04 13:05:30 -0700 | [diff] [blame] | 498 | The method is designed to verify version for following 4 scenarios with |
| 499 | samples of version to update to and expected booted version: |
| 500 | 1. trybot paladin build. |
| 501 | update version: trybot-lumpy-paladin/R27-3837.0.0-b123 |
| 502 | booted version: 3837.0.2013_03_21_1340 |
| 503 | |
| 504 | 2. trybot release build. |
| 505 | update version: trybot-lumpy-release/R27-3837.0.0-b456 |
| 506 | booted version: 3837.0.0 |
| 507 | |
| 508 | 3. buildbot official release build. |
| 509 | update version: lumpy-release/R27-3837.0.0 |
| 510 | booted version: 3837.0.0 |
| 511 | |
| 512 | 4. non-official paladin rc build. |
| 513 | update version: lumpy-paladin/R27-3878.0.0-rc7 |
| 514 | booted version: 3837.0.0-rc7 |
Dan Shi | b95bb86 | 2013-03-22 16:29:28 -0700 | [diff] [blame] | 515 | |
Dan Shi | 7f79551 | 2013-04-12 10:08:17 -0700 | [diff] [blame] | 516 | 5. chrome-perf build. |
| 517 | update version: lumpy-chrome-perf/R28-3837.0.0-b2996 |
| 518 | booted version: 3837.0.0 |
| 519 | |
Dan Shi | 73aa290 | 2013-05-03 11:22:11 -0700 | [diff] [blame] | 520 | 6. pgo-generate build. |
| 521 | update version: lumpy-release-pgo-generate/R28-3837.0.0-b2996 |
| 522 | booted version: 3837.0.0-pgo-generate |
| 523 | |
Dan Shi | b95bb86 | 2013-03-22 16:29:28 -0700 | [diff] [blame] | 524 | When we are checking if a DUT needs to do a full install, we should NOT |
| 525 | use this method to check if the DUT is running the same version, since |
Dan Shi | 190c780 | 2013-04-04 13:05:30 -0700 | [diff] [blame] | 526 | it may return false positive for a DUT running trybot paladin build to |
| 527 | be updated to another trybot paladin build. |
Dan Shi | b95bb86 | 2013-03-22 16:29:28 -0700 | [diff] [blame] | 528 | |
Dan Shi | 190c780 | 2013-04-04 13:05:30 -0700 | [diff] [blame] | 529 | TODO: This logic has a bug if a trybot paladin build failed to be |
| 530 | installed in a DUT running an older trybot paladin build with same |
| 531 | platform number, but different build number (-b###). So to conclusively |
| 532 | determine if a tryjob paladin build is imaged successfully, we may need |
| 533 | to find out the date string from update url. |
Dan Shi | b95bb86 | 2013-03-22 16:29:28 -0700 | [diff] [blame] | 534 | |
| 535 | @returns: True if the DUT's image version (without the date string if |
| 536 | the image is a trybot build), matches the version that the |
| 537 | autoupdater is trying to update to. |
| 538 | |
| 539 | """ |
J. Richard Barnette | ec1de42 | 2013-06-26 15:44:07 -0700 | [diff] [blame] | 540 | # In the local_devserver case, we can't know the expected |
| 541 | # build, so just pass. |
| 542 | if not self.update_version: |
| 543 | return True |
| 544 | |
Dan Shi | b95bb86 | 2013-03-22 16:29:28 -0700 | [diff] [blame] | 545 | # Always try the default check_version method first, this prevents |
| 546 | # any backward compatibility issue. |
| 547 | if self.check_version(): |
| 548 | return True |
| 549 | |
Dan Shi | 190c780 | 2013-04-04 13:05:30 -0700 | [diff] [blame] | 550 | # Remove R#- and -b# at the end of build version |
| 551 | stripped_version = re.sub(r'(R\d+-|-b\d+)', '', self.update_version) |
| 552 | |
Dan Shi | b95bb86 | 2013-03-22 16:29:28 -0700 | [diff] [blame] | 553 | booted_version = self.get_build_id() |
Dan Shi | 190c780 | 2013-04-04 13:05:30 -0700 | [diff] [blame] | 554 | |
Dan Shi | 7f79551 | 2013-04-12 10:08:17 -0700 | [diff] [blame] | 555 | is_trybot_paladin_build = re.match(r'.+trybot-.+-paladin', |
| 556 | self.update_url) |
Dan Shi | 190c780 | 2013-04-04 13:05:30 -0700 | [diff] [blame] | 557 | |
Dan Shi | 7f79551 | 2013-04-12 10:08:17 -0700 | [diff] [blame] | 558 | # Replace date string with 0 in booted_version |
| 559 | booted_version_no_date = re.sub(r'\d{4}_\d{2}_\d{2}_\d+', '0', |
| 560 | booted_version) |
| 561 | has_date_string = booted_version != booted_version_no_date |
| 562 | |
Dan Shi | 73aa290 | 2013-05-03 11:22:11 -0700 | [diff] [blame] | 563 | is_pgo_generate_build = re.match(r'.+-pgo-generate', |
| 564 | self.update_url) |
| 565 | |
| 566 | # Remove |-pgo-generate| in booted_version |
| 567 | booted_version_no_pgo = booted_version.replace('-pgo-generate', '') |
| 568 | has_pgo_generate = booted_version != booted_version_no_pgo |
| 569 | |
Dan Shi | 7f79551 | 2013-04-12 10:08:17 -0700 | [diff] [blame] | 570 | if is_trybot_paladin_build: |
| 571 | if not has_date_string: |
| 572 | logging.error('A trybot paladin build is expected. Version ' + |
| 573 | '"%s" is not a paladin build.', booted_version) |
Dan Shi | 190c780 | 2013-04-04 13:05:30 -0700 | [diff] [blame] | 574 | return False |
| 575 | return stripped_version == booted_version_no_date |
Dan Shi | 73aa290 | 2013-05-03 11:22:11 -0700 | [diff] [blame] | 576 | elif is_pgo_generate_build: |
| 577 | if not has_pgo_generate: |
| 578 | logging.error('A pgo-generate build is expected. Version ' + |
| 579 | '"%s" is not a pgo-generate build.', |
| 580 | booted_version) |
| 581 | return False |
| 582 | return stripped_version == booted_version_no_pgo |
Dan Shi | 7f79551 | 2013-04-12 10:08:17 -0700 | [diff] [blame] | 583 | else: |
| 584 | if has_date_string: |
| 585 | logging.error('Unexpected date found in a non trybot paladin' + |
| 586 | ' build.') |
| 587 | return False |
| 588 | # Versioned build, i.e., rc or release build. |
| 589 | return stripped_version == booted_version |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 590 | |
Sean O | c053dfe | 2010-08-23 18:22:26 +0200 | [diff] [blame] | 591 | |
Sean O'Connor | 5346e4e | 2010-08-12 18:49:24 +0200 | [diff] [blame] | 592 | def get_build_id(self): |
Dale Curtis | 793f912 | 2011-02-04 15:00:52 -0800 | [diff] [blame] | 593 | """Pulls the CHROMEOS_RELEASE_VERSION string from /etc/lsb-release.""" |
| 594 | return self._run('grep CHROMEOS_RELEASE_VERSION' |
| 595 | ' /etc/lsb-release').stdout.split('=')[1].strip() |
Chris Sosa | 6542508 | 2013-10-16 13:26:22 -0700 | [diff] [blame] | 596 | |
| 597 | |
| 598 | def verify_boot_expectations(self, expected_kernel_state, rollback_message): |
| 599 | """Verifies that we fully booted given expected kernel state. |
| 600 | |
| 601 | This method both verifies that we booted using the correct kernel |
| 602 | state and that the OS has marked the kernel as good. |
| 603 | |
| 604 | @param expected_kernel_state: kernel state that we are verifying with |
| 605 | i.e. I expect to be booted onto partition 4 etc. See output of |
| 606 | get_kernel_state. |
| 607 | @param rollback_message: string to raise as a ChromiumOSError |
| 608 | if we booted with the wrong partition. |
| 609 | |
| 610 | @raises ChromiumOSError: If we didn't. |
| 611 | """ |
| 612 | # Figure out the newly active kernel. |
| 613 | active_kernel_state = self.get_kernel_state()[0] |
| 614 | |
| 615 | # Check for rollback due to a bad build. |
| 616 | if (expected_kernel_state and |
| 617 | active_kernel_state != expected_kernel_state): |
Don Garrett | 56b1cc8 | 2013-12-06 17:49:20 -0800 | [diff] [blame] | 618 | |
| 619 | # Kernel crash reports should be wiped between test runs, but |
| 620 | # may persist from earlier parts of the test, or from problems |
| 621 | # with provisioning. |
| 622 | # |
| 623 | # Kernel crash reports will NOT be present if the crash happened |
| 624 | # before encrypted stateful is mounted. |
| 625 | # |
| 626 | # TODO(dgarrett): Integrate with server/crashcollect.py at some |
| 627 | # point. |
| 628 | kernel_crashes = glob.glob('/var/spool/crash/kernel.*.kcrash') |
| 629 | if kernel_crashes: |
| 630 | rollback_message += ': kernel_crash' |
| 631 | logging.debug('Found %d kernel crash reports:', |
| 632 | len(kernel_crashes)) |
| 633 | # The crash names contain timestamps that may be useful: |
| 634 | # kernel.20131207.005945.0.kcrash |
| 635 | for crash in kernel_crashes: |
| 636 | logging.debug(' %s', os.path.basename(crash)) |
| 637 | |
Chris Sosa | 6542508 | 2013-10-16 13:26:22 -0700 | [diff] [blame] | 638 | # Print out some information to make it easier to debug |
| 639 | # the rollback. |
| 640 | logging.debug('Dumping partition table.') |
| 641 | self._run('cgpt show $(rootdev -s -d)') |
| 642 | logging.debug('Dumping crossystem for firmware debugging.') |
| 643 | self._run('crossystem --all') |
| 644 | raise ChromiumOSError(rollback_message) |
| 645 | |
| 646 | # Make sure chromeos-setgoodkernel runs. |
| 647 | try: |
| 648 | utils.poll_for_condition( |
| 649 | lambda: (self.get_kernel_tries(active_kernel_state) == 0 |
| 650 | and self.get_kernel_success(active_kernel_state)), |
| 651 | exception=ChromiumOSError(), |
| 652 | timeout=self.KERNEL_UPDATE_TIMEOUT, sleep_interval=5) |
| 653 | except ChromiumOSError: |
| 654 | services_status = self._run('status system-services').stdout |
| 655 | if services_status != 'system-services start/running\n': |
| 656 | event = ('Chrome failed to reach login screen') |
| 657 | else: |
| 658 | event = ('update-engine failed to call ' |
| 659 | 'chromeos-setgoodkernel') |
| 660 | raise ChromiumOSError( |
| 661 | 'After update and reboot, %s ' |
| 662 | 'within %d seconds' % (event, |
| 663 | self.KERNEL_UPDATE_TIMEOUT)) |