Something broke in the way SSL certs are generated. Disable these
tests for now to unblock the gate.
Change-Id: I9764da7995c198597661ecbea27efba9f76e53e3
Right now two tests mock logging module globally which leads to
another test failure (TestRoutingNotifier.test_notify_filtered)
on my env.
Change-Id: Ifa47ee96dae79ea44dbe17a2d125401589ff13dd
OpenStack projects should be using mock rather than mox.
oslo.messaging was using mox's stubs via oslotest rather than
mock.patch.object. The code is converted to use mock.patch.object
via fixtures.MockPatchObject.
Change-Id: I19490b4e8211c35b237ebfd38bf2f8b8b44cbf61
We log deprecation message even if application doesn't use it.
And we missing the parse method.
This change fixes that.
Change-Id: I476816cb14ac8218feb74e8459cfa5f18fdb6c61
Last oslo.config was magically converting MultiStrOpt "messaging" to ["messaging"],
that was not expected, oslo.messaging have a test that use "messaging"
instead of a list.
This change fixes that.
Change-Id: I65b8cc551dde6f80d979640ba3db097572f3fe9b
When the underlying RPC is configured with rabbit along
with oslo_messaging_notifications, currently the retry is
not supported. There are retry support specific to drivers,
it's better to have the support uniform across the drivers,
by adding configuration retry on the oslo_messaging_notifier.
Closes-bug: 1666364
Change-Id: I8d547ef2a9e8b086f94a1efca17735f3f3cdaac1
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
Use https instead of http to ensure the safety without containing our
account/password information
Change-Id: I0ffb9e2270bbae844c187fdc17610e7805553d66
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
dict doesn't seem always keep order on Python. we have two separate part
to prepare test data[1], calling 'for k, v in self.args.items()' and
'for arg in self.args:' may get data in different order.
This commit makes we collect test data in same order.
Change-Id: I49f0d1b9aed110ad1e59140fd70760682e944d74
Related-Bug: #1677151
The transport_url configuration value provides the username and
password, making the AMQP 1.0 driver's configuration instances of
username and password redundant.
Change-Id: If87d3efe616847625c5ab1329c61f1ad4c421a16
Closes-Bug: #1663721
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
