summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHervé Beraud <hberaud@redhat.com>2020-08-21 14:08:15 +0200
committerHervé Beraud <hberaud@redhat.com>2020-08-27 12:22:41 +0200
commitadd5ab4ecec090efdb9864bc9385f871f2dd082a (patch)
tree5103f641a2c088bc5642fd110063a529cb319531
parent8d78ab2339d5798c7d8f5c49eaf6d2ef18859a07 (diff)
downloadoslo-messaging-add5ab4ecec090efdb9864bc9385f871f2dd082a.tar.gz
Run rabbitmq heartbeat in python thread by default
Removing the experimental nature of this feature and activating it by default. Now to run heartbeat in a green thread users should set this option to False. Also deprecating this option to prepare future removal and force to always run heartbeat in a python thread whatever the context. Change-Id: I32a6c4ad0a456282ec02b5e4c8309489b3c17553
-rw-r--r--oslo_messaging/_drivers/impl_rabbit.py16
-rw-r--r--releasenotes/notes/run-heartbeat-in-pthread-by-default-28637b41ebf500dc.yaml10
2 files changed, 19 insertions, 7 deletions
diff --git a/oslo_messaging/_drivers/impl_rabbit.py b/oslo_messaging/_drivers/impl_rabbit.py
index 2f48541..d5cfc02 100644
--- a/oslo_messaging/_drivers/impl_rabbit.py
+++ b/oslo_messaging/_drivers/impl_rabbit.py
@@ -88,14 +88,16 @@ rabbit_opts = [
help='SSL certification authority file '
'(valid only if SSL enabled).'),
cfg.BoolOpt('heartbeat_in_pthread',
- default=False,
- help="EXPERIMENTAL: Run the health check heartbeat thread "
- "through a native python thread. By default if this "
- "option isn't provided the health check heartbeat will "
- "inherit the execution model from the parent process. By "
- "example if the parent process have monkey patched the "
+ default=True,
+ help="Run the health check heartbeat thread "
+ "through a native python thread by default. If this "
+ "option is equal to False then the health check "
+ "heartbeat will inherit the execution model "
+ "from the parent process. For "
+ "example if the parent process has monkey patched the "
"stdlib by using eventlet/greenlet then the heartbeat "
- "will be run through a green thread."),
+ "will be run through a green thread.",
+ deprecated_for_removal=True),
cfg.FloatOpt('kombu_reconnect_delay',
default=1.0,
deprecated_group='DEFAULT',
diff --git a/releasenotes/notes/run-heartbeat-in-pthread-by-default-28637b41ebf500dc.yaml b/releasenotes/notes/run-heartbeat-in-pthread-by-default-28637b41ebf500dc.yaml
new file mode 100644
index 0000000..383fbda
--- /dev/null
+++ b/releasenotes/notes/run-heartbeat-in-pthread-by-default-28637b41ebf500dc.yaml
@@ -0,0 +1,10 @@
+---
+upgrade:
+ - |
+ The ``[oslo_messaging_rabbit] heartbeat_in_pthread`` config option now
+ defaults to ``True``.
+ Applications will run RabbitMQ heartbeat in a Python thread by default.
+deprecations:
+ - |
+ ``heartbeat_in_pthread`` has been deprecated and will be removed in a
+ future release. If configured, this option should be unset.