From f3525cf3b54bb1b632ab615baff3925179268ace Mon Sep 17 00:00:00 2001 From: Neal Gompa Date: Jun 05 2018 10:56:32 +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. Merges https://pagure.io/pagure/pull-request/3269 Signed-off-by: Neal Gompa Signed-off-by: Pierre-Yves Chibon --- diff --git a/doc/configuration.rst b/doc/configuration.rst index 738a961..d85940a 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 2179403..d57b827 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 5465143..2cdc436 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'