summaryrefslogtreecommitdiffstats
path: root/openstack/common/rpc
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Fix typo made in 372d11e990Eric Windisch2012-07-131-1/+1
| |/ |/| | | | | Change-Id: Iba232257595bb3084052a21a560f2f5551b5f1eb
* | Fix import order of gettextutils in amqp.pyEric Windisch2012-07-131-1/+1
| | | | | | | | Change-Id: I53ca8308fb61c53274ca1e3d801f22c9b65d41aa
* | Add more rpc version docs.Russell Bryant2012-07-131-0/+39
| | | | | | | | | | | | | | This patch adds some additional documentation, including some examples, on making changes to versioned RPC APIs. Change-Id: I5f40b11e07538f36d0f1299f7e822b408bf8e96a
* | Fix missing gettextutils in several modules.Eric Windisch2012-07-122-0/+2
| | | | | | | | | | | | Fixes bug 1021453. Change-Id: I05771a588746a0e43fec0078c5c7152dd9d33b9f
* | Merge "Fix line length for pep8 compliance after cloning into other projects"Jenkins2012-07-112-4/+9
|\ \
| * | Fix line length for pep8 compliance after cloning into other projectsDoug Hellmann2012-07-112-4/+9
| |/ | | | | | | | | | | | | | | | | | | | | There are a few lines that are close to the length limit so they pass the PEP8 check under the common repository but fail the check after the code is cloned into another project such as ceilometer. These changes adjust the lines to give more space to allow the merge into the other projects to go ahead without breaking the pep8 gating tests. Change-Id: I33a05f5b51120114cc8ceffd0bf1b1ef462ed636 Signed-off-by: Doug Hellmann <doug.hellmann@dreamhost.com>
* | Merge "Fix import order."Jenkins2012-07-111-1/+1
|\ \ | |/ |/|
| * Fix import order.Russell Bryant2012-07-021-1/+1
| | | | | | | | | | | | | | Fix import order in rpc.common. They must be alphabetical. Nova tests for this and caught it when i synced in the latest changes. Change-Id: I0dbd6b5004d754d0bd2d1861905db3405c64e3d4
* | Fix Zmq so it doesn't rely on nova flagsJason Kölker2012-07-101-7/+11
| | | | | | | | | | | | | | | | | | * Embeded newlines in logging makes me cry * String formatting uses parens not curly braces * Fix LP1023039 * Hard skip Zmq tests so jenkins passes. Someone will have to fix. Change-Id: I9b12ac47b02edf1d8dba7236550deb8ee33f5e07
* | Switch common files to using jsonutils.Andrew Bogott2012-07-033-8/+8
| | | | | | | | | | | | | | | | For blueprint use-common-jsonutils Pretty much just a search and replace. Change-Id: I66f63e7b0ff82a37d8a6b9acef94f4e10a45bf90
* | Fix missing RPCException from raiseEric Windisch2012-06-281-1/+1
|/ | | | Change-Id: Ia850b07d29459b760d24f9bffdd88e4176cb7faf
* Add missing gettextutils importsDoug Hellmann2012-06-222-0/+2
| | | | | | | | | This change fixes a problem we found in the ceilometer test suite under Python 2.6. The name '_' is not defined globally, so nose reports import errors trying to scan the modules changed here. Change-Id: Ie94cc4effba457342e25f991cf481fc40a1d8cf3
* Avoid erroneous "Unsupported RPC Version" message if method is missingDan Smith2012-06-211-2/+8
| | | | | | | | | | | | | | | | | | The recent RPC versioning support introduced some odd behavior if a non- existent method is called. Instead of reporting that the method does not exist (as the code used to do, in the form of an AttributeError), it now Reports "Unsupported RPC Version". This is highly confusing, as the RPC versions *do* match and is just a result of the logic used to check every advertised RPC proxy for the method. This patch notes if any of the RPC versions matched the requested one, and if so, raises an AttributeError if the method is never found, instead of UnsupportedRpcVersion. Also add unit tests for the cases where a method is missing, both when the API version matches and not. Change-Id: Icb7b13fa890000f1bd03382d5cce747e3311ef6a
* Call exception on the logger, not the logging module.Russell Bryant2012-06-201-1/+1
| | | | Change-Id: Ia3cffe7473c59308f9dcbb40fccb8e2028e2aa96
* Fix import usage in AMQP for rpc.commonJulien Danjou2012-06-201-1/+1
| | | | | | | | | | | The existing line is technically OK, but it's not replaced by update.py on file copy, therefore the import of common.rpc.amqp fails. This fixes this problem by using a different syntax, the one that update.py understand. Change-Id: Id30f581477ab09c2b66fc0d65f1029554b311549 Signed-off-by: Julien Danjou <julien@danjou.info>
* Merge "Fix nova.rpc references in comments."Jenkins2012-06-192-14/+18
|\
| * Fix nova.rpc references in comments.Russell Bryant2012-06-132-14/+18
| | | | | | | | | | | | | | | | Part of blueprint common-rpc. Strip remaining references to 'nova.rpc' that existed in comments. Change-Id: I27885918a5e38eb730504c9e88bdda2b72b23257
* | Merge "rpc: Update rpc_backend handling."Jenkins2012-06-181-2/+8
|\|
| * rpc: Update rpc_backend handling.Russell Bryant2012-06-131-2/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Part of blueprint common-rpc. This patch updates the rpc_backend option handling in a couple of ways. 1) Set the default based on wherever this code has been copied to. 2) If we fail to import the backend, replace 'nova.rpc' with 'nova.openstack.common.rpc' just in case the value came from an older nova configuration file. Backwards compatibility is a good thing. Change-Id: I9ad7c084a2f0813ca1559115e7612f0a24fcce67
* | Update common code to support pep 1.3.Gary Kotton2012-06-188-194/+199
|/ | | | | | bug 1014216 Change-Id: I3f8fa2e11c9d3f3d34fb20f65ce886bb9c94463d
* Add missing ack to impl_qpid.Russell Bryant2012-06-131-1/+6
| | | | | | | | | | | Fix bug 1012374. Johannes Erdfelt pointed out that impl_qpid wasn't acking messages that it received. This turned out to be a nasty oversight, resulting in unbounded message queue growth inside of the python-qpid library. This fixes it. Change-Id: I446530239e16701009c4db8407a344596bb17952
* Add impl_zmq to rpc.Russell Bryant2012-06-112-0/+718
| | | | | | | | | | | | | | | | | Part of blueprint common-rpc. This driver just went into nova.rpc. Pull it in over here. I haven't been able to test this, but I can't get the tests to work in nova, either. There is a bit of a complication with impl_zmq worth mentioning. It requires a service to run (nova/bin/nova-rpc-zmq-receiver). Only once instance of this service should run on a given machine, so it shouldn't be copied into each project that uses rpc. Once openstack-common is an installed library, it can be distributed with that, but until then ... we'll just leave it in nova for now. Change-Id: I2bd067be58d943d4f3a90e013b22282217484872
* Add rpc to openstack-common.Russell Bryant2012-06-069-0/+3025
Implements blueprint common-rpc. This patch imports nova.rpc to openstack-common. All of the necessary changes to the core code to make it compatible with openstack-common have been done in nova, so the only changes made here are to imports. There are more changes made to the tests, but nothing that changes the core functionality of the tests. Change-Id: I17330aa4adfd0f22c449a2376833c7fe9dfd0cf1