Skip to content

Commit

Permalink
Run rabbitmq heartbeat in python thread by default
Browse files Browse the repository at this point in the history
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
  • Loading branch information
4383 committed Aug 27, 2020
1 parent 8d78ab2 commit add5ab4
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 7 deletions.
16 changes: 9 additions & 7 deletions oslo_messaging/_drivers/impl_rabbit.py
Expand Up @@ -88,14 +88,16 @@
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',
Expand Down
@@ -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.

0 comments on commit add5ab4

Please sign in to comment.