summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | Merge "Handle invalid xml request to return BadRequest"Jenkins2012-10-162-1/+34
|\ \ \ \ \ \ \ \
| * | | | | | | | Handle invalid xml request to return BadRequestvijaya-erukala2012-10-092-1/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Handled ExpatError,LookupError to return BadRequest if the user inputs the invalid xml fixes bug 1032092 Change-Id: I7374532d91e3d7675bc80730031a195f50bb2abc
* | | | | | | | | Merge "Set host prior to allocating network information."Jenkins2012-10-161-3/+4
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Set host prior to allocating network information.Brian Elliott2012-10-161-3/+4
| | |_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make sure 'host' field on the instance is set before allocating network information. bug 1065004 Change-Id: I7b929fd50a057c8c69a0b83e07cdc5cd9cb55b45
* | | | | | | | | Merge "Remove db access for block devices and network info on reboot"Jenkins2012-10-165-14/+113
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Remove db access for block devices and network info on rebootDan Smith2012-10-165-14/+113
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes compute/api pass the block device and network info to compute_manager::reboot_instance() so that it doesn't have to hit the database itself. This also extends the reboot tests for the compute manager and api to validate that these two bits of data are making it all the way through the stack. bp no-db-compute Change-Id: I92346d6c821241885b5f4b08fec05f38e6261edf
* | | | | | | | | | Merge "Remove db access for block devices on terminate_instance"Jenkins2012-10-166-29/+86
|\| | | | | | | | | | |/ / / / / / / / |/| | | | | | | |
| * | | | | | | | Remove db access for block devices on terminate_instanceDan Smith2012-10-166-29/+86
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes compute/api pass in an instance's block devices to compute/manager::terminate_instance() so that it doesn't have to hit the database itself. Note that this doesn't remove all database activity in the terminate_instance() path, just the block-device-related bits. bp no-db-compute Change-Id: Ibfb333ef7918e05f0e3634f1fb063f1b02535bff
* | | | | | | | | Imported Translations from TransifexOpenStack Jenkins2012-10-1621-594/+168
| |_|_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ibd2d774602925e5d9863367a42ca79886ea59e2d
* | | | | | | | Merge "Fixes live_migration missing migrate_data parameter in Hyper-V driver"Jenkins2012-10-152-3/+3
|\ \ \ \ \ \ \ \
| * | | | | | | | Fixes live_migration missing migrate_data parameter in Hyper-V driverAlessandro Pilotti2012-10-142-3/+3
| | |_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes Bug #1066513 Live migration on Hyper-V fails due to the parameter "migrate_data" that has been added to the signature of live_migration(...) in nova.virt.ComputeDriver. This fix adds the new parameter. Change-Id: I393f6706bc144838ae232dfb275460620bc203e2
* | | | | | | | Merge "Internationalize nova-manage."Jenkins2012-10-151-36/+43
|\ \ \ \ \ \ \ \
| * | | | | | | | Internationalize nova-manage.Michael Still2012-10-151-36/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This review wraps some strings which were missing out on internationalization. It also adds a slightly more helpful error message for quota key validation. Change-Id: Id6f1f2578b78815b69cf43823d83ad24ca4123aa
* | | | | | | | | Merge "Add api samples to simple tenant usage extension"Jenkins2012-10-1517-0/+319
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Add api samples to simple tenant usage extensionMauro S. M. Rodrigues2012-10-1117-0/+319
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ib6e00e5c049846146619f52732cd06fefac62494
* | | | | | | | | | Imported Translations from TransifexOpenStack Jenkins2012-10-1521-11523/+11936
| |/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I5afda0c4f159ab25a6538b0e79cb1b3859f70f2c
* | | | | | | | | Merge "nova-manage doesn't validate key to update the quota"Jenkins2012-10-142-1/+10
|\ \ \ \ \ \ \ \ \ | |_|/ / / / / / / |/| | | | | | | |
| * | | | | | | | nova-manage doesn't validate key to update the quotavijaya-erukala2012-10-092-1/+10
| | |_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | nova-manage doesn't validate the key value supplied to update the quota, as a result unnecessary records will be created in db and user will be under the impression that quota value got updated. This patch validates the input value given to the key. fixes bug 1064359 Change-Id: I9928f30881aa2780a23005b5f69aa67a44f314c5
* | | | | | | | Merge "Stops compute api import at import time"Jenkins2012-10-125-11/+17
|\ \ \ \ \ \ \ \
| * | | | | | | | Stops compute api import at import timeVishvananda Ishaya2012-10-125-11/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is an automatic import of compute.api at import time whenever anything under compute is imported. This leads to all sorts of weird dependency problems such as the one here: https://review.openstack.org/#/c/14170/ This modifies the import to happen when compute.API() is called, which is how we handle it for network and volume. A few imports were also modified to make it consistent with this new style. Change-Id: Ic748376510aa4cc48dfefc6ec36621d0ae498bfc
* | | | | | | | | Merge "Fix imports in openstack compute tests"Jenkins2012-10-129-191/+174
|\| | | | | | | | | |_|_|_|_|/ / / |/| | | | | | |
| * | | | | | | Fix imports in openstack compute testsVishvananda Ishaya2012-10-129-191/+174
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes imports in a number of tests consistent. There were also various places where files were being stubbed out that werent explicitly imported. This patch fixes those as well. It also moves compute api stubbing to stub the actual compute.api.API class instead of compute.API. Includes a rewrite of test_createserverext to stub the create method instead of doing a bunch of odd setup with a mock compute api. Change-Id: Id8ebb2f65228fafeac69c7ab635ff8909b79c449
* | | | | | | | Merge "Pass correct task_state on snapshot"Jenkins2012-10-121-0/+5
|\ \ \ \ \ \ \ \
| * | | | | | | | Pass correct task_state on snapshotRick Harris2012-10-111-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes an issue where we were not passing the correct task_state to the compute manager thereby triggering an 'unexpected task-state error' when the snapshot finished. This error only manifested itself in multi-cell setups where the compute-manager and compute-api are using separate DB's and therefore must rely on the RPC-layer to enforce consistency. Fixes bug 1065250 Change-Id: Ie07454f0d2075f5c3f2dd53b4e5ff9983ea3241f
* | | | | | | | | Merge "Dis-associate an auto-assigned floating IP should return proper warning"Jenkins2012-10-127-1/+73
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Dis-associate an auto-assigned floating IP should return proper warningSirisha Devineni2012-10-097-1/+73
| | |_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added new exception class CannotDissociateAutoAssignedFloatingIP and raised exception instead of return. Fixes bug 1061499 Change-Id: I348573a235c6b81653837267072b5c48fa15b8af
* | | | | | | | | Merge "Used instance uuid rather than id in remove-fixed-ip"Jenkins2012-10-121-1/+1
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Used instance uuid rather than id in remove-fixed-ipTianTian Gao2012-10-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | remove-fixed-ip called db method fixed_ip_get_by_instance. but fixed_ip_get_by_instance use instance uuid not id, so change to uuid in remove-fixed-ip. Fixed bug: #1065899 Change-Id: Ic7526c7e91933424c04be9bb91d4a98cfedd9b01
* | | | | | | | | | Merge "Migrate DHCP host info during resize"Jenkins2012-10-122-2/+19
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Migrate DHCP host info during resizeJian Wen2012-10-122-2/+19
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update instance host to migration_ref['dest_compute'] before update_dhcp to make its DHCP host info will be written in dhcp-hostsfile. Make similiar change in finish_revert_resize. Fixes bug 1065440 Change-Id: I9d3e0ebf8512f98f2943999e7ad95bc58f2bf585
* | | | | | | | | | Merge "Make run_tests.sh fail if no tests are actually run"Jenkins2012-10-121-1/+12
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Make run_tests.sh fail if no tests are actually runDan Smith2012-10-121-1/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Right now, run_tests seems to report test success if there is a catastrophic failure early in the process. This can happen if you have a python file that fails to parse, or if something else in the test environment setup fails to initialize (such as creating the empty database for the test run). Nosetests will report success, but indicate that zero tests were run. This patch snatches the output of nosetests and, if zero tests ran, reports to the user that something is broken and offers a few suggestions. Change-Id: Ide6fd8b212f0650cb227f60b25d4b4e62ee150ac
* | | | | | | | | | | Merge "read_deleted snapshot and volume id mappings."Jenkins2012-10-121-2/+4
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / |/| | | | | | | | | |
| * | | | | | | | | | read_deleted snapshot and volume id mappings.Adam Gandelman2012-10-111-2/+4
| | |/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since the migration that creates the volume_id_mappings and snapshot_id_mappings tables does not populate the 'deleted' column, queries to this table should not limit results to 'deleted=0'. Limiting to non-deleted rows results in duplicate mappings being created for existing instance mappings after an upgrade, and throws off volume and snapshot ID to UUID mapping. This is a stop-gap measure to ensure avoid serious breakage during an upgrade. The NULL columns in this table are actually unused currently and can be populated in later grizzly migrations, if they end up being used. Fixes bug 1065785. Change-Id: I893c994362a203288b9984f0ede24cbb274bfcc5
* | | | | | | | | | Implement snapshots for raw backendBoris Filippov2012-10-127-28/+199
| |_|_|_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | blueprint snapshots-for-everyone Use simple qemu-img convert for raw snapshots. Polymorphically select snapshot behavior for raw and qcow2. Allow images to be constructed from actual device/file path. Change-Id: I6a57e43c6775c144c41a53382dcc7504ce6d4c43
* | | | | | | | | Make sure sleep can be foundStanislaw Pitucha2012-10-121-1/+1
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some systems have only /usr/bin/sleep and no links. (like Arch) Change-Id: I406dd72ba1a4669966ee58e110aee43ddf723b4a
* | | | | | | | Merge "Clean up imports in test_servers"Jenkins2012-10-111-130/+132
|\ \ \ \ \ \ \ \
| * | | | | | | | Clean up imports in test_serversVishvananda Ishaya2012-10-111-130/+132
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There were a lot of assumed imports in test_servers that were not explicitly being imported. This makes sure all of the used files are actually imported and makes the importing of all of the files consistent. Change-Id: I07d20c44de7a350465dce91fbbe5ac9fd2426d37
* | | | | | | | Merge "Update run_tests.sh pep8 ignore list for pep8 1.2"Jenkins2012-10-111-2/+2
|\ \ \ \ \ \ \ \
| * | | | | | | | Update run_tests.sh pep8 ignore list for pep8 1.2Joe Gordon2012-10-111-2/+2
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Patch I32a8808c6c9cccfedcc4d2a26649333aca1cd713 changed pep8 to 1.2 but did not update the ignore list in run_tests.sh Change-Id: I9d2cab79197ba3cdd7103d4532e28a999c9454f4
* / / / / / / / Revert "Tell SQLite to enforce foreign keys."Joe Gordon2012-10-111-10/+0
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch broke the unit tests. (0 tests were being run) This reverts commit 5b4b7e4083b85bd9c10792cf0adb11dc07e92333. Change-Id: I9b34a4f12c2c17c8f03899f9d92752ec84068be8
* | | / / / / Avoid RPC calls while holding iptables lock.Michael Still2012-10-112-10/+30
| |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This exhibitied itself as very slow instance starts on a Canonical test cluster. This was because do_referesh_security_group_rules() was making rpc calls while holding the iptables lock. This refactor avoids that while making no functional changes (I hope). This should resolve bug 1062314. Change-Id: I36f805bd72f7bd06082cfe96c58d637203bcffb7
* | | | | | Merge "Use paramiko.AutoAddPolicy for the smoketests."Jenkins2012-10-101-1/+1
|\ \ \ \ \ \
| * | | | | | Use paramiko.AutoAddPolicy for the smoketests.Dan Prince2012-10-091-1/+1
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updates the Nova smoketests so that we use Paramiko's auto add ssh key policy. This should help quiet down warnings like this which we previously got when running the EC2 smoketests with Nova: /usr/lib/python2.7/site-packages/paramiko/client.py:95: UserWarning: Unknown ssh-rsa host key for 172.20.0.3: adb1c2f1fe461a35cd01283eac03d6fb Fixes LP Bug #1064550. Change-Id: I1611a3f6b54563b6a1d64c4f8e193a636ccb1c69
* | | | | | Merge "Add api-samples to Used limits extension"Jenkins2012-10-105-0/+254
|\ \ \ \ \ \
| * | | | | | Add api-samples to Used limits extensionMauro S. M. Rodrigues2012-10-095-0/+254
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | Change-Id: I67069637db35eeb5a87026bf62252d006ed11b98
* | | | | | Merge "Proxy floating IP calls to quantum"Jenkins2012-10-105-24/+482
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | |
| * | | | | Proxy floating IP calls to quantumAkihiro MOTOKI2012-10-095-24/+482
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1023169. Also fixes bug 1031119. In Folsom, floating IP support is moved to Quantum. By this commit floating IP calls to nova are proxied to Quantum and nova command can be used to manage floating IPs. pip-requires is also updated and now requires python-quantumclient >=2.1 which supports floating IP feature in Quantum. Change-Id: I2c32948a8fe291601216dc62d546da64a8fb8428
* | | | | Merge "Add Multiple Create extension to API sample tests."Jenkins2012-10-1017-0/+269
|\ \ \ \ \
| * | | | | Add Multiple Create extension to API sample tests.Matthew Treinish2012-10-0517-0/+269
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ifa5cf31cc61e4269502ff692508866948670e46f Signed-off-by: Matthew Treinish <treinish@linux.vnet.ibm.com>