Bug 1344117 - Always print a message when adding `--artifact` to a try push automatically. r=nalexander
authorChris Manchester <cmanchester@mozilla.com>
Thu, 02 Mar 2017 21:21:52 -0800
changeset 374896 2bf7fb2523c3e118d1f11bb87e48d4b5bb1b49d0
parent 374895 e5a3bbe621c9a77a8924819a598dc442545b1c7d
child 374897 15f2040d22ff7cfbc4724cd5a446a03e7398f813
push id10863
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 23:02:23 +0000
treeherdermozilla-aurora@0931190cd725 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersnalexander
bugs1344117
milestone54.0a1
Bug 1344117 - Always print a message when adding `--artifact` to a try push automatically. r=nalexander MozReview-Commit-ID: 7pVIG4v7DHq
testing/mach_commands.py
--- a/testing/mach_commands.py
+++ b/testing/mach_commands.py
@@ -700,26 +700,25 @@ class PushToTry(MachCommandBase):
 
         try:
             msg = at.calc_try_syntax(platforms, tests, talos, builds, paths_by_flavor, tags,
                                      extra, kwargs["intersection"])
         except ValueError as e:
             print(e.message)
             sys.exit(1)
 
-        if kwargs["verbose"]:
-            if self.substs.get("MOZ_ARTIFACT_BUILDS"):
-                if kwargs["no_artifact"]:
-                    print('mozconfig has --enable-artifact-builds but '
-                          '--no-artifact specified, not including --artifact '
-                          'flag in try syntax')
-                else:
-                    print('mozconfig has --enable-artifact-builds; including '
-                          '--artifact flag in try syntax (use --no-artifact '
-                          'to override)')
+        if self.substs.get("MOZ_ARTIFACT_BUILDS"):
+            if kwargs["no_artifact"]:
+                print('mozconfig has --enable-artifact-builds but '
+                      '--no-artifact specified, not including --artifact '
+                      'flag in try syntax')
+            else:
+                print('mozconfig has --enable-artifact-builds; including '
+                      '--artifact flag in try syntax (use --no-artifact '
+                      'to override)')
 
         if kwargs["verbose"] and paths_by_flavor:
             print('The following tests will be selected: ')
             for flavor, paths in paths_by_flavor.iteritems():
                 print("%s: %s" % (flavor, ",".join(paths)))
 
         if kwargs["verbose"] or not kwargs["push"]:
             print('The following try syntax was calculated:\n%s' % msg)