Bug 1207900 - Disable talos-h2 on Win10 production as it takes too long to run; r=armenzg MozReview-Commit-ID: 7JmKthEvsT2

Bug 1419512 - [tryselect] Consolidate subcommand parser retrieval in mach_commands.py, r=armenzg This is a minor cleanup around finding and importing subcommand parser in |mach try|. MozReview-Commit-ID: IHRXXi5mB4G

Bug 1414894 - [tryselect] Don't install shell extensions when bootstrapping fzf for |mach try fuzzy|, r=armenzg Currently the prompts don't make it clear enough that running fzf will mess with your shell settings. This means users could install it without realizing, forget and get confused later on. Rather than try to address this, it's simpler to always skip the shell extensions as |mach try fuzzy| doesn't need them anyway. The extensions are useful, but are better installed via something like |mach bootstrap| which can be tackled in a separate bug. MozReview-Commit-ID: 2kx7UGO5LJ0

Bug 1413374 - [tryselect] Fix |mach try empty| regression and add a test, r=armenzg MozReview-Commit-ID: Iv2RDqqhL57

Bug 1400503 - [tryselect] Add an option to open the presets file in an editor, r=armenzg MozReview-Commit-ID: 2jwdAaNWHH7

Bug 1400503 - [tryselect] Separate 'common_arguments' into groups sub-parsers can opt-in to, r=armenzg This allows subparsers more control over which sets of arguments they need to implement. For example, it doesn't make sense for the 'empty' selector to accept the preset arguments. Now it can opt-out of those and only implement the 'push' arguments. MozReview-Commit-ID: GOfjcFtlfDD

Bug 1406777 - Support --message and --no-push with |mach try empty|, r=armenzg This adds all the common arguments to |mach try empty|, including preset arguments which will just be ignored if specified. The commit message can be changed with: ./mach try empty -m DONTBUILD MozReview-Commit-ID: 5RYfLBOIX1g

Bug 1400469 - Add ability to specify commit message to |mach try|, r=armenzg MozReview-Commit-ID: LWkAEDWn8NC

Bug 1397435 - allow SETA to work on stylo-disabled jobs. r=armenzg

Bug 1395267 - [tryselect] Make default try selector used when running |mach try| without subcommands configurable, r=armenzg To use this, create a ~/.mozbuild/machrc file and add: [try] default = fuzzy Now, running |mach try| without a subcommand will default to the fuzzy selector. The syntax selector can still be run with |mach try syntax|. MozReview-Commit-ID: Ai6f8cW3Swq

Bug 1391675 - [tryselect] Move --no-push into common arguments, r=armenzg The main motivation behind this change is to make testing easier, so e.g: ./mach try fuzzy --no-push and ./mach try syntax --no-push both work the same way. MozReview-Commit-ID: LmjA3Kq7xKN

Bug 1391675 - [tryselect] Make main command and all subcommands use the same argument parser, r=armenzg This will give us a good way to share arguments between subcommands. MozReview-Commit-ID: KmXRj8TBvYK

Bug 1388012 - Ensure a debug artifact build is done when 'debug_build' is present in the mozharness config. r=armenzg MozReview-Commit-ID: Dr029B4jxPH

Bug 1388811 - [try fuzzy] Add ability to run non-interactive fuzzy queries, r=armenzg MozReview-Commit-ID: JojBS8pZHSk

Bug 1306122 - [mozlint] Create a compact formatter that mimics the eslint 'compact' format, r=armenzg MozReview-Commit-ID: 5JJJhMIrMIB

Bug 1384593 - Add an fzf based fuzzy try selector, r=armenzg This try selector works as follows: 1. Generate target tasks (similar to ./mach taskgraph target) 2. Pipe all tasks to fzf (a fuzzy finding binary, this will be bootstrapped if necessary) 3. Allow user to make selection 4. Save selected tasks to 'try_task_config.json'. This is a new try scheduling mechanism built into taskcluster (see bug 1380306). 5. Use `hg push-to-try` (or git-cinnabar) to push the added file to try. This will use a temporary commit, so no trace of 'try_task_config.json' should be left over after use. If you get messages like STOP! No try syntax found, you need to update version-control-tools: ./mach mercurial-setup --update MozReview-Commit-ID: 4xHwZ9fATLv

Bug 1384593 - Abstract version control functionality out of syntax.py to vcs.py, r=armenzg This copies the vcs abstraction from python/mozlint/mozlint/vcs.py. Consumers can call: VCSHelper.create() and that will automatically detect whether we're in hg or git and return the appropriate abstraction class. MozReview-Commit-ID: 4xHwZ9fATLv

