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

Issue 833521 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Whining does not update on moblab

Project Member Reported by mattmallett@chromium.org, Apr 16 2018

Issue description

Tests are not appearing on whining in moblab build R65-10323.71.0. I ran arm.CtsAnimationTestCases which completed with no errors, and did not see it on whining.

Check whining 'tests' link and it gives the error
"""
Chromium OS Test Results (Whining) - Errors Encountered!
Looks like the DB was not updated for at least 7 days
"""

Looking for a root cause, it appears that moblab-whining-update-db is failing with error 
"""
Updating wmatrix database (Mon Apr 16 11:08:47 PDT 2018).
Traceback (most recent call last):
  File "/whining/src/backend/update_rawdb.py", line 14, in <module>
    import whining
ImportError: No module named whining
Traceback (most recent call last):
  File "/whining/src/backend/update_wmdb.py", line 13, in <module>
    import whining
ImportError: No module named whining
"""

Both update tasks are failing because of bad python paths.
 
Status: Started (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 18 2018

Labels: merge-merged-release-R65-10323.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/whining/+/0bf9ccaabac1b38c030ef8374bce05241fbe6e27

commit 0bf9ccaabac1b38c030ef8374bce05241fbe6e27
Author: Matt Mallett <mattmallett@chromium.org>
Date: Wed Apr 18 16:34:24 2018

[moblab] fix wmatrix not updating due to bad imports

wmatrix update scripts try to import whining, but that
package does not exist, and is not used anywhere in
any of the update scripts that imported it. Removing
it and wmatrix updater runs without errors, tests
show up shortly after completion as expected.

BUG= chromium:833521 
TEST=Build and test on local device that recently run
tests appear on wmatrix.

Change-Id: I77d1e8f1b96d693af2d54757b2c7f587274d7396

[modify] https://crrev.com/0bf9ccaabac1b38c030ef8374bce05241fbe6e27/src/backend/update_wmdb.py
[modify] https://crrev.com/0bf9ccaabac1b38c030ef8374bce05241fbe6e27/src/backend/update_rawdb.py
[modify] https://crrev.com/0bf9ccaabac1b38c030ef8374bce05241fbe6e27/src/backend/update_lib.py

Project Member

Comment 3 by bugdroid1@chromium.org, Apr 19 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/whining/+/c325abd3eff318af433a1a5667f5d2745f893945

commit c325abd3eff318af433a1a5667f5d2745f893945
Author: Matt Mallett <mattmallett@chromium.org>
Date: Thu Apr 19 19:46:09 2018

[moblab] fix wmatrix not updating due to bad imports

wmatrix update scripts try to import whining, but that
package does not exist, and is not used anywhere in
any of the update scripts that imported it. Removing
it and wmatrix updater runs without errors, tests
show up shortly after completion as expected.

BUG= chromium:833521 
TEST=Build and test on local device that recently run
tests appear on wmatrix.

Change-Id: I77d1e8f1b96d693af2d54757b2c7f587274d7396
Reviewed-on: https://chromium-review.googlesource.com/1015765
Commit-Ready: Matt Mallett <mattmallett@chromium.org>
Tested-by: Matt Mallett <mattmallett@chromium.org>
Reviewed-by: Jacob Kopczynski <jkop@chromium.org>

[modify] https://crrev.com/c325abd3eff318af433a1a5667f5d2745f893945/src/backend/update_wmdb.py
[modify] https://crrev.com/c325abd3eff318af433a1a5667f5d2745f893945/src/backend/update_rawdb.py
[modify] https://crrev.com/c325abd3eff318af433a1a5667f5d2745f893945/src/backend/update_lib.py

Project Member

Comment 4 by bugdroid1@chromium.org, Apr 24 2018

Labels: merge-merged-release-R66-10452.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/whining/+/519cda0d1b1891d2431cd2e68128d347be792c16

commit 519cda0d1b1891d2431cd2e68128d347be792c16
Author: Matt Mallett <mattmallett@chromium.org>
Date: Wed Apr 18 16:35:31 2018

[moblab] fix wmatrix not updating due to bad imports

wmatrix update scripts try to import whining, but that
package does not exist, and is not used anywhere in
any of the update scripts that imported it. Removing
it and wmatrix updater runs without errors, tests
show up shortly after completion as expected.

BUG= chromium:833521 
TEST=Build and test on local device that recently run
tests appear on wmatrix.

Change-Id: I77d1e8f1b96d693af2d54757b2c7f587274d7396

[modify] https://crrev.com/519cda0d1b1891d2431cd2e68128d347be792c16/src/backend/update_wmdb.py
[modify] https://crrev.com/519cda0d1b1891d2431cd2e68128d347be792c16/src/backend/update_rawdb.py
[modify] https://crrev.com/519cda0d1b1891d2431cd2e68128d347be792c16/src/backend/update_lib.py

Project Member

Comment 5 by bugdroid1@chromium.org, Apr 24 2018

Labels: merge-merged-release-R67-10575.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/whining/+/2d113e67d42b4af6b8fa2d1c8d343517b6435ebe

commit 2d113e67d42b4af6b8fa2d1c8d343517b6435ebe
Author: Matt Mallett <mattmallett@chromium.org>
Date: Wed Apr 18 16:36:01 2018

[moblab] fix wmatrix not updating due to bad imports

wmatrix update scripts try to import whining, but that
package does not exist, and is not used anywhere in
any of the update scripts that imported it. Removing
it and wmatrix updater runs without errors, tests
show up shortly after completion as expected.

BUG= chromium:833521 
TEST=Build and test on local device that recently run
tests appear on wmatrix.

Change-Id: I77d1e8f1b96d693af2d54757b2c7f587274d7396

[modify] https://crrev.com/2d113e67d42b4af6b8fa2d1c8d343517b6435ebe/src/backend/update_wmdb.py
[modify] https://crrev.com/2d113e67d42b4af6b8fa2d1c8d343517b6435ebe/src/backend/update_rawdb.py
[modify] https://crrev.com/2d113e67d42b4af6b8fa2d1c8d343517b6435ebe/src/backend/update_lib.py

Status: Fixed (was: Started)

Sign in to add a comment