Issue metadata
Sign in to add a comment
|
testFindPythonDependencies and testFindPythonDependenciesWithNestedImport failure on chromeos |
||||||||||||||||||||||
Issue descriptionFiled by sheriff-o-matic@appspot.gserviceaccount.com on behalf of oshima@google.com amd64-generic-telemetry:14175 failed Builders failed on: - amd64-generic-telemetry: https://luci-milo.appspot.com/buildbot/chromiumos.chromium/amd64-generic-telemetry/14175
,
Mar 2 2018
,
Mar 2 2018
I'm going to revert and disable the test on chromeos. +achuith for longer term fix (maybe move this module to CrOS? Switch to isolate? :))
,
Mar 2 2018
Yup, we have a bug to move this module to chromeos; just need to find someone to work on it.
,
Mar 2 2018
I don't quite understand why these fail on chromeos but not on other platforms?
,
Mar 2 2018
,
Mar 3 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/03a17b1a9826e1a8dbe0f23ef37d89acb209497b commit 03a17b1a9826e1a8dbe0f23ef37d89acb209497b Author: Robert Iannucci <iannucci@chromium.org> Date: Sat Mar 03 02:54:52 2018 [find_dependencies] Disable tests on CrOS. Bug: 818054 ,818230 Change-Id: Ia3c395bf89845c683bba74b4944e5c673ae3a0b0 Reviewed-on: https://chromium-review.googlesource.com/946747 Commit-Queue: Robbie Iannucci <iannucci@chromium.org> Commit-Queue: Achuith Bhandarkar <achuith@chromium.org> Reviewed-by: Achuith Bhandarkar <achuith@chromium.org> Reviewed-by: Ned Nguyen <nednguyen@google.com> Reviewed-by: David Tu <dtu@chromium.org> Cr-Commit-Position: refs/heads/master@{#540729} [modify] https://crrev.com/03a17b1a9826e1a8dbe0f23ef37d89acb209497b/tools/perf/core/find_dependencies_unittest.py
,
Mar 3 2018
Yeah, I don't either. The only thing I can think of would be that sys.prefix or sys.real_prefix is some subset of /usr/local/telemetry/src/third_party/catapult/telemetry/telemetry/internal/testing/dependency_test_dir/other_animals/cat/cat Which has a lot of telemetry animals and cats...
,
Mar 5 2018
,
Mar 5 2018
+iannucci@, this must be same as 818054?
,
Mar 6 2018
yes, I had un-duped them to keep this one open specifically re: cros, though I didn't close the other one yet.
,
Jun 15 2018
Getting back from leave; I'm not working on this so marking available (lmk if this needs my input though).
,
Oct 18
,
Jan 16
(6 days ago)
,
Jan 16
(6 days ago)
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by osh...@chromium.org
, Mar 2 2018Owner: iannucci@chromium.org
Status: Assigned (was: Available)