summaryrefslogtreecommitdiffstats
path: root/openstack
Commit message (Collapse)AuthorAgeFilesLines
...
| * | Improve millisecond loggingJoe Gordon2013-01-091-3/+3
| | | | | | | | | | | | | | | | | | Always display 3 digits for milliseconds (appends 0s) Change-Id: I3f1461839258be0723e2d3616ec225a830d13029
* | | 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-124-16/+16
|\ \ \
| * | | Fix pep8 E125 errors.Monty Taylor2013-01-114-16/+16
| | |/ | |/| | | | | | | | | | | | | Caesar's wife must be above reproach. Change-Id: Iac85a57e71d403360f1567c07c8699057f0772fb
* / | Simplify version processing.Monty Taylor2013-01-112-197/+61
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* / Revert "Support lookup of value using "group.key""Mark McLoughlin2013-01-081-5/+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 "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 "JSONDictSerializer encode objects to unicode"Jenkins2013-01-081-1/+1
|\ \
| * | JSONDictSerializer encode objects to unicodeSean McCully2013-01-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-2/+2
|\ \ \
| * | | rpc method arguments should be dict, not listEric Windisch2013-01-081-2/+2
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-1/+5
|\ \ \
| * | | Support lookup of value using "group.key"Davanum Srinivas2013-01-021-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | 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 "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-072-0/+136
|\ \ \ \ \
| * | | | | Add common base weigher/weigher handler for filter schedulerZhiteng Huang2013-01-042-0/+136
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-077-0/+423
|\| | | | |
| * | | | | Add common filter/filter handler for filter schedulerZhiteng Huang2013-01-047-0/+423
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | | 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-067-22/+78
|\ \ \ \ \ \
| * | | | | | Optionally add message envelope to notifications.Russell Bryant2013-01-037-22/+78
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "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-041-0/+4
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | |
| * | | | | Make time comparison functions accept stringsDan Smith2013-01-041-0/+4
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch makes is_older_than() and is_newer_than() accept timestamps in string format, as returned from strtime(). Change-Id: I1dbf453cc08bd8aaeb4fee2491a1e8aa74f8bee3
* / | | | Sanitize cells calls.Michael Still2013-01-041-10/+16
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | 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-036-59/+149
|\ \ \ \
| * | | | Add a rpc envelope format version number.Russell Bryant2013-01-026-59/+149
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-033-0/+345
|\ \ \ \
| * | | | Move rootwrap code to openstack.commonThierry Carrez2012-12-203-0/+345
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-031-35/+6
|\ \ \ \ | |_|/ / |/| | |
| * | | fix intermittent failures with deferred version testDavanum Srinivas2012-12-251-35/+6
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-121-7/+12
|\ \ | |/ |/|
| * Make wsgi.Server comply with service.Launcher interfaceMichael Basnight2012-12-101-7/+12
| | | | | | | | | | | | fixes LP 1087369 Change-Id: Icf222b5bffe1f40a5c325ffe2a1b397f22713c00
* | Merge "Add deprecated --logdir common opt"Jenkins2012-12-081-0/+1
|\ \