Bug 1384274 - Set a tooltool auth path for developers. r=armenzg MozReview-Commit-ID: BixDbFJqeVg

Bug 1379163: Implement the "Run Missing Tests" action; r=armenzg,bstack This implements an action to run all test tasks which were optimized out by the decision task. MozReview-Commit-ID: qPflBlxMg7

Bug 1276813 - Retry mozharness failed downloads more persistently; r=armenzg The primary change here is to increase the number of times a failed download is retried, when downloading test_packages.json and test zip files, in hopes of recovering from more temporary service interruptions.

Bug 1324094 - Add correct cache scopes to action tasks r=armenzg,dustin MozReview-Commit-ID: KxtG4cHU0Mm

Bug 1289823 - Add backfilling as an action-task r=armenzg,dustin MozReview-Commit-ID: HALwE6Q0Lch

Bug 1289823 - Make taskcluster action-task more flexible r=armenzg,dustin MozReview-Commit-ID: 8qoJsFeFm6I

Bug 1321872 - Set MOZILLA_OFFICIAL in artifact mozconfigs to allow Talos to run in this configuration. r=armenzg MozReview-Commit-ID: Hy5QTjHWqBc

Bug 1311723 - Look for mozinfo.json in the correct place when running interactive loaners, r=armenzg MozReview-Commit-ID: K8fqlo0x4Oc

Bug 1311723 - Fix wrong binary_path in desktop_unittest.py configs, r=armenzg MozReview-Commit-ID: A08HEzXRf4I

Bug 1311431 - Unset CC/CXX for Mac artifact mozconfigs. r=armenzg MozReview-Commit-ID: 2UdZ9G2dDfP

Bug 1304487 - Make incompatible mozharness actions a no-op when try syntax includes '--artifact'; r=armenzg Rather than clearing actions in volatile_config, add in actions from the artifact config's default_actions. Incompatible actions are then skipped based on 'forced_artifact_build' config value. MozReview-Commit-ID: IZuDvxcQ7cN

Bug 1306421 - Don't use proxxy if running in TaskCluster; r=armenzg Proxxy is only configured in buildbot land. Don't enable it in TaskCluster. Ideally, we'd only enable proxxy if we detect we're in a buildbot environment. But the change in this commit is more conservative and aligns with existing behavior. MozReview-Commit-ID: HBCdQ6MkYGa

Bug 1306421 - Add is_taskcluster to ScriptMixin; r=armenzg So we can detect when we're running on TaskCluster. This will be used to adjust environment settings accordingly. MozReview-Commit-ID: JEG1E3tWsc5

Bug 1305720 - Check for --artifact try syntax regardless of is_automation value; r=armenzg Check try message for --artifact even if fx_desktop_build.py is run with --skip-buildbot-actions We can't rely on buildbot config. Add checks to TryToolsMixin._extract_try_message so that it works even if self.buildbot_config is None. MozReview-Commit-ID: 1xErjuOArBe

Bug 1303305 - Support in-memory unzip on tc win testers; r=armenzg MozReview-Commit-ID: JcGymDuh2s9

Bug 1278702 - Fail early in compiled-code tests with --artifact Try syntax; r=armenzg MozReview-Commit-ID: 84IZn2Qpvk4

Bug 1301431 - Support running Android reftests from a one-click-loaner with mach, r=armenzg This adds the ability to run Android reftests from a taskcluster interactive loaner using mach. MozReview-Commit-ID: 5HMfm64wmK2

Bug 1301431 - Change reftest harness to accept Namespace object instead of kwargs, r=armenzg Reftests hackily create a second argument parser and set defaults on it to get around the fact that the mach command sends the harness a dict rather than a Namespace object. This is bad because: 1. It's much less hacky to just create a Namespace object directly (rather than making a second parser) 2. Most other mach commands actually *do* return a Namespace object, reftest is the odd one out here. So this patch makes the reftest mach commands convert to the Namespace object, which allows us to get rid of the hacks in the reftest harness. This also does some light refactoring of the entry points to the reftest harness so make it more consistent with mochitest and xpcshell. MozReview-Commit-ID: 5HMfm64wmK2

Bug 1252235 - Make try syntax available to mozharness from TaskCluster through an environment variable. r=armenzg MozReview-Commit-ID: HIVIXPQXnQs

