tree cf4880ce37d92f83b193236dd6c8003dbe6780ad
parent 9d5f380f484594f80712e95838a6e18f1d99c968
author Paul Stewart <pstew@chromium.org> 1421952361 -0800
committer ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> 1422042905 +0000

check_ethernet.hook: Refactor and enhance recovery

This CL splits up the recovery steps in check_ethernet.hook so
that less disruptive remidation steps are run before more
disruptive ones.  For example, restarting the connection manager
is likely to disrupt any running network connectivity test that
uses the connection manager.  To mitigate the possibility that
the connection manager is really at fault, we first check to see
if the connection manager is stopped.

Additionally, this change adds a new recovery step that uses
the "reload_network_device" script to virtually unplug and replug
USB Ethernet devices, in most cases performing a power cycle on
the peripheral.  This should hopefully work around failures with
some problematic USB-Etherent devices.

BUG=chromium:445705
TEST=Manual: run with Ethernet disconnected, the driver unloaded,
the connnection manager stopped

Change-Id: I3f213f60f441207a539118ce75e3872040d136bc
Reviewed-on: https://chromium-review.googlesource.com/242583
Reviewed-by: Chris Sosa <sosa@chromium.org>
Commit-Queue: Paul Stewart <pstew@chromium.org>
Tested-by: Paul Stewart <pstew@chromium.org>