Closed
Bug 1223661
Opened 9 years ago
Closed 9 years ago
[gatt] gaia-taskenv, tester images need to be updated to node v4
Categories
(Taskcluster :: Services, defect)
Taskcluster
Services
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla46
People
(Reporter: aus, Assigned: aus)
References
Details
Attachments
(2 files, 1 obsolete file)
Autolander is messed up and can't create new try runs using the original bug. :( +++ This bug was initially created as a clone of Bug #1208230 +++ We want to start using node v4 everywhere and the first step is to get the images updated and uploaded to be ready to be turned on. It's straightforward, and I have the necessary credentials to update the images.
Comment 1•9 years ago
|
||
Comment 2•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Attachment #8685793 -
Attachment is obsolete: true
Assignee | ||
Comment 4•9 years ago
|
||
Attachment #8686190 -
Flags: review?(garndt)
Assignee | ||
Updated•9 years ago
|
Attachment #8685794 -
Flags: review?(gaye)
Attachment #8685794 -
Flags: review?(garndt)
Comment 5•9 years ago
|
||
Comment on attachment 8685794 [details] [review] [gaia] nullaus:bug1223661 > mozilla-b2g:master Left a couple of comments and a question in the PR, but nothing blocking this from landing. I skimmed over the parts related to marionette and some sock-it stuff as I'm not familiar with that.
Attachment #8685794 -
Flags: review?(garndt) → review+
Comment 6•9 years ago
|
||
Comment on attachment 8686190 [details] [diff] [review] Patch - v1 - Update base-test, tester to use node 4.2.2. Review of attachment 8686190 [details] [diff] [review]: ----------------------------------------------------------------- So far I don't see anything obvious that would break, but could we push this path to try and run all tests that require this tester image (pretty much almost everything except fx desktop tests I think). I want to make sure none of those break because of the node change. Also, out of curiosity, what is clang and node-gyp being used for that they have to be installed in the image?
Assignee | ||
Comment 7•9 years ago
|
||
(In reply to Greg Arndt [:garndt] from comment #6) > Comment on attachment 8686190 [details] [diff] [review] > Patch - v1 - Update base-test, tester to use node 4.2.2. > > Review of attachment 8686190 [details] [diff] [review]: > ----------------------------------------------------------------- > > So far I don't see anything obvious that would break, but could we push this > path to try and run all tests that require this tester image (pretty much > almost everything except fx desktop tests I think). I want to make sure > none of those break because of the node change. Indeed. I was planning on doing a full try run today. I'll need to customize which gaia is used during the build but, that's all that I should need to have to do. > > Also, out of curiosity, what is clang and node-gyp being used for that they > have to be installed in the image? sockit-to-me now requires clang to be compiled because node 4+ v8 includes are meant to be used with clang.
Assignee | ||
Comment 8•9 years ago
|
||
Try run -- using all the new stuff! https://treeherder.mozilla.org/#/jobs?repo=try&revision=c793bcb2a622
Comment 9•9 years ago
|
||
Sorry :aus but there seem to be some oranges there. Is that related to any of these changes? Looking at m-c, they look green. I'm not sure what goes on the tests to understand if the node version matters, but I worry about checking in something that might turn orange. If there is anything I can help debug let me know. Also, I believe this image is also used for emulators so I'm not sure if it matters if we run some tests against that as well or if it's good enough using b2g-desktop. Just let me know what I can do to help out.
Comment 10•9 years ago
|
||
is chunk 29 from here intermitten and not caused by your changes? https://treeherder.mozilla.org/#/jobs?repo=try&revision=d2ba56aa3467
Assignee | ||
Comment 11•9 years ago
|
||
(In reply to Greg Arndt [:garndt] from comment #10) > is chunk 29 from here intermitten and not caused by your changes? > https://treeherder.mozilla.org/#/jobs?repo=try&revision=d2ba56aa3467 Indeed, 29 is a known intermittent.
Comment 12•9 years ago
|
||
Comment on attachment 8686190 [details] [diff] [review] Patch - v1 - Update base-test, tester to use node 4.2.2. Review of attachment 8686190 [details] [diff] [review]: ----------------------------------------------------------------- Thanks for all the testing done on this. Looks good to me!
Attachment #8686190 -
Flags: review?(garndt) → review+
Assignee | ||
Comment 13•9 years ago
|
||
Alright everyone, I'm going to try and land this in the next few hours. I'm doing one more round on try with everything rebased to master / latest, just to be certain and then I'll go ahead and push.
Assignee | ||
Comment 14•9 years ago
|
||
Commit (master): https://github.com/mozilla-b2g/gaia/commit/b93f975cc713364b657f125d116f1d98bc47a739 Fixed!
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 15•9 years ago
|
||
https://hg.mozilla.org/integration/b2g-inbound/rev/36228281fd55bdccab7869c8c57b03f4bd2daf5b Bug 1223661 - Update base-tester, tester to node v4.2.2 (LTS). r=garndt
Comment 16•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/36228281fd55
Assignee | ||
Updated•9 years ago
|
Attachment #8685794 -
Flags: review?(gaye)
Comment 17•8 years ago
|
||
Moving closed bugs across to new Bugzilla product "TaskCluster".
Component: TaskCluster → Integration
Product: Testing → Taskcluster
Target Milestone: --- → mozilla46
Updated•5 years ago
|
Component: Integration → Services
You need to log in
before you can comment on or make changes to this bug.
Description
•