Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(479)

Issue 6878082: Skip software verify check. (Closed)

Created:
9 years, 8 months ago by DaleCurtis
Modified:
9 years, 7 months ago
Reviewers:
pauldean, pauldean_chromium, ericli
CC:
chromium-os-reviews_chromium.org, sosa+cc_chromium.org, seano+cc_chromium.org, ericli
Visibility:
Public.

Description

The normal verify_software() functionality in AbstractSSH requires the Autotest directory exists and is writable. Upon loss of the stateful partition, neither of these checks will pass. It's enough to ensure that the stateful partition exists and has space for the stateful and rootfs updates since we're going to blow away the stateful partition anyways. Change-Id: I91629120f6a353904dc479076a5e6b354ce52c45 BUG=none TEST=Ran on cautotest. Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=2468a32

Patch Set 1 #

Patch Set 2 : Revert #

Patch Set 3 : New version. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+28 lines, -2 lines) Patch
M server/hosts/chromiumos_host.py View 1 2 3 chunks +28 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
DaleCurtis
9 years, 8 months ago (2011-04-21 01:33:41 UTC) #1
pauldean_chromium
9 years, 8 months ago (2011-04-21 14:21:42 UTC) #2
On 2011/04/21 01:33:41, dalec wrote:

LGTM.

Powered by Google App Engine
This is Rietveld 408576698