Automatically replace try syntax requesting compiled code tests when pushing an artifact build with |./mach try|

RESOLVED FIXED in Firefox 54

Status

Firefox Build System
General
RESOLVED FIXED
a year ago
5 months ago

People

(Reporter: Chris Manchester (mostly offline July 16-20), Assigned: Chris Manchester (mostly offline July 16-20))

Tracking

(Blocks: 1 bug)

unspecified
mozilla54
Dependency tree / graph

Firefox Tracking Flags

(firefox54 fixed)

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

Bug 1278702 prevents people from inadvertently triggering tests that are guaranteed to fail when run against an artifact build on try, but does so in a way that interrupts the user's workflow. After attempting to do this a number of times I think we should just replace the `-u all` with our best guess of suites that aren't compiled code tests.
Comment hidden (mozreview-request)

Comment 2

a year ago
mozreview-review
Comment on attachment 8842196 [details]
Bug 1343361 - Auomatically replace `-u all` with a set of suites that do not include compiled tests when requesting an artifact build via |./mach try|.

https://reviewboard.mozilla.org/r/116090/#review117666
Attachment #8842196 - Flags: review?(mjzffr) → review+

Comment 3

a year ago
Pushed by cmanchester@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e35efb0da85f
Auomatically replace `-u all` with a set of suites that do not include compiled tests when requesting an artifact build via |./mach try|. r=maja_zf

Comment 4

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/e35efb0da85f
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox54: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54

Updated

5 months ago
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.