summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | | | | | | | | Importing correlation_id middleware from oslo-incubatorAditi Raveesh2013-05-233-0/+30
| | |_|_|_|_|/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Including 'correlation_id' middleware from oslo-incubator. Adding this to openstack-common.conf to copy over the middleware to the location nova/openstack/common/middleware Change-Id: I21b27e8dae585345d03d59c125d9e1bbea95f830 Implements: blueprint cross-service-request-id
* | | | | | | | | | | | | | | Merge "Disallow resize if image not available"Jenkins2013-05-242-0/+39
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / / / / / |/| | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Disallow resize if image not availableNikola Dipanov2013-05-212-0/+39
| | |_|_|_|_|_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a resize is attempted on an instance that was started from an image that has since been deleted, a resize will fail. This change makes error reporting a bit cleaner. This change is needed since in order to actually properly support resize/migrate when the image is deleted - it is necessary for nova to keep a copy of the image metadata and re-use it in case of migration/resize. Fixes bug: 1160773 Fixes bug: 1177001 Change-Id: Ifaea71f79c97046a4cde094e3a5e676772fcceb4
* | | | | | | | | | | | | | Merge "Reset task_state when resetting vm_state to ACTIVE"Jenkins2013-05-241-1/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | Reset task_state when resetting vm_state to ACTIVEChris Behrens2013-05-221-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A virt driver can raise InstanceFaultRollback to indicate that the instance should be reset to ACTIVE. However, the code that resets to ACTIVE does not reset task_state to None. This patch fixes that. Technically, there's currently no bug here, because the compute manager methods that matter here all have the reverts_task_state() decorator to always reset task_state to None on exceptions. However, a bug would be introduced later if we remove those decorators. Change-Id: Ibcf8ae3473f601781d8aae80f88df3da54bbaac8
* | | | | | | | | | | | | | | Merge "hyperv: Move tests under nova/tests/virt/hyperv"Jenkins2013-05-244-2/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / / / / / |/| | | | | | | | | | | | | |
| * | | | | | | | | | | | | | hyperv: Move tests under nova/tests/virt/hypervRick Harris2013-05-234-2/+2
| | |_|_|_|_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ia4ea16e5aa4a48e1f10a52394f7b59fdfdb68d54
* | | | | | | | | | | | | | Merge "Make a few places tolerant of sys_meta being a dict"Jenkins2013-05-233-3/+9
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | Make a few places tolerant of sys_meta being a dictDan Smith2013-05-223-3/+9
| | |_|_|_|_|_|_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will be necessary as we start to introduce objects with real dicts in system_metadata. These are the common spots that need to be ready ahead of time. The rest are at the actual use sites and can/will be replaced when individual uses are converted to objects. Related to bp/unified-internal-objects Change-Id: I8314e0d52ec2ae800765f60ce58ce9b309d7d513
* | | | | | | | | | | | | | Merge "virt: Move generic virt tests to nova/tests/virt/"Jenkins2013-05-236-2/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | virt: Move generic virt tests to nova/tests/virt/Rick Harris2013-05-236-2/+2
| | |/ / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the interest of better organization, this patch moves generic virt testing code into nova/tests/virt. Change-Id: I79473726a848f0abbcb13865efe30e02aea09ae5
* | | | | | | | | | | | | | Merge "Better error message on malformed request url"Jenkins2013-05-231-1/+5
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | Better error message on malformed request urlRick Harris2013-05-231-1/+5
| |/ / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The existing error message just said 'malformed url' which isn't super-helpful. The new version expalins that the project ID in the URL is not matching what's being passed in with the context. Change-Id: I8e5b71fcd51ab3628425b53356f9671f3a2402c0
* | | | | | | | | | | | | | Imported Translations from TransifexOpenStack Jenkins2013-05-2326-10078/+9780
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I0e289eba0a37f9d6068491aefb9261ac81968a02
* | | | | | | | | | | | | | Merge "Fix UnboundLocalError in powervm lvm cleanup code"Jenkins2013-05-232-21/+102
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / |/| | | | | | | | | | | | |
| * | | | | | | | | | | | | Fix UnboundLocalError in powervm lvm cleanup codeMatt Riedemann2013-05-232-21/+102
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | During spawn, if a local volume for the backing block device fails to be created, the cleanup operation for the volume can fail with a "UnboundLocalError: local variable 'disk_name' referenced before assignment" error. The fix is to assign to disk_name earlier and check for None before running cleanup. This patch also adds tests to cover the failure case. Fixes bug 1180955 Change-Id: I0f32df1586d7bd87eccb674fec2a9597b1ce351d
* | | | | | | | | | | | | | Merge "xenapi: Moving tests under tests/virt/xenapi/"Jenkins2013-05-237-401/+377
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | xenapi: Moving tests under tests/virt/xenapi/Rick Harris2013-05-227-401/+377
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have both a tests/xenapi/ and a tests/virt/xenapi/ directory which is confusing. This patch standardizes on tests/virt/xenapi/ which matches the nova/ directory layout. Change-Id: I324a4730e2477aa1863b17c7ea404ce2932a681c
* | | | | | | | | | | | | | | Merge "xenapi: Disable VDI size check when root_gb=0"Jenkins2013-05-232-12/+60
|\| | | | | | | | | | | | | |
| * | | | | | | | | | | | | | xenapi: Disable VDI size check when root_gb=0Rick Harris2013-05-222-12/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Nova uses root_gb=0 to indicate that we should use the image as-is without trying to resize the root disk. Currently the xenapi virt-driver doesn't properly implement this behavior. This patch gets us a step closer by disabling the VDI size check when root_gb=0. We still need to ensure that the disk-resize code handle root_gb=0 as well. References bug 1155113 Change-Id: If91963e079e197a1315c48b8643f3419b42b0b2c
* | | | | | | | | | | | | | | Merge "Remove ImageTooLarge exception"Jenkins2013-05-239-19/+14
|\| | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Remove ImageTooLarge exceptionRick Harris2013-05-229-19/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | InstanceTypeDiskTooSmall and ImageTooLarge are really two sides of the same coin; an image is larger than the root_gb of the instance_type. Rather than have two exceptions for this, this patch consolidates down to just InstanceTypeDiskTooSmall. Change-Id: I401205072c111a960beb2932c6c1889141ae03c3
* | | | | | | | | | | | | | | Merge "Move ImageTooLarge check to Compute API"Jenkins2013-05-235-174/+208
|\| | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Move ImageTooLarge check to Compute APIRick Harris2013-05-225-174/+208
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The existing ImageTooLarge check is in the compute manager, meaning that a validation error puts the instance into an ERROR state without the ability to communicate the fault back to the user. This patch moves the check to the API where we can return the appropriate error to the user. Change-Id: I2183449351e3b70f98d0552ea247184a3641d85a
* | | | | | | | | | | | | | | Merge "Remove locals() usage from nova.virt.libvirt.volume."Jenkins2013-05-231-13/+20
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / / / |/| | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Remove locals() usage from nova.virt.libvirt.volume.Michael Still2013-05-231-13/+20
| | |_|_|_|/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Idda3c0c447bd1f6394805cd6230a32dc22bb6268
* | | | | | | | | | | | | | Merge "Share checks between create and rebuild"Jenkins2013-05-231-31/+42
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / | |/| | | | | | | | | | | |
| * | | | | | | | | | | | | Share checks between create and rebuildRick Harris2013-05-221-31/+42
| | |_|_|_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | DRYs up the checks between create and rebuild in preparation for additional checks going in. Change-Id: I04a8448f7a3a4e1176b40f943ea9c90c06ba8ece
* | | | | | | | | | | | | Merge "Rename requires files to standard names."Jenkins2013-05-237-8/+8
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Rename requires files to standard names.Ruby Loo2013-05-227-8/+8
| | |_|/ / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rename tools/pip-requires to requirements.txt and tools/test-requires to test-requirements.txt. These are standard files, and tools in the general world are growing intelligence about them. Change-Id: I68ece8406fb1d4e082a42db8e76e17b1aaa7e775 Fixes: bug #1179008
* | | | | | | | | | | | | Merge "Added validation for networks parameter value"Jenkins2013-05-232-0/+24
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Added validation for networks parameter valueSatyanarayana Patibandla2013-05-202-0/+24
| | |_|_|_|_|_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A validation check is added for networks parameter value. In server create request body, if networks parameter value is not stored in a list then 'Bad networks format' error is thrown with bad request exception.A new test case is added to verify whether it is returning bad request exception when invalid syntax is used. Fixes: bug #1175683 Change-Id: I457a1626c56281f89d8c4a606ccc0d0e3aea05fa
* | | | | | | | | | | | | Merge "Remove locals() from scheduler filters"Jenkins2013-05-239-14/+27
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Remove locals() from scheduler filtersBelmiro Moreira2013-05-239-14/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Just removed locals() and replaced it with an explicit dicionary. Change-Id: Id476a684abde36238b5abfe8ce4c3ddb0874423b
* | | | | | | | | | | | | Merge "Removed session from fixed_ip_*() functions."Jenkins2013-05-231-34/+35
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Removed session from fixed_ip_*() functions.Victor Sergeyev2013-05-221-34/+35
| | |_|_|_|_|_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removed session objects from public fixed_ip_*() functions in nova/db/sqlalchemy/api module. Minor code refactiring. blueprint db-session-cleanup Change-Id: Ic2a96270745799496e365aee0380ac079e1edeff
* | | | | | | | | | | | | Merge "Imported Translations from Transifex"OpenStack Jenkins2013-05-2226-9859/+8759
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Imported Translations from TransifexOpenStack Jenkins2013-05-2226-9859/+8759
| | |_|_|/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I84eada7abbc7f73514f2b92fc2024fcbd2736af7
* | | | | | | | | | | | | Merge "Make nova-api use servicegroup.API.service_is_up()."Jenkins2013-05-222-5/+14
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Make nova-api use servicegroup.API.service_is_up().Akira Yoshiyama2013-05-192-5/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix bug 1168861 Change-Id: I1e7e8f7519ec627b96736f74ee7123d101973544
* | | | | | | | | | | | | Merge "Make it easier to add namespaced rpc APIs."Jenkins2013-05-222-2/+41
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Make it easier to add namespaced rpc APIs.Russell Bryant2013-05-212-2/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add an additional argument to the base create_rpc_dispatcher() method. When a manager wants to override create_rpc_dispatcher() to add more callbacks in their own rpc namespaces, it can just call the parent class with the additional APIs to include. Change-Id: I9cba8b176b35f55ba9d71365d0a8bf25d2ae311f
* | | | | | | | | | | | | Merge "powervm: Move tests under tests/virt/powervm"Jenkins2013-05-222-0/+16
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | powervm: Move tests under tests/virt/powervmRick Harris2013-05-212-0/+16
| | |_|_|_|/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I9c70a665bce416a46266ba203481b5eac2f62972
* | | | | | | | | | | | Merge "Catch InstanceNotFound in instance_actions GET"Jenkins2013-05-222-1/+13
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Catch InstanceNotFound in instance_actions GETChris Yeoh2013-05-222-1/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Catch the InstanceNotFound exception in the instance_actions index function and handle the error gracefully so a stacktrace is not generated in the log files. This patch does not change the API Fixes bug 1182867 Change-Id: I0764f8abda7176deefe0d103bd86acd0af78780d
* | | | | | | | | | | | Merge "set ERROR state when scheduler hits max attempts"Jenkins2013-05-222-5/+19
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | set ERROR state when scheduler hits max attemptsRyan Moore2013-05-212-5/+19
| | |/ / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Presently when scheduler raises NoValidHost due to max attempts being reached, the instance remains in a build state. Exception handler for NoValidHost in manager.run_instance() needs request_spec[instance_uuids] to know which host to put into an error state in _set_vm_state_and_notify(). schedule_run_instances() was popping instance_uuids from the request_spec prior to a call to _schedule(). Changed pop of instance_uuids prior to call to _schedule() to be a get. Added pop of instance_uuids to beneath call to _schedule() as individual creates do not need them. Change-Id: I9654820e01d5611763e9e673f15f46b947d09e6d Fixes: bug #1182056
* | | | | | | | | | | | Merge "Don't update API cell on get_nwinfo"Jenkins2013-05-224-21/+28
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Don't update API cell on get_nwinfoChris Behrens2013-05-224-21/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Every call to get_instance_nw_info() updates an instance's info_cache. The purpose of this is to make sure we're constantly healing the info_cache table. However, this causes a lot of unneeded updates traveling up to the API cell. Cells has its own periodic task that heals all instance data. Fixes bug 1180304 Change-Id: I07774712a6358999fd1fdd0f5a90c6f05e3d8d6e
generated by cgit (git 2.34.1) at 2026-03-02 09:13:17 +0000