summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | | Merge "Remove XenAPI use of eventlet tpool"Jenkins2012-05-161-11/+3
|\ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / |/| | | | | | | | |
| * | | | | | | | | Remove XenAPI use of eventlet tpoolChris Behrens2012-05-161-11/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | XenAPI implementation does not block, even with it using SSL, therefore there's no need to use eventlet's tpool. Change-Id: I33ef4140068c8af06d27a075f52379581790c647
* | | | | | | | | | Merge "Rearchitect quota checking to partially fix bug 938317."Jenkins2012-05-168-40/+3026
|\ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / |/| | | | | | | | |
| * | | | | | | | | Rearchitect quota checking to partially fix bug 938317.Kevin L. Mitchell2012-05-168-40/+3026
| | |/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a rearchitecting/rewriting of quota handling to correct the quota atomicity issues highlighted by bug 938317. Partially implements blueprint quota-refactor as well. This change is fairly substantial. To make it easier to review, it has been broken up into 3 parts. This is the first part. Change-Id: I805f5750c08de17487e59fe33fad0bed203188a6
* / | | | | | | | Misused and not used config optionsZhongyue Luo2012-05-167-22/+17
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #1000071 [nova.flag.py] stub_network: StrOpt -> BoolOpt [nova.api.sizelimit.py] osapi_max_request_body_size: BoolOpt -> IntOpt [nova.compute.api.py] find_host_timeout: not used [nova.rpc.__init__.py] allowed_rpc_exception_modules: IntOpt -> ListOpt [nova.virt.vmwareapi_conn.py] vmwareapi_api_retry_count: FloatOpt -> IntOpt [nova.volume.driver.py] num_shell_tries: StrOpt -> IntOpt num_iscsi_scan_tries: StrOpt -> IntOpt [nova.volume.driver.py] san_thin_provision: 'true' -> True san_is_local: 'false' -> False Change-Id: I07ab60b1d881c9475f4f2517a432f1e934d8ae52
* | | | | | | | Merge changes I36ff7c05,If43658b9Jenkins2012-05-1623-267/+239
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * changes: Stop using nova.exception from nova.rpc. Make use of openstack.common.jsonutils.
| * | | | | | | Stop using nova.exception from nova.rpc.Russell Bryant2012-05-153-10/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch is a part of continuing to remove dependencies from nova.rpc on the rest of nova. One RPC related exception was defined in nova.exception, so that was moved to nova.rpc.common where the rest of them live. These exceptions were changed to no longer use NovaException as their base. Instead, there is a new RPCException base. One other change that should be reviewed closely is the removal of using nova.exception.wrap_exception() in nova.rpc.amqp. As far as I can tell, this didn't actually do anything since nothing was being passed in to wrap_exception(). Change-Id: I36ff7c05ab0467ad8506b56d561c532eadf8dff8
| * | | | | | | Make use of openstack.common.jsonutils.Russell Bryant2012-05-1521-257/+210
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch imports jsonutils from openstack-common. It removes the equivalent code from nova.utils and then converts the code base to use jsonutils. The primary motivator for this change was to remove the rest of the dependencies from nova.rpc on nova.utils. Change-Id: If43658b9b098ed56cba018c81be268b8c3e2916a
* | | | | | | | Fixed compute periodic task. Fixes bug 973331.Unmesh Gurjar2012-05-169-186/+124
| |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. Moved the poll_unconfirmed_resizes implementation to compute manager (as it is not hypervisor specific). 2. Added unit test case coverage. Change-Id: Ic58502f44326c5d59439087b4fa9ae7aa79c1e17
* | | | | | | Merge "Fix bug 988034 - Quantum Network Manager - not clearing ips"Jenkins2012-05-153-8/+16
|\ \ \ \ \ \ \
| * | | | | | | Fix bug 988034 - Quantum Network Manager - not clearing ipsSomik Behera2012-05-153-8/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added a line in nova_ipam_lib.QuantumNovaIPAMLib init method to ensure we clean up IPs and explanation of whats going on. Also, nuked release_fixed_ip as that is not needed in QuantumManager anymore. Change-Id: Ic2ad1e87441c448c10ff097b28d2b6347eef5401
* | | | | | | | Merge "Alphabetize imports in nova/api/"Jenkins2012-05-1527-33/+33
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
| * | | | | | | Alphabetize imports in nova/api/Joe Gordon2012-05-1527-33/+33
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | Change-Id: I2a265ffded604265bdb9c75db36112aebf1e5270
* | | | | | | Merge "Alphabetize imports in nova/virt/"Jenkins2012-05-1511-23/+23
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | Alphabetize imports in nova/virt/Joe Gordon2012-05-1411-23/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ifca489da67cc834284ee2f2beeaafb28f0638f7e
* | | | | | | Merge "Alphabetize imports"Jenkins2012-05-1523-31/+31
|\ \ \ \ \ \ \ | |_|_|_|/ / / |/| | | | | |
| * | | | | | Alphabetize importsJoe Gordon2012-05-1423-31/+31
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Except for nova/api/, nova/tests/, nova/virt/ In preparation for automated checking being enabled (N306) Change-Id: I5af70ea13b470ce58695ef0a5e0ff9da3ac1b135
* | | | | | Merge "Pass 'nova' project into ConfigOpts."Jenkins2012-05-151-0/+2
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | |
| * | | | | Pass 'nova' project into ConfigOpts.Rick Harris2012-05-141-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 999305 The problem was that, post refactoring, `ConfigOpts.find_file` was no longer searching `/etc/nova` for config files. Change-Id: I3dee8d8f24ae3912e07dcb9d19086d55eb98ccaa
* | | | | | Merge "Create an internal key pair API."Jenkins2012-05-157-128/+255
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Create an internal key pair API.Dan Prince2012-05-147-128/+255
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Creates an internal key pair API and update the EC2 and OS API's to use it. This de-duplicates some of the code used to manage keypairs across the APIs. Fixes LP Bug #998059. Change-Id: I10d58d7ce68cc2b993c72b6639f66c72def3bfbc
* | | | | Adding notifications for volumesCraig Vyvial2012-05-149-3/+323
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added notifications for volumes have been added with tests. This includes create/delete/exists events for volumes. blueprint nova-notifications Change-Id: I21b74974fac22c3621ccf7564dc5c0d339f8751a
* | | | Merge "Avoid setting up DHCP firewall rules with FlatManager"Jenkins2012-05-145-14/+28
|\ \ \ \
| * | | | Avoid setting up DHCP firewall rules with FlatManagerhua zhang2012-05-145-14/+28
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #704737 With FlatManager, ensure that the network info's dhcp_server value is not set and use that key to decide in the compute service whether DHCP firewall rules should be added. Change-Id: I8183a6fa3881adea1a09f3f1a29442e6b7a919ce
* | | | Merge "Make allocation failure a bit more friendly"Jenkins2012-05-141-6/+12
|\ \ \ \
| * | | | Make allocation failure a bit more friendlyChuck Short2012-05-141-6/+12
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the user allocates the wrong address display a message about incorrect address rather than a traceback. Fixes LP: #957707 Change-Id: I8ccd3056f755afb9f873a25fd494428e481ff3a2 Signed-off-by: Chuck Short <chuck.short@canonical.com>
* | | | Merge "Report memory correctly on Xen. Fixes bug 997014"Jenkins2012-05-141-6/+29
|\ \ \ \
| * | | | Report memory correctly on Xen. Fixes bug 997014Alvaro Lopez Garcia2012-05-101-6/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | /proc/meminfo may show wrong values for the memory when using Xen, so this fix computes the memory quering libivrt. Change-Id: I188e2d34bcee13954653b93b9b816cf4530b8859
* | | | | Migrate missing license infoZhongyue Luo2012-05-149-0/+60
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #998965 Some modules in nova/db/sqlalchemy/migrate_repo are missing license information Change-Id: I40efeb2f72b08e117b3a1b1fe25e51d8b4fa9700
* | | | Merge "Fix libvirt Connection.get_disks method"Jenkins2012-05-132-18/+63
|\ \ \ \
| * | | | Fix libvirt Connection.get_disks methodJulien Danjou2012-05-112-18/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The get_disks() method is not used anywhere yet, and is not tested properly, but it does not work as reported in bug #998089. This patch add unit test for it and fixes it so it's now possible to retrieve the list of device name associated with a libvirt domain. Change-Id: If09fb683364174fe02cbcbec4c6bbcda991287d6 Signed-off-by: Julien Danjou <julien.danjou@enovance.com>
* | | | | Merge "Create a utf8 version of the dns_domains table."Jenkins2012-05-111-0/+145
|\ \ \ \ \
| * | | | | Create a utf8 version of the dns_domains table.Dan Prince2012-05-111-0/+145
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The old dns_domains table was encoded in the 'latin1' charset and has its primary key length set to 512. This is too long to be a valid pkey in a proper 'utf8' encoded table and seems to cause a slew of errors like: 1) Dumping a database with mysqldump and trying to import it fails because this table is latin1 but fkeys to utf8 tables (projects). 2) Trying to alter the old dns_domains table fails with errors like: mysql> ALTER TABLE dns_domains DROP PRIMARY KEY; ERROR 1025 (HY000): Error on rename of './nova/#sql-6cf_855'.... This patch creates a new migration that re-creates the dns_domains table with a shorter 'domain' column which works with the utf8 charset. Fixes LP Bug #993663. Change-Id: I2060d39cbdabd20f410ebd501160a04c51641755
* | | | | Merge "fix bug 992008, we should config public interface on compute"Jenkins2012-05-112-4/+17
|\ \ \ \ \
| * | | | | fix bug 992008, we should config public interface on computePeng Yong2012-05-082-4/+17
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I35a7d63c9613161f9fea052edf39759e4e2c6227
* | | | | | Merge "Added img metadata validation. Fixes bug 962117."Jenkins2012-05-115-6/+50
|\ \ \ \ \ \
| * | | | | | Added img metadata validation. Fixes bug 962117.Unmesh Gurjar2012-05-115-6/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. Added validation for image metadata. 2. Renamed the method 'check_img_metadata_quota_limit' to 'check_img_metadata_quota_limit' since it also validates the image metadata. 3. Added unit test cases. Change-Id: I0e9dad97c03070363d14977897701e146870e41a
* | | | | | | Merge "Use default qemu-img cluster size in libvirt connection driver"Jenkins2012-05-113-2/+3
|\ \ \ \ \ \ \
| * | | | | | | Use default qemu-img cluster size in libvirt connection driverJim Fehlig2012-05-113-2/+3
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The recommended cluster size [1] for qcow2 is 64k, which is the default when invoking 'qemu-img create ...'. Remove the cluster_size option from qemu-img invocation. [1] http://www.mail-archive.com/openstack@lists.launchpad.net/msg11371.html V2: Fix test. Change-Id: Iafc290720a820cb5f57ded10a2f584fdc5cd5b49
* | | | | | | Merge "Setup logging, particularly for keystone middleware"Jenkins2012-05-112-0/+4
|\ \ \ \ \ \ \
| * | | | | | | Setup logging, particularly for keystone middlewareAlessio Ababilov2012-05-112-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 979282 and bug 988951 Change-Id: I895d00ac9648e8b3cb3ad1bf141c9bfe02ef3680
* | | | | | | | Merge "Destroy system metadata when destroying instance"Jenkins2012-05-111-0/+5
|\ \ \ \ \ \ \ \
| * | | | | | | | Destroy system metadata when destroying instanceChris Behrens2012-05-111-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | instance system_metadata was not being marked as deleted when an instance is marked as deleted. Fixes bug 997849 Change-Id: I876b402ca33f4574bb426615f6c467ded6908fc2
* | | | | | | | | Merge "Store image properties with instance system_metadata"Jenkins2012-05-116-27/+100
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Store image properties with instance system_metadataChris Behrens2012-05-116-27/+100
| | |_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In implementing adding 'image_meta' to instance notifications, this was committed: https://review.openstack.org/#/c/7309 commit 53adfd289bf7226f1c6f59c17608e42f3083f130 However, it's buggy because an instance could have been deleted after we built from it... and bin/instance-usage-audit uses an admin context with which we cannot query glance. This stores image properties with instances in nova as system_metadata and notifications will use that data. Fixes bug 997833 Change-Id: I50575969b5cb28adaae9a713e749dc486772c417
* | | | | | | | | Merge "Provide a transition to new .info files."Jenkins2012-05-112-1/+30
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Provide a transition to new .info files.Michael Still2012-05-112-1/+30
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I2fb6ef93c0652a797dba72cf75fb78baaa9c9c5a
* | | | | | | | | Merge changes I144e5331,I96a7222aJenkins2012-05-113-54/+25
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * changes: Remove unnecessary stubout_loopingcall_start Actually use xenapi fake setter
| * | | | | | | | Remove unnecessary stubout_loopingcall_startJohannes Erdfelt2012-05-112-13/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The xenapi driver no longer uses utils.LoopingCall Change-Id: I144e5331e871570d8d22b87844ecebb63ed02697
| * | | | | | | | Actually use xenapi fake setterJohannes Erdfelt2012-05-113-41/+25
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was implemented (with a small bug), but never used. Also, lean on the xenapi fake over using stubs in the migration tests to ensure we test more code. Change-Id: I96a7222a017302332bd6a3600fa645a74c502944