⚓ T192166 Drop HHVM support from MediaWiki

Status	Assigned	Task
Stalled	None	T216734 Drop devDependency on phpunit-mock-objects, which is abandoned and warns against use
Open	None	T233012 Make MediaWiki core compatible with PHP 7.4
Open	None	T192167 Upgrade PHPUnit from 4/6 to 8
Resolved	Reedy	T196206 Bump symfony libraries when we no longer need hhvm support
Resolved	Daimona	T220589 Support phan in MediaWiki's default composer.json
Resolved	Daimona	T231710 Recommend PHP7-only features after dropping HHVM support
Resolved	aaron	T232128 Make MultiHttpClient use CURLMOPT_MAX_HOST_CONNECTIONS and reuse connections
Resolved	MaxSem	T234236 Drop HHVM support from MediaWiki-Vagrant
Stalled	None	T227242 Deploy WebAuthn to Wikimedia Wikis
Open	Daimona	T231636 Butcher the list of phan suppressions in MW core
Open	None	T235266 Composer Library Updates
Resolved	Reedy	T235233 Stop using PHP5 versions of ruflin/elastica and elasticsearch/elasticsearch
Open	None	T234512 Archive HHVM Phabricator project
Open	None	T235160 Decide whether extensions should set a PHP version requirement in composer.json
Resolved	Jdforrester-WMF	T192166 Drop HHVM support from MediaWiki
Resolved	Jdforrester-WMF	T234062 ci-src-setup job (used by mediawiki-core-php72-phan-docker) is still running on PHP 7.0.33
Resolved	Jdforrester-WMF	T234384 Drop HHVM from CI
Open	jijiki	T229792 Remove HHVM from production

Liuxinyu970226 removed a subscriber: Liuxinyu970226.Oct 1 2018, 11:42 PM

While we'll have to drop HHVM support soon, we're migrating to PHP 7 even sooner. So this isn't currently blocking PHP 7.0 support.

Per T176370, the initial production roll-out will likely use PHP 7.2, or maybe PHP 7.3 even (we're currently evaluating the viability of fast-tracking that and do the gradual roll-out straight from HHVM to PHP 7.3 for user traffic). More details and question at T176370, but it does mean we won't drop HHVM until after PHP 7.3 support is finalised.

Jdforrester-WMF added a subtask: T176370: Migrate to PHP 7 in WMF production.Oct 12 2018, 6:49 PM

cscott added a parent task: T204945: Deprecate one of the Preprocessor implementations for 1.34.Oct 18 2018, 6:37 PM

GTirloni removed a subscriber: GTirloni.Dec 20 2018, 6:46 PM

Reedy mentioned this in T212642: Quiz extension incompatibility with HHVM: RuntimeException: Requested template, Quiz, is not callable.Dec 28 2018, 6:15 PM

Krinkle removed a parent task: T204945: Deprecate one of the Preprocessor implementations for 1.34.Feb 5 2019, 12:00 AM

Anomie mentioned this in T216102: Determine which PHP version to target with Parsoid.Feb 14 2019, 4:45 PM

Jdforrester-WMF added a parent task: T220589: Support phan in MediaWiki's default composer.json.Apr 11 2019, 12:14 AM

Change 434973 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/core@master] [DNM][WIP][GOSH] Drop HHVM support from MediaWiki

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

Jdforrester-WMF mentioned this in T216165: Drop PHP 7.0 support from MediaWiki.Jul 6 2019, 12:44 AM

Krinkle mentioned this in T119818: hhvm unable to stat files.Jul 11 2019, 10:45 PM

Michael mentioned this in T228855: Fix broken WikibaseQualityConstraints tests.Jul 24 2019, 4:19 PM

cscott mentioned this in T213494: Installing composer modules for deployment.Jul 31 2019, 9:34 PM

There is now the Hiera key profile::mediawiki::install_hhvm that let's us remove HHVM entirely from appservers individually.

Simetrical mentioned this in T228342: Define criteria for setting explicit PHP support target for MediaWiki.Aug 13 2019, 11:16 AM

abi_ mentioned this in T231270: Failing build process: Variadic params with type constraints are not supported in non-Hack files.Aug 27 2019, 7:15 AM

Daimona mentioned this in T225730: Reduce runtime of MW shared gate Jenkins jobs to 5 min.Aug 29 2019, 6:09 PM

Daimona added a parent task: T231710: Recommend PHP7-only features after dropping HHVM support.Aug 31 2019, 8:28 AM

Daimona added a parent task: T231636: Butcher the list of phan suppressions in MW core.Sep 3 2019, 4:45 PM

Change 534521 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[integration/config@master] [DNM] layout: Drop HHVM testing from all quibble jobs

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

Reedy added a parent task: T232128: Make MultiHttpClient use CURLMOPT_MAX_HOST_CONNECTIONS and reuse connections.Sep 10 2019, 3:22 PM

kostajh added a subscriber: kostajh.Tue, Sep 24, 11:07 AM

Will this happen in 1.35?

