summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Merge "Refactor Xen Vif drivers. Fixes LP907850"Jenkins2011-12-223-36/+56
|\ \ \ | |/ / |/| |
| * | Refactor Xen Vif drivers. Fixes LP907850Jason Kölker2011-12-223-36/+56
| | | | | | | | | | | | | | | | | | | | | | | | * Allows Vif subclasses to have an __init__ for setting up state * Allows Vif subclasses to pass arbitrary kwargs to plug/unplug should they require more than what the interface specifies Change-Id: I816aa1537d21005d4332af3477d9fd12f177f326
* | | Merge "Bug#898257 support handling images with libguestfs"Jenkins2011-12-223-2/+99
|\ \ \
| * | | Bug#898257 support handling images with libguestfsPádraig Brady2011-12-223-2/+99
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | http://libguestfs.org/ provides both utilities and libraries to manipulate image files containing various operating systems. It supports various image file formats and so will expand the formats and guest types supported by openstack. It does have extra overhead in that it starts a VM to access the image. This has both advantages and disadvantages. Also qemu-nbd is not supported on some systems like RHEL 6. * nova/virt/disk/api.py (img_handlers): Add guestfs to the default list of access methods to try, to act as a fallback. * nova/virt/disk/guestfs.py: A new plugin class to provide support for libguestfs mounting. Note we use the guestmount utility, as a non root user, so the user will need the ability to use fusermount, which is often provided by being a member of the 'fuser' group. In future we might use the guestfs python module to give greater granularity of control over the image. Change-Id: I2e22c9d149fff7a73cd8cebaa280d68d3fb9096c
* | | | Merge "Bug#898257 abstract out disk image access methods"Jenkins2011-12-2212-153/+467
|\| | |
| * | | Bug#898257 abstract out disk image access methodsPádraig Brady2011-12-2212-153/+467
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rather than providing two mutually exlusive image access methods (loop and qemu-nbd), try each in turn. This is to prepare for a follow up patch which will add libguestfs as a method to try. * nova/virt/mount.py: A new Mount class to abstract the devce allocation, partition mapping and file sys mounting, for each access type. * nova/virt/disk/loop.py: A specialization of the base Mount class to provide loop back mounting support. * nova/virt/disk/nbd.py: A specialization of the base Mount class to provide qemu-nbd mounting support. * nova/virt/disk/api.py: A new file containing the nova.virt.disk module interface. (img_handlers): A new list of access methods to try, with the order being honored. (_DiskImage): An internal helper class that uses the plugin classes above, to provide the operations available on a disk image file. When mounting, iterate over each access method until one succeeds. If a hint is provided about a CoW format image, the list of methods to try will be reduced accordingly. Note expected errors are no longer raised as exceptions during mounting. Instead, on failure to mount an image, errors are collated and raised. Interveining errors are logged in debug mode for successful mounts. * nova/virt/libvirt/connection.py: Adjust the function parameter names to be more general, rather than referencing specific implementations like 'nbd' and 'tune2fs'. Simplify the destroy_container() by storing and passing back a reference to the _DiskImage object, which has the necessary state to unmount. * nova/utils.py (trycmd): A helper function to both deal with, commands that issue ignorable warnings to stderr, and commands that EXIT_SUCCESS while issuing errors to stderr. nova/virt/xenapi/vm_utils.py: Adjust for the moved virt.disk package Change-Id: If3a4b1c8f4e2f2e7300a21071340dcc839cb36d7
* | | | Merge "Move 'actions' subresource into extension"Jenkins2011-12-226-57/+128
|\ \ \ \
| * | | | Move 'actions' subresource into extensionBrian Waldon2011-12-226-57/+128
| |/ / / | | | | | | | | | | | | | | | | | | | | Related to blueprint separate-nova-adminapi Change-Id: I6929c5c467f339bc91ac8185d6217a55a5369a58
* | | | Merge "Remove unneeded broken test case."Jenkins2011-12-221-37/+0
|\ \ \ \ | |_|_|/ |/| | |
| * | | Remove unneeded broken test case.Mark Washenberger2011-12-211-37/+0
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | This test case appears to be broken and unneeded. It is trying to test nova.tests.rpc.fake which does not exist. It appears to me that what this is supposed to be doing is already being covered by nova/tests/rpc/test_fake.py. Change-Id: I49259841e988b50784f637dac016477f246674f5
* | | Merge "Move createBackup server action into extension"Jenkins2011-12-224-201/+202
|\ \ \
| * | | Move createBackup server action into extensionBrian Waldon2011-12-224-201/+202
| | |/ | |/| | | | | | | | | | | | | Related to blueprint separate-nova-adminapi Change-Id: Ie962202cd1e7f6c648e397ea311e4ec59b56df1b
* | | Merge "Make os-server-diagnostics extension admin-only"Jenkins2011-12-221-1/+1
|\ \ \
| * | | Make os-server-diagnostics extension admin-onlyBrian Waldon2011-12-221-1/+1
| | |/ | |/| | | | | | | | | | | | | | | | Related to blueprint separate-nova-adminapi. This was overlooked in the initial commit that added this extension Change-Id: I4ed1cdb3b40eb54ab224d3b167e7ffcb31983f5b
* | | Use SQLAlchemy to drop foreign key in DB migrateCor Cornelisse2011-12-222-5/+12
| |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | A foreign key constraint needs to be removed in order to succeed in the DB migration. An earlier submitted fix for this uses a mysql drop statement. I think it's cleaner to use sqlalchemy to do this, as it's done this way in other version migrations as well. Fix for bug 907254. Update: IBM from 1928 called, they want their 80 CPL limit back. - on a more serious note, fixed jenkins PEP8 failure Update2: Replaced tab by a spaces Update3: Don't do Foreign Key stuff if the engine is sqlite Update4: Add myself to the Authors file Update5: With the correct mail address Change-Id: Ib021e0ddb2c80ee63888435a9e3761c053534160
* | Merge "Allow accessIPv4 and accessIPv6 on rebuild action"Jenkins2011-12-224-115/+113
|\ \ | |/ |/|
| * Allow accessIPv4 and accessIPv6 on rebuild actionBrian Waldon2011-12-214-115/+113
| | | | | | | | | | | | Fixes bug 893243 Change-Id: If320ead6f25613725faf6c01c4a87b496ef88961
* | Fix spelling typos in commentsJoe Gordon2011-12-211-2/+2
|/ | | | Change-Id: I7d8011f2c44ee597f323c8e1de090d33bf63577b
* Move 'diagnostics' subresource to admin extensionBrian Waldon2011-12-216-10/+109
| | | | | | Related to blueprint separate-nova-adminapi Change-Id: Ibbb2e4d638c16e3209ca4b3d71892a5d7e874ca4
* Merge "Cleaning up imports in compute and virt"Jenkins2011-12-2116-43/+19
|\
| * Cleaning up imports in compute and virtBrian Waldon2011-12-2116-43/+19
| | | | | | | | Change-Id: Ib3ea4014a37b852e713f8a683238a7c158272229
* | Merge "Make reroute_compute use functools.wraps. Fixes LP bug #906945."Jenkins2011-12-211-0/+1
|\ \ | |/ |/|
| * Make reroute_compute use functools.wraps. Fixes LP bug #906945.Dan Prince2011-12-211-0/+1
| | | | | | | | Change-Id: I043b6543e79f9cc6c7a32c9952113ee009b31377
* | Merge "Cleaning up imports in nova.api"Jenkins2011-12-2119-40/+24
|\ \
| * | Cleaning up imports in nova.apiBrian Waldon2011-12-2119-40/+24
| |/ | | | | | | Change-Id: I2779c1fbff7344370ce17c6562086e29329b6e65
* | Merge "Removing extra code from servers controller"Jenkins2011-12-211-31/+8
|\ \ | |/ |/|
| * Removing extra code from servers controllerBrian Waldon2011-12-211-31/+8
| | | | | | | | Change-Id: Ifbc494be980cc6ef1df83b8a27bd45f62388e4b4
* | Merge "Generate instance faults when instance errors"Jenkins2011-12-212-37/+185
|\ \ | |/ |/|
| * Generate instance faults when instance errorsAlex Meade2011-12-212-37/+185
| | | | | | | | | | | | | | Currently adds all exceptions having to do with a specific instance that are generated in compute manager to the instance faults table. Change-Id: Idf92348c6896b0bf0f26d3a0dbd4a48a4c1d4329
* | Merge "Makes disassociate by timeout work with multi-host"Jenkins2011-12-211-8/+17
|\ \ | |/ |/|
| * Makes disassociate by timeout work with multi-hostVishvananda Ishaya2011-12-161-8/+17
| | | | | | | | | | | | | | | | * fixes bug 898485 * updates timeout query to key off of instance host as well * removes unused status=1 in related query Change-Id: Ia11b3d4d3db930eb31ccc2f74635971ba6edc32c
* | Clarify NoValidHost messagesJoe Gordon2011-12-201-3/+3
| | | | | | | | | | | | "All hosts have too many cores" isn't that clear Change-Id: I11102fb5c86966f2b7888d9cba29aed040dd8f4e
* | Fix one last bug in os-console-output extensionBrian Waldon2011-12-202-4/+10
| | | | | | | | | | | | Actually fixes bug 907083 Change-Id: Ia57d316db0c79d7e78ef3225e77cd95589ac68de
* | Fix os-console-output extension integrationBrian Waldon2011-12-202-12/+46
| | | | | | | | | | | | Fixes bug 907083 Change-Id: Ia57d316db0c79d7e78ef3225e77cd95589ac68df
* | Merge "Improve the minidns tests to handle zone matching."Jenkins2011-12-202-14/+23
|\ \
| * | Improve the minidns tests to handle zone matching.Andrew Bogott2011-12-202-14/+23
| | | | | | | | | | | | | | | | | | For blueprint public-and-private-dns. Change-Id: I88e906e2ce031b6cf60ff0d16a20aadfdd515723
* | | Set Location header in server create and rebuild actionsJohannes Erdfelt2011-12-202-0/+51
| | | | | | | | | | | | | | | | | | | | | | | | The API documentation states that the Location header should be filled with the URL of the server. This change implemnts this for both the create and rebuild actions. Change-Id: Ic0372aeaea58c9288065e2cfc222ca9257d35c87
* | | Consistently use REBUILDING vm_stateJohannes Erdfelt2011-12-203-5/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is both a vm_state and task_state called REBUILDING, which was used inconsistently. This resulted in a rebuild action returning "ACTIVE" for the status instead of "REBUILD" like the API documentation states. This change drops the task_state in favor of making all of the code use vm_state Change-Id: I56b4b6fd4d02b75c6270fc91cdec1902e7794cf1
* | | Merge "Make UUID format checking more correct."Jenkins2011-12-202-2/+11
|\ \ \
| * | | Make UUID format checking more correct.Russell Bryant2011-12-202-2/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch uses the same logic from glance to verify if a UUID is properly formed. It just tries to build a uuid.UUID from the string. A test case has been added which previously would have failed, but now passes. A second test case has been added that verifies that the result of the gen_uuid() function is considered valid, as well. Change-Id: I33fe37b23ef03eda0f0e9d69afb601ca260aabde
* | | | Merge "Set min_ram and min_disk on snapshot"Jenkins2011-12-202-1/+155
|\ \ \ \ | |_|/ / |/| | |
| * | | Set min_ram and min_disk on snapshotAlex Meade2011-12-202-1/+155
| |/ / | | | | | | | | | | | | | | | Fixes bug 884269 Change-Id: Id92727ea09bf73ab8d6201f0aece8018de4b674b
* | | Merge "Properly passes arg to run_iscsiadm to fix logout"Jenkins2011-12-203-13/+32
|\ \ \
| * | | Properly passes arg to run_iscsiadm to fix logoutVishvananda Ishaya2011-12-163-13/+32
| | |/ | |/| | | | | | | | | | | | | | | | | | | * fixes 905471 * includes test to verify expected calls * elevates context as expected for volume_api.detach Change-Id: I0c2a5822d45df6f82f103f49ce8fcfe4a38a0049
* | | Remove unused FLAGS.block_sizeJulien Danjou2011-12-201-2/+0
| | | | | | | | | | | | | | | Change-Id: I41ba92f7443dba94e9e749c916833325f46fb750 Signed-off-by: Julien Danjou <julien.danjou@enovance.com>
* | | Merge "Creating mechanism that loads Admin API extensions"Jenkins2011-12-2010-77/+69
|\ \ \ | |_|/ |/| |
| * | Creating mechanism that loads Admin API extensionsBrian Waldon2011-12-1910-77/+69
| | | | | | | | | | | | | | | | | | Related to blueprint separate-nova-adminapi Change-Id: I2491267e370f053b5ae4e844aba1d82c3b0f0d86
* | | Merge "Add a console output action to servers"Jenkins2011-12-208-8/+164
|\ \ \
| * | | Add a console output action to serversJake Dahn2011-12-198-8/+164
| |/ / | | | | | | | | | | | | | | | | | | Relates to blueprint osapi-console-log and bug 876809. Adds equivalent of euca-get-console-output to openstack api as an extension. Change-Id: Ia71361ebbec820616a3007e216b0b9ff98d43541
* | | Merge "Imported Translations from Launchpad"OpenStack Jenkins2011-12-192-181/+2983
|\ \ \ | |/ / |/| |