Bug 1510829 - Upload AWSY artifacts even if processing fails. r=bc

It's useful to get DMD reports even if we end up with negative
heap-unclassified. This catches any assertions during processing and makes
sure we move the reports to the upload dir even if there's a failure.

--HG--
extra : rebase_source : 3268d7b8f7f33c36c9b59bfdd5f181d80a0c4201
This commit is contained in:
Eric Rahm 2018-12-03 11:50:45 -08:00
Родитель 5c2dc1d41d
Коммит 0d6916073e
1 изменённых файлов: 23 добавлений и 18 удалений

Просмотреть файл

@ -80,6 +80,7 @@ class AwsyTestCase(MarionetteTestCase):
def tearDown(self): def tearDown(self):
MarionetteTestCase.tearDown(self) MarionetteTestCase.tearDown(self)
try:
self.logger.info("processing data in %s!" % self._resultsDir) self.logger.info("processing data in %s!" % self._resultsDir)
perf_blob = process_perf_data.create_perf_data( perf_blob = process_perf_data.create_perf_data(
self._resultsDir, self.perf_suites(), self._resultsDir, self.perf_suites(),
@ -90,7 +91,11 @@ class AwsyTestCase(MarionetteTestCase):
with open(perf_file, 'w') as fp: with open(perf_file, 'w') as fp:
json.dump(perf_blob, fp, indent=2) json.dump(perf_blob, fp, indent=2)
self.logger.info("Perfherder data written to %s" % perf_file) self.logger.info("Perfherder data written to %s" % perf_file)
except:
raise
finally:
# Make sure we cleanup and upload any existing files even if there
# were errors processing the perf data.
if self._dmd: if self._dmd:
self.cleanup_dmd() self.cleanup_dmd()