This gives an error about /usr/bin/env: 'python': No such file or directory so it's probably been broken since the upgrade two weeks ago.
Description
Details
Related Objects
Event Timeline
This feature might have been non-operational even before the upgrade. It's not advertised and we don't know of any complaints. Worth looking into but not high priority.
Change 993759 had a related patch set uploaded (by EoghanGaffney; author: EoghanGaffney):
[operations/puppet@production] [phabricator] Fix commenting on tasks by email
Change 993766 had a related patch set uploaded (by Paladox; author: Paladox):
[operations/puppet@production] Phabricator: switch python to python3 in phab_epipe
Change 993766 abandoned by Paladox:
[operations/puppet@production] Phabricator: switch python to python3 in phab_epipe
Reason:
https://gerrit.wikimedia.org/r/c/operations/puppet/+/993759
Change 993759 merged by EoghanGaffney:
[operations/puppet@production] [phabricator] Fix commenting on tasks by email
The patch did get the script to run, but there seems to be an error from Phabricator in addition to this:
[2024-01-29 19:28:10] EXCEPTION: (RuntimeException) Undefined index: subject at [<arcanist>/src/error/PhutilErrorHandler.php:251] arcanist(), ava(), phorge(), translations(), wmf-ext-misc() #0 PhutilErrorHandler::handleError(integer, string, string, integer, array) called at [<phorge>/scripts/mail/mail_handler.php:58]
What's documented is creating tasks by email, which also fails but in a different way:
A message that you sent could not be delivered to one or more of its recipients. This is a permanent error. The following address(es) failed: general@phabricator.wikimedia.org (generated from task@phabricator.wikimedia.org) local delivery failed The following text was generated during the delivery attempt: ------ general@phabricator.wikimedia.org (generated from task@phabricator.wikimedia.org) ------ /usr/local/bin/phab_epipe.py:57: DeprecationWarning: The SafeConfigParser class has been renamed to ConfigParser in Python 3.2. This alias will be removed in future versions. Use ConfigParser directly instead. parser = configparser.SafeConfigParser() Error parsing message Please contact a WMF Phabricator admin. [2024-02-06 15:22:26] EXCEPTION: (RuntimeException) Undefined index: subject at [<arcanist>/src/error/PhutilErrorHandler.php:251] arcanist(), ava(), phorge(), translations(), wmf-ext-misc() #0 PhutilErrorHandler::handleError(integer, string, string, integer, array) called at [<phorge>/scripts/mail/mail_handler.php:58]
/usr/local/bin/phab_epipe.py:57: DeprecationWarning: The SafeConfigParser class has been renamed to ConfigParser in Python 3.2. This alias will be removed in future versions. Use ConfigParser directly instead.
Fixed by rOPUP1f87f1edb83b535ac424a793d8fe0ef848f670ee merged today.
Please contact a WMF Phabricator admin.
I'm curious if upstream https://we.phorge.it/rPa69db10c5e15be4b7d851099b48c30f7cfe11046 and https://we.phorge.it/rP186768ccfd8895af80137bc3c6f41281ed744cfc and https://we.phorge.it/rP644f179dd27a78b91c88366b9432f5faaeb2bbb4 might change anything once deployed here, given the script is using stdin
Still an issue after deploying rOPUP1f87f1edb83b535ac424a793d8fe0ef848f670ee...should check again after pulling a newer upstream (see previous comment) to have at least more correct error output than now:
Error parsing message Please contact a WMF Phabricator admin. [2024-04-29 12:42:57] EXCEPTION: (RuntimeException) Undefined index: subject at [<arcanist>/src/error/PhutilErrorHandler.php:263] arcanist(), ava(), phorge(), translations(), wmf-ext-misc() #0 PhutilErrorHandler::handleError(integer, string, string, integer, array) called at [<phorge>/scripts/mail/mail_handler.php:58]