summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Add support for retrieving group keysshared-key-msgSimo Sorce2013-08-192-25/+104
| | | | | | | | 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-195-14/+293
| | | | | | | | | | | | | | 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-192-0/+22
|\
| * Make ZeroMQ based RPC consumer threads more robustEric Windisch2013-07-232-0/+22
| | | | | | | | | | | | | | | | | | Consumers could potentially raise an uncaught exception and exit prematurely. Related to bug 1189711 Change-Id: I9cb32d8530f3a1dd623e193476e02b990b40b466
* | Bump hacking to 0.7.0Zhongyue Luo2013-08-1726-206/+204
| | | | | | | | | | | | Fixes H202, H602 errors Change-Id: Ic3045f15116d1d7a28b63bc94a0a341b9304ffdd
* | exception: removeJulien Danjou2013-08-1612-316/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-162-0/+8
|\ \
| * | py3kcompat: Add urljoin compatibilityChuck Short2013-08-142-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | Add urljoin for python2/python3 compatibility, used by python-keystoneclient. Change-Id: Ied0841bf6b34d0087002c77915031156767b3d8d Signed-off-by: Chuck Short <chuck.short@canonical.com>
* | | Merge "Replace using tests.utils with openstack.common.test"Jenkins2013-08-1627-75/+70
|\ \ \
| * | | Replace using tests.utils with openstack.common.testAlexander Gorodnev2013-08-0927-75/+70
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is the first step to replace using tests.utils with openstack.common.test. All these tests don't use mock objects, stubs, config files and use only BaseTestCase class. Change-Id: I511816b5c9e6c5c34ebff199296ee4fc8b84c672 bp: common-unit-tests
* | | | Merge "Allow mapping _ to lazy gettext path"Jenkins2013-08-152-1/+53
|\ \ \ \
| * | | | Allow mapping _ to lazy gettext pathMatt Odden2013-08-142-1/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "Move `./run_tests.sh` to Oslo"Jenkins2013-08-156-1/+737
|\ \ \ \ \
| * | | | | Move `./run_tests.sh` to OsloVictor Sergeyev2013-08-136-1/+737
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Moved Nova `./run_tests.sh` script to `tools/run_tests_common.sh` file. This script run unit-tests and can be reused in other projects. Added script `./run_tests.sh` with project specific variables to run `tools/run_tests_common.sh` script. Added required `tools/*` files for pretty colorizer outpoot. Modified `tests/utils.py` file - added work with OS_TEST_TIMEOUT, OS_STDOUT_CAPTURE and OS_STDERR_CAPTURE environment variables from .testr.conf blueprint run-tests-script Change-Id: I057e8cf2c74d160402cb881fc761623c74905b7e
* | | | | Merge "Fix wrong argument in openstack common policy"Jenkins2013-08-152-1/+6
|\ \ \ \ \
| * | | | | Fix wrong argument in openstack common policyAlexander Gordeev2013-08-072-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "Update .mailmap"Jenkins2013-08-151-0/+1
|\ \ \ \ \ \
| * | | | | | Update .mailmapYufang Zhang2013-08-131-0/+1
| | |/ / / / | |/| | | | | | | | | | | | | | | | Change-Id: I9ea7398281fe57c02df74854ecac8431125638e0
* | | | | | 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
* | | | | | Unset OS_xx variable before generate configurationMehdi Abaakouk2013-08-141-0/+3
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When generate_sample.sh generate the configuration of the keystone middleware part, it use OS_XXXXX environment variable if this one is set instead of the default one. This change unset this variables, before generate_sample.sh inspect the code to always have the default value. Change-Id: I4f41eece3d93633fcc07274f533adfa1887325fc
* | | | | 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-122-2/+18
|\ \ \ \ \
| * | | | | Ensure context type is handled when using to_dictLance Bragstad2013-08-102-2/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | | Don't attempt to patch eventlet without the patchMonty Taylor2013-08-081-9/+10
| |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | Some projects, like horizon, do not carry the eventlet patch. Change-Id: I24ffad9ab7852be44e5d6c8805aa0a47647e21cc
* | | | | Merge "Add a fixture for dealing with config"Jenkins2013-08-083-0/+90
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Add a fixture for dealing with configAlexander Gorodnev2013-08-073-0/+90
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-082-7/+49
|\ \ \ \ \
| * | | | | Fix missing argument bug in oslo common policyVictor Sergeyev2013-08-012-7/+49
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-083-4/+27
|\ \ \ \ \
| * | | | | Helper function to sanitize db url credentialsZhongyue Luo2013-08-023-4/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-083-0/+279
|\ \ \ \ \ \
| * | | | | | Move sqlalchemy migration from NovaVictor Sergeyev2013-07-253-0/+279
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
generated by cgit (git 2.34.1) at 2026-01-01 22:05:38 +0000