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

Issue 761720 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

"ImportError: No module named psutil" due to vpython rollout

Project Member Reported by oprypin@chromium.org, Sep 4 2017

Issue description

Hi,

Previously I reported such a bug for LUCI emulation mode: https://bugs.chromium.org/p/chromium/issues/detail?id=759571

However, now we are getting the same error in our normal tryjobs, effectively blocking commit queue.

https://build.chromium.org/p/tryserver.webrtc/builders/android_rel/builds/27452/steps/android_junit_tests
Traceback (most recent call last):
  File "/b/c/b/android_rel/src/build/android/test_runner.py", line 40, in <module>
    from pylib.base import test_run_factory
  File "/b/c/b/android_rel/src/build/android/pylib/base/test_run_factory.py", line 11, in <module>
    from pylib.local.device import local_device_gtest_run
  File "/b/c/b/android_rel/src/build/android/pylib/local/device/local_device_gtest_run.py", line 21, in <module>
    from pylib.local import local_test_server_spawner
  File "/b/c/b/android_rel/src/build/android/pylib/local/local_test_server_spawner.py", line 8, in <module>
    from devil.android import forwarder
  File "/b/c/b/android_rel/src/third_party/catapult/devil/devil/android/forwarder.py", line 10, in <module>
    import psutil

 
kjellander@ found this 
https://chromium.googlesource.com/chromium/tools/build/+/b4b5fdc638270e08fc8131b878f28a55b2450a35 - we think this was the change that started the problem.
Owner: oprypin@chromium.org
Status: Started (was: Untriaged)
I also found the src/.vpython file sitting here https://cs.chromium.org/chromium/src/.vpython
We'll try adding a similar file to our WebRTC repo.
Blocking: 759571
Project Member

Comment 4 by bugdroid1@chromium.org, Sep 4 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/external/webrtc.git/+/678636bf7ce90d8f0dfb587ea880101184cd7a9d

commit 678636bf7ce90d8f0dfb587ea880101184cd7a9d
Author: oprypin <oprypin@webrtc.org>
Date: Mon Sep 04 08:04:16 2017

Add psutil to vpython dependencies (used on builder bots)

This is copied from https://chromium-review.googlesource.com/639357

BUG= chromium:761720 
NOTRY=True
R=kjellander@webrtc.org
TBR=kwiberg@webrtc.org

Review-Url: https://codereview.webrtc.org/3008063002
Cr-Commit-Position: refs/heads/master@{#19646}

[add] https://crrev.com/678636bf7ce90d8f0dfb587ea880101184cd7a9d/.vpython
[modify] https://crrev.com/678636bf7ce90d8f0dfb587ea880101184cd7a9d/OWNERS

Blocking: -759571
Status: Fixed (was: Started)
Summary: "ImportError: No module named psutil" due to vpython rollout (was: "ImportError: No module named psutil" in buildbot tryjobs (master.tryserver.webrtc))
Adding the .vpython was enough here. Not really blocking  bug 759571  then.
Project Member

Comment 6 by bugdroid1@chromium.org, Sep 7 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/libyuv/libyuv.git/+/50e4465cb4de6cb15292f2ec47f2568d31b0b9da

commit 50e4465cb4de6cb15292f2ec47f2568d31b0b9da
Author: Henrik Kjellander <kjellander@chromium.org>
Date: Thu Sep 07 07:21:25 2017

Add psutil to vpython dependencies (used on builder bots)

This is copied from https://chromium-review.googlesource.com/639357
and should solve the psutil missing import problems
happening during Android test execution.

BUG= chromium:761720 
NOTRY=True
TBR=fbarchard@chromium.org

Change-Id: I36c4febd6f2dcf6419dbc846ee4717b22e2525e6
Reviewed-on: https://chromium-review.googlesource.com/654600
Reviewed-by: Henrik Kjellander <kjellander@chromium.org>
Commit-Queue: Henrik Kjellander <kjellander@chromium.org>

[add] https://crrev.com/50e4465cb4de6cb15292f2ec47f2568d31b0b9da/.vpython

Sign in to add a comment