New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 755078 link

Starred by 1 user
Status: Fixed
Owner:
█ yhong@chromium.org
Last visit > 30 days ago
Closed: Sep 7
Cc:
marcochen@chromium.org
chromeos-factory-eng@google.com
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

factory: Goofy should load model specific test list by default.

Project Member Reported by hungte@chromium.org, Aug 14 2017 
Part of coral work.

Currently Goofy tries to load 'main' if no active test list found.
I think we should change it to load in following order:
 main_$(mosys platform model)
 main
 generic_main
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 17 2017 

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/overlays/overlay-coral-private/+/ac8640e8fdb304ce17db2660bb3b3391d0996d05

commit ac8640e8fdb304ce17db2660bb3b3391d0996d05
Author: Hung-Te Lin <hungte@chromium.org>
Date: Thu Aug 17 03:37:05 2017

Comment 2 by yhong@chromium.org, Aug 25 2017

Status: Started (was: Untriaged)

Comment 3 by hungte@chromium.org, Aug 25 2017 

Let's postpone this one.

I'd like to see how the selection script for coral works, since we are not sure if in the future, there will be more projects that need this or not.

So let's try to make things less complicated.
Project Member

Comment 4 by bugdroid1@chromium.org, Aug 25 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/factory/+/65b3cfe04c2bd218548253c8b0d8167101f01896

commit 65b3cfe04c2bd218548253c8b0d8167101f01896
Author: Yong Hong <yhong@chromium.org>
Date: Fri Aug 25 08:19:13 2017

hwid: Replace the method of obtaining project name.

This change replaces the source of project name from the result
of command `mosys platform chassis` by the result of command
`mosys platform model`.

BUG= chromium:755078 
TEST=maually test

Change-Id: Ie3440b4fe18234abd7f0ba419f1671eebbdea723
Reviewed-on: https://chromium-review.googlesource.com/616411
Commit-Ready: Yong Hong <yhong@chromium.org>
Tested-by: Yong Hong <yhong@chromium.org>
Reviewed-by: Hung-Te Lin <hungte@chromium.org>

[modify] https://crrev.com/65b3cfe04c2bd218548253c8b0d8167101f01896/py/hwid/v3/common.py

Comment 5 by hungte@chromium.org, Sep 7

Status: Fixed (was: Started)
I believe we're already loading right test list right?
►

Sign in to add a comment