have marionette-socket-host kick off runner-service

RESOLVED FIXED

Status

Testing Graveyard
JSMarionette
RESOLVED FIXED
4 years ago
4 months ago

People

(Reporter: mdas, Unassigned)

Tracking

Trunk
x86
Mac OS X

Details

This bug will move the python runner-service code to mocha-socket-host instead of in the Makefile.
Summary: have mocha-socket-host kick off runner-service → have marionette-socket-host kick off runner-service
(In reply to Malini Das [:mdas] from comment #0)
> This bug will move the python runner-service code to mocha-socket-host
should be marionette-socket-host
all these parts have landed.
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
I'm sorry to remove some of the fun, but you can't just call "npm install" in the Makefile, and increase versions in package.json.

The main reason is that in TBPL we need things to come from mozilla servers only. That means we have another git repository [1] that is mirrored on git.mozilla.org. The repository in git.mozilla.org is used in TBPL, and the one in github is used otherwise.

[1] https://github.com/mozilla-b2g/gaia-node-modules

That means you can't just do what you did in that patch, because this will cause all sorts of issues.
I think we should backout and have this reviewed by a build peer (these days, it's :ricky), and I'd also like to have a feedback from :kgrandon here.
(In reply to Julien Wajsberg [:julienw] from comment #4)
> I'm sorry to remove some of the fun, but you can't just call "npm install"
> in the Makefile, and increase versions in package.json.
> 
> The main reason is that in TBPL we need things to come from mozilla servers
> only. That means we have another git repository [1] that is mirrored on
> git.mozilla.org. The repository in git.mozilla.org is used in TBPL, and the
> one in github is used otherwise.
> 
> [1] https://github.com/mozilla-b2g/gaia-node-modules
> 
> That means you can't just do what you did in that patch, because this will
> cause all sorts of issues.
> I think we should backout and have this reviewed by a build peer (these
> days, it's :ricky), and I'd also like to have a feedback from :kgrandon here.

Right, but tbpl currently won't hit that part of the Makefile, since it will only install the extra packages if BUILDAPP=device, and we don't run device builds in tbpl. I've filed https://bugzilla.mozilla.org/show_bug.cgi?id=1089712 to address the case when we do want to run tests on device in tbpl (this bug is needed before we can update gaia-node-modules with the new package, since they rely on libzmq)

As for the increased package.json version, we already updated the gaia-node-modules repo with the respective changes https://github.com/mozilla-b2g/gaia-node-modules/commits/master, they were done on Nov 5.

Updated

4 months ago
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.