deploy_production_local.py should remove .pyc files after syncing code. |
||||||||||
Issue descriptionWe should remove stale .pyc files after syncing code. They can cause removed libraries to still appear to be present, and can be corrupt in a variety of ways. I believe it the correct fix is to do this right after the "repo_sync()" method in deploy_production_local.py. Something similar to: find -iname \*.pyc | xargs --no-run-if-empty rm
,
Jun 29 2016
,
Nov 21 2016
I'm working on a few cosmetic test_push changes this week.
,
Jan 17 2017
I think this is a good candidate for the coming Fixit. This has broken test_push multiple times, and we have had to rollback changes because we weren't sure if the same issue will occur in prod.
,
Jan 23 2017
,
Jan 31 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/269242ad597f79932d08bc27699799992663fd10 commit 269242ad597f79932d08bc27699799992663fd10 Author: Shuqian Zhao <shuqianz@chromium.org> Date: Mon Jan 30 23:35:33 2017 [autotest] remove .pyc files during server deployment Remove stale .pyc files after syncing code. BUG= chromium:616277 TEST=Test locally Change-Id: Ic3a61ad72ab31f345c4dfc11357ad6e48b9ac443 Reviewed-on: https://chromium-review.googlesource.com/434859 Tested-by: Shuqian Zhao <shuqianz@chromium.org> Reviewed-by: Don Garrett <dgarrett@chromium.org> Reviewed-by: Shuqian Zhao <shuqianz@chromium.org> [modify] https://crrev.com/269242ad597f79932d08bc27699799992663fd10/site_utils/deploy_server_local.py
,
Jan 31 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/ec26bd73d1a30576b501728797e3949c4f3be249 commit ec26bd73d1a30576b501728797e3949c4f3be249 Author: Shuqian Zhao <shuqianz@chromium.org> Date: Tue Jan 31 18:17:25 2017 [autotest] remove .pyc files during server deployment Remove stale .pyc files after syncing code. BUG= chromium:616277 TEST=Test locally and unittest Change-Id: Ie7cc55d71b1ccc84efe7704d60db476e284cf662 Reviewed-on: https://chromium-review.googlesource.com/435041 Reviewed-by: Shuqian Zhao <shuqianz@chromium.org> Tested-by: Shuqian Zhao <shuqianz@chromium.org> [modify] https://crrev.com/ec26bd73d1a30576b501728797e3949c4f3be249/site_utils/deploy_server_local_unittest.py [modify] https://crrev.com/ec26bd73d1a30576b501728797e3949c4f3be249/site_utils/deploy_server_local.py
,
Feb 1 2017
,
Apr 17 2017
,
May 30 2017
,
Aug 1 2017
,
Oct 14 2017
|
||||||||||
►
Sign in to add a comment |
||||||||||
Comment 1 by autumn@chromium.org
, Jun 2 2016