summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | Merge "Log when release file lock"Jenkins2013-01-121-0/+5
|\ \ \ \ \
| * | | | | Log when release file lockJoe Gordon2013-01-091-0/+5
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We log when waiting for a file lock, when getting one, so lets log when releasing one. Change-Id: I1838c424fda74c04e9c2107db08226670e8372f3
* | | | | Merge "Fix pep8 E125 errors."Jenkins2013-01-126-18/+18
|\ \ \ \ \
| * | | | | Fix pep8 E125 errors.Monty Taylor2013-01-116-18/+18
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | Caesar's wife must be above reproach. Change-Id: Iac85a57e71d403360f1567c07c8699057f0772fb
* | | | | Merge "Simplify version processing."Jenkins2013-01-125-203/+70
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Simplify version processing.Monty Taylor2013-01-115-203/+70
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-082-11/+1
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "Add zmq receiver binary"Jenkins2013-01-081-0/+76
|\ \
| * | Add zmq receiver binaryEric Windisch2013-01-081-0/+76
| | | | | | | | | | | | | | | | | | | | | | | | | | | The impl_zmq rpc driver requires a binary running on all hosts. Previously residing in nova, this binary is being moved into oslo-incubator. Change-Id: I3f38374b71794029550147ed4fc75f0fd63adf75
* | | Merge "Use package for writing versioninfo."Jenkins2013-01-081-2/+2
|\ \ \
| * | | Use package for writing versioninfo.Steve Baker2013-01-041-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This currently fails for client projects as python_package is the name of the project, while package is where versioninfo is written to. This doesn't affect server openstack projects because python_package == package. This hasn't affected any client projects yet because most likely none of them call version.canonical_version_string() in their setup.py Change-Id: Ia97e5fae378c47909f3dbc842dc83e408af68671
* | | | 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-082-1/+12
|\ \ \ \ \
| * | | | | JSONDictSerializer encode objects to unicodeSean McCully2013-01-032-1/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-082-15/+39
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | |
| * | | | | rpc method arguments should be dict, not listEric Windisch2013-01-082-15/+39
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-082-1/+11
|\ \ \ \ \
| * | | | | Support lookup of value using "group.key"Davanum Srinivas2013-01-022-1/+11
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "Fix serialization in impl_zmq."Jenkins2013-01-081-7/+30
|\ \ \ \ \
| * | | | | Fix serialization in impl_zmq.Russell Bryant2013-01-071-7/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the rpc message envelope patch went in, the serialization done in impl_zmq was replaced with what is done in rpc.common. Instead, it just needs to be done as an additional stage of serialization. The reason is that the amqp drivers can handle passing dicts into the messaging libraries. The zeromq library can not handle that, so it has to ensure it is a string before passing it down to the library. This patch restores the original (unmodified) _serialize() and _deserialize() methods in impl_zmq that were removed and calls them as appropriate. Change-Id: I09cac780c9a9b15390229c63d71e360120548b3d
* | | | | | Merge "Make WebOb version specification more flexible"Jenkins2013-01-081-1/+1
|\ \ \ \ \ \ | |/ / / / / |/| | | | / | | |_|_|/ | |/| | |
| * | | | Make WebOb version specification more flexibleDoug Hellmann2013-01-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change the WebOb version to >=1.0.8 as an temporary measure to allow the actual version update to 1.2.3 to roll out across the projects one at a time without breaking the integration tests. Change-Id: Ie08a97a31eaae0b1245b94f85e0f926512e4d513 Signed-off-by: Doug Hellmann <doug.hellmann@dreamhost.com>
* | | | | Merge "Close kombu.connection.BrokerConnection correctly"Jenkins2013-01-071-1/+1
|\ \ \ \ \
| * | | | | Close kombu.connection.BrokerConnection correctlyHengqing Hu2012-12-281-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make kombu.connection.BrokerConnection use release() to close the connection. It has no such attribute as close(), since AttributeError is in connection_errors, the fault was previous concealed by the following catch statement. Change-Id: I9f7da2c64f9d408aa2a9ddf723ea83291887cff6
* | | | | | Merge "Add common base weigher/weigher handler for filter scheduler"Jenkins2013-01-075-0/+209
|\ \ \ \ \ \
| * | | | | | Add common base weigher/weigher handler for filter schedulerZhiteng Huang2013-01-045-0/+209
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-0712-3/+1127
|\| | | | | |
| * | | | | | Add common filter/filter handler for filter schedulerZhiteng Huang2013-01-0412-3/+1127
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | | Merge "Add a latex command to our sphinx builders."Jenkins2013-01-071-1/+9
|\ \ \ \ \ \
| * | | | | | Add a latex command to our sphinx builders.Monty Taylor2013-01-041-1/+9
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The libra project added this locally and use it for some docs they make. It seemed like something we could add what we provide globally. Change-Id: I219829693568362bbe01354a8f477a10031754c4
* | | | | | 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 "Enable millisecond logging by default"Jenkins2013-01-071-5/+6
|\ \ \ \ \ \
| * | | | | | Enable millisecond logging by defaultJoe Gordon2013-01-031-5/+6
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Related to Ic28867ae9a436e81c7f2fcf79f40a1ecc251072c Change-Id: I135213cad3ec21edf8934c1094ee84ac93ed4561
* | | | | | Merge "Fix debug message in multicall of rpc.amqp"Jenkins2013-01-061-1/+1
|\ \ \ \ \ \
| * | | | | | Fix debug message in multicall of rpc.amqpHengqing Hu2012-12-271-1/+1
| | |_|/ / / | |/| | | | | | | | | | | | | | | | Change-Id: Ib5ed65df937e9fe4e36cdf7d0ce92b627f22dbd3
* | | | | | Merge "Optionally add message envelope to notifications."Jenkins2013-01-069-26/+129
|\ \ \ \ \ \
| * | | | | | Optionally add message envelope to notifications.Russell Bryant2013-01-039-26/+129
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "Ensure proxy callback threads exit during tests"Jenkins2013-01-053-0/+24
|\ \ \ \ \ \
| * | | | | | Ensure proxy callback threads exit during testsMark McLoughlin2013-01-033-0/+24
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On the consumer side, each message is consumed in a separate thread but we currently have no way of waiting for all these threads to complete. In BaseRpcTestCase.test_call_timeout(), we complete the thread while there is still a message handler running on the other side. This can mean the consumer attempting to send its reply while other tests are running, which clearly is a bad thing. Make the connection close() and reset() methods in the kombu and qpid drivers wait on proxy object threads before returning to avoid this situation. Change-Id: I8fb84b1dc7f31196ef27a5d1f8c4a3346b03e0ae
* | | | | | Use basestring instead of str for type check.Flaper Fesp2013-01-041-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I1dbf453cc08bd8aaeb4fee2491a1e8aa74f8bee3 introduced some checks to is_older_than and is_newer_than functions against str instead of basestring. I'm aware that basestring doesn't exist in Python3 but since it isn't a target right now and there's no "standard" way to support it right now in openstack. Change-Id: I8d67d384637dfdde3f5621052d8511e7b6ecab7b
* | | | | | Merge "Make time comparison functions accept strings"Jenkins2013-01-042-8/+34
|\ \ \ \ \ \
| * | | | | | Make time comparison functions accept stringsDan Smith2013-01-042-8/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
generated by cgit (git 2.34.1) at 2025-09-28 19:13:06 +0000