summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | Remove image service show_by_name methodBrian Waldon2012-06-278-43/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This method isn't worth keeping around. It's used in one place and consists of a try/except block. We also shouldn't encourage clients of GlanceImageService to depend on finding a single unique image by a non-unique attribute. Change-Id: I02347adef7bc7ac70407226ea150000e55a798bc
* | | | | | | | Merge "Cleanup of image service code"Jenkins2012-06-2835-187/+177
|\| | | | | | | | |_|_|/ / / / |/| | | | | |
| * | | | | | Cleanup of image service codeBrian Waldon2012-06-2735-187/+177
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Remove image_service flag * Move nova.image.fake to nova.tests.image.fake * Move nova.image.get_default_image_service to nova.image.glance.get_default_image_service * Move nova.image.get_image_service to nova.image.glance.get_remote_image_service * Related to bp integrate-python-glanceclient Change-Id: Iea6db7898328a9060fb88586e042efbc0a4351fc
* | | | | | Merge "Swap VDI now uses correct name label."Jenkins2012-06-271-1/+1
|\ \ \ \ \ \
| * | | | | | Swap VDI now uses correct name label.Rick Harris2012-06-271-1/+1
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1018596 Change-Id: I788a762e3d3362c04c7d6986808d13f77d9c5a5e
* | | | | | Merge "Switch to using new config parsing for vm_vdi_cleaner.py"Jenkins2012-06-271-74/+45
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Switch to using new config parsing for vm_vdi_cleaner.pyJohannes Erdfelt2012-06-271-74/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1018166 Also uses CONF.verbose instead of passing verbose flag everywhere Change-Id: Ifee832dd087cae8c36b7b7a4f08b2e22ef7f06a3
* | | | | | Merge "Add missing netaddr import"Jenkins2012-06-271-0/+1
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Add missing netaddr importMark Washenberger2012-06-271-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It looks like 345439f8 brought in netaddr dependencies without importing the library itself. Change-Id: Iabd58dd540e1d0c83482cb60538de7494099b34d
* | | | | | Merge "Avoid casts in network manager to prevent races"Jenkins2012-06-271-2/+2
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Avoid casts in network manager to prevent racesVishvananda Ishaya2012-06-261-2/+2
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | * Part of fix for bug 1017418 Change-Id: I3251f02fc5fcb8305f12b6b8a9644232b9146db2
* | | | | Merge "Make nova list/show behave nicely on instance_type deletion"Jenkins2012-06-272-1/+19
|\ \ \ \ \
| * | | | | Make nova list/show behave nicely on instance_type deletionChris Behrens2012-06-272-1/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | One shouldn't remove an instance_type from the DB outright... but we should make the code behave nicely in the rare case. Fixes bug 1018500 Change-Id: If742070d7982889a56709ae058cd106e3a63114b
* | | | | | Merge "refactor libvirt from connection -> driver"Jenkins2012-06-277-112/+129
|\ \ \ \ \ \
| * | | | | | refactor libvirt from connection -> driverSean Dague2012-06-277-112/+129
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | part of bp:virt-driver-cleanup part d) of the blueprint is to get rid of the connection terminology and use driver instead. This does this for the libvirt driver and associated tests. Also add LibvirtDriver import to __init__ so that nova.virt.libvirt.LibvirtDriver can be used as the compute_driver string Change-Id: I13c701c390784fa1f7809705741abb46e40973be
* | | | | | | Merge "Fix db calls for snaphsot and volume mapping"Jenkins2012-06-273-18/+23
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | |
| * | | | | | Fix db calls for snaphsot and volume mappingVishvananda Ishaya2012-06-253-18/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fix misuse of project_only * Includes failing tests * Adds missing call to db/api.py * Fixes bug 1017816 Change-Id: I993089ba031a05e7ab0ea91c279a2ac47593eda3
* | | | | | | Merge "Stop nova_ipam_lib from changing the timeout setting"Jenkins2012-06-271-11/+8
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | |
| * | | | | | Stop nova_ipam_lib from changing the timeout settingVishvananda Ishaya2012-06-261-11/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fixes incorrect logic for when timeout should be used. We want to timeout along with dhcp, and release immediately without dhcp * Also makes ipam_lib release fixed_ips after floating ips, as this is the way that the internal network managers do it. * Fixes bug 1014769 Change-Id: I687fac314264b8a89da128ed77c02d9b1ca140ff
* | | | | | | Merge "Avoid reset on hard reboot if not supported."Jenkins2012-06-271-6/+19
|\ \ \ \ \ \ \
| * | | | | | | Avoid reset on hard reboot if not supported.Dan Prince2012-06-261-6/+19
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updates the libvirt connection so that it avoids using virt_dom.reset(0) when performing a hard reboot if it is not supported. Libvirt reset requires at least libvirt 0.9.7. When reset is not supported it uses the older method of recreating the domain. This patch also sets the libvirt min version back to 0.9.6. Fixes LP Bug #1017593. Change-Id: I6c46c03fad9de11ae3cc814b06dd8e13bb66de2d
* | | | | | | Merge "Adds missing 'owner' attribute to image."Jenkins2012-06-272-3/+9
|\ \ \ \ \ \ \
| * | | | | | | Adds missing 'owner' attribute to image.MotoKen2012-06-272-3/+9
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1018216. Change-Id: I5540e0b8ab1df1904778e3edc250369b0930987e
* | | | | | | Merge "metadata: cleanup pubkey representation"Jenkins2012-06-272-5/+25
|\ \ \ \ \ \ \
| * | | | | | | metadata: cleanup pubkey representationScott Moser2012-06-222-5/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous code provided a '_name' entry in a dict to represent this, but then appended "=_name" to the key rather than populating _name with the full value to be rendered. This just changes the value in '_name' for public key to be '0=keyname' rather than 'keyname'. It makes the rendering simpler by just using the _name. It also adds a test for this strange behavior of the metadata service. Change-Id: I3ef67d85354bf7ac8abaaf900ddfdd2e945aa9af
* | | | | | | | Merge "do sync before fusermount to avoid busyness"Jenkins2012-06-271-0/+1
|\ \ \ \ \ \ \ \
| * | | | | | | | do sync before fusermount to avoid busynessAlessio Ababilov2012-06-211-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix bug #1013689 Change-Id: I8ae786e105c08e1b5011a1b2a1dac048a710a1cb
* | | | | | | | | Merge "Improve filter_scheduler performance"Jenkins2012-06-275-12/+17
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Improve filter_scheduler performanceChris Behrens2012-06-265-12/+17
| | |_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add 'columns_to_join' keyword arg to instance_get_all() and have the scheduler host_manager only join 'instance_type'. It doesn't use anything else. Fixes bug 1017769 Change-Id: Ia2c0956d2453f00e26eb89d9496a32ba15d970aa
* | | | | | | | | Merge "Remove unnecessary queries for network info in notifications"Jenkins2012-06-271-1/+1
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Remove unnecessary queries for network info in notificationsChris Behrens2012-06-261-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bandwidth_usage() tries to query network info if "not network_info" when looking at instance['info_cache']. [] is a valid entry for network_info (on build)... so don't query network when we see it. Fixes bug 1017770 Change-Id: Ia7291dbbcc25c04eff16eb048068f879e18f53a4
* | | | | | | | | | Merge "Remove extra DB calls for instances from OS API extensions"Jenkins2012-06-277-95/+125
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Remove extra DB calls for instances from OS API extensionsChris Behrens2012-06-267-95/+125
| | |_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow the core API to store DB results so that extensions can use data already retrieved within the same API request...eliminating extra expensive DB calls. This implements storing of instances only, so far. Fixes bug 1017756 Change-Id: I98a2e3323f5ea69ab9e7470f95d821790510989e
* | | | | | | | | | Merge "Re-factor instance DB creation"Jenkins2012-06-274-99/+185
|\ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / | |/| | | | | / / / | |_|_|_|_|_|/ / / |/| | | | | | | |
| * | | | | | | | Re-factor instance DB creationChris Behrens2012-06-264-99/+185
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch speeds up the DB creation process considerably in deployments with a considerable amount of instances. It also cleans up a lot of the code which ends up creating the instance DB record. The biggest improvement in this patch is removing unnecessary joins in security_groups DB calls. This also reduces the number of DB calls needed to create an instance DB record in general. Side effect of this patch is the default 'display_name' for an instance when it (or hostname) is not specified is now 'Server <uuid>' vs 'Server <id>'. Because of the use of 'id', it required creating the DB record, then updating the record later after we new the 'id'. This is gone. Fixes bug 1017722 Change-Id: I9b7d48644a7abe075545c2c11399351b6a37939c
* | | | | | | | Merge "Allow single uuid to be specified for affinity."Jenkins2012-06-272-1/+30
|\ \ \ \ \ \ \ \ | |_|_|_|_|/ / / |/| | | | | | |
| * | | | | | | Allow single uuid to be specified for affinity.Vishvananda Ishaya2012-06-262-1/+30
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fixes bug 1017988 * Includes failing test for same_host and different_host Change-Id: I91263512a9f76436dce5a8f00d4d97f841d08fec
* | | | | | | Merge "Allow access to metadata server '/' without IP check"Jenkins2012-06-261-0/+4
|\ \ \ \ \ \ \
| * | | | | | | Allow access to metadata server '/' without IP checkZhiteng Huang2012-06-261-0/+4
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow access to '/' of metadata server to get version information without checking source IP, which was how previous release behave. Fix bug 1017759. Change-Id: I3edc2d3808b0ebb2064513a59e6a3fc9e212b283
* | | | | | | Merge "Removes utils.logging_error (no longer used)"Jenkins2012-06-261-13/+0
|\ \ \ \ \ \ \
| * | | | | | | Removes utils.logging_error (no longer used)Dan Prince2012-06-261-13/+0
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | Change-Id: Iea03d4a87452af3fe1bfb8bd5359bcd17fb561be
* | | | | | | Merge "Removes utils.fetch_file (no longer used)."Jenkins2012-06-261-5/+0
|\ \ \ \ \ \ \
| * | | | | | | Removes utils.fetch_file (no longer used).Dan Prince2012-06-251-5/+0
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | Change-Id: I75828a1e40f1daa3a15221f1ab8de484e6bb03ef
* | | | | | | Merge "fix libvirt get_memory_mb_total() with xen"Jenkins2012-06-261-14/+2
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | |
| * | | | | | fix libvirt get_memory_mb_total() with xenPádraig Brady2012-06-251-14/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 5b93a576 failed to remove the 'staticmethod' decorator from the method, to allow it to access object variables. Also there is no need to have the libvirt getInfo() call be xen specific. In the LXC/QEMU/UML drivers this call uses GNULIB's physmem module, which is cross-OS portable. Change-Id: I565eda78f6c468cf6bd227756cfc068725069218
* | | | | | | Merge "Fix invalid variable reference."Jenkins2012-06-262-1/+2
|\ \ \ \ \ \ \
| * | | | | | | Fix invalid variable reference.David Shrewsbury2012-06-262-1/+2
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Correct variable name is 'source_type', but just happened to work because the default value for the variable is also 'mount'. Change-Id: Ifba9ce0ef3052e1b5e07f90d236bfcf24ca6a608
* / | | | | | Fix several PEP-8 issuesSascha Peilicke2012-06-2611-17/+17
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As found by pep8 version 1.2. Change-Id: I5d40294416db0410fa7d9c82ea8efe8efadd0f58
* | | | | | Merge "Another killfilter test fix for Fedora 17."Jenkins2012-06-261-1/+1
|\ \ \ \ \ \
| * | | | | | Another killfilter test fix for Fedora 17.Dan Prince2012-06-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updates killfilter test to use /usr/bin/sleep in addition to /bin/sleep. Fixes LP Bug #1017585. Change-Id: Ied0a8837fb0e7ca7fa3629859fbd67f71e5d80dc