Closed Bug 1282764 Opened 9 years ago Closed 9 years ago

Integrate running of Loop's tests with Taskcluster

Categories

(Hello (Loop) :: Client, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

We're looking at replacing Travis with Taskcluster as it will allow us to use a wider variety of platforms and browsers, especially given the Docker integration. I've got a branch together which gets us initial integration, there's some rough edges, but bug 1282762 can track additional work on cleaning those up.
Comment on attachment 8765897 [details] [review] [loop] mozilla:taskcluster-experimental > mozilla:master Notes: - I've versioned the docker files and tagged them in my docker account. - I think its in a reasonable state where we can push this to master, and start seeing the effects on PRs etc. - There's a bunch of stuff that would need to be addressed before we can turn off travis (see bug 1282762). - In particular, there's intermittent failures, however I think we'll have to go with "ignore task cluster for landings" initially, and fix those up when we can. - The PR is split up into a few parts, the last two are the main work, the first two are more clean up to make doing the images & commands simpler. - You can see the output of taskcluster by going to the commits for a branch, e.g. https://github.com/mozilla/loop/commits/taskcluster-experimental then select the latest commit, go to the bottom and select the link. I believe for PRs taskcluster should hook into the normal system as per Travis, but by landing it we'll find out and see how the integration goes.
Attachment #8765897 - Flags: review?(edilee)
Attachment #8765897 - Flags: review?(dpreston)
Attachment #8765897 - Flags: review?(dmose)
Attachment #8765897 - Flags: review?(dcritchley)
Attachment #8765897 - Flags: review?(fernando.campo)
Attachment #8765897 - Flags: review?(b.mcb)
Comment on attachment 8765897 [details] [review] [loop] mozilla:taskcluster-experimental > mozilla:master I read over the code and didn't see any obvious typos. Most of the code is taken from mozilla-central and slightly modified. The tests run and pass. r+
Attachment #8765897 - Flags: review?(dpreston) → review+
Attachment #8765897 - Flags: review?(fernando.campo)
Attachment #8765897 - Flags: review?(edilee)
Attachment #8765897 - Flags: review?(dmose)
Attachment #8765897 - Flags: review?(dcritchley)
Attachment #8765897 - Flags: review?(b.mcb)
Blocks: 1285974
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: