summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | Merge "Split xenapi agent code out to nova.virt.xenapi.agent"Jenkins2012-07-113-200/+274
|\ \ \ \ \ \ \
| * | | | | | | Split xenapi agent code out to nova.virt.xenapi.agentJohannes Erdfelt2012-07-103-200/+274
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This refactoring makes the split between driver methods and internal methods clearer. It also makes all agent methods private instead of making some private, some public and some both. Change-Id: I6c6b405942e47a85816e10224d4e6d986a8b06dc
* | | | | | | | Merge "Use compute_api.get_all in affinity filters."Jenkins2012-07-111-4/+9
|\ \ \ \ \ \ \ \
| * | | | | | | | Use compute_api.get_all in affinity filters.Dan Prince2012-07-111-4/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updates the affinity filters so they make a single compute API call to lookup instance host information rather than single lookups for each UUID. This resolves a potential performance issue which can cause a scheduler to hang while processing requests which contain large numbers of UUID's in the scheduler_hints. Fixes LP Bug #1017795. Change-Id: I30f434faf109058573ee41c4a6abce2e48939e8d
* | | | | | | | | Remove unused find_data_files function in setup.pyMonty Taylor2012-07-111-13/+0
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | find_data_files was copied in a while back to work around setuptools deficiencies. It has since made its way into distribute, and we are using it from there - thus do not need a copy in our tree anymore. Change-Id: I8bc76a201c325571601ac8da292ebd5690668065
* | | | | | | | Merge "fix metadata file injection with xen"Jenkins2012-07-111-1/+1
|\ \ \ \ \ \ \ \
| * | | | | | | | fix metadata file injection with xenPádraig Brady2012-07-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The metadata parameter was misordered in commit 1463839f Fixes Bug: 1022036 Change-Id: Ica0e2f51b6852c8572bdc3c995cc8c2588b9a25c
* | | | | | | | | Merge "Removed a bunch of cruft files."Jenkins2012-07-119-156/+0
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Removed a bunch of cruft files.Monty Taylor2012-07-079-156/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ohai. We dont't uses any of theses anymores. Let's just make buh-bye. Change-Id: Ib81c92ab9196552150d81d158befb0747966965d
* | | | | | | | | | Merge "Fixes api fails to unpack metadata using cinder"Jenkins2012-07-111-3/+3
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / |/| | | | | | | | |
| * | | | | | | | | Fixes api fails to unpack metadata using cinderJohn Griffith2012-07-101-3/+3
| | |_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix for bug 1023027 Change-Id: I2d19f461f3734518717b8b54d9d3428b75a2a44d
* | | | | | | | | Merge "Avoid lazy-loading errors on instance_type"Jenkins2012-07-111-3/+3
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Avoid lazy-loading errors on instance_typeVishvananda Ishaya2012-07-091-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When instances are launched with a source group, the instance_type is not lazy_loaded in properly. The issue is intermittent because if another query has already loaded the instance with a joined instance_type or the instance has not yet been expunged from the session, then the load succeeds. Therefore this failure only appears under heavy load. Adding an explicit joindload fixes the issue. Change-Id: I450c598c31c416c7be341cbb8cd14e33c6bb2aec
* | | | | | | | | | Merge "Remove deprecated auth code"Jenkins2012-07-1122-3193/+0
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Remove deprecated auth codeBrian Waldon2012-07-0922-3193/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Remove nova.auth.manager -- AuthManager, User, Role, Project, etc. * Remove nova.auth.dbdriver * Remove nova.auth.ldapdriver * Remove nova.auth.signer * Remove arbitrary scripts and schemas in nova/auth/ * Remove nova/auth/novarc.template * Remove or update affected tests * Related to bp remove-deprecated-auth Change-Id: Ide0fefd0ddf79ae1b3bb74cb242c2893575839e7
* | | | | | | | | | | Merge "Update common setup code to latest."Jenkins2012-07-108-133/+227
|\ \ \ \ \ \ \ \ \ \ \ | | |_|_|/ / / / / / / | |/| | | | | | | | |
| * | | | | | | | | | Update common setup code to latest.Monty Taylor2012-07-078-133/+227
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This gets us up to date with common/setup.py and replaces custom nova autodoc generation with the port of that code found in common. Change-Id: I2a1c5d2c0fdcf40dbea50cc123b537adb068cdc2
* | | | | | | | | | | Merge "ensure libguestfs has completed before proceeding"Jenkins2012-07-101-1/+12
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | ensure libguestfs has completed before proceedingPádraig Brady2012-07-101-1/+12
| | |_|_|_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a more correct fix than the previous sync(1) which probably only changed timings, masking the real issue. The FUSE issue is referenced here: http://code.google.com/p/s3ql/issues/detail?id=159 which in turn references this thread on the subject from 2006: http://thread.gmane.org/gmane.comp.file-systems.fuse.devel/3903 And another case from 2008: http://permalink.gmane.org/gmane.comp.file-systems.fuse.devel/6502 Fixes bug: 1013689 Change-Id: I36fd11d5f01562f65a6b6f07e759ea066490b067
* | | | | | | | | | | Merge "flags documentation to deprecate connection_type"Jenkins2012-07-102-3/+6
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | flags documentation to deprecate connection_typeSean Dague2012-07-102-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | connection_type in nova.conf is deprecated for Folsom, provide additional documentation in the flags to indicate this, and the new compute_driver options Part of bp:virt-driver-cleanup Change has DocImpact for Folsom Change-Id: Iaff658771b1e992304454368e66ee53f9a0480ff
* | | | | | | | | | | | Merge "Remove deprecated auth docs"Jenkins2012-07-102-277/+0
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Remove deprecated auth docsBrian Waldon2012-07-102-277/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Related to bp remove-deprecated-auth Change-Id: I2637188e75e1881e6cc5af09d86eaf45581b3e9e
* | | | | | | | | | | | Merge "Switch to common notifiers."Jenkins2012-07-1028-482/+180
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Switch to common notifiers.Andrew Bogott2012-07-0628-482/+180
| | |_|_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Added notifier files to openstack-common.conf, merged from common. - Import notifiers from common rather than nova. - Removed nova-specific notifier code. - Update some driver paths to refer to the new location within nova. Should be a no-op, as the common notification code is freshly copied from Nova. Change-Id: Icdf892bc3826b683fc74f22ead00939beda2859f
* | | | | | | | | | | | Merge "Raise Failure exception when setting duplicate other_config key"Jenkins2012-07-101-0/+3
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Raise Failure exception when setting duplicate other_config keyJohannes Erdfelt2012-07-101-0/+3
| | |_|_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When https://review.openstack.org/7971 got merged (and fixed in https://review.openstack.org/8504), it introduced a bug where a key in the xenapi other_config would get set twice. The real xenapi would raise a Failure exception in that case. So make sure the fake module also does that to ensure similar bugs will get caught in the future. Change-Id: Ie0151ffaf45dade930c3e8ff1279bfeab537bab2
* | | | | | | | | | | Merge "Set the default CPU mode to 'host-model' for Libvirt KVM/QEMU guests"Jenkins2012-07-102-3/+57
|\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | Set the default CPU mode to 'host-model' for Libvirt KVM/QEMU guestsDaniel P. Berrange2012-07-062-3/+57
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Historically Nova has not set any CPU model for guests started with the libvirt driver. This means they are all using the per-hyervisor default settings for CPU model. With KVM/QEMU guests the model was traditionally a very conserative choice which exposed minimal features. This is significantly limits the performance of applications. Further the model has changed over time, so the exact default model is unpredictable. Switching Nova to use the host CPU model by default should improve the out of the box performance & give a known setup. This does not impact migration compatibility, since the migration code is already doing comparison checks against the source and destination host CPU model, regardless of the actual model used in the guest. In the future the migration code should be tweaked so that it actually compares the current guest CPU model, against the target host CPU model, which would potentially broaden the migration compatibility pool. With this patch there is a new libvirt_cpu_mode="none" which can be used to explicitly prevent any CPU model setting in the instance XML. The default value None, will now default to "none" for LXC/UML/etc, and "host-model" for QEMU/KVM Fixes: bug #1003373 Implements: blueprint libvirt-xml-cpu-model Change-Id: I5b96e4532b6a960e1608dd6e19ae9d194379fb6a Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
* | | | | | | | | | | Merge "refactor baremetal/proxy => baremetal/driver"Jenkins2012-07-104-7/+10
|\ \ \ \ \ \ \ \ \ \ \ | | |_|/ / / / / / / / | |/| | | | | | | | |
| * | | | | | | | | | refactor baremetal/proxy => baremetal/driverSean Dague2012-07-104-7/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Part of bp:virt-driver-cleanup Make the baremetal driver consistent in naming as a driver instead of a proxy / connection. Change-Id: I75d7d90bd8139842b588f1fafb1267511f29a1fb
* | | | | | | | | | | Merge "Properly track VBD and VDI connections in xenapi fake"Jenkins2012-07-102-12/+42
|\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | Properly track VBD and VDI connections in xenapi fakeJohannes Erdfelt2012-07-052-12/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The fake xenapi module correctly destroys VBDs when destroying a VM but it won't destroy VBDs when destroying a VDI. If an exception occurs during a spawn when running test_xenapi, then another exception can occur which obscures the original (and likely more important) exception. This can make troubleshooting test suite failures more difficult. Change-Id: I19aad7f39b0707893a04784faef90815f7b0effb
* | | | | | | | | | | refactor xenapi/connection => xenapi/driverSean Dague2012-07-108-8/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Part of bp:virt-driver-cleanup Make the xenapi driver consistent in naming as a driver instead of a connection. Change-Id: Ibb1485515ff073dd0265969fdcfbfc2665b93fb7
* | | | | | | | | | | refactor vmwareapi_conn => vmwareapi/driverSean Dague2012-07-105-6/+8
| |_|_|/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Part of bp:virt-driver-cleanup Make the vmwareapi driver consistent in naming as a driver instead of a connection. Get all the vmware driver parts into the same directory Change-Id: I7c7e7c391086cf2fc32e8cb85d557005d417042f
* | | | | | | | | | Merge "Fallback to fakelibvirt in test_libvirt.py test suite"Jenkins2012-07-102-49/+41
|\ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / | |/| | | | | | | |
| * | | | | | | | | Fallback to fakelibvirt in test_libvirt.py test suiteDaniel P. Berrange2012-07-062-49/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the native 'libvirt' module is not available on the host, currently test_libvirt.py stubs it out to ''. While this has sort-of worked for some test cases, others have had to be configured to skip. As the complexity of the libvirt driver increases, ever more tests will need to be skipped. There is, however, the fakelibvirt.py module which provides a dummy impl of the libvirt APIs for use in test_virt_drivers.py With a few enhancements this fake impl is good enough to use with test_libvirt.py too. The constructor of libvirtError is changed to accept the 'msg' as first arg to match the regular libvirt module. A few missing constants are added. The migrateToURI method is stubbed out All the previously skipped tests can now be enabled as normal. Change-Id: I15efbef8c676cc0e263a9c9bdd7b14dfb3771105 Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
* | | | | | | | | | Merge "Make pep8 test work on Mac"Jenkins2012-07-101-1/+1
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / |/| | | | | | | | |
| * | | | | | | | | Make pep8 test work on MacYun Mao2012-07-101-1/+1
| | |_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | find . -executable is not available on Mac OS X. We switch back to the old -perm syntax to find files to test. Change-Id: Iea58a249f282c98ae6cffbe8e6359f70f8b1d63f
* / | | | | | | | Remove deprecated auth-related api extensionsBrian Waldon2012-07-095-546/+0
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Delete the accounts and users extensions * Delete associated tests * Related to bp remove-deprecated-auth Change-Id: I429551dbb39ec569c6d2c229fa0c3cafd9e56f11
* | | | | | | | Merge "Boot from volume for Xen"Jenkins2012-07-106-35/+141
|\ \ \ \ \ \ \ \
| * | | | | | | | Boot from volume for XenRenuka Apte2012-07-096-35/+141
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements bp xenapi-boot-from-volume Ensure boot from volume works for XenAPI (tested using XenServer). 1. Add strip_prefix to block_device to make the command mountpoint agnostic. 2. Pass block device and delete on terminate information to driver layer. 3. Add ability to recognize and boot from the specified nova volume. Change-Id: If117087086eab809217d2b173f921bf9319a52c7
* | | | | | | | | Merge "Fix KeyError 'key_name' when KeyPairExists raised."Jenkins2012-07-101-2/+1
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Fix KeyError 'key_name' when KeyPairExists raised.Dan Prince2012-07-091-2/+1
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updates the Nova compute API to pass a key_name arg to the KeyPairExists constructor. This fixes an issue where a KeyError exception would get logged to Nova's api.log file if a KeyPairExists exception was thrown. Also removes the un-needed exception explanation argument from the KeyPairExists constructor. Fixes LP Bug #1022611 Change-Id: I83ecf3d392fef97649b298d1b732e1ff51145ae4
* | | | | | | | | Merge "Fetch kernel/ramdisk images directly"Jenkins2012-07-101-6/+5
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / |/| | | | | | | |
| * | | | | | | | Fetch kernel/ramdisk images directlyJohannes Erdfelt2012-07-091-6/+5
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Kernel and ramdisk images don't have a VDI allocated to them, so printing the UUID in _fetch_image will show None. [...] Fetched VDIs of type 'ramdisk' with UUID 'None' [...] Change-Id: I1113606a97b48518e1db61f737633140b0f5eccd
* | | | | | | | Merge "Ignore failure to delete kernel/ramdisk in xenapi driver"Jenkins2012-07-091-2/+11
|\ \ \ \ \ \ \ \
| * | | | | | | | Ignore failure to delete kernel/ramdisk in xenapi driverJohannes Erdfelt2012-07-091-2/+11
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1022681 If deleting the kernel/ramdisk fails because the files don't exist anymore, then ignore the error and continue. This will ensure that the instance will get destroyed properly even if the files were deleted outside of nova. Change-Id: I5d1f95ea2a6f552c48efbb9e92bb36767df19e34
* / / / / / / / Fix 'instance %s: snapshotting' log message.Dan Prince2012-07-091-1/+1
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update the Nova compute manage to remove the '%s:' string from the log message. Nova already logs teh instance UUID for this log message because we pass instance_uuid=instance_uuid to the logger so this extra string isn't required. Change-Id: I3fa276347d246e440917ac12d4fc5f88d90d7468
* | | / / / / Make reboot work for halted xenapi instancesChris Behrens2012-07-084-7/+69
| |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1022199 This also will catch exceptions and make sure to reset the task_state, while still generating an instance fault. Change-Id: I122a1422b8e5731bc484414736ab44e60d4c9830