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)
Hello (Loop)
Client
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 1•9 years ago
|
||
| Assignee | ||
Comment 2•9 years ago
|
||
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)
| Assignee | ||
Updated•9 years ago
|
Attachment #8765897 -
Flags: review?(fernando.campo)
Attachment #8765897 -
Flags: review?(b.mcb)
Comment 3•9 years ago
|
||
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+
| Assignee | ||
Updated•9 years ago
|
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)
| Assignee | ||
Comment 4•9 years ago
|
||
https://github.com/mozilla/loop/commit/89f3e1b43f336a0a39198f068062c492922b5e74
https://github.com/mozilla/loop/commit/369410e9e8fcfb379659e2b3adfd528dc263e33d
https://github.com/mozilla/loop/commit/1dc38adc84a94cf557d453d8a9926f492d657abe
https://github.com/mozilla/loop/commit/07a4ab1f38eafd7346d5cc520e10148c89058bae
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•