summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | | | | | | | | Move get_table() from test_migrations to sqlalchemy.utilsBoris Pavlovic2013-05-132-83/+86
| | |_|_|_|_|_|_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | get_table() is very useful in different places. So it is better to move it from test_migrations to sqlalchemy.utils. blueprint db-common-migration-and-utils Change-Id: I1191b95aede8688a2b38ac4dd0f2e9108dda1559
* | | | | | | | | | | | | | | Merge "Show the cause of virt driver error"Jenkins2013-05-131-2/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / / / |/| | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Show the cause of virt driver errorStanislaw Pitucha2013-05-131-2/+2
| | |_|_|_|_|_|_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make sure that the virt driver loading errors are shown with the stacktrace attached. Otherwise the source of an import error can be completely hidden. This is important here, since virt driver can fail due to various issues, like quantum vif problem. Change-Id: Icb90d348e4a4ddef5768ca32836bd95d94981499
* | | | | | | | | | | | | | Merge "Convert to using newly imported processutils."Jenkins2013-05-1310-139/+112
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | Convert to using newly imported processutils.Michael Still2013-05-0810-139/+112
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change moves nova over to using the newly imported trycmd() and ssh_execute() implementations from oslo's processutils. Change-Id: Ied2bb1eeb0cbdc6903a4028a60c89ca9e23656b4
* | | | | | | | | | | | | | | Merge "Import new additions to oslo's processutils."Jenkins2013-05-131-0/+66
|\| | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Import new additions to oslo's processutils.Michael Still2013-05-081-0/+66
| | |_|/ / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | trycmd() and ssh_execute() have been moved into oslo. Copy the oslo version here. Conversion will occur in a later review. Change-Id: I69926af33ff824f712ae2b8e4023233afe1dff2f
* | | | | | | | | | | | | | Merge "Remove duplicate gettext.install() calls"Jenkins2013-05-135-16/+0
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / / |/| | | | | | | | | | | | |
| * | | | | | | | | | | | | Remove duplicate gettext.install() callsMark McLoughlin2013-05-085-16/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When commit 799a925 moved us to console scripts entry points, we moved the gettext.install() call into nova/cmd/__init__.py but forgot to remove it from some of the commands. Since that move, we've switched to using gettextutils.install() which supports a $NOVA_LOCALEDIR environment variable ... so calling install() from these commands will actually break that support. Change-Id: I1f959704551b35e95c5470a7aef900312db2e223
* | | | | | | | | | | | | | Merge "API extensions framework for v3 API"Jenkins2013-05-1311-0/+471
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | API extensions framework for v3 APIChris Yeoh2013-05-1311-0/+471
| | |_|/ / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is the initial patch for the new extension framework to be used by the Nova v3 API. It will only be used by v3 API extensions and not v2 API extensions. v3 API extensions will only use this plugin framework and will not be compatible with the old one. - Sets up a /v3 url prefix - Looks in an entry point namespace of nova.api.extensions - The fixed_ips extensions is ported to /v3 as an example of a resource extensions. Required changes are very minor. - All extensions must derive from the V3APIExtensionBase class - Drops updated field from extensions, replaced with version field - Ports tests for fixed_ips extension - None of the core has been ported in this patch, though the example extension works without it. The intent is to port the core code over as plugins as well. There will still be a conceptual core however I don't think we need a separate directory for core. This is the first of a series of patches to add support for the new extension framework. Future direction including support for controller extensions, removal of extension code in core code etc can be seen here: https://github.com/cyeoh/nova/tree/v3_api_extension_framework Partially implements blueprint v3-api-extension-framework Change-Id: I88aa6353ad1d74cac51abbb6aac7274b1567485a
* | | | | | | | | | | | | | Merge "Add cpuset attr to vcpu conf in libvirt xml"Jenkins2013-05-134-8/+186
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / |/| | | | | | | | | | | | |
| * | | | | | | | | | | | | Add cpuset attr to vcpu conf in libvirt xmlWangpan2013-05-074-8/+186
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently the instances can use all of the pcpu of compute node, the host may become slow when vcpus of instances are busy, so we need to pin vcpus to the specific pcpus of host instead of all pcpus. Also added a cache of total vcpus num in libvirt driver here. DocImpact: please refer to the bp page. Implements blueprint: instance-vcpu-pin Change-Id: I6b6e805c15b249cc5e7517e26619d79bb0183de8
* | | | | | | | | | | | | | Merge "Add sqlalchemy migration utils.create_shadow_table method"Jenkins2013-05-112-13/+142
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | Add sqlalchemy migration utils.create_shadow_table methodBoris Pavlovic2013-05-092-13/+142
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This method allows to create shadow table by existing table. Two use cases: a) create shadow table by table name b) create shadow table from table instance blueprint db-improve-archiving Change-Id: Ib3a3797782b89a9a89baa60738edcabde7fe1299
* | | | | | | | | | | | | | | Merge "Add sqlalchemy migration utils.check_shadow_table method"Jenkins2013-05-114-10/+129
|\| | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Add sqlalchemy migration utils.check_shadow_table methodBoris Pavlovic2013-05-094-10/+129
| | |_|_|_|/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This method check that our table with name `xxx`: 1) Has shadow table `shadow_xxx` 2) All columns in `xxx` are also in `shadow_xxx` (with same type) 3) There is no extra columns in `shadow_xxx` Will be used to check that all shadow tables are up-to-date. blueprint db-improve-archiving Change-Id: I0b366afe1b7d8703ab9c49a68977202b82d844fc
* | | | | | | | | | | | | | Merge "Update rootwrap with code from oslo."Jenkins2013-05-113-6/+8
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | Update rootwrap with code from oslo.Monty Taylor2013-05-093-6/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I30f1b8773840c098f1e38d09bf13f0863035289e
* | | | | | | | | | | | | | | Merge "Imported Translations from Transifex"OpenStack Jenkins2013-05-1023-2091/+2393
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | Imported Translations from TransifexOpenStack Jenkins2013-05-1023-2091/+2393
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ic7342f9da05f435b7beeae516dccfb102741e62c
* | | | | | | | | | | | | | | | Merge "Cleanup for test_create_server_with_deleted_image"Jenkins2013-05-101-0/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | Cleanup for test_create_server_with_deleted_imageChris Yeoh2013-05-101-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds a cleanup for test_create_server_with_deleted_image so that the test resets the test image to the active state after the test finishes. Otherwise other tests which use the same image after this test can fail. Change-Id: Iedd039af54d2233d1afbc7fe2d21d1b3594c47b1
* | | | | | | | | | | | | | | | | Merge "Check cached SSH connection in PowerVM driver"Jenkins2013-05-104-18/+98
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | Check cached SSH connection in PowerVM driverMatt Riedemann2013-05-094-18/+98
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The PowerVM operator code caches an SSH connection to the hypervisor which can become invalid if the connection to the hypervisor is not terminated cleanly, e.g. the hypervisor is rebooted while the compute node is connected to it. This code checks an existing SSH connection object to see if it's transport is still alive and if not, re-establishes the connection before attempting to run SSH commands on the hypervisor. Also added some basic unit tests to cover the new check_connection function in nova.virt.powervm.common. Fixes bug 1177104 Change-Id: I08079cf0d9e60e1d8902d32d684d979b06f7f287
* | | | | | | | | | | | | | | | | | Merge "Check QCOW2 image size during root disk creation"Jenkins2013-05-102-0/+29
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | Check QCOW2 image size during root disk creationChet Burgess2013-05-102-0/+29
| | |/ / / / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | glance can only tell us the size of the file, not the virtual size of the QCOW2. As such we need to check the virtual size of the image once its cached and ensure it's <= to the flavor's root disk size. Change-Id: I833467284126557eb598b8350a84e10c06292fa9 Fixes: bug 1177830
* | | | | | | | | | | | | | | | | | Merge "Allow different paths for deploy-helper helpers."Jenkins2013-05-101-5/+5
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | Allow different paths for deploy-helper helpers.Robert Collins2013-05-111-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Different distros (or even distro releases) put iscsiadm in potentially arbitrary paths. It is better to use rootwraps' own search than to hardcode the path. Fixes bug 1178612 Change-Id: I690788b666a84e599504690d0e2d087eaa66a4c8
* | | | | | | | | | | | | | | | | | | Merge "Show exception details for failed deploys."Jenkins2013-05-101-1/+1
|\| | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | Show exception details for failed deploys.Robert Collins2013-05-111-1/+1
| | |_|_|_|_|_|_|/ / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Without some detail errors are inscrutable and impossible to debug. Fixes bug 1178608 Change-Id: I0ed6164e8689c63bd194924d3f7cb50b49feb921
* | | | | | | | | | | | | | | | | | Merge "Add unitests for VMware vif, and fix code logical error."Jenkins2013-05-103-18/+109
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | Add unitests for VMware vif, and fix code logical error.Yaguang Tang2013-05-083-18/+109
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix code logical error that cause variable referenced before assignment. Add unittests for vmwareapi vif. Fix bug #1177635 Change-Id: I6656e8b77417061eca2a56d50820f5385c0ae987
* | | | | | | | | | | | | | | | | | | Merge "Allow a floating IP to be associated to a specific fixed IP"Jenkins2013-05-1031-14/+311
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | Allow a floating IP to be associated to a specific fixed IPPhil Day2013-05-1031-14/+311
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current floating IP API extension only accepts the instance ID and the floating address to be assigned. Where the instance is connected to more than one network the behaviour is to associated the floating IP with the first fixed IP of the instances (and to log a warning) This change introduces a new extension which wehn loaded adds a fixed IP address as an optional parameter, allowing the floating IP to be associated with a specific fixed IP. Without this extension, or without the optional parameters, the API behaviour is unchanged. If specified the fixed IP must be associated with the instance. DocImpact Implements blueprint multi-nic-floating-ip-assignment Change-Id: I9241137ad794cdf7f452ed84e9445f0e11fdd44e
* | | | | | | | | | | | | | | | | | | | Merge "Make resize/migrated shared storage aware"Jenkins2013-05-102-6/+70
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | Make resize/migrated shared storage awareRafi Khardalian2013-05-092-6/+70
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1177247 Added some logic to check for whether or not we are on a shared filesystem and set shared_storage accordingly. We perform similar checks in other areas of the code, typically through RPC calls. However, all the resize/migrate code is slated to be refactored for Hava, so the idea was to keep this patch as minimally intrusive as possible. When shared_storage is true, we pass that on to the cleanup call so that it no longer executes an rm via SSH, which was ultimately destroying the original instance directory. Change-Id: Ie9decedd373c000211c171df64e1e96fe78e5081
* | | | | | | | | | | | | | | | | | | | Merge "Remove invalid block_device_mapping volume_size of ''"Jenkins2013-05-102-0/+30
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | Remove invalid block_device_mapping volume_size of ''David Ripton2013-05-082-0/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #1171190 Parsing block_device_mapping from strings sometimes results in volume_size being set to '' or u''. This causes a problem when inserting into PostgreSQL, which insists that integer columns actually contain integers. Remove invalid device_size before inserting or updating block_device_mapping database records. Change-Id: If9132a96604b5b0e3a6df96624e565e016d5c9a3
* | | | | | | | | | | | | | | | | | | | | fix non reporting of failures with floating IP assignmentPádraig Brady2013-05-101-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It seems that commit essex-1-1003-g9728ae5 inadvertently removed a 'raise', thus hiding all processutils.ProcessExecutionError exceptions apart from the one indicating "Interface not found". Fixes bug: 1178658 Change-Id: I2bcf8c3a19ebebc81a38ce6bf7b54065d62834c0
* | | | | | | | | | | | | | | | | | | | | Merge "Adds instance root disk size checks during resize"Jenkins2013-05-103-43/+90
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | | Adds instance root disk size checks during resizeAlessandro Pilotti2013-05-083-43/+90
| | |_|_|_|_|_|_|_|_|/ / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug: #1163844 The Hyper-V driver does not support disk resizes to a smaller size. This patch verifies the disk size compatibility before the migration starts in order to avoid data losses. Change-Id: Ie99bf8779d583e97b911c9a136cee1bca9a1ecdc
* | | | | | | | | | | | | | | | | | | | | Merge "Enable live block migration when using iSCSI volumes"Jenkins2013-05-0910-78/+338
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | | Enable live block migration when using iSCSI volumesBob Ball2013-05-0710-78/+338
| | |_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix for bug 1160323. DocImpact Depends on a version of XAPI supporting relax-xsm-sr-check. Currently no release versions support this, so the change at https://github.com/xen-org/xen-api/pull/1116 would need to be applied to the source to enable this. XenServer 6.2 is expected to support this mode, and possibly some future hotfixes for XenServer 6.1. It also depends on a configuration change which must be documented and added to devstack: * Set "relax-xsm-sr-check = true" in /etc/xapi.conf This commit makes the following changes: * Attach the SR on the destination host prior to migrate * Returns the SR-ref from the pre_migrate call (API change) * Populates the XS VDI map with the sr-ref on the destination host * Removes the connection to the SR from the source host post-migrate * Added plugin to determine if iSCSI block migration is enabled Associated tempest test at https://review.openstack.org/#/c/26478/ Change-Id: I917d9cf094190d636f4b9e14f6c8e728ff85af0e Fixes: bug 1160323
* | | | | | | | | | | | | | | | | | | | | Merge "Fix zookeeper import and tests"Jenkins2013-05-092-4/+4
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | | Fix zookeeper import and testsAndrew Laski2013-05-082-4/+4
| | |_|_|_|_|_|_|_|_|_|/ / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Import zookeeper.membership rather than zookeeper.membersip. Also fixed some issues with setting up the tests for the zookeeper servicegroup driver. Config options were not being set before initializing the driver leading to failures. There is no added test for this because the bug is indistinguishable from not having the zookeeper python modules installed, which leads to skipping these servicegroup tests. Bug 1177776 Change-Id: Idd6dca2e03169399b930cc1fc1a85636497cb0b5
* | | | | | | | | | | | | | | | | | | | | Merge "Fixes KeyError bug with network api associate"Jenkins2013-05-092-2/+16
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | | Fixes KeyError bug with network api associateChris Yeoh2013-05-082-2/+16
| | |_|_|_|_|_|_|_|_|/ / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug in network api associate function where it would always raise a KeyError when associating or disassociating a project. Fixes bug 1171284 Change-Id: Iae0e57e0a961b8a3377b38dad72094188755a3e8
* | | | | | | | | | | | | | | | | | | | | Merge "Record smoketest dependency on gFlags"Jenkins2013-05-091-0/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|/ / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | Record smoketest dependency on gFlagsRick Harris2013-05-091-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The smoketests currently have a dependency on gFlags. Long term, this should be converted to use openstack.common.cfg; but in the near term, we should record this existing dependency. Fixes bug 1174953 Change-Id: I465abc6a4a2e8f7ebd1fe70ad239c760960a82ed