summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | | | | | Merge "Move to a more canonicalized output from qemu-img info."Jenkins2012-11-016-57/+302
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Move to a more canonicalized output from qemu-img info.Joshua Harlow2012-10-316-57/+302
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move to a form that is all lower cased, dashes->underscores, underscores instead of spaces which allows for better integration with python. Also make the parser more robust to failures when encountering new fields such as snapshot lists. Provide a new qemu img info object that can be used to do the parsing and access the underlying attributes. Change-Id: Ie098dbd9f06dd4ef966768e2caa128f1d09b019c
* | | | | | | | | | | | | Merge "Make bdms primitive in rpcapi.terminate_instance"Jenkins2012-11-013-6/+37
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Make bdms primitive in rpcapi.terminate_instanceNikola Dipanov2012-11-013-6/+37
| | |_|_|_|_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch converts bdms (a list of block device mappings) passed to nova.compute.rpcapi.terminate_instance to primitive types that are safe to be sent to the rpc backend. Also fixes the fake BDM classes used in: * nova.tests.api.ec2.test_cloud * nova.tests.api.openstack.compute.test_server_actions to pass through josnutils.to_primitive properly and not confuse it due to wrongly delegated dotted lookups fixes bug #1073240 Change-Id: I329a48c1a53dde5895a329b39f32854c286de071
* | | | | | | | | | | | Merge "Fix copy-paste bug in block_device_info_generation"Jenkins2012-11-012-7/+9
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Fix copy-paste bug in block_device_info_generationVishvananda Ishaya2012-10-312-7/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: If45a20d9e7cc0c332e3a3df1eaf22457f505acea
* | | | | | | | | | | | Remove duplicate api_paste_config setting.Clark Boylan2012-10-311-1/+0
|/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | nova/tests/fake_flags.py does not need to set the api_paste_config option twice. Remove the duplicate entry. Change-Id: Id0d95d222a9c57c88abb3071c0b682f644747e20
* | | | | | | | | | | Merge "Fixes hypervisor based image filtering on Hyper-V"Jenkins2012-10-316-112/+184
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Fixes hypervisor based image filtering on Hyper-VAlessandro Pilotti2012-10-316-112/+184
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes Bug #1073547 Nova scheduler's ImagePropertiesFilter based filtering requires the "supported_instances" stat returned by the driver's get_host_stats method. This fix implements the get_host_stats method, providing also some refactoring in the process by adding a HostOps class. Change-Id: Ia6e73eb3e8ff0be028854fe4fd6d8e305fa1d504
* | | | | | | | | | | | Merge "Add rxtx_factor to network migration logic"Jenkins2012-10-316-60/+111
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Add rxtx_factor to network migration logicKevin L. Mitchell2012-10-316-60/+111
| | |_|_|_|_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, when an instance is migrated from one host to another, the only action the network API is able to take is moving floating IP addresses. We also need to be able to update the rxtx_factor value to update QoS settings. This change makes sure that the network APIs will be passed the rxtx_factor value during a migration. Change-Id: If4e1e96b84ee082c2d277c3d91da7df0325fc1ff
* | | | | | | | | | | | Merge "Remove redundant code from PowerVM driver"Jenkins2012-10-311-14/+1
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Remove redundant code from PowerVM driverDan Smith2012-10-311-14/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch removes the dead instance cleanup from the init_host() method in the PowerVM driver. Not only do I think that it's redundant with the rest of init_host() in compute_manager(), it's also examining the instance task_state, which I think is a layering violation. Tangentially related to bp/no-db-compute Change-Id: Ie7eb4c07caa162f32021ccdbed51db950d30e2c7
* | | | | | | | | | | | Merge "Use env to set environ when starting dnsmasq"Jenkins2012-10-313-8/+11
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Use env to set environ when starting dnsmasqAndrew Laski2012-10-303-8/+11
| | |_|_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Explictly use env to setup needed environment variables for dnsmasq when starting to avoid problems with subprocess mis-interpreting the first command line argument as the executable name. Also update DnsmasqFilter to accept any command that starts with env followed by a set of args that include an equals sign, as long as the next arg is the dnsmasq command. fixes bug #1073253 Change-Id: I8ac08ba2d2309934a67ed2cb28049ed5d3277d63
* | | | | | | | | | | Merge "Add scheduler retries for prep_resize operations."Jenkins2012-10-316-119/+445
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ / / / |/| | | | | | | | | |
| * | | | | | | | | | Add scheduler retries for prep_resize operations.Brian Elliott2012-10-316-119/+445
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Retry operations are subject to race conditions for compute resources. prep_resize can race for resources on the destination host and revert_resize can race for resources on the original source host. This type of problem is handled for new instance build requests by the resource tracker in compute. As a first step to extending the resource tracker to cover resizes, this patch adds scheduler retries around the prep_resize operation. bug 1065267 Change-Id: I1b32c0a282772d9580e322b50990932016058329
* | | | | | | | | | | Merge "Add call to reset quota usage"Jenkins2012-10-314-9/+97
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / |/| | | | | | | | | |
| * | | | | | | | | | Add call to reset quota usageKevin L. Mitchell2012-10-314-9/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are certain circumstances where we may need to indicate an uncertainty in a user's current usage of a given resource, thus forcing a recount of those resources currently in use. This adds a QUOTAS.usage_reset() call which can be used to do this. It works by resetting the current usage to -1, which will force the usage record to be refreshed the next time that user reserves those resources. Change-Id: I4a40c8edcc78ed9a7d18a293d4d7d6418b04b7f2
* | | | | | | | | | | Merge "Clean up xenapi VM records on failed disk attaches"Jenkins2012-10-315-68/+127
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / / |/| | | | | | | | | |
| * | | | | | | | | | Clean up xenapi VM records on failed disk attachesChris Behrens2012-10-305-68/+127
| | |_|/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Breaks the create_vm_step up so that we can clean up the VM record in case of disk attach failures. Fixes bug 1073219 Change-Id: I54e28e2cce6fe4f29a03566eff998479410f7555
* | | | | | | | | | Merge "fetch_images() method no more needed"Jenkins2012-10-313-26/+10
|\ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / |/| | | | | | | | |
| * | | | | | | | | fetch_images() method no more neededÉdouard Thuleau2012-10-263-26/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Nova Glance wrapper returns all images metadata if the parameter 'limit' is not define. It doesn't split the image metadata list. So the sub method fetch_image is no more needed. We can delete it. I added a check in Nova Glance wrapper to be sure the parameter 'marker' or another one aren't set to 'None' (like in bug #1065053). Fixes LP bug #1070904 Change-Id: I24ecc4adf158401f60b508aa3a20bd4c91bfa92b
* | | | | | | | | | Merge "Clean up quantumv2.get_client"Jenkins2012-10-311-12/+11
|\ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / |/| | | | | | | | |
| * | | | | | | | | Clean up quantumv2.get_clientZhongyue Luo2012-10-291-12/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Makes the get_client function more readable Change-Id: I90a1e529308d0f52d1315e57cc412b40426d50a6
* | | | | | | | | | Merge "Remove nose detailed error reporting."Jenkins2012-10-311-1/+0
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / |/| | | | | | | | |
| * | | | | | | | | Remove nose detailed error reporting.Clark Boylan2012-10-301-1/+0
| | |_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Nosetests' detailed error reporting breaks when running testtools tests. Disable this plugin to get proper error reporting. Change-Id: I4898f567e8588e1dd61940486dfc515e459adc34
* | | | | | | | | Merge "refactor: extract method vm_ref_or_raise"Jenkins2012-10-313-8/+69
|\ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / |/| | | | | | | |
| * | | | | | | | refactor: extract method vm_ref_or_raiseMate Lakat2012-10-313-8/+69
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Related to bp xenapi-volume-drivers Imporve the code by extract method, and testing. Change-Id: I8caca173266890bd99a360671e7af86759b89201
* | | | | | | | | Merge "Fix VM deletion from down compute node"Jenkins2012-10-313-45/+134
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Fix VM deletion from down compute nodeJoe Gordon2012-10-303-45/+134
| | |/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * free network resources * free volume resources * delete.start and delete.stop notifications added * Handle network deallocate in multi_host mode Fixes bug 1067214 Co-authored-by: Vishvananda Ishaya <vishvananda@gmail.com> Change-Id: I0d4a7dc5836d39e405824528de214f23b214849f
* | | | | | | | | Force earlier version of sqlalchemyChris Yeoh2012-10-311-1/+1
| |/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Set the version of sqlalchemy to be earlier than the 0.8 beta as it is incompatible with the latest available sqlalchemy-migrate and as a result run_tests.sh does not run any tests. Change-Id: I9c5515b76d35e4999f81d71e82f818685efe54ea
* | | | | | | | Merge "pep8 fixes for nova-manage"Jenkins2012-10-301-3/+4
|\ \ \ \ \ \ \ \
| * | | | | | | | pep8 fixes for nova-manageSean Dague2012-10-301-3/+4
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | nova-manage is now getting pep8 scanned, so these fixes are needed to make things happy. Change-Id: I68b1d17fc06c2b2580907eb950b6087e499d65f9
* | | | | | | | Merge "remove session parameter from fixed_ip_get"Jenkins2012-10-301-9/+14
|\ \ \ \ \ \ \ \
| * | | | | | | | remove session parameter from fixed_ip_getDevananda van der Veen2012-10-271-9/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove session parameter from public method fixed_ip_get. This requires some changes to floating_ip_disassociate, but doesn't appear to affect anything else. Change-Id: I435bfc4f2f4175049fd164fea37b4724871e1eb1
* | | | | | | | | Remove database usage from libvirt check_can_live_migrate_destinationDan Smith2012-10-3010-40/+99
| |/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While libvirt is the only one that actually uses the compute host info during this check, I think that it's an entirely reasonable thing to pass to each hypervisor when doing the "can I migrate to this host" check. Thus, I've added it excplicitly, without a gating check of the capabilities. Related to bp:no-db-compute Change-Id: I7049879fcec3894165f1bece9beeb045ed78c94b
* | | | | | | | Merge "refactor: extract _terminate_volume_connections"Jenkins2012-10-301-16/+13
|\ \ \ \ \ \ \ \
| * | | | | | | | refactor: extract _terminate_volume_connectionsMate Lakat2012-10-301-16/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Extract _terminate_volume_connections method, to remove code duplication. Change-Id: Ib9bede24d051700fe297fdd5a619f4a3a21bb88e
* | | | | | | | | Merge "Ensures compute_driver flag can be used by bdm"Jenkins2012-10-302-5/+5
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Ensures compute_driver flag can be used by bdmJohn Garbutt2012-10-292-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1063851 by moving compute_driver flag from compute/manager.py to the global flags.py Change-Id: I06891fd65550b63ffe83da18b6af41cf46938667
* | | | | | | | | | Merge "improve sessions around compute_node_*"Jenkins2012-10-301-19/+18
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | improve sessions around compute_node_*Devananda van der Veen2012-10-291-19/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removed 'session' parameter from public method compute_node_get, and replaced private calls with new method _compute_node_get. Removed unused session paramter from other compute_node_* methods. Change-Id: I943306be014b61d8818fdbe7891f6ad5ef934751
* | | | | | | | | | | Merge "details the reboot behavior that a virt driver should follow."Jenkins2012-10-301-0/+6
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | details the reboot behavior that a virt driver should follow.Armando Migliaccio2012-10-261-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixes bug 1000774 Change-Id: I5a1cd62181d6f3772b890006ae73113eee0a0e0e
* | | | | | | | | | | | Merge "Support for nova client list hosts with specific zone"Jenkins2012-10-302-11/+30
|\ \ \ \ \ \ \ \ \ \ \ \ | | |_|_|_|_|_|_|_|_|_|/ | |/| | | | | | | | | |
| * | | | | | | | | | | Support for nova client list hosts with specific zoneivan-zhu2012-10-252-11/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements one workitem of blueprint apis-for-nova-manage This add an optional QUEY_STRING(zone=xxx) in /v1.1/{tenant_id}/os-hosts. So we can also send /v1.1/{tenant_id}/os-hosts?zone=xxx now. It will only return the hosts that availability_zone=xxx. Change-Id: I89f2c0bf50f8ea1ef9f06da664f5c4b24098e5df
* | | | | | | | | | | | Merge "More robust checking for empty requested_networks"Jenkins2012-10-301-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | More robust checking for empty requested_networksAndrew Laski2012-10-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's possible that requested_networks will be empty but not None. Especially since novaclient defaults to an empty list. This avoids attempting an unnecessary validation. Change-Id: I131dbb3f6cc6167e150d4a46386e6b0b51035579
* | | | | | | | | | | | | Merge "xenapi: refactor: Agent class"Jenkins2012-10-302-118/+130
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / / / / |/| | | | | | | | | | | |