We deprecate and change parameter enforce_type's default value
to True[1], so don't need usage enforce_type.
[1] https://review.openstack.org/328692
Change-Id: Ib89cb9bc8afa13e13350e80ec9210ab9e878201f
When running the tests with concurrency=1, 2 tests fail with:
AttributeError: Mock object has no attribute 'debug'
and
AssertionError: Expected 'notify' to be called once. Called 0 times
Mocking the LOG object solves the problem.
Change-Id: Ie7f4448a103fae448123a05bc92e961aac00d6ec
Closes-Bug: #1660393
This uses the underlying x-ssl option that will attempt an SSL
connection to the server using the system's CA bundle.
Change-Id: I73f3b43b4ebc0c4ce4c0ba43b5a84361fb308686
* add application scenario to help of config option
'executor_thread_pool_size'
* Move eventlet patched checking to constructor
Change-Id: I9dc7c993c8947b51952365cc5e7b6f2daae6bb70
We deprecated RequestContextSerializer in 404bebcca and
it's not used by any project, so we can remove it safely now.
Change-Id: I0300f5d2aaa85d515c0437e0e69399b9f5bad09a
The rightmost '@' in a url netloc field separates the user:password
from the hostname:port. Using the first may cause the separation to
occur in the username or password field (if unencoded).
Change-Id: I7b02d8e0cbdf875ea80a55065ce98fa73696e05f
Closes-Bug: 1675423
rpc_response_timeout is not present when sending notifications. All
RPC call requests have a timeout attribute - use that. In the case
where there is no timeout, use retry value if present.
Change-Id: I27a3dd0962a06f15e85e9c9c8c24aa7786c71056
Closes-Bug: 1675510
zmq_options.register_opts() requires a TransportURL in the parameter
list. This patch creates an empty TransportURL and passes it since
there is no actual URL available.
Change-Id: I5954e430eedc5904594ee3be49d460de4b99d89a
Closes-Bug: #1675425
oslo.messaging supports three types of executors: blocking, eventlet
and threading, so it's good to list all of them in docstring.
Change-Id: Ief14d54b5f029cf3e9c79334e405dca5cf2dc13d
oslo.messaging deprecated transport aliases[1] since 5.2.0+
that is the minimal version supported for stable/newton.
This commit deprecates aliases kwargs of TransportURL() and
get_transport(), then we can remove the parameter aliases
in the future version.
Related-Bug: #1424728
[1]I314cefa5fb1803fa7e21e3e34300e5ced31bba89
Change-Id: I0c96979dacd9eb05b4215c687b5e34e7b3475ddb
With pbr 2.0 and Sphinx 1.5, the setting for treat sphinx warnings as
errors is setting warning-is-error in build_sphinx section. Migrate
the setting from the old warnerrors one.
The history document is removed because some of the commit messages used
in ChangeLog were being identified as invalid markup and there doesn't
appear to be anyway to edit there retroactively nor disable warnings on
a specific file.
Change-Id: I79e7ac56d5af1151865686761f3d40a11efbf472
The 'list-modules' directive used in some docs does not appear to be
Python 3 compatible, yielding the following error:
TypeError: unorderable types: NoneType() < str()
Simply use Python 2.7 until such a time as someone wants to investigate
why this happens.
Change-Id: I0be19b056e1c2272df3666626fbd17ec78017bce
debtcollector's decorator updated_kwarg_default_value is used to indicate
default value will be changed in future version. We introduced parameter
access_policy and add itfor constructor method of RPCDispatcher in
d3a8f280ebd6fd12865fd20c4d772774e39aefa2. Other services usually call
get_rpc_server method instead of calling constructor method of
RPCDispatcher. get_rpc_server method constructs RPCDispatcher instance
in [1], then FutureWarning will be generated even we set parameter
access_policy explicitly. Need move the decorator updated_kwarg_default_value
to method get_rec_server to avoid the case.
Closes-Bug: #1661572
[1] f1efaf1b6e/oslo_messaging/rpc/server.py (L200).
Change-Id: I3214045856d0bbd05a0000eeafb769e4697cd34c
15cd99050c6d2714b90059e0faad9f9e3409eaaa added this alias
to workaround an issue with neutron using the internal
path in liberty. That was fixed in neutron in mitaka:
225663888e71d44e9732951f515da925b8799b74
The alias was left for liberty, but liberty is end of life
now so we can remove this.
Change-Id: I145921b98d48e03df38ebcbaebc1a4375911f9ff
Since pbr already landed and the old version of hacking seems not
work very well with pbr>=2, we should update it to match global
requirement.
Partial-Bug: #1668848
Change-Id: I29b64d2f8d6e3305d052562f160050fdd05c2a55
Generating the payload of a notificatiton might be expensive due
to database access. This expensive step is unnecessary if the
notification driver is set to noop because it means that the
generated payload will not be sent.
This patch makes it possible for the user of the notifier to check
if the notifier will emit any notification before the payload
is generated.
Cinder already implemented similar behavior in
I77f655d3ef90088ce71304da5d4ea7b543991e90 and nova also plans
to do it in I3e6741d59df49e1e58409314008c2ed609fdedc1
Change-Id: Ib992f5c20fef85224fb00823e1d8d9c6cff19bec
This makes the SSL-related parameters to be similar to the ones used by
the AMQP and pika drivers. This will enable easier configuration of
these parameters if the transport URL is used. And easier migration from
one driver to the other when needed.
Change-Id: Ic32b2cb253fa0dc43aad7226b24919b7e588faa9
When driver load we allow to override option unrelated to the driver and
to set option useless for the driver.
This change validates the query string when the driver load to report as
soon as possible invalid options. And allow to override only option
of the driver option group (ie: [oslo_messaging_<driver_name>].
Related-bug: #1666903
Change-Id: Iaf23f773279c10bf37d545883ada7c2f6a9ffbbf