Bug 1286075: allow optimization of tasks whose dependencies have not been optimized; r=armenzg MikeLing initially did this in bug 1287018. The intent of this conditional was to make optimization faster by not even checking most tasks, based on the assumption that if the prerequisite to a task has changed (for example, a docker image or a build), then naturally we will want to execute that task. However, as we have developed actual optimization methods, this has proven not to be the case: we might want to optimize a test out if its inputs have not changed, even if a new installer has been built. Similarly, SETA may optimize tasks out even if their inputs have changed. MozReview-Commit-ID: LgHET3Z84GB

Bug 1278698 - Get forced artifact build working in TaskCluster vs Buildbot; r=armenzg Override actions specified in TC task definition with default_actions from artifact config Clear MOZ_SIGN_CMD when enable_signed is set to False. MozReview-Commit-ID: FmI6lu5EKWg

Bug 1278698 - Make mozharness build scripts trigger artifact builds when specified via try syntax; r=armenzg MozReview-Commit-ID: JDieAcHgpSy

Bug 1278698 - Get forced artifact build working in TaskCluster vs Buildbot; r=armenzg Override actions specified in TC task definition with default_actions from artifact config Clear MOZ_SIGN_CMD when enable_signed is set to False. MozReview-Commit-ID: FmI6lu5EKWg

Bug 1278698 - Make mozharness build scripts trigger artifact builds when specified via try syntax; r=armenzg MozReview-Commit-ID: JDieAcHgpSy

Bug 1300163 - Add support for Android mochitests to interactive loaners, r=armenzg This will allow developers to request a loaner for an Android mochitest job and then use |mach mochitest| to run tests. MozReview-Commit-ID: 4lsKGpizfH7

Bug 1300163 - Reuse the 'e10s' value from the mozharness config on interactive loaners, r=armenzg I noticed that if you request a loaner for a non-e10s job then run mach, it will be run with e10s enabled. The mach command should accurately reflect the type of job that got requested. This patch grabs the 'e10s' argument from the mozharness localconfig.json and uses that. MozReview-Commit-ID: 4lsKGpizfH7

Bug 1300163 - Lazy load certain mach context attributes using the 'key' mechanism, r=armenzg You can set attributes on a mach context by using the 'key' argument to the context_handler. Basically, whatever gets returned by the handler when <key> is passed in, will get set (i.e cached) on the context object for fast retrieval next time. This is a way to lazy load these attributes. Previously I was setting functions like 'find_firefox()' on the context object, and then having the mach_commands call that directly. But this way is much cleaner. Now, the loaded 'mozharness_config' can be stored as an attribute on the context. Also 'find_firefox()' is now an attribute called 'firefox_bin'. MozReview-Commit-ID: 4lsKGpizfH7

Bug 1300163 - Fix hidden tracebacks in mochitest runtests.py, r=armenzg I had a mistake in my code that resulted in an exception being thrown when starting the mochitest servers. However, the log just showed a timeout. It was very confusing. After some digging, I found that there's a large try/finally block (with no except) in runtests.py. The exception wasn't being displayed because of the finally block. This try/finally block had an inner try/except/finally that wrapped part of the code. This patch simply removes that inner block and dedents the 'except' clause to attach to the original try/finally. MozReview-Commit-ID: 4lsKGpizfH7

Bug 1296067 - Add mach test package integration for marionette-test, r=armenzg This makes it possible to run |mach marionette-test| from a test package (and therefore also an interactive loaner). MozReview-Commit-ID: Lxhe8KMQaWq

Bug 1293397: fix fake to make tests pass? r=armenzg MozReview-Commit-ID: AuQBk2w8Ka1

Bug 1289879 - Activate the mozharness virtualenv when running tests on interactive workers, r=armenzg There are some packages like 'requests' that are bundled in the mozharness venv, but not in the test package. It would be easy to manually add these to sys.path in the mach bootstrap script, but it's much nicer to simply activate this virtualenv in the first place. MozReview-Commit-ID: 8xeJEIgUbLj

Bug 1288827 - Allow running mochitest from test-package without specifying --appname, r=armenzg Because it is now possible for options.app to get set after 'parse_args' time, we need to make sure the argument validation happens later. To accomplish this we pass in the parser instance to 'run_test_harness' and do parser.validate there. This unfortunately requires some minor uses of global to accomplish easily due to how mach handles parsers. MozReview-Commit-ID: s3Js1aZlSE

Bug 1288827 - Allow running mochitest from test-package without specifying --appname, r=armenzg Because it is now possible for options.app to get set after 'parse_args' time, we need to make sure the argument validation happens later. To accomplish this we pass in the parser instance to 'run_test_harness' and do parser.validate there. This unfortunately requires some minor uses of global to accomplish easily due to how mach handles parsers. MozReview-Commit-ID: s3Js1aZlSE