Page MenuHomePhabricator

Clean up mediawiki-config from HHVM-related configuration
Closed, ResolvedPublic

Description

Currently, there are quite a lot of special cases that are not needed anymore.


Status as of 2019-12-05:

Code search results

  • Mention in composer.lock from sebastian/environment which should go away in future when we bump PHPUnit.
  • Mentions in the upstream mongofill package used by Performance.
  • Documentation mention in 404.php and README which can be updated (but to what?).
  • Documentation mention in ServiceConfig.php of hhvm-fatal-error.php which has been… removed now?
  • Used in arclamp.php which presumably is also used by PHP72?
  • Documention mention in profiler.php abouse historical uses, and can stay.

Event Timeline

Change 542178 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[operations/mediawiki-config@master] tests: Declare strict types for the static test now HHVM is gone

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

Change 542179 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[operations/mediawiki-config@master] Wikibase: Don't check to shard wmgWBSharedCacheKey for HHVM any more

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

Change 542180 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[operations/mediawiki-config@master] Don't check to shard static config cache for HHVM any more

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

Change 542181 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[operations/mediawiki-config@master] Drop HHVM special-case for SVG converter, no longer used

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

Change 542182 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[operations/mediawiki-config@master] Drop special-case for PHP7 in PHPAutoPrepend, now always used

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

Change 542183 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[operations/mediawiki-config@master] Drop nutcracker indirection for HHVM servers, just point to localhost

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

Change 542184 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[operations/mediawiki-config@master] Drop HHVMRequestInit, never called

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

Change 542185 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[operations/mediawiki-config@master] Drop HHVM XHProf and Arclamp code, no longer called

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

Change 542178 merged by jenkins-bot:
[operations/mediawiki-config@master] tests: Declare strict types for the static test now HHVM is gone

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

Change 542179 merged by jenkins-bot:
[operations/mediawiki-config@master] Wikibase: Don't check to shard wmgWBSharedCacheKey for HHVM any more

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

Change 542180 merged by jenkins-bot:
[operations/mediawiki-config@master] Don't check to shard static config cache for HHVM any more

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

Change 542181 merged by jenkins-bot:
[operations/mediawiki-config@master] Drop HHVM special-case for SVG converter, no longer used

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

Change 542182 merged by jenkins-bot:
[operations/mediawiki-config@master] Drop special-case for PHP7 in PHPAutoPrepend, now always used

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

Change 542183 merged by jenkins-bot:
[operations/mediawiki-config@master] Drop nutcracker indirection for HHVM servers, just point to localhost

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

Change 542185 merged by jenkins-bot:
[operations/mediawiki-config@master] Drop HHVM XHProf and Arclamp code, no longer called

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

Krinkle subscribed.

Moving from tracked under "Stuff part of deploying PHP 7 properly" to instead above "Stuff unblocked by not having HHVM".

Change 553144 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[operations/mediawiki-config@master] CommonSettings: Drop Scribunto special-case for HHVM, never reached

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

Change 542184 merged by jenkins-bot:
[operations/mediawiki-config@master] Drop HHVMRequestInit, never called

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

Change 553144 merged by jenkins-bot:
[operations/mediawiki-config@master] CommonSettings: Drop Scribunto special-case for HHVM, never reached

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

Mentioned in SAL (#wikimedia-operations) [2019-11-26T16:36:27Z] <jforrester@deploy1001> Synchronized wmf-config/CommonSettings.php: Drop Scribunto special-case for HHVM, never reached T235142 (duration: 00m 52s)

Change 554836 had a related patch set uploaded (by Krinkle; owner: Krinkle):
[operations/mediawiki-config@master] profiler: Remove arclamp.php (was for HHVM)

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

Change 554836 merged by jenkins-bot:
[operations/mediawiki-config@master] profiler: Remove arclamp.php (was for HHVM)

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

Change 565687 had a related patch set uploaded (by Dzahn; owner: Dzahn):
[operations/puppet@production] admin::ori: remove hhvm related commands

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

Change 565688 had a related patch set uploaded (by Dzahn; owner: Dzahn):
[operations/puppet@production] deployment-prep: remove install_hhvm from Hiera for mediawiki-parsoid10

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

Change 565688 merged by Dzahn:
[operations/puppet@production] deployment-prep: remove install_hhvm from Hiera for mediawiki-parsoid10

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

Change 565687 merged by Dzahn:
[operations/puppet@production] admin::ori: remove hhvm related commands

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

Mentioned in SAL (#wikimedia-releng) [2021-04-16T19:11:05Z] <Krinkle> Remove profile::mediawiki::install_hhvm: false Hiera config in Horizon for deployment-prep. This variable is no longer used. ref T235142

Krinkle claimed this task.
Krinkle added a project: Performance-Team.

Change 858441 had a related patch set uploaded (by Krinkle; author: Krinkle):

[operations/mediawiki-config@master] build: Update to PHPUnit 9.5 and declare php requirement

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

Change 858441 merged by jenkins-bot:

[operations/mediawiki-config@master] build: Update to PHPUnit 9.5 and declare php requirement

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