summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* Rework zmq setup and cleanupMark McLoughlin2013-01-151-1/+0
| | | | | | | | | | | | | | | Register config options at import time and remove the register_opts() function. It seems this isn't being used. Add accessor methods for ZMQ_CTXT and matchmaker so that they are lazily created rather than created at import time. Remove the rpc_zmq_matchmaker override in the tests since MatchMakerLocalhost is the default anyway. We were setting it with an incorrect value, but only after the matchmaker had already been created. Change-Id: I2520252c64d70e4c4903e34d07952fed43e70ebe
* Merge "Implement importutils.try_import."Jenkins2013-01-151-0/+8
|\
| * Implement importutils.try_import.Dan Prince2013-01-141-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds a new (simple) importutils.try_import function which can return a module name or a default (by default this is None). This should help clean up some of our try: import foo except... blocks in modules. This commit also drops the dependency on python-extras which is no longer needed. Fixes LP Bug #1099501. Change-Id: I8591f79983bdad67a50d1aaae6dce8428dfce084
* | reusable ZmqProxy service initializationEric Windisch2013-01-131-27/+2
|/ | | | | | | | | | | Move near-identical proxy setup code from oslo-zmq-receiver and test_zmq into ZmqProxy's consume_in_thread method Provides code reuse and simplifies the tests and receiver binary. Change-Id: I1324eacfa2a456599d5fd462b6476ddf659e95c4
* Merge "Verbose should not enable debug level logging"Jenkins2013-01-131-1/+7
|\
| * Verbose should not enable debug level loggingDavanum Srinivas2013-01-121-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | Fixes LP #989269 Currently setting --verbose in will still allow DEBUG level message to be logged to python logger object. we need to check for --debug first (set DEBUG level), then --verbose (set INFO level) and if neither is set then set default to WARNING DocImpact Change-Id: Ic9e3cb5979b2d7283552ad3a461870373f45a239
* | Merge "Fix pep8 E125 errors."Jenkins2013-01-121-1/+1
|\ \
| * | Fix pep8 E125 errors.Monty Taylor2013-01-111-1/+1
| | | | | | | | | | | | | | | | | | Caesar's wife must be above reproach. Change-Id: Iac85a57e71d403360f1567c07c8699057f0772fb
* | | Merge "Simplify version processing."Jenkins2013-01-121-3/+4
|\ \ \ | |_|/ |/| |
| * | Simplify version processing.Monty Taylor2013-01-111-3/+4
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | In response to the cessation of per-commit tarballs produced by the CI systems, simplify the version processing code. We did many many many complicated things to support producing release artifacts that were not actually supported release artifacts. After a bunch of issues emerged, it was determined that the best way forward was to stop producing quasi-not-really non-release tarballs. If people want to consume versions of openstack that are not released versions, the git repos are always available. Additionally, this patch removes versioninfo and just uses PKG-INFO and pkg_resources to handle version processing. Change-Id: I5c799f3520adb2ba5288d852543706d81a92f8a1
* / Address test_zmq setup/teardown racesEric Windisch2013-01-101-16/+17
|/ | | | | | | | | | | | Each test now consumes a new port number (9500+), and socket directory based on an auto-incrementing global counter (TESTCNT). Also makes minor cleanup to test receiver setup logic. fixes bug 1096223 Change-Id: Ie6a88c535c81f2fc12b97d32c2674a6184325d2a
* Revert "Support lookup of value using "group.key""Mark McLoughlin2013-01-081-6/+0
| | | | | | | | | | | | | | | This reverts commit 525ac47. There are already two ways to reference an option in a group: CONF.group.key CONF[group].key Adding a third variant doesn't seem ideal. Also, for the specific case of LazyPluggable in Nova, we can easily just pass an optional config group name to the constructor. Change-Id: I1a29a18d90e8af3ce2563bd1b3eeb64422140016
* Merge "Fix publish_errors unit test"Jenkins2013-01-081-8/+9
|\
| * Fix publish_errors unit testMark McLoughlin2013-01-031-8/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The unit test for publish_errors=True causes logging to be spewed from other tests because it configures the default root logger rather than creating its own logger like other logging tests do. This makes the unit tests completely silent again. Also, it hooked into the rpc notifier to catch notifications when it can simple hook into the no-op notifier like other tests. Finally, it used stubout to override the value of config options rather than the BaseTestCase.config() helper method. Change-Id: I13313fcd9ad8f9e5bcb9af90652b5dde5c3feec9
* | Merge "JSONDictSerializer encode objects to unicode"Jenkins2013-01-081-0/+11
|\ \
| * | JSONDictSerializer encode objects to unicodeSean McCully2013-01-031-0/+11
| |/ | | | | | | | | | | | | | | | | | | Averts raising ValueError, Circular Reference Detected exception. Add additional test to JSONDictSerializerTest testing that JSONDictSerializer correctly serializes objects into unicode repr. and not raise error Fixes: bug #1089100 Change-Id: Ifdb0562c7c43ab66617dddb65f16f893df2f4895
* | Merge "rpc method arguments should be dict, not list"Jenkins2013-01-081-13/+37
|\ \
| * | rpc method arguments should be dict, not listEric Windisch2013-01-081-13/+37
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | default arguments were being set to a list instead of a dict. This caused problems if the args key was missing in the msg argument to the call/cast methods. Adds tests to check if RPC cast/call work without arguments. Changes and documents the _test_cast method arguments, allowing for more flexibility and code reuse. fixes bug 1089750 Change-Id: I854dc3fcad8936695feb4414f3862672cf98c5d1
* | Merge "Support lookup of value using "group.key""Jenkins2013-01-081-0/+6
|\ \
| * | Support lookup of value using "group.key"Davanum Srinivas2013-01-021-0/+6
| |/ | | | | | | | | | | | | | | | | | | | | | | Let us check if the opt_name has a '.', if it does then split it into a group/key and try lookup using that combination. Since LazyPluggable uses "CONF[self.__pivot]" if we just add this capability to cfg, we get "LazyPluggable doesn't support option groups" for free. Fixes LP #1093043 Change-Id: I9cedcf22014038e9fe4ed5e66ca5427aa99b5091
* | Merge "Add common base weigher/weigher handler for filter scheduler"Jenkins2013-01-072-0/+67
|\ \
| * | Add common base weigher/weigher handler for filter schedulerZhiteng Huang2013-01-042-0/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Filter scheduler is being used for more than one core projects (Nova and Cinder as of writing), the implementation shared a lot of common code. This patch is to move base weigher (weighing function), weigher handler for filter scheduler into oslo to reduce possible porting. implement bp: common-weights Change-Id: I2d1b37438663b53e035cc262875a283e5e2ee970
* | | Merge "Add common filter/filter handler for filter scheduler"Jenkins2013-01-073-0/+689
|\| |
| * | Add common filter/filter handler for filter schedulerZhiteng Huang2013-01-043-0/+689
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Filter scheduler is being used for more than one core projects (Nova and Cinder as of writing), the implementation shared a lot of common code. This patch moves base filter/filter handler class as well as common filter implementation for filter scheduler into oslo to reduce possible porting. implement bp: common-filters Change-Id: If0b1dee79c410c98e152230b55c1ec5dbcdef27c
* | | Eliminate sleep in the lockutils test case (across processes)Davanum Srinivas2013-01-071-32/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fork many processes and try to lock the same set of files using flock without blocking and make sure we wait for all the processes to finish as well. Fixes LP #1068316 Change-Id: I09964b2c5af63f31b5ddee1f18eaf646f8d8ba58
* | | Merge "Optionally add message envelope to notifications."Jenkins2013-01-062-4/+51
|\ \ \
| * | | Optionally add message envelope to notifications.Russell Bryant2013-01-032-4/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When message envelopes were added for rpc messages, notifications were explicitly left out. This was due to the fact that notifications are consumed outside of OpenStack. However, notification consumers will likely want to eventually start taking advantage of some of the features that the envelope is a prerequisite for, such as message signing. This patch adds a new notification driver that produces message based notifications including the message envelope. Related to blueprint version-rpc-messages. Change-Id: Iaa8b437e6b4f64053f4b02bf486516f60bb020e9
* | | | Merge "Fix test cases in tests.unit.test_service"Jenkins2013-01-051-28/+26
|\ \ \ \
| * | | | Fix test cases in tests.unit.test_serviceHengqing Hu2013-01-031-28/+26
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Be patient in test_service while waiting service to be spawned or killed, because: * Service spawning has a limit of one process a second. * A test server in the cloud might have limited resources, the time there might be several times slower. Change-Id: Idab1d1158ecc7564d0b1f9cdd32496496ff5b3f8
* | | | Merge "Make time comparison functions accept strings"Jenkins2013-01-041-8/+30
|\ \ \ \
| * | | | Make time comparison functions accept stringsDan Smith2013-01-041-8/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch makes is_older_than() and is_newer_than() accept timestamps in string format, as returned from strtime(). Change-Id: I1dbf453cc08bd8aaeb4fee2491a1e8aa74f8bee3
* | | | | Merge "Disable lockutils test_synchronized_externally"Jenkins2013-01-041-0/+2
|\ \ \ \ \
| * | | | | Disable lockutils test_synchronized_externallyMark McLoughlin2013-01-041-0/+2
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're seeing this test fail very regularly in Jenkins and it's become a serious distraction for everyone. Disable the test temporarily and use a bug #1095957 to track that the test itself needs fixing. Change-Id: I0645d8f5f740d40f83d82e5ef7a048f33a44ac2b
* | | | | Merge "Fix timezone handling in timeutils tests"Jenkins2013-01-041-10/+7
|\ \ \ \ \ | |/ / / / |/| / / / | |/ / /
| * | / Fix timezone handling in timeutils testsDan Smith2013-01-031-10/+7
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The tests for timeutils use some canned datetime values that include timezones, and override the result of utcnow() with the same. This is not correct because the real utcnow() returns naive datetime objects, so tests that assume the contrary are incorrect. This came about because I was unable to satisfy both the oslo unit tests and the nova ones with the same code. That happened because the oslo ones override utcnow() with an invalid result, and the nova tests use the real one. Change-Id: I683818f77ce0d3e3b2f07c0326d7a7196f3346c3
* / | Sanitize cells calls.Michael Still2013-01-041-0/+47
|/ / | | | | | | | | | | | | Cells puts the data to be sanitized deeper into the message. We should get this into oslo ready for when cells lands. Change-Id: Ic5d111f11c979a179e0edcb314feb3f9d93f66ce
* | Merge "Add a rpc envelope format version number."Jenkins2013-01-032-7/+36
|\ \
| * | Add a rpc envelope format version number.Russell Bryant2013-01-022-7/+36
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds a message envelope that includes a envelope format version number. This message envelope will allow us to embed additional metadata later on, such as a signature for the message payload. Up to this point, we've deferred message serialization as a responsibility of the messaging library we're using by passing it a message as Python types and letting it deal with how to pass it over a network. This patch adds json serialization in the rpc layer of the application message payload before passing the message down into the messaging library. There are some benefits to be gained by doing a pass at serialization ourselves. As an example, we occasionally hit serialization bugs that only affect some messaging drivers. The kombu driver has always had a nice advantage. It uses anyjson internally for serializing messages, which we hook into using our jsonutils module. When there is a problem serializing, we automatically use to_primitive() to fix it. This patch allows all drivers to take advantage of this automatic message fix-up. This also creates a convenient common hook point for messages coming in and out of the system, regardless of the driver in use. While this changes the base format of the messages sent between nodes, it has been done in a backwards compatible manner. The new message format will not be used by default. The idea is that all nodes will be upgraded to a version that is capable of receiving the new format (Grizzly) before switching it on. We will turn it on post-Grizzly. Implement blueprint version-rpc-messages. Change-Id: Ib6b2d11ca42abaa64c40986d72233e7048e504a0
* | Merge "Move rootwrap code to openstack.common"Jenkins2013-01-031-0/+202
|\ \
| * | Move rootwrap code to openstack.commonThierry Carrez2012-12-201-0/+202
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Copies current nova-rootwrap code to openstack.common, so that it can be reused by Cinder and Quantum. Implements blueprint common-rootwrap. Before it can be used in projects, update.py needs to grow the capability to deploy files in bin/ and etc/, as well as replacing a placeholder text by the destination project name in source files and binary names. In this proposed version, the placeholder text is "oslo". Change-Id: I8655d5b3cccacd1cc2225aa539339fb478615422
* / fix intermittent failures with deferred version testDavanum Srinivas2012-12-251-4/+23
|/ | | | | | | | | | | | | | Need to get rid of the deferred string approach as the problem was overriding string methods is hard and flaky. There is just no other documented argparse hook to allow deferring of the version calculation as version passed in has to be a str or buffer. Added a test to try longer string and even run it in a loop just to be sure. Fixes LP #1087575 Change-Id: Icf83944d67b8ce5d3230627f3de10501da3276f2
* Merge "Make wsgi.Server comply with service.Launcher interface"Jenkins2012-12-121-3/+3
|\
| * Make wsgi.Server comply with service.Launcher interfaceMichael Basnight2012-12-101-3/+3
| | | | | | | | | | | | fixes LP 1087369 Change-Id: Icf222b5bffe1f40a5c325ffe2a1b397f22713c00
* | Merge "Add deprecated --logdir common opt"Jenkins2012-12-081-2/+11
|\ \
| * | Add deprecated --logdir common optMark McLoughlin2012-12-071-2/+11
| | | | | | | | | | | | | | | | | | | | | | | | --logfile and --logdir are aliases Nova has for the --log-file and --log-dir. If we're to support --logfile as a deprecated common option, we should do the same for --logdir. Change-Id: I16485a93070d9ad7789a287d5b035c6f270ffead
* | | Merge "Allow remote stdlib exceptions to be deserialized"Jenkins2012-12-081-0/+14
|\ \ \
| * | | Allow remote stdlib exceptions to be deserializedMark McLoughlin2012-12-051-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #1086798 Add 'exceptions' to allowed_rpc_exception_modules so that stdlib exceptions (like NotImplementedError) can be deserialized from a RemoteError. Change-Id: I57f40fed2a86cc08fb64b81d648c71cd7db8027c
* | | | Merge "Add deprecated --logfile common opt."Jenkins2012-12-071-0/+11
|\ \ \ \ | | |/ / | |/| |
| * | | Add deprecated --logfile common opt.Dan Prince2012-12-061-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds a deprecated common options for --logfile which is an alias for --log_file. This resolves some backwards compatability issues with the most recent oslo common code where --logfile was no longer a valid opt. Change-Id: I17b1277da94a2d81ae439d650a6d7321420dfe14
* | | | Merge "Allow nova and others to override some logging defaults"Jenkins2012-12-062-0/+74
|\ \ \ \