Page MenuHomePhabricator

Split visualeditor Vagrant role
Closed, ResolvedPublic

Description

The visualeditor Vagrant role pulls in a bunch of extensions, and (more problematically) the Parsoid, RESTBase and Citoid services (Parsoid could probably be discarded now; that's T259989: Support RESTBase using Parsoid/PHP in MediaWiki-Vagrant), which have conflicting Node.js requirements with the eventgate role (T217113: MediaWiki-Vagrant should use the same Node.js version as Wikimedia production) and are generally fragile. While that might be a worthwhile tradeoff for VisalEditor development, other roles which only need VisualEditor's basic functionality (e.g. flow or growthexperiments) should be able to install that without pulling in all the optional services. Also, we expect barebones VisualEditor to be widely used by third parties, so having a role for testing it is valuable.

Thus, the role should be split into a "basic" and "development" version, similar to how it was done for the flow role (T133629).

Event Timeline

Once this is done we should audit other extensions/roles that depend on VE:

  • ContentTranslation
  • DiscussionTools

Change 620776 had a related patch set uploaded (by Gergő Tisza; owner: Gergő Tisza):
[mediawiki/vagrant@master] Split visualeditor role

https://gerrit.wikimedia.org/r/620776

Change 620776 merged by jenkins-bot:
[mediawiki/vagrant@master] Split visualeditor role

https://gerrit.wikimedia.org/r/620776

Tgr claimed this task.