summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | Implement replay detection.Eric Windisch2013-02-193-22/+99
| |/ / / | | | | | | | | | | | | | | | | | | | | Bumps the envelope revision to 2.1 Change-Id: Ib0260a0c62e3d312d2e3448a125bed64d861319e
* | | | Merge "Avoid using cfg in install_venv_common"Jenkins2013-02-201-20/+7
|\ \ \ \
| * | | | Avoid using cfg in install_venv_commonMark McLoughlin2013-02-191-20/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Several people have complained that requiring oslo-config to be installed in order to run install_venv seems like a chicken and egg problem. We're only using cfg for a boolean CLI option which is way overkill. The truth is that cfg is really only interesting if you want to process options from both the command line and configuration files. Clearly that's not the case here. Just use argparse instead of cfg. Change-Id: Ib602686b554c3548f4be3d4a69d128c5fef7e8cf
* | | | | Merge "Add function for listing native threads to eventlet backdoor"Jenkins2013-02-191-0/+8
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Add function for listing native threads to eventlet backdoorDaniel P. Berrange2013-02-181-0/+8
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The eventlet backdoor has a 'pgt' function for listing green threads and their stack traces. This adds a new 'pnt' function for doing the same with native threads. Change-Id: If6dcfd8dde61c96adfc6e052e5ec7db82914cd55 Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
* | | | Support RPC envelopes in impl_zmqEric Windisch2013-02-192-28/+77
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch began as a set of tests verifying the functionality of sending and receiving RPC envelopes when using impl_zmq. It was discovered that when enabled, RPC envelopes were not actually working, The ZeroMQ driver includes its own envelopes. This patch introduce versioning to that envelope, eliminating the previously reserved 'style' field. A new iteration of the zeromq-envelope is introduced, 'impl_zmq_v2'. It specifies that the zeromq-envelope should be followed by an unpacked array representing key value pairs of the standard RPC Envelope. Because the key-values of the RPC Envelope can be successfully transformed with bytes(), this prevents the need to double-serialize the content traversing the message bus. Also removes some unused imports. Closes bug 1123709 Closes bug 1055446 Change-Id: Ib04e3d092c9596146f1048d3502ac248496d313b
* | | | Merge "Add ConfigFilter wrapper class"Jenkins2013-02-195-0/+423
|\ \ \ \ | |_|/ / |/| | |
| * | | Add ConfigFilter wrapper classMark McLoughlin2013-02-185-0/+423
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements blueprint cfg-filter-view At the moment, if a module requires a configuration option from another module, we do: CONF.import_opt('source.module', 'option_name') but, in fact, all options from the imported module are available for use. The new ConfigFilter class makes it possible to enforce which options are available within a module e.g. with CONF = cfgfilter.ConfigFilter(cfg.CONF) CONF.import_opt('foo', 'source.module') CONF.register_opt(StrOpt('bar')) then the foo and bar options would be the only options available via this CONF object while still being available via the global cfg.CONF object. Change-Id: Ie3aa2cd090a626da8afd27ecb78853cbf279bc8b
* | | | blueprint amqp-rpc-fast-reply-queueRaymond Pekowski2013-02-184-81/+517
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For AMQP based RPC, specifically RabbitMQ and Qpid, this change replaces the dynamically created RPC call reply queue with a single queue that is created on the first RPC call and used on all subsequent calls. It provides backward compatibility on the callee side by recognizing downlevel callers and on the caller side by adding a config option to revert to the old dynamically created queue based upon the msg_id. Change-Id: Idb09a71472866bd3950f58d4f7f45a3181eb40fc
* / / Make update.py work with oslo.configDoug Hellmann2013-02-181-0/+9
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | As we move modules out of openstack.common into the oslo namespace package, we need to prevent update.py from breaking any import statements using them. Rather than build a super complicated regex to only detect the proper instances of "oslo" to replace in places like the defaults for option settings, just do the simple translation between oslo -> base and then go back and fix the modules we know have been moved. Change-Id: I9686b94b4c7df8542964b30ba15f0a91d2b3d722 Signed-off-by: Doug Hellmann <doug.hellmann@dreamhost.com>
* | Merge "Bump eventlet to 0.12.0"Jenkins2013-02-181-1/+1
|\ \
| * | Bump eventlet to 0.12.0Eric Windisch2013-02-011-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | This version has important fixes required for ZeroMQ (spurious wakeups). Fixes bug 1065532. Change-Id: I16b9220db1f5bf16a24644887963ce18e4e5ecbd
* | | Use oslo-config-2013.1b3Mark McLoughlin2013-02-1747-3858/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The cfg API is now available via the oslo-config library, so switch to it and remove the copied-and-pasted version. Add the 2013.1b3 tarball to tools/pip-requires - this will be changed to 'oslo-config>=2013.1' when oslo-config is published to pypi. This will happen in time for grizzly final. Remove the 'deps = pep8==1.3.3' and 'deps = pyflakes' from tox.ini as it means all the other deps get installed with easy_install which can't install oslo-config from the URL. Change-Id: I4815aeb8a9341a31a250e920157f15ee15cfc5bc
* | | Change the check for the existence of .git directory.Robert Myers2013-02-152-5/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In common setup the check for the .git directory is too restrictive. Instead of checking that it is a directory just check to see if it exists. That way if the project is part of a submodule it will continue to work correctly. Change-Id: If6b6531ab5778ac17537e3f18bde1844620c8316 Fixes: bug 1126416
* | | returncode for Popen is valid only after communicate callDavanum Srinivas2013-02-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | Found when testing a bug in cinder (1125416), code relying on throw_on_error won't work because returncode is None if checked before the communicate() method is called Change-Id: I8c9dd00396346ec3ad7bbe1dc17643c385da8d6f
* | | Merge "Fix Lack of reconnecting operation when using rabbitmq mirrored queue"Jenkins2013-02-141-1/+2
|\ \ \
| * | | Fix Lack of reconnecting operation when using rabbitmq mirrored queueKei Masumoto2013-02-131-1/+2
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | On using Mirrored Queue feature in impl_kombu.py, there is a case that messages are lost because amqp client does not handle exceptions properly when rabbitmq is dead on the way to wait return value of call(). This patch fix this and enables ampq client reconnecting slave rabbitmq. Fixes bug 1102051 Change-Id: Ia7a1b9067f7ea4639195a1548de29e0364368e51
* | | Merge "Cleanup docstring and print in install_venv_common"Jenkins2013-02-141-2/+2
|\ \ \
| * | | Cleanup docstring and print in install_venv_commonMatthew Treinish2013-02-111-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit cleans up usage in the docstring for: run_command_with_code() to be more explicit in the description of where the commands are being run. It also updates the prints to be consistent in the usage of venv vs virtualenv. These changes were recommended in the review for change: I68fe3449ed0fcc7fdfb84af2dcc77a39ebde7f01 Change-Id: I3cdb3d991503446d37c8fa6a3547585a4f8beb6a
* | | | Merge "Add join_consumer_pool() to RPC connections"Jenkins2013-02-147-8/+188
|\ \ \ \
| * | | | Add join_consumer_pool() to RPC connectionsDoug Hellmann2013-02-137-8/+188
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | blueprint move-listener-framework-oslo bug 1047015 bug 1111632 Ceilometer and Quantum use private methods of the RPC connection object to configure themselves to listen to a queue shared among a group of workers. This change adds a public method to the RPC connection to support this use case, without resorting to using private API calls. Change-Id: I3a89f1dfdcf8accca70cf305f7a31315bea093d8 Signed-off-by: Doug Hellmann <doug.hellmann@dreamhost.com>
* | | | Merge "Cleanup imports"Jenkins2013-02-141-3/+0
|\ \ \ \
| * | | | Cleanup importsEric Windisch2013-02-141-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bringing this into Nova, I found that it failed their pep8 checks (which are NOT ignored for the oslo imported binaries) Change-Id: I4fb7e32bc7e83faf4131302a06c4d9c2c16cab70
* | | | | Merge "Rename oslo-zmq-receiver to oslo-rpc-zmq-receiver"Jenkins2013-02-141-0/+0
|\| | | |
| * | | | Rename oslo-zmq-receiver to oslo-rpc-zmq-receiverEric Windisch2013-02-141-0/+0
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | Would not copy without the name change. Also, this will result in this file maintaining its name in nova, where it already exists. Change-Id: Ic204be899921ce63b769194474c31e1640391197
* | | | Don't use subprocess for testing excepthookJason Kölker2013-02-131-18/+18
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | * The previous test tested the funtionality of the excepthook using subprocess.popen. We don't need to test the interpreter, just test that our excepthook is installed and test that it functions as expected. * Fix Bug 1124617 Change-Id: Ifd8d35b842b913003100097d917d30bf66e5cb7a
* | | Merge "sort options to make --help output prettier"Jenkins2013-02-131-1/+1
|\ \ \
| * | | sort options to make --help output prettierYAMAMOTO Takashi2013-02-131-1/+1
| |/ / | | | | | | | | | | | | | | | | | | | | | explicitly sort options when adding them to argparse. it's a bit silly to print them in a dict iteration order. Change-Id: Id508331d7ee3b24e76be7fa958d27d29905bd3d2 Signed-off-by: YAMAMOTO Takashi <yamamoto@valinux.co.jp>
* / / to_primitive imposes what seems to be an arbitary data structurePhil Day2013-02-122-3/+36
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | depth of 3, but there is at least on case in Nova (Security group Rules) which requires a depth beyond this. https://bugs.launchpad.net/nova/+bug/1118608 Specifically security_group_rule_get_by_security_group returns a set of rules which have the structure: rule -> grantee_group -> Instance -> Instance_type Rather than just bumping the depth limit, which might break some other user of to_primitive we make it a specific parameter that defaults to the current value but can be over-ridden when required and log a warning when the depth is exceeded Change-Id: I1eaebd484e20cb2eae09a693289709973de9943c
* | Merge "Allow running test in uninstalled source tree."Jenkins2013-02-112-7/+7
|\ \
| * | Allow running test in uninstalled source tree.Monty Taylor2013-02-102-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | It was pointed out that we missed the use-case that running tests in a fresh tree checkout should work - and we missed that by reaching too far into the setup module interface. Shame on us. Change-Id: Ic0056a3113b308f063d927484e1e4a44c9316a2d
* | | Merge "Support testing args for LocalhostMatchMaker."Jenkins2013-02-113-9/+12
|\ \ \
| * | | Support testing args for LocalhostMatchMaker.Eric Windisch2013-02-103-9/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The matchmaker should return the correct direct topics used by the RPC driver. The LocalhostMatchMaker was always returning 'localhost', even though our tests might use '127.0.0.1' instead. Now, the LocalhostMatchmaker will use whatever value it is told should be considered local. MatchMakerLocalhost and LocalhostExchange now take an optional host parameter. impl_zmq._get_matchmaker now passes arguments to the selected matchmaker module. The test_zmq now initializes the matchmaker, using the host '127.0.0.1' Change-Id: I8daa2c0668f1d717eb95ab56009612b8b60f0a15
* | | | Merge "Add a fixture for dealing with mock patching."Jenkins2013-02-111-0/+51
|\ \ \ \
| * | | | Add a fixture for dealing with mock patching.Monty Taylor2013-02-071-0/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Quantum uses a pattern all over their test base, which can be collapsed down into a simple re-usable fixture. Change-Id: I5944505ce44ce8b79a685c3ea392f001307b5319
* | | | | Merge "Update .coveragerc"Jenkins2013-02-111-0/+6
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Update .coveragercAlessio Ababilov2013-02-061-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Set up proper source and omit options. Change-Id: Ib14ff98218d23db694c1614aa3b4225a8d5dd847 Implements: blueprint update-coveragerc
* | | | | Merge "Implements import_group"Jenkins2013-02-093-0/+56
|\ \ \ \ \
| * | | | | Implements import_groupZhongyue Luo2013-01-313-0/+56
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Created import_group method in cfg.py Added testcases Fixes bug #1107843 Change-Id: I427d4069dacbb6e586687370adbb08b5d50f7b63
* | | | | Merge "Update HACKING.rst per recent changes"Jenkins2013-02-092-4/+11
|\ \ \ \ \
| * | | | | Update HACKING.rst per recent changesZhongyue Luo2013-02-042-4/+11
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added "is not" usage with examples Fixed "not in" usage description https://review.openstack.org/#/c/20576/ and https://review.openstack.org/#/c/20879/ Fixes a previous "not X in Y" grep corner case Change-Id: I92ae642887de734a8562a528003d8092f5ff27d4
* | | | | Merge "Exchanges should return directed topics."Jenkins2013-02-082-4/+24
|\ \ \ \ \
| * | | | | Exchanges should return directed topics.Eric Windisch2013-02-012-4/+24
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Exchanges in the matchmaker are supposed to return a directed topic as the first element in returned tuples. Adds a test to ensure this is so. Change-Id: I6c62f325914faa19216d5362c6d8ed0288942110
* | | | | Merge "Allow to_primitive to ignore datetimes"Jenkins2013-02-082-21/+17
|\ \ \ \ \
| * | | | | Allow to_primitive to ignore datetimesJoe Gordon2013-02-072-21/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In preparation for having nova.db.api not return any sqlalchemy objects. nova.db.api will return only primitives, except for datetime.datetime objects. Uses functools.partial to make code DRYING Partially implements bp db-api-cleanup Change-Id: I9980d8c4e20b05bbe734cf90ac209e4e7e89befb
* | | | | | Use importutils.try_import() for MySQLdbMark McLoughlin2013-02-072-12/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a common pattern we've adopted elsewhere, so use it here too. Change-Id: I2e0947b4857005b478e796e9c9a7c05ea1f0d926
* | | | | | Fix sorting in test-requiresMark McLoughlin2013-02-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ic45ac7c85dbe2834d7b325ba8babf71429b277b0
* | | | | | Minor tweak to make update.py happyMark McLoughlin2013-02-072-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fairly lame to do this instead of just fixing update.py, but we seem to do it this way everywhere else, so ... Change-Id: Ie1cebcfec57c6138b9b726d7d3bc6e0ea456f146
* | | | | | Remove pointless use of OpenStackExceptionMark McLoughlin2013-02-071-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The use of the base exception does nothing for the test but means the "dead module walking" exception module. Change-Id: Ibd7bb8aa526be366f005ab5e12fd4da89a976c57
* | | | | | Remove unused context from test_sqlalchemyMark McLoughlin2013-02-071-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I0b3c7341e91cfc57aed625754bc9bf009ef70b14
generated by cgit (git 2.34.1) at 2025-09-26 13:01:41 +0000