Failure to import selenium in client side ebuild |
||||
Issue descriptionI am trying to add a client side test to the autotest-chrome-9999.ebuild. https://chromium-review.googlesource.com/#/c/348960/ pre-cq continues to fail on build_packages. https://uberchromegw.corp.google.com/i/chromiumos.tryserver/builders/pre-cq/builds/32881 https://uberchromegw.corp.google.com/i/chromiumos.tryserver/builders/compile-only-pre-cq/builds/43770 autotest-chrome-0.0.1-r5928: Traceback (most recent call last): autotest-chrome-0.0.1-r5928: File "/build/rambi/tmp/portage/chromeos-base/autotest-chrome-0.0.1-r5928/work/autotest-work/client/bin/setup_job.py", line 75, in init_test autotest-chrome-0.0.1-r5928: exec import_stmt + '\n' + init_stmt in locals_dict, globals_dict autotest-chrome-0.0.1-r5928: File "<string>", line 1, in <module> autotest-chrome-0.0.1-r5928: File "/build/rambi/tmp/portage/chromeos-base/autotest-chrome-0.0.1-r5928/work/autotest-work/client/site_tests/network_CastTDLS/network_CastTDLS.py", line 10, in <module> autotest-chrome-0.0.1-r5928: from selenium.webdriver.common.keys import Keys autotest-chrome-0.0.1-r5928: ImportError: No module named selenium.webdriver.common.keys
,
Jul 7 2016
Yes. It's called virtual/host-sdk or something like that. Lots of dependencies are added there. You probably need to add selenium there. (unit tests on the builders run in the chroot, so that should be enough)
,
Jul 12 2016
Tien, do you need more help on this? Or is this resolved now?
,
Aug 3 2016
Ping. Is this problem resolved now?
,
Aug 4 2016
Hi Aviv, just to confirm, do you mean this file? https://cs.corp.google.com/chromeos_public/src/third_party/chromiumos-overlay/virtual/target-chromium-os-sdk/target-chromium-os-sdk-1.ebuild
,
Aug 4 2016
Looks like Aviv is OOO - Simran, could you confirm?
,
Aug 5 2016
Yup thats the ebuild.
,
Aug 11 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/391b9bf5834ddf2cb4f3b1c4515c078c21af07ad commit 391b9bf5834ddf2cb4f3b1c4515c078c21af07ad Author: Tien Chang <tienchang@google.com> Date: Wed Aug 10 17:48:25 2016 ebuild: Add network_CastTDLS autotest and add selenium package dependency. BUG= chromium:626145 TEST=None Change-Id: I667ed4788d33901ef9ce03514826f9b8757115f3 Reviewed-on: https://chromium-review.googlesource.com/367536 Commit-Ready: Tien Chang <tienchang@chromium.org> Tested-by: Tien Chang <tienchang@chromium.org> Reviewed-by: Clifford Cheng <cliffordcheng@chromium.org> Reviewed-by: Dan Shi <dshi@google.com> [modify] https://crrev.com/391b9bf5834ddf2cb4f3b1c4515c078c21af07ad/virtual/target-chromium-os-sdk/target-chromium-os-sdk-1.ebuild [modify] https://crrev.com/391b9bf5834ddf2cb4f3b1c4515c078c21af07ad/chromeos-base/autotest-chrome/autotest-chrome-9999.ebuild [rename] https://crrev.com/391b9bf5834ddf2cb4f3b1c4515c078c21af07ad/virtual/target-chromium-os-sdk/target-chromium-os-sdk-1-r65.ebuild
,
Aug 12 2016
,
Aug 18 2016
Test ran (albeit failed) so marking as Verified. http://cautotest/afe/#tab_id=view_job&object_id=73776182
,
Aug 29 2016
Tien, does that mean all the TDLS tests should be working now?
,
Aug 29 2016
Hi Clifford - TDLS tests should be running now. |
||||
►
Sign in to add a comment |
||||
Comment 1 by sbasi@chromium.org
, Jul 7 2016