)]}' { "commit": "1ab7a663fc01c4ff9dcf0be0ce42341a6aa054c8", "tree": "5fe654e63f4c69757f01de7466299f43bcd276c0", "parents": [ "d3209c140a2172a4ea1412511311cde20a72c620" ], "author": { "name": "Julius Werner", "email": "jwerner@chromium.org", "time": "Fri Nov 30 22:06:29 2012 -0800" }, "committer": { "name": "Julius Werner", "email": "jwerner@chromium.org", "time": "Wed Jan 16 18:07:28 2013 -0800" }, "message": "recover_duts: ping SSH clients to check connectivity\n\nrecover_duts\u0027 check_ethernet.hook tests for network connectivity by\npinging the system\u0027s default gateway on any device that starts with\n\"eth\". This is obviously a problem when the default route does not use\nsuch a device, as it can happen during some autotests (like\npower_LoadTest) that use the cros/backchannel.py script to switch their\ndefault route to wireless.\n\nWhat we really want to know is if the autotest server is still reachable\nand can control the device. Autotest uses an SSH connection to control\nits client, so this patch changes recover_duts to search for an active\nSSH connection and ping that remote client first. This will work with\nbackchannel tests as it specifically ensures to keep routes to subnets\nthat have active SSH connections on the wired device. If no active SSH\nconnection is detected, the hook will fall back to the old method.\n\nBUG\u003dchromium-os:36454\nTEST\u003dSubmit this patch and rejoice as all the power_LoadTest runs in the\nlab magically start to work again for the first time in months.\n\nOriginal-Change-Id: I0066ccebe7a2f77744f2104555160c08feb7c108\nSigned-off-by: Julius Werner \u003cjwerner@chromium.org\u003e\nReviewed-on: https://gerrit.chromium.org/gerrit/39036\nReviewed-by: Chris Sosa \u003csosa@chromium.org\u003e\n(cherry picked from commit b47fd1c69651818b34f351f375798d73d9bec510)\n\nChange-Id: I11027e61dd445e04bfe0d9b4a829e060e5bbbd69\nReviewed-on: https://gerrit.chromium.org/gerrit/39260\nReviewed-by: Chris Sosa \u003csosa@chromium.org\u003e\nCommit-Queue: Julius Werner \u003cjwerner@chromium.org\u003e\nTested-by: Julius Werner \u003cjwerner@chromium.org\u003e\n", "tree_diff": [ { "type": "modify", "old_id": "9c4f7e1528cbee9e0b5cdf47c89f055a39511593", "old_mode": 33261, "old_path": "recover_duts/hooks/check_ethernet.hook", "new_id": "2ee8a4874f13762c4f58b24fa5b9ef04fd008a46", "new_mode": 33261, "new_path": "recover_duts/hooks/check_ethernet.hook" } ] }