commit 346cb4d72731fd373521d11e49ae0186caa69859
author Jacob Kopczynski <jkop@google.com> Thu Feb 20 12:57:51 2020 -0800
committer Commit Bot <commit-bot@chromium.org> Mon Feb 24 23:05:19 2020 +0000
tree b088d694422c5c61ed9d5476cb4d676496fcf65a
parent ea7738d04284e6a9c5048a5a56484d26ed934c40

Clarify stale TODO

Too dangerous to touch but at least can be clear about that being the
only blocker

BUG=chromium:1054564
TEST=comment only

Change-Id: I41bcfd87a46f139a9cbfe01f60f608908fb7f7d5
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/2067354
Tested-by: Jacob Kopczynski <jkop@chromium.org>
Auto-Submit: Jacob Kopczynski <jkop@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>
Commit-Queue: Jacob Kopczynski <jkop@chromium.org>

server/control_segments/cleanup

diff --git a/server/control_segments/cleanup b/server/control_segments/cleanup
index 2c7aca8..9e57a60 100644
--- a/server/control_segments/cleanup
+++ b/server/control_segments/cleanup
@@ -26,11 +26,11 @@
         host = hosts.create_host(machine, try_lab_servo=True)
         with metrics.SecondsTimer(DURATION_METRIC,
                                   fields={'dut_host_name': hostname}):
-            # Try to save /var/log files. If the dut is not sshable, try to restart
-            # with servo. This is a temp fix to collect log for test failed with dut
+            # Try to save /var/log files. If the dut is not sshable, try to
+            # restart with servo, to collect logs for test failed with dut
             # not returning from reboot.
-            # TODO(dshi): This temp fix should be removed after crash collect work
-            # is completed, crbug.com/336985
+            # TODO: This is a temp fix and should no longer be necessary
+            # Depended on crbug.com/336985, fixed in 2015
             try:
                 host.ssh_ping()
             except error.AutoservSshPingHostError: