Page MenuHomePhabricator

Remove pear/mail packages from WMF MW app servers
Open, NormalPublic

Description

Following up T97454... We should remove php-mail and php-mail-mime from WMF MW app servers

Event Timeline

Reedy created this task.May 23 2018, 11:34 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMay 23 2018, 11:34 AM

Change 434710 had a related patch set uploaded (by Reedy; owner: Reedy):
[operations/puppet@production] Remove pear packages from MW Application Servers

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

MoritzMuehlenhoff triaged this task as Normal priority.May 28 2018, 11:13 AM
Vvjjkkii renamed this task from Remove pear/mail packages from WMF MW app servers to jgcaaaaaaa.Jul 1 2018, 1:09 AM
Vvjjkkii raised the priority of this task from Normal to High.
Vvjjkkii updated the task description. (Show Details)
Vvjjkkii removed subscribers: gerritbot, Aklapper.
CommunityTechBot renamed this task from jgcaaaaaaa to Remove pear/mail packages from WMF MW app servers.Jul 2 2018, 4:41 AM
CommunityTechBot lowered the priority of this task from High to Normal.
CommunityTechBot updated the task description. (Show Details)
CommunityTechBot added subscribers: gerritbot, Aklapper.
Tgr added a subscriber: Tgr.Feb 4 2019, 11:39 PM

Per T215126: PHP warning on some Echo email sending attempts due to mismatching PEAR file versions, we are actually loading the PEAR version of classes in production, even though the Composer version is included in mediawiki/vendor, causing confusion and sometimes bugs.

Tgr added a comment.Feb 5 2019, 12:19 AM

Note that the pear/mail files (the ones we get from composer as well) start with require_once 'PEAR.php'; require_once 'Mail/mimePart.php'; and the Debian package is the only way those requires can be fulfilled, so removing the packages will probably break email sending. This seems like a Composer bug, filed as T215224: PEAR PHP classes are loaded from system packages instead of Composer packages in WMF production.

Change 434710 had a related patch set uploaded (by Krinkle; owner: Reedy):
[operations/puppet@production] Remove pear packages from MW Application Servers

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

jijiki added a subscriber: jijiki.
Joe added a subscriber: Joe.Jul 23 2019, 2:33 PM

@Tgr do you see any reason not to uninstall those packages? I will for now just remove them from puppet, and uninstall them on a few selected appservers only.

Change 434710 merged by Giuseppe Lavagetto:
[operations/puppet@production] Stop installing pear packages on MW Application Servers

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

Tgr added a comment.Jul 23 2019, 2:38 PM

@Tgr do you see any reason not to uninstall those packages? I will for now just remove them from puppet, and uninstall them on a few selected appservers only.

In theory uninstalling should be fine now that T215224: PEAR PHP classes are loaded from system packages instead of Composer packages in WMF production has been fixed.

Mentioned in SAL (#wikimedia-operations) [2019-07-23T15:08:33Z] <_joe_> uninstalling php-pear, php-mail, php-mail-mime from mw1267 T195364

jijiki moved this task from Backlog to Next up on the serviceops board.Sep 13 2019, 4:43 AM

@Joe should I remove php-pear, php-mail, php-mail-mime from the rest of the fleet?