summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | Silence exceptions from qpid connection.close()David Ripton2013-05-311-1/+7
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1185465 Don't log exceptions that happen during connection.close(). They're not surprising (a closed socket can't be read or written), don't require sysadmin attention, and provide red herrings in the logs to distract from finding real root causes of problems. Change-Id: Ie414ba3f4ff1f1dee13e6168ac2e28dab535581b
* | | | | Merge "Reduce duplicated code related to policies"Jenkins2013-05-315-107/+337
|\ \ \ \ \
| * | | | | Reduce duplicated code related to policiesFlaper Fesp2013-05-215-107/+337
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds some logic that is currently duplicated throughout Openstack. As part of this de-duplication, the patch also modifies current implementation. Major Changes: * check, set_rules, reset, init are now part of the Enforcer class * check was renamed into enforce * init was renamed into load_rules * It is now possible to load multiple files and have per instance rules instead of global rules. * There's a global instance of the Enforcer class that can be used as main enforcer. from openstack.common import policy ENFORCER = policy.ENFORCER ENFORCER.enforce(rule, target, creds) Minor Changes: * Added do_raise to the enforce method * Enforcer instance is now passed to the Check call. NOTE: If / once this patch gets in, I'll update other projects and port them to the latest version. Change-Id: Ife909bdf3277ef33c2fb1eae16ae261fa6374c63
* | | | | | Merge "Use zero-copy in ZeroMQ proxy (zmq-rpc-receiver)"Jenkins2013-05-311-14/+7
|\ \ \ \ \ \
| * | | | | | Use zero-copy in ZeroMQ proxy (zmq-rpc-receiver)Eric Windisch2013-05-231-14/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now, messages will not be copied into the Python memory space from the C library. The 'topic' key *will* be copied, to make use of several string/bytes functions that are available to us. (Potentially, we might be able to work with this as a memoryview in the future as a further optimization?) Implements blueprint zmq-zero-copy Change-Id: I20a700867ac642b1a1788b31e3a21f8a299c97c8
* | | | | | | Merge "Use mox object from moxstubout fixture"Jenkins2013-05-313-6/+3
|\ \ \ \ \ \ \ | |_|_|_|/ / / |/| | | | / / | | |_|_|/ / | |/| | | |
| * | | | | Use mox object from moxstubout fixtureMark McLoughlin2013-05-203-6/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The moxstubout fixture creates a mox object which we can re-use in tests. It also ensures verify() is called, which it clearly wasn't in the case of the qpid tests. Change-Id: Ibf3326f18b160865012d40a5841b65438262c4df
* | | | | | Merge "Enable hacking H403 test"Jenkins2013-05-3013-19/+34
|\ \ \ \ \ \
| * | | | | | Enable hacking H403 testSergey Lukjanov2013-05-3013-19/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | H403 multi line docstring end on new line Change-Id: I33249651026b54ef346214965e909835288bb14e
* | | | | | | Merge "Enable hacking H401 test"Jenkins2013-05-308-11/+10
|\| | | | | |
| * | | | | | Enable hacking H401 testSergey Lukjanov2013-05-308-11/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | H401 docstring should not start with a space Change-Id: I761665ea9aff1f55de9f629ff8d108ba141eaf79
* | | | | | | Merge "Enable hacking H201 test"Jenkins2013-05-302-3/+3
|\| | | | | |
| * | | | | | Enable hacking H201 testSergey Lukjanov2013-05-292-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | H201 no 'except:' at least use 'except Exception:' Change-Id: Ibfab55ace63636d3507fbdad46d1436b8ec83475
* | | | | | | Merge "Ignore backup files in .gitignore"Jenkins2013-05-291-1/+2
|\ \ \ \ \ \ \
| * | | | | | | Ignore backup files in .gitignoreAlessio Ababilov2013-05-221-1/+2
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | Change-Id: I71af28ec8e75f321961826e2653fee9b99a87c1c
* | | | | | | Merge "Remove rootwrap from IPC directory creation"Jenkins2013-05-291-11/+10
|\ \ \ \ \ \ \
| * | | | | | | Remove rootwrap from IPC directory creationEric Windisch2013-05-231-11/+10
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removes processutils/rootwrap dependency. Fixes bug 1160420 and partially fixes 1180631. The attempt will still be made to create /var/run/openstack, but will give a reasonable error when it happens. If the directory exists, but permissions are denied, we should also give a reasonable error. A patch will be submitted to devstack that will set and create the IPC directory as necessary. Packagers should do this as well. Change-Id: I7814320adbfd0b2fe54cc2e523452c18ab2e7687
* | | | | | | Merge "Use stubout in test_correlation_id"Jenkins2013-05-291-3/+2
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | |
| * | | | | | Use stubout in test_correlation_idMark McLoughlin2013-05-201-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Best to use stubout for stubbing out methods like this. That way they get properly unstubbed on failure. Change-Id: I61dc96d01e331e8d811b8f24137e8f738cf09215
* | | | | | | Merge "Fix tox coverange env"Jenkins2013-05-291-1/+2
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | |
| * | | | | | Fix tox coverange envMark McLoughlin2013-05-201-1/+2
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're not setting the VIRTUAL_ENV environment variable, so 'tox -ecover' is failing because patch_tox_venv uses the variable. Change-Id: I3898d0dd1f8a9bf4216253bb2c7d8491e1e8ce56
* | | | | | Fixes email typo in MAINTAINERSZhongyue Luo2013-05-281-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I7ba0b9435d12064c4ab0e169fbc08a0c704ae78d
* | | | | | Merge "Import oslo.config configuration file generator"Jenkins2013-05-283-0/+259
|\ \ \ \ \ \
| * | | | | | Import oslo.config configuration file generatorJulien Danjou2013-05-283-0/+259
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is stolen from Nova with a fix stolen from https://review.openstack.org/#/c/30251/ Obviously the final goal is to stop copying this file into each project and start improving it, to move it finally in oslo.config. Change-Id: Ie81a5c3db1dc6092bcc6179ab0ab9499239f0bbc Signed-off-by: Julien Danjou <julien@danjou.info>
* | | | | | Merge "Changed processing unique constraint name."Jenkins2013-05-281-3/+4
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Changed processing unique constraint name.Victor Sergeyev2013-05-281-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Function `_raise_if_duplicate_entry_error()` modified respectively to new unique constraint name convention. We found that current constraint name convention allows us create constraints with duplicate names. It happens if we will add constraints in different tables with same column names (for example, `name`, `deleted`). In this case we can not create new constraint due to mysql limitation (it requires unique constraint name for database) To solve this issue unique constraint name convention was changed from "uniq_c1_x_c2_x_c3" to "uniq_t$c1$c2$c3" where `t` it is table name and columns `c1`, `c2`, `c3` are in UniqueConstraint. Function `_raise_if_duplicate_entry_error()` parse error from database and provides us correct column names in error message. So now we receive correct error message from function. Change-Id: I4a507deba97c499fcd9738d980e36ca0f9454b71 Fixes: bug 1182054
* | | | | | Merge "Add support for requirements.txt."Jenkins2013-05-271-2/+14
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | |
| * | | | | Add support for requirements.txt.Monty Taylor2013-05-181-2/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In prep for moving the projects to requirements.txt and test-requirements.txt, we need to have our legacy install_venv codebase support finding requirements in the right place. Change-Id: I0f300c6bb1792c81e2133a41a5631f2dd4c021ba
* | | | | | Merge "Specify database group instead of DEFAULT"Jenkins2013-05-225-41/+162
|\ \ \ \ \ \
| * | | | | | Specify database group instead of DEFAULTDavanum Srinivas2013-05-205-41/+162
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | At the request of Quantum folks, Let us switch from DEFAULT to database for the database related options. This will help with migration etc. DocImpact Fixes LP# 1171837 Change-Id: If602a6a7cc0f2a202632dd14574fea60dce4b589
* | | | | | Merge "Enable hacking H702 localization test"Jenkins2013-05-212-2/+3
|\ \ \ \ \ \
| * | | | | | Enable hacking H702 localization testJoe Gordon2013-05-202-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix and enable test for 'formatting operation should be outside of localization method call' Change-Id: I0e707f9d4aacf75f9a9239b323a57d7d39f30c84
* | | | | | | Merge "Enable hacking H703: Multiple positional placeholders"Jenkins2013-05-212-3/+4
|\| | | | | |
| * | | | | | Enable hacking H703: Multiple positional placeholdersJoe Gordon2013-05-202-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix and enable test for multiple positional placeholders in localizations Change-Id: Ie3b5e3113cd0f3340b7b0f5774001893b1e6857c
* | | | | | | Add generic serialization supportDan Smith2013-05-215-5/+175
| |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds a generic serialization model that the RpcProxy and RpcDispatcher will use to serialize and deserialize arguments and return values. The base definition is provided, as well as a default NoOpSerializer which performs no special action (retaining the existing behavior). Related to blueprint rpc-object-serialization Change-Id: I0a33baddee3e28dfc47100eb3216b679558b0bdd
* | | | | | Allow RPC_API_NAMESPACE on RpcProxy objectsChris Behrens2013-05-202-5/+19
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows one to subclass RpcProxy for use with a particular namespace without having to repeatedly pass the namespace into the make_namespaced_msg() method. This new RPC_API_NAMESPACE attribute will be used for the make_msg() call. It is defaulted to None to match previous behavior. Change-Id: Ied62df839cda0be6f9f9b060bbfc22e086f61be8
* | | | | Merge "Add Middleware for correlation_id"Jenkins2013-05-203-0/+91
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Add Middleware for correlation_idAditi Raveesh2013-05-153-0/+91
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The middleware generates the 'correlation_id' if it is not already present in the request header. This 'correlation_id' can be used to enable more effective debugging of requests that span multiple services. Implements: blueprint create-a-unified-correlation-id Change-Id: Ibc06a5b05b628e0b640e9299c79127ecbf9c684e
* | | | Merge "Add hacking extension to flake8"Jenkins2013-05-202-1/+5
|\ \ \ \ | | |/ / | |/| |
| * | | Add hacking extension to flake8Joe Gordon2013-05-152-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Include hacking in test-requires and disable any failing tests. Once failing tests are fixed they can be enabled to gate on them. Change-Id: Id6a228441ec94d5d0bee1dba05118bb35ecaa001
* | | | Migrate to pbr.Monty Taylor2013-05-178-876/+50
| |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | Also, remove the files associated with this that should not be pulled from oslo-incubator any longer. Closes bug 1179007 Change-Id: Id2e882837bdde51a60562e8cdd207c7ff0c88a6a
* | | Merge "Break out common functionality for filters and weights"Jenkins2013-05-174-42/+53
|\ \ \
| * | | Break out common functionality for filters and weightsMatthew Sherborne2013-05-154-42/+53
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the context of openstack.common.scheduler: base_filter.BaseFilterManager and base_weight.BaseWeightManager had duplicated code. This patch breaks that code out into a shared base class: base_handler.BaseHandler Change-Id: Ie5232c6dd9876a75170beda71adde90fc6bb4153
* | | Merge "Extract matchmaker_ring to own module"Jenkins2013-05-174-81/+123
|\ \ \
| * | | Extract matchmaker_ring to own moduleEric Windisch2013-05-154-81/+123
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The base matchmaker module should not have all the matchmakers implemented within. Instead, we desire to break them out. For purposes of deprecation, logs a warning when importing from the ZeroMQ driver. Change-Id: I76936edd97309621a9c28363dccd792293a1c0d3
* | | Merge "Pass backdoor_port to services being launched"Jenkins2013-05-162-1/+12
|\ \ \
| * | | Pass backdoor_port to services being launchedBrian Elliott2013-05-092-1/+12
| | |/ | |/| | | | | | | Change-Id: I7227648f9b9e1d3e30f4b2d03964ee5c625a1024
* | | Merge "Add DynamicLoopCall timers to ThreadGroups"Jenkins2013-05-162-0/+57
|\ \ \
| * | | Add DynamicLoopCall timers to ThreadGroupsBrian Elliott2013-05-092-0/+57
| |/ / | | | | | | | | | Change-Id: I99a67355a7af054c17d3f6807609dbb11ad40631
* | | Merge "Remove the notifier and its dependencies from log.py"Jenkins2013-05-163-14/+38
|\ \ \