Page MenuHomePhabricator

Remove reference to Elastica\Type from CirrusSearch and related extensions and upgrade to Elastica 7.1.5
Closed, ResolvedPublic5 Estimated Story Points

Description

The class Elastica\Type is being removed from Elastica so we should stop using it.

  • remove references to Elastica\Type from all supported MW extensions
  • upgrade mediawiki-vendor to depend on Elastica 7.1.5
  • upgrade the MW Elastica extension to depend on Elastica 7.1.5

Notes:

  • production and CI is still running against PHP 7.2 (elasticsearch/elasticsearch v7.11.0 must be forced because this is the last version explicitly supporting this version)

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 790731 had a related patch set uploaded (by DCausse; author: DCausse):

[mediawiki/extensions/WikibaseCirrusSearch@master] Remove reference to Elastica\Type

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

Change 790732 had a related patch set uploaded (by DCausse; author: DCausse):

[mediawiki/extensions/GeoData@master] Remove reference to Elastica\Type

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

Change 790734 had a related patch set uploaded (by DCausse; author: DCausse):

[mediawiki/extensions/CirrusSearch@master] [WIP] First pass at removing obvious Type usages

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

Change 790988 had a related patch set uploaded (by DCausse; author: DCausse):

[mediawiki/extensions/Wikibase@master] Fix cirrussearch stubs

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

Change 791634 had a related patch set uploaded (by DCausse; author: DCausse):

[mediawiki/vendor@master] Upgrade to Elastica 7.1.5

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

Change 791407 had a related patch set uploaded (by DCausse; author: DCausse):

[mediawiki/extensions/CirrusSearch@master] [WIP] Migrate to Elastica 7.1.5

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

Change 790731 merged by jenkins-bot:

[mediawiki/extensions/WikibaseCirrusSearch@master] Remove reference to Elastica\Type

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

Change 790734 merged by jenkins-bot:

[mediawiki/extensions/CirrusSearch@master] First pass at removing obvious Type usages

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

Change 791406 had a related patch set uploaded (by DCausse; author: DCausse):

[mediawiki/extensions/Elastica@master] [WIP] Switch to Elastica 7.1.5

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

dcausse renamed this task from Remove reference to Elastica\Type from CirrusSearch and related extensions to Remove reference to Elastica\Type from CirrusSearch and related extensions and upgrade to Elastica 7.1.5.May 16 2022, 8:38 AM
dcausse updated the task description. (Show Details)

Change 792114 had a related patch set uploaded (by DCausse; author: DCausse):

[mediawiki/extensions/CirrusSearch@es710] Migrate to Elastica 7.1.5

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

Change 791407 abandoned by DCausse:

[mediawiki/extensions/CirrusSearch@master] [WIP] Migrate to Elastica 7.1.5

Reason:

moved to the es710 branch

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

Gehel set the point value for this task to 5.May 16 2022, 3:29 PM
Gehel moved this task from Incoming to In Progress on the Discovery-Search (Current work) board.

Change 790732 merged by jenkins-bot:

[mediawiki/extensions/GeoData@master] Remove reference to Elastica\Type

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

Change 792652 had a related patch set uploaded (by Ladsgroup; author: DCausse):

[mediawiki/extensions/GeoData@wmf/1.39.0-wmf.12] Remove reference to Elastica\Type

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

Change 792652 merged by jenkins-bot:

[mediawiki/extensions/GeoData@wmf/1.39.0-wmf.12] Remove reference to Elastica\Type

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

Mentioned in SAL (#wikimedia-operations) [2022-05-18T09:06:45Z] <ladsgroup@deploy1002> Synchronized php-1.39.0-wmf.12/extensions/GeoData/includes/Searcher.php: Backport: [[gerrit:792652|Remove reference to Elastica\Type (T308044)]] (duration: 00m 52s)

Change 790988 abandoned by Ebernhardson:

[mediawiki/extensions/Wikibase@master] Fix cirrussearch stubs

Reason:

I9ea6a539f2a2e438584436fef7ce464314d9dcea changed wikibase to clone dependencies instead of using stubs, this is no longer necessary.

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

Change 823717 had a related patch set uploaded (by Ebernhardson; author: Ebernhardson):

[mediawiki/vendor@es7] Set branch to es7

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

Change 823736 had a related patch set uploaded (by Ebernhardson; author: Ebernhardson):

[mediawiki/vendor@es7] Upgrade to Elastica 7.11.0

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

Change 823717 abandoned by Ebernhardson:

[mediawiki/vendor@es7] Set branch to es7

Reason:

branch must be es710, not es7, to match the extension repositories

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

Change 823736 abandoned by Ebernhardson:

[mediawiki/vendor@es7] Upgrade to Elastica 7.1.5

Reason:

branch must be es710, not es7, to match the extension repositories

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

Change 823760 had a related patch set uploaded (by Ebernhardson; author: Ebernhardson):

[mediawiki/vendor@es710] Set branch to es710

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

Change 823761 had a related patch set uploaded (by Ebernhardson; author: Ebernhardson):

[mediawiki/vendor@es710] Upgrade to Elastica 7.1.5

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

Change 791634 abandoned by Ebernhardson:

[mediawiki/vendor@master] Upgrade to Elastica 7.11.0

Reason:

supserseded by I960c480fb83014b48116d9ece4f6473b647110e9. Vendor will be merged into a branch to allow cross-repo CI testing prior to full
production deployment.

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

Change 823760 merged by Ebernhardson:

[mediawiki/vendor@es710] Set branch to es710

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

Change 823761 merged by Ebernhardson:

[mediawiki/vendor@es710] Upgrade to Elastica 7.1.5

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

Change 791406 merged by jenkins-bot:

[mediawiki/extensions/Elastica@master] Switch to Elastica 7.1.5

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

Change 792114 merged by jenkins-bot:

[mediawiki/extensions/CirrusSearch@es710] Migrate to Elastica 7.1.5

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

EBernhardson subscribed.

afaict there are no more patches to review here. The re-tagging to wmf.26 was mistaken, the merge to Elastica master branch was supposed to be merged to es710 branch. The master branch change was reverted.

Change 827544 had a related patch set uploaded (by Ebernhardson; author: Ebernhardson):

[mediawiki/vendor@master] Upgrade to Elastica 7.1.5

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

Gehel claimed this task.

Change 827544 merged by Ebernhardson:

[mediawiki/vendor@master] Upgrade to Elastica 7.1.5

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