Page MenuHomePhabricator

Make ircecho much better
Closed, InvalidPublic

Description

Current code is a fair bit of a mess, and the architecture is also a bit 'eeeeughh'.

Related Objects

Event Timeline

yuvipanda raised the priority of this task from to Needs Triage.
yuvipanda updated the task description. (Show Details)
yuvipanda added subscribers: yuvipanda, ori, faidon.
Andrew triaged this task as Low priority.Apr 6 2015, 4:33 PM
Andrew set Security to None.

@Krenair I don't know what the difference is, though the service is called ircecho and that spawns udpmxircecho

kraz:~$ grep -i exec /etc/systemd/system/ircecho.service
ExecStart=/usr/local/bin/udpmxircecho.py

They're completely separate scripts reading data to broadcast in different ways and are set up to send to different networks. There's also tcpircbot.

alex@alex-laptop:~/Development/Wikimedia/Operations-Puppet (master)$ find modules -name '*ircecho*'
modules/mw_rc_irc/files/systemd/ircecho.service
modules/mw_rc_irc/files/upstart/ircecho.conf
modules/mw_rc_irc/files/udpmxircecho.py
modules/ircecho
modules/ircecho/files/ircecho
modules/ircecho/files/ircecho.init

Perhaps we should rename some of them to avoid confusion (e.g. "udpmxircecho" as "rcbot"?) and start being consistent in our naming?

Yes, and also merge ircecho with tcpircbot. I've had patches open to work on that for over 6 months (one of which is still WIP waiting for the dependency).

Danny_B renamed this task from Make ircecho much better (Tracking) to Make ircecho much better.Jul 15 2016, 11:17 PM
Danny_B edited projects, added IRCecho; removed Tracking-Neverending.

If you think none of the information in the comments above is worth to be transferred to task on its own, feel free to close this task, please.

Marostegui subscribed.

Closing this task as it doesn't provide much details on what actually needs to be done.
If someone feels this needs to stay open, please reopen.