Mathoid's deployment is pretty much a one-off, but we have now some standardised components for Node.JS deploys, most notably the service-template-node and the service::node Puppet type With its adoption of the service template, Mathoid becomes a candidate for standardisation.
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Declined | Physikerwelt | T87390 Math extension should not use a wmflabs.org address as default $wgMathMathMLUrl | |||
| Resolved | • mobrovac | T102030 Document and hook up public mathoid end point in RB | |||
| Resolved | • mobrovac | T97124 Standardise Mathoid's deployment |
@Physikerwelt Am I correctly understanding that https://github.com/physikerwelt/mathoid-server is simply an updated version of https://gerrit.wikimedia.org/r/#/admin/projects/mediawiki/services/mathoid ?
PR for bringing mathoid-server up to date with the service-template-node is available here.
@mobrovac I recently tried to build with ppa with a makefile that calls npm install. However this still fails with a out of memory extension... Therefore all the node modules (incl. binary modules) need to be included in the reposory, which is does not really seem to be the right way to use git.
Do you have a soultion for that or is the additional deploy module the solution?
Which one? :) services/mathoid or services/mathoid/deploy? The latter has not been created yet
Yes, we are aware of that problem. That's why the second deploy repo is for. Basically the workflow is:
Cf. https://wikitech.wikimedia.org/wiki/User:Mobrovac/Service_Deployment for an extensive list of deployment steps
Thank you very much for the excellent documentation. This simplifies the process significantly (since I don't have to care about ppa debian repos).
Originally I was asking to be added to the request on here
https://www.mediawiki.org/wiki/Git/New_repositories/Requests
but now I thinking that I don't need access there.
Change 167413 had a related patch set uploaded (by Alexandros Kosiaris):
mathoid to service::node
The mathoid module has been update to use service::node in in https://gerrit.wikimedia.org/r/167413. Unfortunately this can not be merged before the mathoid code is modified to work with the assumptions of service::node. Those seem to be:
@akosiaris, yup, I am aware of that. The deploy repo has been created (yet need to structure it and import the src and modules). As for
@akosiaris, yup, I am aware of that (should have probably mentioned in the task desc that there has to be an order to things here).
Those seem to be:
- being able to accept a -c parameter for the configfile.
- presence of src/server.js instead of mathoid.js
These are covered in https://github.com/physikerwelt/mathoid-server/pull/5
- a mathoid/deploy repo
The repo has been created. As soon as the above PR is merged, we can import that into mediawiki/services/mathoid and then structure the deploy repo properly.
@akosiaris , mediawiki/services/mathoid and mediawiki/services/mathoid/deploy and now set up as needed. I also took the liberty to improve the mathoid to service::node patch, which should now work as advertised. Could you please test it and confirm?
Change 211667 had a related patch set uploaded (by Mobrovac):
Update mathoid to 7e184cf
Change 215016 had a related patch set uploaded (by Mobrovac):
Mathoid: update Vagrant puppet module using service::node
The deployment has been completed, and we now have a standardised Mathoid deployment workflow. Thanks to everyone that contributed to this!
I'm keeping this open until https://gerrit.wikimedia.org/r/#/c/215016/ is merged to keep track.
Change 215016 merged by jenkins-bot:
Mathoid: update Vagrant puppet module using service::node