New issue
Advanced search Search tips

Issue 726778 link

Starred by 2 users
Status: Verified
Owner:
cmt...@chromium.org
Closed: May 2017
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

NIghtly lumpy test trying to use Android manifest and failing.

Project Member Reported by cmt...@chromium.org, May 26 2017 

Our lumpy nightly tests are failing with increasing frequency, trying to use a manifest that does not exist in ChromeOS.  On investigation, it looked like the cros_utils/manifest_versions.py:TimeToVersion is using the soft link in LKGM/lkgm.xml to decide which manifest to download, and that version is frequently pointing to something in android-LKGM-candidates, so we are trying to find a manifest version that did not actually exist in the ChromeOS paladins tree.   This needs to be fixed.
Project Member

Comment 1 by bugdroid1@chromium.org, May 31 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/toolchain-utils/+/8ab698a91e101c218e58a67d810d1ae0691d7b29

commit 8ab698a91e101c218e58a67d810d1ae0691d7b29
Author: Caroline Tice <cmtice@google.com>
Date: Wed May 31 21:15:47 2017

Verify manifest is in paladin before trying to use it.

With increasing frequency the lumpy nightly test fails very early
because it ends up trying to get a manifest that only exists in
Android, not in ChromeOS.  This CL adds code to verify the manifest
version actually exists in the paladins list before trying to use it.

BUG= chromium:726778 
TEST=Tested this code in the nightly build tests and it worked.

Change-Id: I86f8eb9c8e28fd4a404bc63d49d8d9fb27de8b3b
Reviewed-on: https://chromium-review.googlesource.com/517266
Commit-Ready: Caroline Tice <cmtice@chromium.org>
Tested-by: Caroline Tice <cmtice@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>

[modify] https://crrev.com/8ab698a91e101c218e58a67d810d1ae0691d7b29/setup_chromeos.py
[modify] https://crrev.com/8ab698a91e101c218e58a67d810d1ae0691d7b29/cros_utils/manifest_versions.py

Comment 2 by cmt...@chromium.org, May 31 2017 

 Issue 715849  has been merged into this issue.

Comment 3 by cmt...@chromium.org, May 31 2017

Status: Fixed (was: Assigned)

Comment 4 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 5 by abod...@chromium.org, Aug 3 2017

Status: Verified (was: Fixed)
Closing. Please reopen it if its not fixed. Thanks!
►

Sign in to add a comment