summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | 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
* | | | | | Merge "Allow update.py copy config files and binaries"Jenkins2013-01-041-6/+22
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Allow update.py copy config files and binariesMark McLoughlin2012-12-201-6/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements blueprint common-binaries Add support to update.py for copying config files and binaries matching etc/oslo/$module*.conf bin/oslo-$module* into the destination project after replacing 'oslo' with the destination project name. This is specifically intended for rootwrap but may be applicable in other cases in the future. Change-Id: If8a3a815ef452b0dd73ed28cb61a7ff16a9dee5c
* | | | | | Sanitize cells calls.Michael Still2013-01-042-10/+63
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-038-66/+185
|\ \ \ \ \
| * | | | | Add a rpc envelope format version number.Russell Bryant2013-01-028-66/+185
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-036-0/+699
|\ \ \ \ \ | | |/ / / | |/| | |
| * | | | Move rootwrap code to openstack.commonThierry Carrez2012-12-206-0/+699
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | Merge "Don't use exclusive queues for fanout and direct"Jenkins2013-01-031-4/+4
|\ \ \ \
| * | | | Don't use exclusive queues for fanout and directVishvananda Ishaya2012-12-281-4/+4
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The use of exclusive queues can cause issues on reconnect if the rabbit server hasn't deleted the queue already. We might as well use exclusive=False so that we can reconnect to the queue if necessary. Fixes bug 1094358 Change-Id: I35bd903737d6054e8856b0003b0b8af97fd21c91
* | | | Merge "fix intermittent failures with deferred version test"Jenkins2013-01-032-39/+29
|\ \ \ \ | |_|/ / |/| | |
| * | | fix intermittent failures with deferred version testDavanum Srinivas2012-12-252-39/+29
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* / / Don't rely on os.wait() blockingDavanum Srinivas2013-01-021-1/+8
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before this fix which was included in eventlet 0.10.0: https://bitbucket.org/which_linden/eventlet/issue/92/eventletgreen-override-of-oswaitpid the os.wait() wrapper in eventlet would not block if no child processes had exited. This was clearly buggy behaviour and os.wait() no blocks correctly. However, it appears it now does not return if the parent process was interrupted by a signal. This means that you can't kill the parent process and have the child processes cleaned up gracefully. To avoid all these issues, switch to non-blocking wait behaviour. We use WNOHANG with waitpid() to poll for an exited child process and, if there is none, we yield to other green threads and sleep for a short period. This means excessive CPU usage, though, which will be tracked by bug #1095346. Check pid to avoid some unnecessary warning/info messages in the logs Fixes LP #1094076 Change-Id: I783fac652376c2daa9d591403b9f4c1fe9523043
* | Merge "Register the control_exchange RPC option"Jenkins2012-12-182-13/+9
|\ \ | |/ |/|
| * Register the control_exchange RPC optionMark McLoughlin2012-12-112-13/+9
| | | | | | | | | | | | | | | | | | | | | | Fixes bug #1031725 Have the control_exchange option be registered by Oslo, but allow projects to override the default of 'openstack' by doing e.g.: rpc.set_defaults(control_exchange='nova') Change-Id: Ic6e534d9a70a1cfc94207b78caa57e897cf78f63
* | Merge "Make wsgi.Server comply with service.Launcher interface"Jenkins2012-12-122-10/+15
|\ \ | |/ |/|
| * Make wsgi.Server comply with service.Launcher interfaceMichael Basnight2012-12-102-10/+15
| | | | | | | | | | | | fixes LP 1087369 Change-Id: Icf222b5bffe1f40a5c325ffe2a1b397f22713c00
* | Merge "Add deprecated --logdir common opt"Jenkins2012-12-082-2/+12
|\ \
| * | Add deprecated --logdir common optMark McLoughlin2012-12-072-2/+12
| | | | | | | | | | | | | | | | | | | | | | | | --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-082-0/+15
|\ \ \
| * | | Allow remote stdlib exceptions to be deserializedMark McLoughlin2012-12-052-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-072-1/+13
|\ \ \ \ | | |/ / | |/| |
| * | | Add deprecated --logfile common opt.Dan Prince2012-12-062-1/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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