Page MenuHomePhabricator

Bundle Echo extension with MediaWiki
Open, Stalled, MediumPublic

Description

  • Passed security review or already Wikimedia deployed
  • Voting CI structure tests
  • Runs MediaWiki-CodeSniffer
  • Runs phan
  • Supports MySQL, SQLite, and Postgres (if there are schema changes)
  • GPL v2 or later compatible license
  • Extension's default configuration provides optimal experience
  • Tested with web installer

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 8 2018, 6:25 PM
Tgr added a subscriber: Tgr.Apr 8 2018, 8:56 PM
Legoktm updated the task description. (Show Details)Apr 9 2018, 2:21 AM
Restricted Application added a project: Growth-Team. · View Herald TranscriptSep 2 2018, 10:33 AM
SBisson moved this task from Inbox to External on the Growth-Team board.Sep 7 2018, 12:19 PM
Tgr updated the task description. (Show Details)Oct 11 2018, 11:14 PM

License is MIT which is compatible with GPL 2+.

MGChecker updated the task description. (Show Details)Oct 12 2018, 9:10 AM
CCicalese_WMF renamed this task from Bundle Echo extension with MW 1.32 to Bundle Echo extension with MediaWiki.Nov 10 2018, 6:49 PM

What is the situation with respect to database support?

CCicalese_WMF triaged this task as Medium priority.Sep 13 2019, 4:48 PM

@Anomie, do you have any insight into the database support in Echo?

Reedy added a subscriber: Reedy.Sep 13 2019, 5:05 PM

@Anomie, do you have any insight into the database support in Echo?

I think sqlite and mysql are good. Postgres if it works, is more by accident (ie the sql works there too) than any sort of planning

Postgres if it works, is more by accident (ie the sql works there too) than any sort of planning

It almost certainly doesn't work then, since for example the syntax for defining auto-incrementing ID fields is different, PG doesn't have "unsigned", and other incompatibilities.

It might work if someone were to write the schema for it and wire it into EchoHooks::onLoadExtensionSchemaUpdates(). MediaWiki's DQL and DML abstractions are pretty good if used right, it's just the DDL that's almost completely not abstracted.

Jdforrester-WMF changed the task status from Open to Stalled.Sep 17 2019, 2:15 PM
Jdforrester-WMF edited projects, added MW-1.35-release; removed MW-1.34-release.
Jdforrester-WMF added a subscriber: Jdforrester-WMF.

Based on the determination above, and that clearly we are not going to get DB schemata written, tested, and merged in the next 11 days when there are no volunteers who care about PG support around, I'm moving this and related tasks to 1.35 and marking as Stalled.

Krinkle moved this task from Blocker to Bundling on the MW-1.35-release board.Oct 11 2019, 10:45 PM

Change 565045 had a related patch set uploaded (by Legoktm; owner: Jack Phoenix):
[mediawiki/extensions/Echo@master] PostgreSQL support

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

Akuckartz added a subscriber: Akuckartz.

I seriously doubt anyone is going to fix CentralAuth to work with postgres any time soon. :-(

Tgr added a comment.Jul 8 2020, 10:26 AM

I seriously doubt anyone is going to fix CentralAuth to work with postgres any time soon. :-(

Why is that a dependency? CentralAuth is not bundled; if Echo works in Postgres as part of the bundle but doesn't work when random extension X is also added to the bundle, that should not be a blocker IMO.

I seriously doubt anyone is going to fix CentralAuth to work with postgres any time soon. :-(

Why is that a dependency? CentralAuth is not bundled; if Echo works in Postgres as part of the bundle but doesn't work when random extension X is also added to the bundle, that should not be a blocker IMO.

If you want to write a set of config in CI that knows to pull the CentralAuth dependency from Notifications but only when running the postgresql DB back-end, and convince RelEng to allow that complexity into their stack, be my guest.

Legoktm moved this task from Blocker to Bundling on the MW-1.36-release board.Jul 15 2020, 4:30 AM
Aklapper removed a subscriber: Anomie.Fri, Oct 16, 5:02 PM