similar to https://github.com/nolanlum/kochira/blob/master/kochira/services/web/youtube.py

We copied this from kochira, the old unofficial IRC bot that used to be in the channel, but it's in pretty poor taste and we should replace it.

Some possibilities:

• X is the worst
• X went to stanford
• X has outdated wordpress
• X voted for trump
• X called us "the ofc"
• X killed net neutrality
• X uses windows
• X caused a kernel panic

any other suggestions?

We don't necessarily need to pick just one; it'd be cool to have a list of lots of these and have each !sux invocation randomly pick 4 to use.

Let's bounce users to the right resource. It's a better experience than an inexplicable no.

Most of the work will be in ocflib. I'll make an issue there as well.

This was discussed at BoD yesterday.

I proposed we use RT's REST API to post to a specific channel whenever a new help ticket is posted. In addition, we could have create post to #rebuild when a security ticket is sent (since those are higher priority).

We could also have create ping a group (root? sms? help queue group?) for security tickets especially.

it raises some exception and the bot crashes

I believe we can do this by using a different buffer, like the following (taken from https://pypi.python.org/pypi/irc under "Decoding Input"):

from jaraco.stream import buffer
irc.client.ServerConnection.buffer_class = buffer.LenientDecodingLineBuffer

Otherwise, if invalid tokens are given, we get this error:

Exception in thread Thread-2:
Traceback (most recent call last):
 File "/usr/lib/python3.5/threading.py", line 914, in _bootstrap_inner
   self.run()
 File "/usr/lib/python3.5/threading.py", line 862, in run 
   self._target(*self._args, **self._kwargs)
 File "/opt/ircbot/venv/lib/python3.5/site-packages/irc/bot.py", line 332, in start
   super(SingleServerIRCBot, self).start()
 File "/opt/ircbot/venv/lib/python3.5/site-packages/irc/client.py", line 1195, in start
   self.reactor.process_forever()
 File "/opt/ircbot/venv/lib/python3.5/site-packages/irc/client.py", line 273, in process_forever
   consume(infinite_call(one))
 File "/opt/ircbot/venv/lib/python3.5/site-packages/more_itertools/recipes.py", line 159, in consume
   deque(iterator, maxlen=0)
 File "/opt/ircbot/venv/lib/python3.5/site-packages/jaraco/itertools.py", line 386, in <genexpr>
   return (f() for _ in itertools.repeat(None))
 File "/opt/ircbot/venv/lib/python3.5/site-packages/irc/client.py", line 254, in process_once
   self.process_data(i)
 File "/opt/ircbot/venv/lib/python3.5/site-packages/irc/client.py", line 218, in process_data
   c.process_data()
 File "/opt/ircbot/venv/lib/python3.5/site-packages/irc/client.py", line 535, in process_data
   for line in self.buffer:
 File "/opt/ircbot/venv/lib/python3.5/site-packages/jaraco/stream/buffer.py", line 104, in lines
   self.handle_exception()
 File "/opt/ircbot/venv/lib/python3.5/site-packages/jaraco/stream/buffer.py", line 102, in lines
   yield line.decode(self.encoding, self.errors)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xcc in position 511: unexpected end of data

Traceback (most recent call last):
 File "/usr/lib/python3.5/runpy.py", line 193, in _run_module_as_main
   "__main__", mod_spec)
 File "/usr/lib/python3.5/runpy.py", line 85, in _run_code
   exec(code, run_globals)
 File "/opt/ircbot/ircbot/ircbot.py", line 367, in <module>
   main()
 File "/opt/ircbot/ircbot/ircbot.py", line 361, in main
   raise RuntimeError('Thread exited: {}'.format(thread))
RuntimeError: Thread exited: <Thread(Thread-2, stopped daemon 140446527051520)>

Certain modules should work with private messaging. Not account approval or that kind of stuff, but things like ping or shruuuug, whatever help output we add as a part of #1, rackspace monitoring status, commands like that.

since many plugins are quite similar and have similar backing tables, it might make sense to consolidate them into a single table. For example, quotes and inspire, what_is and macros, etc. etc. might be a decent starter ticket project for new staff.

useful to know, probably would involve adding some new ocflib functions

Currently, all recent messages in all channels are stored together, which sometimes causes weird behavior when using regex substitution and a message from another channel matches it instead. Recent messages should be grouped by channel instead.

