summaryrefslogtreecommitdiffstats
path: root/openstack
Commit message (Collapse)AuthorAgeFilesLines
* Add support for retrieving group keysshared-key-msgSimo Sorce2013-08-191-24/+103
| | | | | | | | Group keys are keys used by all the hosts listening to a specific topic. The unqualified topic name is the group name. Example: conductor, scheduler, ... Change-Id: Ia5bcdd140cef16e07d4d8ab4030d1ac5cb59212c
* RPC: Add support for optional message signingSimo Sorce2013-08-194-13/+119
| | | | | | | | | | | | | | Wires in SecureMessage so that signing can be optionally used. Signing messages is always attempted but failing to create a signed message is currently not fatal. In case of failure the message is sent unsigned. Once all services has been converted to support signing, we can start switching on mandatory signing. Currently the service name needs to be a global due to the way services are constructed. Change-Id: Ie565bbaf61a69a4e526e78e723efecb5e2712877
* Adjust KDS communication protocolSimo Sorce2013-08-191-4/+4
| | | | | | | The KDS review process has determined some changes to the API. This patch fixes KDSClient to interoperate properly. Change-Id: I888e8a41573f171048356a9d277de1d7610537ce
* Merge "Make ZeroMQ based RPC consumer threads more robust"Jenkins2013-08-191-0/+1
|\
| * Make ZeroMQ based RPC consumer threads more robustEric Windisch2013-07-231-0/+1
| | | | | | | | | | | | | | | | | | Consumers could potentially raise an uncaught exception and exit prematurely. Related to bug 1189711 Change-Id: I9cb32d8530f3a1dd623e193476e02b990b40b466
* | exception: removeJulien Danjou2013-08-164-156/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch drops the obsolete openstack.common.exception module. Most project should define their own exception based on their API and the context they evolve in. Many projects don't use this, only neutron, cinder and heat used it. I've copy pasted the few exceptions that they were using, but that's still by far less code than this whole file. Ultimately that will let them a chance to remove or shring their own exception.py. I don't think most projects should have one anyway. Change-Id: Ia8b2b29bd443233def324e97de7342c2634bccff Closes-Bug: #1208734
* | Merge "py3kcompat: Add urljoin compatibility"Jenkins2013-08-161-0/+2
|\ \
| * | py3kcompat: Add urljoin compatibilityChuck Short2013-08-141-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | Add urljoin for python2/python3 compatibility, used by python-keystoneclient. Change-Id: Ied0841bf6b34d0087002c77915031156767b3d8d Signed-off-by: Chuck Short <chuck.short@canonical.com>
* | | Merge "Allow mapping _ to lazy gettext path"Jenkins2013-08-151-1/+17
|\ \ \
| * | | Allow mapping _ to lazy gettext pathMatt Odden2013-08-141-1/+17
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | Since nova has switched from using gettext.install for importing the _ function to explicitly importing the _ function from gettextutils, we need to be able to enable our deferred translation code path on the gettextutils._ function. Change-Id: I7fec00810ea57c985e1f43c644913a9980b35655
* | | Merge "Fix wrong argument in openstack common policy"Jenkins2013-08-151-1/+1
|\ \ \
| * | | Fix wrong argument in openstack common policyAlexander Gordeev2013-08-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fully fixes wrong param passed to the exception constructor from oslo.config occurring in Enforcer._get_policy_path Closes-Bug: #1208397 Change-Id: I28b11178808b447555e6e6e63383904749c77b18
* | | | Merge "Fixes files with wrong bitmode"Jenkins2013-08-152-2/+0
|\ \ \ \ | |_|/ / |/| | |
| * | | Fixes files with wrong bitmodeZhongyue Luo2013-08-132-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | Some modules have bitmode 755. Changed to 644 Change-Id: I2084779186fcbd5ee2010451531355a0c67897f5
* | | | Merge "Allow passing a logging level to processutils.execute"Jenkins2013-08-131-3/+8
|\ \ \ \ | |/ / / |/| | |
| * | | Allow passing a logging level to processutils.executeDavid Ripton2013-08-081-3/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, processutils.execute always logged what commands were run at DEBUG level. This is usually fine, but sometimes the caller wants to log particularly important commands at INFO level instead, and oslo should allow that. Change-Id: I89af6fdc0bb11a4134aaaca417a54ba3b515ca7e
* | | | Fix case error in qpid exchange type "direct"David Ripton2013-08-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Most places correctly have "direct", but one place has "Direct" instead, which is incorrect. Fixes bug #1211338 Change-Id: I2516317ff7f6ff18ee0a99a60d0c058ed309d4da
* | | | Merge "Ensure context type is handled when using to_dict"Jenkins2013-08-121-2/+7
|\ \ \ \
| * | | | Ensure context type is handled when using to_dictLance Bragstad2013-08-101-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Handle the case where the context passed into def pack_context() is a dictionary. If a dictionary is passed in, we don't need to call to_dict before updating the msg. fixes bug 1208971 Change-Id: I2ce0b28f97634e717868e0ee5525189338d4981c
* | | | | Merge "Fix capitalization, it's OpenStack"Jenkins2013-08-124-4/+4
|\ \ \ \ \
| * | | | | Fix capitalization, it's OpenStackAndreas Jaeger2013-08-114-4/+4
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | This capitalization shows up in the manual as well. Let's be consistent. Change-Id: I282d4ae3311bc3fcda6be8e5a1fceecfc8ddb115
* | | | | Merge "middleware: code cleanup"Jenkins2013-08-112-8/+1
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | middleware: code cleanupJulien Danjou2013-08-082-8/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The removed code doesn't serve anything. Change-Id: I04b0cf7abc465de0115345018d2157da67f6985c
* | | | | Merge "Allow non-use of cfg.CONF in securemessage"Jenkins2013-08-091-4/+6
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Allow non-use of cfg.CONF in securemessageMark McLoughlin2013-08-071-4/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If someone passes a ConfigOpts instance other than cfg.CONF, we need to register the group and options with it. Change-Id: I10908d5e29d8a871ae29a5d3fa3f415e487f23b8
* | | | | Merge "Add a fixture for dealing with config"Jenkins2013-08-081-0/+45
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Add a fixture for dealing with configAlexander Gorodnev2013-08-071-0/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Many tests change config's variables directly in CONF object. This fixture allows to override the values in more convenient way and restores config after test completion. Blueprint: common-unit-tests Change-Id: I5535c534425278d8804ca9fe84846e15ac898db0
* | | | | Merge "Fix missing argument bug in oslo common policy"Jenkins2013-08-081-2/+2
|\ \ \ \ \
| * | | | | Fix missing argument bug in oslo common policyVictor Sergeyev2013-08-011-2/+2
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | AndCheck() and OrCheck() classes in common policy module missing a argument when calls rules in __call__() method. Current patch fixed it by adding the necessary argument. Tests added. Fixed bug 1194354 Change-Id: I9a6e7d04b06ef61216cb7dbfd291c855b799b621
* | | | | Merge "Helper function to sanitize db url credentials"Jenkins2013-08-082-4/+13
|\ \ \ \ \
| * | | | | Helper function to sanitize db url credentialsZhongyue Luo2013-08-022-4/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The database url is sanitized in logfiles because of security issues. However the connected url itself is useful information to devs and admins. This patch provides a helper function to sanitize only the credentials in a database url. All projects must process the CONF.database.connection value using "sanitize_db_url" when updating the db package Fixes bug #1076833 Change-Id: Id6cf7b120ef6c3fcda7f33fd26676b62a4475bb2
* | | | | | Merge "Move sqlalchemy migration from Nova"Jenkins2013-08-082-0/+125
|\ \ \ \ \ \
| * | | | | | Move sqlalchemy migration from NovaVictor Sergeyev2013-07-252-0/+125
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added functions for work with migration scripts to openstack/common/db/sqlalchemy/migration.py file. This code can be used in Nova, Cinder, Glance, etc. Added new DbMigrationError exception to openstack/common/db/exception.py file. Tests added. blueprint test-migrations Change-Id: Ia63f7fb89b0a6baacf3fd424c0db09c74680af4a
* | | | | | Merge "Add common part of test-related tools to oslo"Jenkins2013-08-081-0/+52
|\ \ \ \ \ \
| * | | | | | Add common part of test-related tools to osloAlexander Gorodnev2013-08-061-0/+52
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add BaseTestCase class - testtools wrapper; Blueprint: common-unit-tests Change-Id: I5bce04f2ee55f678ea5a2a2c76f5837454ed7bb7
* | | | | | Merge "Some nitpicky securemessage cleanups"Jenkins2013-08-071-6/+6
|\ \ \ \ \ \
| * | | | | | Some nitpicky securemessage cleanupsMark McLoughlin2013-08-071-6/+6
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove redundant parenthesis and use a consistent commenting style. Some redundant parenthesis are retained to avoid this pep8 silliness: https://github.com/jcrocholl/pep8/issues/126 i.e. to avoid E125 we have the choice of: if (md['source'] != source or md['destination'] != target or md['expiration'] < time.time()): raise InvalidEncryptedTicket(md['source'], md['destination']) or: if ((md['source'] != source or md['destination'] != target or md['expiration'] < time.time())): raise InvalidEncryptedTicket(md['source'], md['destination']) the latter is my preference. Change-Id: Ic890bfc787a17fc77821e01e586806d1b558d530
* | | | | | Merge "Remove redundant global keyword in securemessage"Jenkins2013-08-071-1/+0
|\ \ \ \ \ \
| * | | | | | Remove redundant global keyword in securemessageMark McLoughlin2013-08-071-1/+0
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The global keyword is only needed where we are assigning to the variable in the method. Change-Id: If9f3fcc5323e2a154db1f5b457ce3d5079f83253
* | | | | | Merge "Don't shadow cfg import in securemessage"Jenkins2013-08-071-6/+6
|\ \ \ \ \ \
| * | | | | | Don't shadow cfg import in securemessageMark McLoughlin2013-08-071-6/+6
| |/ / / / / | | | | | | | | | | | | | | | | | | Change-Id: I03ed9f1cbbc08dea43a5c93d6997eb4e633dfed4
* / / / / / Clarify precedence of secret_key_fileMark McLoughlin2013-08-071-2/+4
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If secret_key_file is set, then secret_key is ignored. Clarify this in the help strings. Change-Id: Ie10ebb7c76e274b31ab9826a81373518c9b09e0a
* | | | | Merge "RPC: Add MessageSecurity implementation"Jenkins2013-08-071-0/+518
|\ \ \ \ \
| * | | | | RPC: Add MessageSecurity implementationSimo Sorce2013-08-051-0/+518
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add Message Security[1] class and exceptions Also add process-local Key Storage and Caching facility. [1] https://wiki.openstack.org/wiki/MessageSecurity Change-Id: Ieced7ab6a9209c3e0291d26376a8d35c25175002
* | | | | | Merge "Add missing exceptions for per user quota"Jenkins2013-08-061-29/+31
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Add missing exceptions for per user quotaSergey Skripnick2013-08-051-29/+31
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was missing exceptions QuotaExists and ProjectUserQuotaNotFound Rename message format property from `message' to `msg_fmt' in QuotaException Change-Id: I93f621fae342e78280bf50eebf5b0bca9d281ed3
* | | | | Merge "Fix Message format-string parsing"Jenkins2013-08-051-1/+1
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Fix Message format-string parsingJohn Warren2013-08-051-1/+1
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The parsing in openstack/common/gettextutils.Message._save_dictionary_parameter does not account for two or more format directives in direct succession, e.g.: %(key1)s%(key2)s causing key/value pairs to not be copied during a mod operation. Fixes bug 1207789 Change-Id: I56940003de3e6e2fa7dd08604d04f4c57586b0a3
* | | | Merge "python3: Import translation layer"Jenkins2013-08-052-0/+64
|\ \ \ \
| * | | | python3: Import translation layerChuck Short2013-08-042-0/+64
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Python3 reorganized the standard library and moved several functions to different modules. Six provides a consistent interface to them through the fake six.moves module. However, the urlparse, urllib2, etc modules have been combined into one module which Six does not support so do it the old fashioned way. Change-Id: I9a865ef1fc4de6f94cc3ccf26db0483f2bdc13df Signed-off-by: Chuck Short <chuck.short@canonical.com>