Bug 1506928 - [raptor] Handle "MOZ_UPLOAD_DIR" as optional in Output class. r=rwood
☠☠ backed out by f478c61c1bc9 ☠ ☠
authorHenrik Skupin <mail@hskupin.info>
Mon, 01 Apr 2019 22:21:07 +0200
changeset 469762 ab985d0cbffd4602bbbd513b8717c53879ef7933
parent 469761 6ad35d665d4b843f340b7f90e4b858a74866e505
child 469763 9ca5edb93d09b69fa866e9ca42c7bdb38ecf2d65
push id112819
push userhskupin@mozilla.com
push dateWed, 17 Apr 2019 11:09:58 +0000
treeherdermozilla-inbound@98c7524647ae [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: