recover_duts: reimplement as shell script |
||
Issue descriptionIn certain autotest failure cases /var gets wiped and with it the Python interpreter, in such a situation recover_duts.py can't be executed. The script is fairly simple and doesn't really benefit from being implemented in Python, reimplement it as shell script.
,
Sep 17
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/c722720815348f89a861e5a77a3bc5a13db3042d commit c722720815348f89a861e5a77a3bc5a13db3042d Author: Matthias Kaehlcke <mka@chromium.org> Date: Mon Sep 17 20:17:29 2018 recover_duts: switch to shell script recover_duts.py has been reimplemented as shell script, install the new version. BUG= chromium:882015 TEST=see CL:1213409 CQ-DEPEND=CL:1213409 Change-Id: Ic260bdbec0782c890e482ef73fed1b008c872824 Signed-off-by: Matthias Kaehlcke <mka@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/1217023 Reviewed-by: Douglas Anderson <dianders@chromium.org> Reviewed-by: Grant Grundler <grundler@chromium.org> [modify] https://crrev.com/c722720815348f89a861e5a77a3bc5a13db3042d/chromeos-base/recover-duts/recover-duts-9999.ebuild
,
Sep 17
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform/crostestutils/+/ea94e325ebe45d445bd032b278daaa91e7cdacc4 commit ea94e325ebe45d445bd032b278daaa91e7cdacc4 Author: Matthias Kaehlcke <mka@chromium.org> Date: Mon Sep 17 20:17:28 2018 recover_duts: Reimplement as shell script In certain autotest failure cases /var gets wiped and with it the Python interpreter, in such a situation recover_duts.py can't be executed. The script is fairly simple and doesn't really benefit from being implemented in Python, reimplement it as shell script. BUG= chromium:882015 TEST=touch /mnt/stateful_partition/.labmachine rmmod r8152 # or applicable ethernet driver # wait up to 90s => eth0 is brought up again CQ-DEPEND=CL:1217082 Change-Id: I07a4610056c83e88f1b19e7e2f8b7767f81254e2 Signed-off-by: Matthias Kaehlcke <mka@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/1213409 Reviewed-by: Douglas Anderson <dianders@chromium.org> [add] https://crrev.com/ea94e325ebe45d445bd032b278daaa91e7cdacc4/recover_duts/recover_duts.sh [delete] https://crrev.com/584d04568c8b135e3b24fe3ff372bd6e22078c6e/recover_duts/recover_duts.py [modify] https://crrev.com/ea94e325ebe45d445bd032b278daaa91e7cdacc4/recover_duts/README
,
Sep 17
|
||
►
Sign in to add a comment |
||
Comment 1 by bugdroid1@chromium.org
, Sep 17