On Tue, 26 Apr 2016 01:07:41 +0200
Pierre-Yves Chibon <pingou(a)pingoured.fr> wrote:
On Mon, Apr 25, 2016 at 01:37:58PM -0600, Kevin Fenzi wrote:
> Greetings.
>
> Right now we send fedmsgs for most lists. There's a small few lists
> that we don't want to send any messages for as they might have
> sensitive items discussed on them.
>
> There's pretty much no way I can think of to tell this trait from
> mailman (we have 'advertised', but thats not at all the same
> thing).
>
> See tickets:
>
https://fedorahosted.org/fedora-infrastructure/ticket/5030
>
https://fedorahosted.org/fedora-infrastructure/ticket/5258
>
> So, the best we can do is maintain a list of these and update it on
> request. (Thanks to decause for the initial patch which I expanded
> on).
>
> I'd like to apply the following patch and run the mailman
> playbooks:
>
> diff --git a/roles/mailman/files/fedmsg-plugin-conf.py
> b/roles/mailman/files/fedmsg-plugin-conf.py index 3ee3567..e1f7732
> 100644 --- a/roles/mailman/files/fedmsg-plugin-conf.py
> +++ b/roles/mailman/files/fedmsg-plugin-conf.py
> @@ -1,8 +1,10 @@
> config = {
> - # These are mailing lists that we don't publish to fedmsg
> (because spam)
> + # These are mailing lists that we don't publish to fedmsg
> (because spam or privacy) 'mailman.excluded_lists': [
I there a new line character missing here?
Not in the real patch/diff... my mailer must have decided to do
something weird. ;(
In the real patch there is a newline there. ;)
kevin