commit 1aef2f3c8605be1eeae80edfc03763a220448954
author Allen Li <ayatane@chromium.org> Tue May 16 18:32:09 2017 -0700
committer chrome-bot <chrome-bot@chromium.org> Tue Jun 13 19:36:59 2017 -0700
tree 94a2351a67b40afd8fbd26f046e7018b048b96f8
parent 44969e3e324a5b1b55ff629b1b940a4e556a7027

[autotest] Remove failure handling in archive step

Since the archive step is now "true", it will/should never fail.

BUG=chromium:699275
TEST=None

Change-Id: I0465eb87f7d7f5d7d72fc344308db0b1a1dd38d1
Reviewed-on: https://chromium-review.googlesource.com/507019
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

scheduler/postjob_task.py

diff --git a/scheduler/postjob_task.py b/scheduler/postjob_task.py
index 9438277..60e3b07 100644
--- a/scheduler/postjob_task.py
+++ b/scheduler/postjob_task.py
@@ -381,12 +381,4 @@
 
     def epilog(self):
         super(ArchiveResultsTask, self).epilog()
-        if not self.success and self._paired_with_monitor().has_process():
-            failed_file = os.path.join(self._working_directory(),
-                                       self._ARCHIVING_FAILED_FILE)
-            paired_process = self._paired_with_monitor().get_process()
-            self._drone_manager.write_lines_to_file(
-                    failed_file, ['Archiving failed with exit code %s'
-                                  % self.monitor.exit_code()],
-                    paired_with_process=paired_process)
         self._set_all_statuses(self._final_status())