From 44202fb688806a4b1aeb0c044719fc52adfbccde Mon Sep 17 00:00:00 2001 From: Monty Taylor Date: Thu, 11 Jul 2013 23:51:13 -0400 Subject: [PATCH] Add #openstack-infra to eavesdrop There have been general requests for more logging of more things. I think we might have a longer discussion there. But recently there have been specific requests about -infra. I think that since discussions around the gate and rather wide-spread project concerns wind up happening in infra, we should definitely start logging it, and then we can talk about whether other team channels should be logged or not. Made this use an array of channels rather than a string for sanity. Change-Id: I3f3e82aa86155d1ac3010ca3c3ade006b7a26793 --- modules/meetbot/templates/supybot.conf.erb | 2 +- modules/openstack_project/manifests/eavesdrop.pp | 8 +++++++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/modules/meetbot/templates/supybot.conf.erb b/modules/meetbot/templates/supybot.conf.erb index 4c22d5a41e..3f32674603 100644 --- a/modules/meetbot/templates/supybot.conf.erb +++ b/modules/meetbot/templates/supybot.conf.erb @@ -63,7 +63,7 @@ supybot.networks.<%= network %>.servers: <%= server %> # # Default value: ### -supybot.networks.<%= network %>.channels: <%= channels %> +supybot.networks.<%= network %>.channels: <% channels.each do |channel| %> <%= channel %> <% end %> ### # Determines what key (if any) will be used to join the channel. diff --git a/modules/openstack_project/manifests/eavesdrop.pp b/modules/openstack_project/manifests/eavesdrop.pp index a1712c56c8..922c6aedfe 100644 --- a/modules/openstack_project/manifests/eavesdrop.pp +++ b/modules/openstack_project/manifests/eavesdrop.pp @@ -22,7 +22,13 @@ class openstack_project::eavesdrop ( nickpass => $nickpass, network => 'FreeNode', server => 'chat.freenode.net:7000', - channels => '#openstack #openstack-dev #openstack-meeting #openstack-meeting-alt', + channels => [ + '#openstack', + '#openstack-dev', + '#openstack-infra', + '#openstack-meeting', + '#openstack-meeting-alt', + ], use_ssl => 'True' }