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 345869 2bf7fb2523c3e118d1f11bb87e48d4b5bb1b49d0
parent 345868 e5a3bbe621c9a77a8924819a598dc442545b1c7d
child 345870 15f2040d22ff7cfbc4724cd5a446a03e7398f813
push id31451
push usercbook@mozilla.com
push dateMon, 06 Mar 2017 09:52:09 +0000
treeherdermozilla-central@7099e03837e8 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersnalexander
bugs1344117
milestone54.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 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)