summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorClark Boylan <clark.boylan@gmail.com>2018-10-15 15:38:16 -0700
committerClark Boylan <clark.boylan@gmail.com>2018-10-15 15:38:16 -0700
commitfefaae11098b060c368993144a37a144afe9ba90 (patch)
treef1cb727d130de834a75a85a5dcd755f6d3a3f47d
parent57a669d35dc70d21be0b799e893535fc26d95c6d (diff)
Don't autojoin channelsHEADmaster
Don't pass the channels list to super in the irc bot class init method because this appears to cause autojoining to happen which breaks because we have too many channels. Instead let the event driven processing join and part channels as needed to make the least recently used channel rotation with channel limit work. Change-Id: I447e75e84526d08fb9ec88eaae9a90f950dc5349
Notes
Notes (review): Code-Review+2: Jeremy Stanley <fungi@yuggoth.org> Code-Review+2: Monty Taylor <mordred@inaugust.com> Workflow+1: Monty Taylor <mordred@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 16 Oct 2018 01:05:00 +0000 Reviewed-on: https://review.openstack.org/610745 Project: openstack-infra/gerritbot Branch: refs/heads/master
-rwxr-xr-xgerritbot/bot.py3
1 files changed, 1 insertions, 2 deletions
diff --git a/gerritbot/bot.py b/gerritbot/bot.py
index d6cc226..1185a0e 100755
--- a/gerritbot/bot.py
+++ b/gerritbot/bot.py
@@ -103,8 +103,7 @@ class GerritBot(SASL, SSL, irc.bot.SingleServerIRCBot):
103 server_list=[(server, port)], 103 server_list=[(server, port)],
104 nickname=nickname, 104 nickname=nickname,
105 realname=nickname, 105 realname=nickname,
106 ident_password=password, 106 ident_password=password)
107 channels=channels)
108 self.all_channels = {} 107 self.all_channels = {}
109 for name in channels: 108 for name in channels:
110 self.all_channels[name] = Channel(name) 109 self.all_channels[name] = Channel(name)