Remove devserver package in site-packages of autotest? |
||
Issue descriptionseems only one function (gsutil_util.GetGSNamesWithWait) in devserver package is used in autotest, and that function has already been replaced by gs.GSContext().GetGsNamesWithWait in chromite. So possibly we could remove devserver package in site-packages.
,
Aug 18 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/8fe92ca280428c8abf2524efa38bceb85048bf53 commit 8fe92ca280428c8abf2524efa38bceb85048bf53 Author: xixuan <xixuan@chromium.org> Date: Fri Aug 18 05:41:10 2017 autotest: Remove devserver gs dependency. This CL removes the last user for devserver gsutil package. BUG= chromium:709650 TEST=Ran test_image.gs_ls(pattern, archive_url, False) locally before/after changes, and compare the outputs. Ran unittest. Change-Id: I91413e6698c80066c85bd48295285e00ad894bdf Reviewed-on: https://chromium-review.googlesource.com/619496 Commit-Ready: Xixuan Wu <xixuan@chromium.org> Tested-by: Xixuan Wu <xixuan@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org> [modify] https://crrev.com/8fe92ca280428c8abf2524efa38bceb85048bf53/site_utils/autoupdate/test_image.py [modify] https://crrev.com/8fe92ca280428c8abf2524efa38bceb85048bf53/site_utils/autoupdate/full_release_test_test.py
,
Aug 24 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/523638859a12ad816475f859ec33b145e91400a2 commit 523638859a12ad816475f859ec33b145e91400a2 Author: Xixuan Wu <xixuan@chromium.org> Date: Thu Aug 24 00:01:23 2017 autotest: remove devserver server-side package from site-packages. BUG= chromium:709650 TEST=Run ./utils/build_externals.py locally. Change-Id: I2a7d41bc673195456cfc587bc4d0d023af51e567 Reviewed-on: https://chromium-review.googlesource.com/629157 Commit-Ready: Xixuan Wu <xixuan@chromium.org> Tested-by: Xixuan Wu <xixuan@chromium.org> Reviewed-by: Don Garrett <dgarrett@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org> [modify] https://crrev.com/523638859a12ad816475f859ec33b145e91400a2/utils/external_packages.py
,
Mar 19 2018
|
||
►
Sign in to add a comment |
||
Comment 1 by xixuan@chromium.org
, Aug 16 2017Status: Started (was: Untriaged)