Skip to content

Conversation

@dati18
Copy link
Contributor

@dati18 dati18 commented Jan 28, 2026

Bug: T415664

Copy link
Contributor

@tarrow tarrow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know it's just a WIP but here's my thoughts:

  • let's go ahead and remove all the ES6 stuff
    • the actual ES6 deployment
    • the special ES6 transport
    • any ES6 related environment variables
    • resolve any code that had special behaviour when those variables were set
    • remove from the docs any references to ES6

@dati18 dati18 requested a review from tarrow February 3, 2026 08:57
@dati18 dati18 changed the title WIP: Switch default cluster to ES7 Switch default cluster to ES7 Feb 3, 2026
- MW_WRITE_ONLY_ELASTICSEARCH_HOST=elasticsearch-7.svc
- MW_WRITE_ONLY_ELASTICSEARCH_PORT=9200
- MW_WRITE_ONLY_ELASTICSEARCH_ES6=false
- MW_DEFAULT_ELASTICSEARCH_ES6=false
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How come this line remains? I thought this PR was removing ES6.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah can you test it? Whenever I remove it, everything goes "kaputt"

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just tried commenting out MW_DEFAULT_ELASTICSEARCH_ES6 and followed the instructions in https://github.com/wbstack/mediawiki/pull/525/changes. The first time didn't seem to work, but after throwing everything away (docker compose down --volumes) and trying again I was able to successfully create some items and do a case-insensitive search for them. The output at http://localhost:9200/mwdb_somedb1_content_first/_search also looked sensible.

Have you tried recreating your local set-up? What specifically is "kaputt"?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants