summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Adds tests for isolated_hosts_filter"Jenkins2013-04-292-35/+63
|\
| * Adds tests for isolated_hosts_filterMatthew Sherborne2013-04-262-35/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The isolated_hosts_filter takes two configuration options. This patch adds three tests defining the behaviour for when one or more of these variables are missing. This patch speeds things up a bit when the isolated_images option is missing, making it do less dict lookups to come up with the same answer. Matrix of isolated_hosts_filter changes: +----------------------+--------------------+--------------+ | Config options | Before patch | After | +----------------------+--------------------+--------------+ | 0 + 0 | No filtering | Same as B4 | | | no tests | Tested | +----------------------+--------------------+--------------+ | 0 + iso_image | Block iso'd images | Same as B4 | | | no tests | Tested | +----------------------+--------------------+--------------+ | iso_host + 0 | Block iso'd images | Same as b4 | | | no tests | Tested | +----------------------+--------------------+--------------+ | iso_host + iso_image | Block matched h+i | Same as b4 | | | Tested | Tested | +----------------------+--------------------+--------------+ Change-Id: Ibe6725e39a771b31fdf49513335275deba33e96b
* | Merge "Fix up regression tester"Jenkins2013-04-291-64/+92
|\ \
| * | Fix up regression testerJoe Gordon2013-04-171-64/+92
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Clean up to support pydoc * Support -h * Make it actually work * Improve docstrings * Add copyright Change-Id: I977df71e8213e39e9eebf9cc56b2dd7625959870
* | | Merge "Fixes race condition of deleting floating ip"Jenkins2013-04-292-5/+31
|\ \ \
| * | | Fixes race condition of deleting floating ipWangpan2013-04-262-5/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #1172599 reproduce steps: 1. create an instance 2. get a floating ip from pool 3. associate the ip to instance created at step 1 4. delete the instance and the ip at a very little interval reason may be that: 1. the action of delete ip find the ip is associated to a fixed ip at first(at api level) 2. and then the fixed ip is released when the instance is deleted 3. so nova-network manager findes this issue and raises exception.FloatingIpNotAssociated(address=floating_address) Change-Id: I253d0b240451e8a728532639e9857894f6fc6870
* | | | Merge "Fix missing kernel output via VNC/Spice on boot"Jenkins2013-04-291-5/+7
|\ \ \ \
| * | | | Fix missing kernel output via VNC/Spice on bootNicolas Simonds2013-04-251-5/+7
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug 1164021 Use both ttys0 and tty0 as console options to the kernel during boot. Change-Id: I0365ff7cad021fb6ca551651d00d58297087e95e
* | | | Merge "Support FlatDHCP network for VMware Hyper."Jenkins2013-04-292-30/+29
|\ \ \ \
| * | | | Support FlatDHCP network for VMware Hyper.Yaguang Tang2013-04-242-30/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fix bug #1171783 Change-Id: Ibb75c8e3174283f0f5e94bacfd61ee93e8bd6227
* | | | | Merge "Add missing snapshot image properties for VMware Hyper."Jenkins2013-04-291-1/+5
|\ \ \ \ \
| * | | | | Add missing snapshot image properties for VMware Hyper.Yaguang Tang2013-04-291-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix bug #1174199 Change-Id: I50929ed2e66052a37bb5a94cd3863baaebda1e28
* | | | | | Fix flakey TestS3ImageService bugChris Yeoh2013-04-291-1/+1
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test_show_translates_image_state_correctly sometimes fails on some systems. This is caused because the ec2utils cache is reset after the test rather than before the test (but after the images are manually added to the db in setUp). This change resets the ec2utils cache in the correct place Fixes bug 1174266 Change-Id: I2ff9b71c932c913ca0423b1de4c04ca7d9de59cf
* | | | | Merge "Completes the power_state mapping from compute driver and manager"Jenkins2013-04-284-1/+92
|\ \ \ \ \
| * | | | | Completes the power_state mapping from compute driver and managerDavid Peraza2013-04-124-1/+92
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1166906 Added extra states to constants module together with test cases checking every state map. Also added some debug statements under blockdev and operator to be able to spot obvious issues with images at deploy time. Change-Id: I77ebb4d417a3875b761f5fb220d9274b79611ff0
* | | | | | Merge "Map internal S3 image state to EC2 API values"Jenkins2013-04-282-1/+52
|\ \ \ \ \ \
| * | | | | | Map internal S3 image state to EC2 API valuesChris Yeoh2013-04-182-1/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes the EC2 API so it maps internal S3 image state values to ones defined by the EC2 API rather than returning the internally used values. Fixes bug #1074904 Change-Id: Iabbda0e5fcbe4d572c76367c6f98d4bece050e73
* | | | | | | Imported Translations from TransifexOpenStack Jenkins2013-04-2822-4562/+4837
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I1d5e362f2e4e57cb8c754f27fc27fed328989597
* | | | | | | Merge "set bdm['volume_id'] to None rather than delete it"Jenkins2013-04-261-1/+1
|\ \ \ \ \ \ \
| * | | | | | | set bdm['volume_id'] to None rather than delete itDongdong Zhou2013-04-261-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bdm is a sqlalchemy model object, it doesn't make sense to delete an attribute, so just set it to None Fix bug 1166160 Change-Id: I41017af602f0c8369d442b4a6936e1a829d2f9a9
* | | | | | | | Merge "Send a instance create error notification"Jenkins2013-04-263-6/+89
|\ \ \ \ \ \ \ \
| * | | | | | | | Send a instance create error notificationBrian Elliott2013-04-263-6/+89
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Send a 'create.error' notification if instance creation (run_instance) fails. This will make it easier for external notification consuming systems to learn when instance build attempts fail within the nova-compute layer. blueprint create-error-notification Change-Id: I62d7c9c80c51241bf124509af7cdd8484d9ea2d3
* | | | | | | | | Merge "Refactor _run_instance() to unify control flow"Jenkins2013-04-263-116/+161
|\| | | | | | | |
| * | | | | | | | Refactor _run_instance() to unify control flowBrian Elliott2013-04-263-116/+161
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | _run_instance() had multiple points where various classes of errors were handled. This patch classifies the possible error/exit conditions into the following categories: * success * rescheduled - an error occurred, but the build will be returned to scheduled for another attempt. * aborted - build was aborted for a non-error. (e.x. instace was deleted during spawn) * fatal - general case of uncaught exceptions resulting in the instance going to error. blueprint create-error-notification Change-Id: I3f01753bde8af0750ba9067c8a98ded45d5a0eb7
* | | | | | | | Merge "Fix fixed_ip_count_by_project in DB API"Jenkins2013-04-262-3/+1
|\ \ \ \ \ \ \ \
| * | | | | | | | Fix fixed_ip_count_by_project in DB APIDmitry Spikhalskiy2013-04-222-3/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix "by project" filtering in fixed_ip_count_by_project method of SQLAlchemy DB API implementation. Fixes: bug #1171161 Change-Id: Id408aba6db4dd953945feba36d17b056266ca4b7
* | | | | | | | | Merge "Wrong proxy port in nova.conf for Spice proxy"Jenkins2013-04-263-5/+5
|\ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / |/| | | | | | | |
| * | | | | | | | Wrong proxy port in nova.conf for Spice proxyÉmilien Macchi2013-04-263-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If we look at nova/cmd/spicehtml5proxy.py, we use 6082 port for Spice proxy. In nova.conf, the default port for html5proxy_base_url is 6080 which fails console service. Fix bug #1172957 Change-Id: I52ca37f76e7a6abd085b079ae42e7914cb619ad5
* | | | | | | | | Merge "Imported Translations from Transifex"OpenStack Jenkins2013-04-2622-9773/+10033
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ / |/| | | | | | | |
| * | | | | | | | Imported Translations from TransifexOpenStack Jenkins2013-04-2622-9773/+10033
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I5a7d1d82cea596426c5a5caeaa5d4a0389e833db
* | | | | | | | Merge "Allow listing fixed_ips for a given compute host."Jenkins2013-04-266-13/+123
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
| * | | | | | | Allow listing fixed_ips for a given compute host.Michael Still2013-04-156-13/+123
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | nova.db.fixed_ip_get_all_by_instance_host was removed in commit bb867ce3948ddc23cf928ca3dda100a1a977896a, but was still used in nova-manage. This patch works around the removal of the foreign key that led to this method being removed. Resolves bug 1163660. Change-Id: Ieccd37c752e245558827615c098604d11ad3d945
* | | | | | | | Merge "Fix bug in db.instance_type_destroy"Jenkins2013-04-252-1/+26
|\ \ \ \ \ \ \ \
| * | | | | | | | Fix bug in db.instance_type_destroyBoris Pavlovic2013-04-252-1/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix db.instance_type_destroy() Corresponding instance_type_projects weren't deleted Fix db.instance_type_acces_get_by_flavor_id method Return [] list if there is no such instance_type instead of AttributeError Fixes bug 1172153 Change-Id: Ib1a9a23aa6e26711c198bf4a5059797044c44426
* | | | | | | | | Merge "Move get_backdoor_port to base rpc API."Jenkins2013-04-2538-232/+56
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Move get_backdoor_port to base rpc API.Russell Bryant2013-04-2438-232/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Each service implemented the get_backdoor_port method individually. This patch moves the implementation of this method to the base rpc API instead, and removes the now unnecessary code from each of the services. The server side method was left on all of the managers for rpc backwards copmatibility. They can be removed on the next major rpc version bump of those APIs. Part of blueprint base-rpc-api. Change-Id: Ia8838fafd80eb86a1c2d66f5e97370042d8d8c53
* | | | | | | | | | Merge "Deprecate conductor ping method."Jenkins2013-04-254-20/+10
|\| | | | | | | | |
| * | | | | | | | | Deprecate conductor ping method.Russell Bryant2013-04-234-20/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Deprecate the ping() method of the conductor API in favor of the one that comes with the base rpc API. Part of blueprint base-rpc-api. Change-Id: Iea46b7f6ce70f4919d0121433af2fd64d6346941
* | | | | | | | | | Merge "Add an rpc API common to all services."Jenkins2013-04-2510-9/+136
|\| | | | | | | | |
| * | | | | | | | | Add an rpc API common to all services.Russell Bryant2013-04-2310-9/+136
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds an rpc API that is exposed by all services. The methods in this API exist in their own namespace and are versioned independently of the main API for the service. The first method for this API is a simple ping() method. This method exists in the conductor rpc API already, and could be more generally useful. Other methods will be added in later patches. The base rpc API will be exposed from all services automatically unless they override the create_rpc_dispatcher method in the base manager class. All services need to pass a service_name into the base manager constructor. Some services already did this, but now it's needed for all of them. Implements blueprint base-rpc-api. Change-Id: I02ab1970578bc53ba26461b533d06d1055c2d88e
* | | | | | | | | | Merge "Fix powervm driver resize instance error"Jenkins2013-04-252-18/+63
|\ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / |/| | | | | | | | |
| * | | | | | | | | Fix powervm driver resize instance errorguohliu2013-04-242-18/+63
| | |_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | During a resize operation in powervm, the logical volume of the instance will be copied and compressed to a file. Current logic decompresses it twice before copying this file to a new volume which will cause the new instance to not boot properly. Fixes bug #1169848 Change-Id: I91f536919511aafbf7dbe14abbee48dbceb189b2
* | | | | | | | | Merge "Translate NoMoreFloatingIps exception"Jenkins2013-04-252-7/+20
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Translate NoMoreFloatingIps exceptionChris Yeoh2013-04-182-7/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the pool of floating ips is exhausted the NoMoreFloatingIps exception is raised but not handled properly in the floating ips extension resulting in the exception propagating further up and causing a stack trace to be logged. This change translates the exception explicitly into a HTTPNotFound, preserving the current REST API behaviour. It fixes a bug where the pool name was not correctly inserted into the message returned if allocation from a specific pool was requested. Fixes bug 1169811 Part of blueprint no-stacktraces-in-logs Change-Id: I8f35d25d065bb1fa709cff6f59841ac8c86658bd
* | | | | | | | | | Merge "Import and convert to oslo loopingcall."Jenkins2013-04-2414-156/+208
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Import and convert to oslo loopingcall.Michael Still2013-04-1614-156/+208
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Import the oslo looping call implementation (which is a copy of nova's), delete nova's local copy, convert all users to the new location. It should be noted that the oslo implementation of FixedIntervalLoopingCall measures time from the start of the periodic task, not the end, so periodic tasks will run with a constant frequency instead of the frequency changing depending on how long the periodic task takes to run. Change-Id: Ia62ce1988f5373c09146efa6b3b1d1dc094d50c4
* | | | | | | | | | Merge "Add unit tests for /db/api.py#fixed_ip_*"Jenkins2013-04-241-75/+221
|\ \ \ \ \ \ \ \ \ \ | | |_|_|_|/ / / / / | |/| | | | | | | |
| * | | | | | | | | Add unit tests for /db/api.py#fixed_ip_*Dmitry Spikhalskiy2013-04-221-75/+221
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements: some tests for db methods fixed_ip_* for blueprint db-api-tests Change-Id: I51fe24600b7df2b529350ad70939a01f60bd3c9c
* | | | | | | | | | Merge "Move db.instance_type_extra_specs_* to db.instance_type_* methods"Jenkins2013-04-242-101/+95
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Move db.instance_type_extra_specs_* to db.instance_type_* methodsBoris Pavlovic2013-04-242-101/+95
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All instance_type_* methods should be one after another blueprint db-cleanup Change-Id: Icc69c45dbcb3dcea5f81266a11f726b5b3215e27