Bug 1506928 - [raptor] Handle "MOZ_UPLOAD_DIR" as optional in Output class. r=rwood
authorHenrik Skupin <mail@hskupin.info>
Wed, 24 Apr 2019 19:06:16 +0000
changeset 530012 ec3fe3df0985e35f654dbf5854a19fce8b79fe44
parent 530011 6f64c339a2a8f08693ff62512d6f730b2425dc1d
child 530013 97795bbf4e35329511b1371da8f5a7c0c8a7d87a
push id11265
push userffxbld-merge
push dateMon, 13 May 2019 10:53:39 +0000
treeherdermozilla-beta@77e0fe8dbdd3 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersrwood
bugs1506928
milestone68.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1506928 - [raptor] Handle "MOZ_UPLOAD_DIR" as optional in Output class. r=rwood Differential Revision: https://phabricator.services.mozilla.com/D25750
testing/raptor/raptor/output.py
--- a/testing/raptor/raptor/output.py
+++ b/testing/raptor/raptor/output.py
@@ -731,17 +731,17 @@ class Output(object):
                         screenshot['screenshot'],
                         screenshot['test_name'],
                         screenshot['page_cycle']))
 
         self.summarized_screenshots.append("""</table></body> </html>""")
 
     def output(self, test_names):
         """output to file and perfherder data json """
-        if os.environ['MOZ_UPLOAD_DIR']:
+        if os.getenv('MOZ_UPLOAD_DIR'):
             # i.e. testing/mozharness/build/raptor.json locally; in production it will
             # be at /tasks/task_*/build/ (where it will be picked up by mozharness later
             # and made into a tc artifact accessible in treeherder as perfherder-data.json)
             results_path = os.path.join(os.path.dirname(os.environ['MOZ_UPLOAD_DIR']),
                                         'raptor.json')
             screenshot_path = os.path.join(os.path.dirname(os.environ['MOZ_UPLOAD_DIR']),
                                            'screenshots.html')
         else: