openstack/oslo.messaging
Code Issues Proposed changes

Touch up grammar in warning messages

Browse Source 
Change-Id: I556e6e646417be3732a7c31987ec964f9808f079
This commit is contained in:
Doug Hellmann 2014-12-04 16:04:37 -05:00
parent eb21f6b263
commit 66db2b310d
3 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
doc/source/transport.rst
View File

@ -16,15 +16,15 @@ Transport
.. autofunction:: set_transport_defaults
About fork oslo.messaging transport object
------------------------------------------
Forking Processes and oslo.messaging Transport objects
------------------------------------------------------
oslo.messaging can't ensure that forking a process that shares the same
transport object is safe for the library consumer, because it relies on
different 3rd party libraries that don't ensure that too, but in certain
case/driver it works:
different 3rd party libraries that don't ensure that. In certain
cases, with some drivers, it does work:
* rabbit: works only if no connection have already been established.
* qpid: doesn't work (qpid library have a global state that use fd
that can't be resetted)
* qpid: doesn't work (The qpid library has a global state that uses
file descriptors that can't be reset)
* amqp1: works

2
oslo/messaging/_drivers/impl_qpid.py
View File

@ -590,7 +590,7 @@ class Connection(object):
# https://github.com/celery/kombu/blob/master/kombu/transport/
# qpid_patches.py#L67
LOG.warn("Process forked! "
"This can results to unpredictable behavior. "
"This can result in unpredictable behavior. "
"See: http://docs.openstack.org/developer/"
"oslo.messaging/transport.html")
self._initial_pid = current_pid

2
oslo/messaging/_drivers/impl_rabbit.py
View File

@ -559,7 +559,7 @@ class Connection(object):
current_pid = os.getpid()
if self._initial_pid != current_pid:
LOG.warn("Process forked after connection established! "
"This can results to unpredictable behavior. "
"This can result in unpredictable behavior. "
"See: http://docs.openstack.org/developer/"
"oslo.messaging/transport.html")
self._initial_pid = current_pid