I think we're trying to make it happen as part of 1.34, might need to be backported at the last minute.

Even we don't make it, what will happen is that we drop support in documentation, but that it might (accidentally) work during the first .0 release, but we have no intention of supporting it officially. Once WMF has migrated away, all testing on master and REL1_34+ will cease, possibly for older branches as well (to be determined).

In T192166#5520176, @Krinkle wrote:

I think we're trying to make it happen as part of 1.34, might need to be backported at the last minute.

Huh, that soon? I thought we would deprecate HHVM support in 1.34, and remove it in 1.35.

Even we don't make it, what will happen is that we drop support in documentation, but that it might (accidentally) work during the first .0 release, but we have no intention of supporting it officially. Once WMF has migrated away, all testing on master and REL1_34+ will cease, possibly for older branches as well (to be determined).

Good, thanks. So, basically, we can update the docs and unblock these patches as soon as 1.34 is cut?

Once HHVM is completely and disabled from WMF production servers relating to MediaWiki and e.g. 1-2 weeks have passed, then we can immediately remove support for it in master.

For documentation it is fine to already remove it. It has been stated for several releases already that HHVM only works for WMF and will be removed as soon as the migration is completed. We will not be supporting HHVM for the full 1.34 release cycle.

Krinkle added a project: TechCom.Wed, Sep 25, 8:10 PM

Krinkle moved this task from Inbox to Watching on the TechCom board.

Not HHVM-related, but posting here before I forget: https://codesearch.wmflabs.org/search/?q=void%20PHP%20*7.%2Barning&i=nope&files=&repos= can also be cleaned up.

Change 539326 had a related patch set uploaded (by Giuseppe Lavagetto; owner: Giuseppe Lavagetto):
[operations/puppet@production] mediawiki: remove the PHP/HHVM conditionals from the code

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

In T192166#5520214, @Daimona wrote:

In T192166#5520176, @Krinkle wrote:

I think we're trying to make it happen as part of 1.34, might need to be backported at the last minute.

Huh, that soon? I thought we would deprecate HHVM support in 1.34, and remove it in 1.35.

It was never really supported, and deprecated in 1.31. It's only dragged on for so long because of Wikimedia production.

Good, thanks. So, basically, we can update the docs and unblock these patches as soon as 1.34 is cut?

No.

Daimona added a subtask: T234062: ci-src-setup job (used by mediawiki-core-php72-phan-docker) is still running on PHP 7.0.33.Fri, Sep 27, 5:57 PM

Lucas_Werkmeister_WMDE added a subscriber: Lucas_Werkmeister_WMDE.Mon, Sep 30, 2:48 PM

Mainframe98 mentioned this in T234236: Drop HHVM support from MediaWiki-Vagrant.Mon, Sep 30, 3:54 PM

Mainframe98 added a parent task: T234236: Drop HHVM support from MediaWiki-Vagrant.

Jdforrester-WMF added a subtask: T234384: Drop HHVM from CI.Tue, Oct 1, 9:45 PM

Jdforrester-WMF changed the status of subtask T234384: Drop HHVM from CI from Open to Stalled.

Jdforrester-WMF changed the status of subtask T234384: Drop HHVM from CI from Stalled to Open.Wed, Oct 2, 6:52 PM

Change 540488 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/core@master] Drop HHVM support from MediaWiki

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

Change 540488 merged by jenkins-bot:
[mediawiki/core@master] Drop HHVM support from MediaWiki

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

ReleaseTaggerBot added a project: MW-1.35-notes (1.35.0-wmf.1; 2019-10-08).Wed, Oct 2, 11:00 PM

Change 540502 had a related patch set uploaded (by MaxSem; owner: MaxSem):
[mediawiki/vagrant@master] Drop HHVM support

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

Krinkle mentioned this in T207523: JsonContent incorrectly reports invalid JSON is valid, only on HHVM.Thu, Oct 3, 2:32 AM

Krinkle mentioned this in T76831: HHVM's xhprof_frame_begin() calls from XhprofFrame constructor closed too soon.Thu, Oct 3, 3:23 AM

Krinkle mentioned this in T133674: HHVM is leaking memory on the API appservers.

Krinkle mentioned this in T91056: Some letters (initial characters of page titles) not being correctly capitalised.Thu, Oct 3, 3:27 AM

Krinkle mentioned this in T229379: On zh.wikipedia.org, MediaWiki wrongly interprets an article about main pages as the wiki's main page.

Krinkle mentioned this in T178242: On HHVM, "CACHE_MEMCACHED" shortcut should select native client (default PHP implementation has issues).Thu, Oct 3, 3:30 AM

Krinkle mentioned this in T157213: HHVM and PHP handle non-static input to call_user_func_array() different.

Krinkle mentioned this in T153881: Don't require eval() (via TemplateParser) to run MediaWiki.

Krinkle mentioned this in T78558: HHVM segfault when not passing parameter by reference when required.Thu, Oct 3, 3:32 AM

