From 6d0468ec6a0bd5c074a0e283bd875975be331222 Mon Sep 17 00:00:00 2001 From: Neal Gompa Date: Jun 04 2018 17:13:49 +0000 Subject: Disable sending FedMsg notifications by default Pagure does not currently mandate FedMsg integration to be configured for initial setup, but if it isn't installed and configured, a lot of regular tasks break due to tracebacks from missing or broken FedMsg setups. This was necessary for fixing Pagure on Mageia and openSUSE. Signed-off-by: Neal Gompa --- diff --git a/doc/configuration.rst b/doc/configuration.rst index c01bb6d..c630885 100644 --- a/doc/configuration.rst +++ b/doc/configuration.rst @@ -953,7 +953,7 @@ FEDMSG_NOTIFICATIONS This configuration key allows to turn on or off notifications via `fedmsg `_. -Defaults to: ``True``. +Defaults to: ``False``. ALWAYS_FEDMSG_ON_COMMITS diff --git a/files/pagure.cfg.sample b/files/pagure.cfg.sample index 6a919f6..443fefa 100644 --- a/files/pagure.cfg.sample +++ b/files/pagure.cfg.sample @@ -20,6 +20,9 @@ SECRET_KEY='' #DB_URL=postgres://user:pass@host/db_name DB_URL = 'sqlite:////var/tmp/pagure_dev.sqlite' +### Send FedMsg notifications of events in pagure +FEDMSG_NOTIFICATIONS = False + ### The FAS group in which the admin of pagure are ADMIN_GROUP = ['sysadmin-main'] diff --git a/pagure/default_config.py b/pagure/default_config.py index 2450bb0..727abca 100644 --- a/pagure/default_config.py +++ b/pagure/default_config.py @@ -31,6 +31,9 @@ INSTANCE_NAME = 'Pagure' # DATAGREPPER_URL = 'https://apps.fedoraproject.org/datagrepper' # DATAGREPPER_CATEGORY = 'pagure' +# Send FedMsg notifications of events in pagure +FEDMSG_NOTIFICATIONS = False + # The FAS group in which the admin of pagure are ADMIN_GROUP = 'sysadmin-main'