summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | Merge "Actual fix for bug 931608."Jenkins2012-02-231-4/+5
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | |
| * | | | | | | | Actual fix for bug 931608.Joe Gordon2012-02-171-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | -x --stop is a nose flag Provide short term solution to deal with '--hide-elapsed' Can be tested by adding a assertTrue(False) in a test and running ./run_tests.sh -x Change-Id: I7357e7cdaa3d6370ed111d6209917a833d084db3
* | | | | | | | | blueprint host-aggregates: host maintenance - xenapi implementationArmando Migliaccio2012-02-2314-166/+321
| |_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this changeset introduces the following: - refactoring around host-related operations for xenapi - increased test coverage - first cut at implementing host evacuation for a XS/XCP host Change-Id: I8509cdde95f6777ecfa928663b0c4bedbccf5d38
* | | | | | | | Merge "Make database downgrade works"Jenkins2012-02-23105-2293/+5134
|\ \ \ \ \ \ \ \
| * | | | | | | | Make database downgrade worksHengqing Hu2012-02-17105-2293/+5134
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #854905 Use sqlalchemy reflection whenever possible: http://sqlalchemy-migrate.readthedocs.org/en/latest/versioning.html Work around sqlalchemy-migrate sqlite 'bool column not deletable' issue: http://code.google.com/p/sqlalchemy-migrate/issues/detail?id=143 Add following sql scripts for sqlite: 002_sqlite_downgrade.sql 015_sqlite_downgrade.sql 033_sqlite_downgrade.sql 050_sqlite_downgrade.sql 068_sqlite_downgrade.sql Work around sqlalchemy-migrate sqlite 'table with foreign key column not deletable' issue: http://code.google.com/p/sqlalchemy-migrate/issues/detail?id=94 Add following sql scripts for sqlite: 003_sqlite_downgrade.sql 006_sqlite_downgrade.sql 007_sqlite_downgrade.sql 012_sqlite_upgrade.sql 013_sqlite_downgrade.sql 020_sqlite_downgrade.sql 030_sqlite_downgrade.sql 038_sqlite_downgrade.sql 042_sqlite_downgrade.sql 053_sqlite_downgrade.sql 067_sqlite_downgrade.sql Work around sqlalchemy-migrate 'migrate drops engine reference' issue: http://code.google.com/p/sqlalchemy-migrate/issues/detail?id=72 Add following sql scripts for long primary key to work with utf-8 mysql table: 072_mysql_upgrade.sql Add following sql scripts for postgresql: 002_postgresql_downgrade.sql Add snake walk test cases for database migration based on glance migration test. Change-Id: Ib454ecb4662bbf47736c1b12d9a4f969f180ceb6
* | | | | | | | | Merge "Speeding up resize down with sparse_copy"Jenkins2012-02-231-6/+54
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Speeding up resize down with sparse_copyRick Harris2012-02-221-6/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 939084 Change-Id: I78f7476adda427d729b67e7c32efc2a4dd2d7234
* | | | | | | | | | Merge "Move get_info to taking an instance."Jenkins2012-02-2314-74/+68
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Move get_info to taking an instance.Michael Still2012-02-2214-74/+68
| | |_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a first step towards using uuids in this interface instead of instance names (which are synonyms for instance ids). Change-Id: I54b32a5020b0dbc21ba7156ed38ed188c483086b
* | | | | | | | | | Remove network_api fallback for info_cache from APIsChris Behrens2012-02-2210-470/+332
| |/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 932395 OS API and EC2 would query the network API when instance['info_cache'] had no network info.. or network info was an empty list. The ideal was to fall back to querying the network IP in case the cache was busted. However, reality says this case is extremely common as it is the case when instances are freshly built and haven't had network info assigned yet. The calls to network API are expensive and goes against the whole idea of this "cache". So, this patch removes the fallback to querying the network API. In its place, it adds a periodic task to the compute manager to periodically sync the cache entry. Since I had to fix a number of tests, I consolidated some things there as well with regards to instance stubbing. Change-Id: I493f811bcba4e99ac6a5756bcab473557d5c0104
* | | | | | | | | Merge "Add Nexenta volume driver."Jenkins2012-02-224-0/+680
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Add Nexenta volume driver.Yuriy Taraday2012-02-234-0/+680
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Covers blueprint nexenta-volume-driver. Change-Id: Iac30886981355f99e450a7ffbca24e7c23e4e97d
* | | | | | | | | | Merge "refactor a conditional for testing and understanding"Jenkins2012-02-222-35/+61
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | refactor a conditional for testing and understandingAaron Lee2012-02-222-35/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | _cleanup_running_deleted_instances was getting difficult to understand, so I extracted a method that contained the calculation about what is to be deleted. Also, added tests for that method. update: _shutdown_instance has an arity of three, the existing code gives four arguments, one was passed through to driver.destroy, which now has a default argument that defaults to True, the passed value. update 2: Doh! pep8 Change-Id: I1a32512a4e0d80ba4dcc911b96790c29c1f36710
* | | | | | | | | | | Merge "Disable usb tablet support for LXC."Jenkins2012-02-221-1/+1
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|/ / |/| | | | | | | | | |
| * | | | | | | | | | Disable usb tablet support for LXC.Chuck Short2012-02-221-1/+1
| | |/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | libvirt does not support usb tablets. Disable it if running libvirt-lxc. Fixes bug 935744 Change-Id: Ic648df4b3a5b66da089c7153971f0e8dd53bacdd Signed-off-by: Chuck Short <chuck.short@canonical.com>
* | | | | | | | | | Merge "Return 40x for flavor.create duplicate."Jenkins2012-02-222-3/+33
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Return 40x for flavor.create duplicate.Anthony Young2012-02-222-3/+33
| | |/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fixes bug 938194 * Return error 409 instead of 400 Change-Id: Ia3c597dc996d88e7026f76d1104058259c96301c
* | | | | | | | | | Merge "Improve unit test coverage per bug/934566."Jenkins2012-02-222-17/+163
|\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | | |
| * | | | | | | | | Improve unit test coverage per bug/934566.Michael Still2012-02-232-17/+163
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bug/934566 identified an error which should have been caught by more complete unit test coverage. This review extends unit test coverage to include handle_base_image and is one of a series I will be sending. Change-Id: I287fc50ea6a92239f11a107f65da84d3ff0c8b3b
* | | | | | | | | Merge "nova-manage: Fix 'fixed list'"Jenkins2012-02-221-2/+4
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | |
| * | | | | | | | nova-manage: Fix 'fixed list'Adam Gandelman2012-02-221-2/+4
| | |_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix 'nova-manage fixed list' Fixes bug 938702 Change-Id: I65a0a57ac8e16b605eeee963146d4dba582dc077
* | | | | | | | Merge "Add 'nova-manage export auth'"Jenkins2012-02-222-8/+129
|\ \ \ \ \ \ \ \
| * | | | | | | | Add 'nova-manage export auth'Brian Waldon2012-02-212-8/+129
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Introduce ability to dump auth data to json blob for consumption by keystone * Implements bp keystone-export-rewrite Change-Id: Ic8c0ade8350daa449fb6df7ad3c55c3ae2b0c2c7
* | | | | | | | Merge "Don't store connection pool in RpcContext."Jenkins2012-02-222-34/+12
|\ \ \ \ \ \ \ \
| * | | | | | | | Don't store connection pool in RpcContext.Russell Bryant2012-02-212-34/+12
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix bug 934575. As Chris pointed out in the bug, there is a downside to this approach to fixing the bug, in that a manager will no longer be able to use context.reply(). However, it wasn't being used at all, and it's no loss in functionality. A remote method can still return multiple values (in response to a multicall()) by using yield. Change-Id: I0e5aff2e8a40ffd8390c0e19d89dd17e60a74130
* | | | | | | | Merge "Add exception SnapshotIsBusy to be handled as VolumeIsBusy."Jenkins2012-02-223-0/+53
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | |
| * | | | | | | Add exception SnapshotIsBusy to be handled as VolumeIsBusy.Yuriy Taraday2012-02-213-0/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is helpful when driver can not delete snapshot if it has volumes created from it. Add tests for both VolumeIsBusy and SnapshotIsBusy cases. Change-Id: I3fcc6b1ffdfbc7e22ed30ac832db500eb3260d04
* | | | | | | | Merge "Fix WeightedHost"Jenkins2012-02-222-2/+22
|\ \ \ \ \ \ \ \
| * | | | | | | | Fix WeightedHostMark Washenberger2012-02-222-2/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | WeightedHosts don't have zone attributes. Let me know if I'm missing something and should be adding a hasattr check instead of removing the zone reference. Change-Id: If76f23fa46c965fc9004e35dd10da33e2bbb48dc
* | | | | | | | | Merge "Don't allow EC2 removal of security group in use."Jenkins2012-02-229-1/+113
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Don't allow EC2 removal of security group in use.Russell Bryant2012-02-149-1/+113
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix bug 817872. This patch modifies the behavior of removing security groups via the EC2 API to better match the EC2 API spec. The EC2 documentation says that a group that is still in use can not be removed. A new function has been added to the db API to find out whether a particular security group is still in use. "In use" is defined as applied to an active instance, or applied to another group that has not been deleted. Unit tests have been updated to ensure that an error is raised when these conditions are hit. Change-Id: I5b3fdf1da213b04084fe266c1a6ed92e01cf1e19
* | | | | | | | | | Merge "blueprint host-aggregates: improvements and clean-up"Jenkins2012-02-2210-37/+119
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | blueprint host-aggregates: improvements and clean-upArmando Migliaccio2012-02-2210-37/+119
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changeset addresses a number of issues found during testing: - avoid name conflicts during aggregate creation (see db/* changes) - avoid masking of XenAPI.Failure if pool-join fails (see plugins/* changes) - preserve VM placement decisions made during scheduling (see xenapi/vmops.py) - ensure plugins are called on the right hosts in XS pools (see xenapi_con.py) - stores master uuid in aggregate metadata for use in VM live migration and raise InvalidAction rather than Aggregate error if we attempt to remove a mster (see xenapi/pool.py and compute/manager.py) - clean-up of unit tests Change-Id: I881a94d87efe1e81bd4f86667e75f5cbee50ce91
* | | | | | | | | | | Merge "Support fixed_ip range that is a subnet of the network block"Jenkins2012-02-224-10/+18
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | Support fixed_ip range that is a subnet of the network blockJustin Santa Barbara2012-02-214-10/+18
| | |_|_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This enables flat deployments where the instances live on the 'general' network. For example, a typical home router uses 192.168.0.0/16 for the private network. Cloud machines can be assigned 192.168.100.0/24. An example network create command: nova-manage network create private 192.168.0.0/16 1 65536 \ --fixed_cidr=192.168.100.0/24 --gateway=192.168.1.1 Change-Id: I17edd81e9bc21ca3320233b606c99e03e25201bc
* | | | | | | | | | Merge "Clarify use of Use of deprecated md5 library"Jenkins2012-02-222-1/+2
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Clarify use of Use of deprecated md5 libraryJoe Gordon2012-02-212-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See Invalid bug 937463 for more details Also change run_tests.sh to ignore more '*.pyc' files Change-Id: I8c8ded902833fe4268966ff7a9630daaaad43a19
* | | | | | | | | | | Merge "Makes HTTP Location Header return as utf-8 as opposed to Unicode."Jenkins2012-02-221-1/+1
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|/ |/| | | | | | | | | |
| * | | | | | | | | | Makes HTTP Location Header return as utf-8 as opposed to Unicode.Jason Cannavale2012-02-211-1/+1
| | |/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 938114 Change-Id: Ie8a495a8743d2f261fc8f9600cfa1df28a2d5b1e
* | | | | | | | | | Merge "Rework base file checksums."Jenkins2012-02-223-13/+51
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Rework base file checksums.Michael Still2012-02-223-13/+51
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The libvirt image cache manager wants to verify that the images it is managing have not become corrupt. We therefore write checksums for these images to disk and verify them as part of the cache management periodic task. This checksumming was originally done as part of the setup for a new virtual machine. This has been refactored so that generating a checksum on a large file will not delay the startup of a VM. Note that these checksums are intended for detecting corruption, not malicious system admins. Change-Id: I781877b342207cb2cf03eb48bd89456846cbf487
* | | | | | | | | | Merge "Backslash continuations (nova.api.openstack)"Jenkins2012-02-2222-64/+68
|\ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / |/| | | | | | | | |
| * | | | | | | | | Backslash continuations (nova.api.openstack)Zhongyue Luo2012-02-2022-64/+68
| | |_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #925286 Backslash continuations removal for package nova.api.openstack Change-Id: I204b6d53bea224859e23771d5f4c96c1aff6fd03
* | | | | | | | | Merge "blueprint host-aggregates: host maintenance"Jenkins2012-02-225-179/+204
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | blueprint host-aggregates: host maintenanceArmando Migliaccio2012-02-215-179/+204
| | |/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | First cut at implementing host maintenance (aka host evacuation). This allows zero-downtime upgrades of the hosts by moving VMs off of to another host to carry out hypervisor upgrades. A number of issues have been addressed in this changeset: - improved the semantic of update operation on hosts (as per dabo comment) - refactored host-related operations into a separate class in to improve readability/maintainability - refactored test_hosts to reduce duplicated code - added first stub of host-maintenance operation Change-Id: I933f7cb8736e56c9ecea5255936d8826ef6decec
* | | | | | | | | Merge "xenapi: nova-volume support for multiple luns"Jenkins2012-02-222-2/+16
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | xenapi: nova-volume support for multiple lunsRenuka Apte2012-02-212-2/+16
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I012fe76a8e2fb4ddd3c24c5fd43a9b2126d2773d
* | | | | | | | | Merge "Scheduler notifications added."Jenkins2012-02-226-8/+80
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Scheduler notifications added.Sandy Walsh2012-02-216-8/+80
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Re-added lost fake virt hypervisor info (required for local dev). Added multiple exchange topic support to RabbitNotifier. Change-Id: I33cc9076ee35061c21852cabba3275006fc87b86