Krinkle mentioned this in T126473: CI slaves should have HHVM call the exception user handler so we have useful stack trace on fatal errors.

Krinkle mentioned this in T115781: Grep for func_get_arg()/func_get_args() and fix an HHVM reference bugs.Thu, Oct 3, 3:38 AM

Bugreporter mentioned this in T234512: Archive HHVM Phabricator project.Thu, Oct 3, 8:32 AM

Peachey88 added a subtask: T234512: Archive HHVM Phabricator project.Thu, Oct 3, 9:23 AM

Change 540767 had a related patch set uploaded (by MaxSem; owner: MaxSem):
[mediawiki/php/wikidiff2@master] Drop HHVM support

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

Change 540767 merged by jenkins-bot:
[mediawiki/php/wikidiff2@master] Drop HHVM support

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

Daimona updated the task description. (Show Details)Sun, Oct 6, 8:38 AM

Change 541141 had a related patch set uploaded (by MaxSem; owner: MaxSem):
[mediawiki/extensions/wikihiero@master] Remove HHVM check

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

Change 541143 had a related patch set uploaded (by MaxSem; owner: MaxSem):
[mediawiki/core@master] Hard-deprecate wfIsHHVM()

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

Change 426165 abandoned by MaxSem:
WIP: kill HHVM specific hacks

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

Change 541141 merged by jenkins-bot:
[mediawiki/extensions/wikihiero@master] Remove HHVM check

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

MaxSem mentioned this in rEHIEdc5dda074699: Remove HHVM check.Mon, Oct 7, 2:11 PM

Change 541266 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/core@REL1_34] Drop HHVM support from MediaWiki

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

Change 541266 merged by jenkins-bot:
[mediawiki/core@REL1_34] Drop HHVM support from MediaWiki

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

ReleaseTaggerBot added a project: MW-1.34-notes.Mon, Oct 7, 3:00 PM

Change 541143 merged by jenkins-bot:
[mediawiki/core@master] Hard-deprecate wfIsHHVM()

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

I'm declaring this Resolved.

In T192166#5552909, @Jdforrester-WMF wrote:

I'm declaring this Resolved.

Macro itshappening:

Ladsgroup changed the status of subtask T234512: Archive HHVM Phabricator project from Stalled to Open.Mon, Oct 7, 8:03 PM

Change 540502 merged by jenkins-bot:
[mediawiki/vagrant@master] Drop HHVM support

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

Jdforrester-WMF closed subtask T234062: ci-src-setup job (used by mediawiki-core-php72-phan-docker) is still running on PHP 7.0.33 as Resolved.Thu, Oct 10, 6:54 PM

Reedy added a parent task: T235233: Stop using PHP5 versions of ruflin/elastica and elasticsearch/elasticsearch.Thu, Oct 10, 11:34 PM

Change 434973 merged by jenkins-bot:
[mediawiki/core@master] PHPVersionCheck: Simplify warning as part of dropping HHVM support

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

ReleaseTaggerBot edited projects, added MW-1.35-notes (1.35.0-wmf.2; 2019-10-15); removed MW-1.35-notes (1.35.0-wmf.1; 2019-10-08).Fri, Oct 11, 2:00 AM

Krinkle removed a parent task: T192167: Upgrade PHPUnit from 4/6 to 8.Fri, Oct 11, 5:56 PM

Krinkle removed a parent task: T231636: Butcher the list of phan suppressions in MW core.Sat, Oct 12, 4:51 PM

Krinkle mentioned this in T231636: Butcher the list of phan suppressions in MW core.

Krinkle mentioned this in T235266: Composer Library Updates.Sat, Oct 12, 4:53 PM

Krinkle mentioned this in T227242: Deploy WebAuthn to Wikimedia Wikis.

Krinkle added a parent task: T227242: Deploy WebAuthn to Wikimedia Wikis.

Drawing the line between "stuff that happens post-HHVM" and "stuff required for WMF to start using PHP7" at this task. It's useful to track al the things the migration unblocked, so I've re-added them as parent tasks here (sorry about the noise), and instead broke the link with T176370 in order to keep the tree somewhat sane.

Krinkle added a parent task: T192167: Upgrade PHPUnit from 4/6 to 8.Sat, Oct 12, 4:57 PM

Krinkle added a parent task: T231636: Butcher the list of phan suppressions in MW core.

Krinkle added a parent task: T235266: Composer Library Updates.

Krinkle removed a subtask: T234512: Archive HHVM Phabricator project.Sat, Oct 12, 5:00 PM

Krinkle added a parent task: T234512: Archive HHVM Phabricator project.

Change 539326 merged by Effie Mouzeli:
[operations/puppet@production] mediawiki: remove the PHP/HHVM conditionals from the code

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

Jdforrester-WMF removed a subtask: T235160: Decide whether extensions should set a PHP version requirement in composer.json.Tue, Oct 15, 3:17 PM

Jdforrester-WMF added a parent task: T235160: Decide whether extensions should set a PHP version requirement in composer.json.