We should just make a function that is available to all plugins that add 0-length spaces to a message for everything that could be a nick. This is currently done (badly) in turing. (See #91.) We should do it better and probably add it to some other commands (such as !quote).

maybe at the bottom?

e.g. "create: help"

Our current weather API is no longer working. We need to find a new API and use that instead.

https://github.com/asottile/ircbot/commit/54105df6914d444e539c94738ec26964be2597ce had to be done to get this to work.

When the command !shorturl add ^fieldunits <redacted> was run, the following error was reported.

A problem was encountered and reported via ocflib:

ircbot exception in shorturls/add: shorturl '^fieldunits' contains illegal characters

Traceback (most recent call last):
File "/opt/ircbot/ircbot/ircbot.py", line 250, in on_pubmsg
listener.fn(self, msg)
File "ircbot/plugin/shorturls.py", line 45, in add
add_shorturl(ctx, slug, target)
File "/opt/ircbot/venv/lib/python3.7/site-packages/ocflib/misc/shorturls.py", line 38, in add_shorturl
_validate_slug(slug)
File "/opt/ircbot/venv/lib/python3.7/site-packages/ocflib/misc/shorturls.py", line 22, in _validate_slug
raise ValueError("shorturl '{}' contains illegal characters".format(slug))
ValueError: shorturl '^fieldunits' contains illegal characters

This should work just like the Marathon plugin. Perhaps the kubernetes-deploy kubernetes-restart command could be useful here?

we currently have create expand rt, discourse, stackoverflow, etc. inline on IRC. considering how many tweets get linked on IRC, we may want to consider expanding them inline as well.

https://airnowapi.org provides API access to AQI data.

I think it would be nice to include this information in weather, but perhaps a separate command would be better? I have no strong feeling.

You can sign up for an account and you get a GUID(?) API key.

Calls look like:

http://www.airnowapi.org/aq/observation/zipCode/current/?format=application/json&zipCode=94720&distance=25&API_KEY=<REDACTED>

The output changes based on region (ugh) but gives either the catagory (Unhealthy/Moderate/etc) or the AQI value.

You can also get data based on lat/lon information.

Exported from https://ocf.io/rt/6832

It would be nice to see the service statuses change automatically instead of having to do create: status each time. It would probably be easiest to store the last status output and only print out a change if the new status fetched is any different.

Move this method to a shared file and use it both in !turing and !quote.

with some authenticated jenkins webhooks we could let IRC privileged ops rebuild projects without having to log in and navigate the awful jenkins interface. we already allow the marathon equivalent of this in create: restart <x>

Now that we are using kubernetes logging, the container should not use stdin2syslog as in

This should allow us to see logs by running something like kubectl -n app-ircbot get logs deployment/deployment instead of having to connect to democracy to see logs.

As suggested as part of the feedback for #17, splitting up the different ircbot responses into separate functions would be a good thing to do to refactor how the bot works.

next in the line of ircbot functionality to steal from Facebook, we should have a plugin that files GitHub issues against the mentioned repository using the GH API, e.g.

!fileatask ircbot make a fileatask plugin
!fileatask ocfweb migrate to kubernetes

When expanding tweets, if the tweet contains a link, create should also expand it inline as it would with other links shared by users in the channel.

#167 implements basic locking around the say function. This should be expanded to make sure we lock every time the connection is accessed (including things like setting the topic or getting the current oper list).

it might be nice to have a small plugin that checks if the status blog has been updated in the past couple minutes and if so, updates #rebuild with a link to new posts

Like d#999 or rt#9999, k#99 should link to ocf.io/k/99 and provide a short description of the task.

For example, if someone writes:

Create: who is in the lab

This won't work, since ircbot requires a case-sensitive matching. Sometimes people capitalize the name, so we should still support this case.

slack users cannot ping create directly which is a bit sad.

tanya (or any equivalent) solves this: https://github.com/nolanlum/tanya

it takes a long time to load all the pictures for macros so it might be a nice idea to lazy load them to save bandwidth. might need javascript for this though :(

#113 caused jinja to hard fail since the show_topic method didn't have a docstring. This is especially annoying because Marathon uses the webserver to healthcheck the bot—so a broken webserver leaves ircbot in a deploy loop. We can't reasonably expect people to remember to docstring methods when writing plugins. We should have make test run a smoke test that ensures the webserver 200s to catch these errors.