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

Issue 658794 link

Starred by 2 users

Issue metadata

Status: Archived
Owner:
Last visit > 30 days ago
Closed: Oct 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Moblab on 53 stable channel doesnt show wmatrix link

Project Member Reported by krk@chromium.org, Oct 24 2016

Issue description

Moblab is at http://100.96.57.118/afe/#tab_id=job_list&state_filter=all&type_filter=all
and runs 8530.95.0. This version should have the wmatrix link show up in AFE via CL https://chromium-review.googlesource.com/#/c/346496/

This seems limited to the moblab mentioned above. I tested it on my moblab at 100.96.57.46 and it shows up for me. My moblab runs the same version as above.
 

Comment 1 by ntang@google.com, Oct 24 2016

there is a "wmatrix_url=/wmatrix" missing in the shadow configuration file. For people who upgrade to M53, may need to add this to their /usr/local/autotest/shadow.ini manually. Clean install is not a problem.

I will add a fix in M54.

Comment 2 by jean@chromium.org, Oct 24 2016

so this is caused by my installed M52 and then AU to M53? I just checked my moblab in PEng lab and wmatrix was correctly shown, but i think we always do remote cros flash for milestone release testing instead of AU. http://100.96.48.80/afe/#tab_id=job_list&state_filter=all&type_filter=all

Comment 3 by krk@chromium.org, Oct 25 2016

Labels: -Pri-3 M-54 Pri-1
Upping priority since this is needed for 54 and I am planning a stable push next week.

Please cherrypick the fix into 55 as well.

Comment 4 by krk@chromium.org, Oct 25 2016

Status: Assigned (was: Untriaged)

Comment 5 by ntang@google.com, Oct 26 2016

Labels: Merge-Request-54
Project Member

Comment 6 by bugdroid1@chromium.org, Oct 26 2016

Labels: merge-merged-release-R54-8743.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/412e5bcf93f82f8f0e293761c6ad081d72214154

commit 412e5bcf93f82f8f0e293761c6ad081d72214154
Author: Michael Tang <ntang@google.com>
Date: Wed Oct 26 17:15:34 2016

Fixed the wmatrix setting in moblab config.

BUG= chromium:658794 
TEST=locally.

Change-Id: I5d10aa099312700adb04761add39a5317e0d75ec
Reviewed-on: https://chromium-review.googlesource.com/403509
Reviewed-by: Mohan Konduri <krk@chromium.org>
Reviewed-by: Michael Tang <ntang@chromium.org>
Commit-Queue: Michael Tang <ntang@chromium.org>
Tested-by: Michael Tang <ntang@chromium.org>
Trybot-Ready: Michael Tang <ntang@chromium.org>

[modify] https://crrev.com/412e5bcf93f82f8f0e293761c6ad081d72214154/moblab_config.ini

Comment 7 by krk@chromium.org, Oct 26 2016

Labels: -Merge-Request-54 Merge-Approved-54

Comment 8 by ntang@google.com, Oct 26 2016

Labels: Merge-Request-55

Comment 9 by dimu@chromium.org, Oct 27 2016

Labels: -Merge-Request-55 Merge-Approved-55 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M55 (branch: 2883)

Comment 10 by krk@chromium.org, Oct 27 2016

Reminder: This needs to be cherrypicked into 55 and into master.
Project Member

Comment 11 by bugdroid1@chromium.org, Oct 28 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/9833adb40279863ef14a5836f5e423969c40211c

commit 9833adb40279863ef14a5836f5e423969c40211c
Author: Michael Tang <ntang@google.com>
Date: Wed Oct 26 17:15:34 2016

Fixed the wmatrix setting in moblab config.

BUG= chromium:658794 
TEST=locally.

Change-Id: I5d10aa099312700adb04761add39a5317e0d75ec
Reviewed-on: https://chromium-review.googlesource.com/403610
Commit-Ready: Michael Tang <ntang@chromium.org>
Tested-by: Michael Tang <ntang@chromium.org>
Reviewed-by: Michael Tang <ntang@chromium.org>
Reviewed-by: Mohan Konduri <krk@chromium.org>

[modify] https://crrev.com/9833adb40279863ef14a5836f5e423969c40211c/moblab_config.ini

Project Member

Comment 12 by bugdroid1@chromium.org, Oct 28 2016

Labels: merge-merged-release-R55-8872.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/e022f7ad3c5e748e3a8fb7e569db4f1db327640e

commit e022f7ad3c5e748e3a8fb7e569db4f1db327640e
Author: Michael Tang <ntang@google.com>
Date: Wed Oct 26 17:15:34 2016

Fixed the wmatrix setting in moblab config.

BUG= chromium:658794 
TEST=locally.

Change-Id: I5d10aa099312700adb04761add39a5317e0d75ec
Reviewed-on: https://chromium-review.googlesource.com/403512
Reviewed-by: Michael Tang <ntang@chromium.org>
Reviewed-by: Mohan Konduri <krk@chromium.org>
Commit-Queue: Michael Tang <ntang@chromium.org>
Tested-by: Michael Tang <ntang@chromium.org>
Trybot-Ready: Michael Tang <ntang@chromium.org>

[modify] https://crrev.com/e022f7ad3c5e748e3a8fb7e569db4f1db327640e/moblab_config.ini

Project Member

Comment 13 by sheriffbot@chromium.org, Oct 30 2016

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 14 by krk@chromium.org, Oct 30 2016

Labels: -Hotlist-Merge-Approved -Merge-Approved-54 -merge-merged-release-R54-8743.B -Merge-Approved-55 -merge-merged-release-R55-8872.B

Comment 15 by ntang@google.com, Oct 31 2016

Status: Fixed (was: Assigned)

Comment 16 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 17 by dchan@google.com, May 30 2017

Labels: VerifyIn-60
Labels: VerifyIn-61

Comment 19 by dchan@chromium.org, Oct 14 2017

Status: Archived (was: Fixed)

Sign in to add a comment