summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | | | | | | Eliminate race conditions in floating associationVishvananda Ishaya2012-12-205-38/+83
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes associating and disassociating floating ips atomic and idempotent. This means multiple concurrent messages will not leave behind iptables rules and concurrent request will not cause odd failures. Fixes bug 1092762 and bug 1092761. Change-Id: Idbcad6c1d2a3d4881cf7180b848ed3844fac4054
* | | | | | | | | | | | | | Merge "Add html reports to report action in coverage extension."Jenkins2013-01-047-7/+32
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | Add html reports to report action in coverage extension.Matthew Treinish2012-12-207-7/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes a few minor cleanups and 2 additions to the coverage extension. First, it adds a new optional field to the report action: 'html'. This will generate an html report instead of either the text or xml reports that the extension can generate. The second change is that it adds a return for the stop action. Now upon successful completion of the stop action the directory where the coverage data files will be returned. Change-Id: If1aa25fc7237e9bb5100d2a4a8e560f0a68eba61
* | | | | | | | | | | | | | | Merge "Fix error in test_get_all_by_multiple_options_at_once()"Jenkins2013-01-041-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | Fix error in test_get_all_by_multiple_options_at_once()Boris Pavlovic2012-12-281-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We should not use 0 as id. It will conflict with db-unique-keys. Since soft_delete() stores `id` value in `deleted` column, and checking whether entry is deleted softly is made through comparison of `id` and `deleted` column values, created entry will look like already deleted softly. blueprint db-unique-keys Change-Id: Icb7a4880886301a3cc72c6ea0e455b867952812e
* | | | | | | | | | | | | | | | Merge "Fix race condition of resize confirmation"Jenkins2013-01-042-3/+12
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | Fix race condition of resize confirmationWangpan2012-12-192-3/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, the db update of migration status is after the db update of instance's vm_state in _finish_resize, so a race condition occurs when an instance's vm_state is updated but the migration status doesn't, and at this moment a resize confirmation is performed, then an exception 'MigrationNotFoundByStatus' is raised. Fixes: bug #1066140 Change-Id: I1dac98cc99d3491c92faaad2b9dcc40d7c0524b5
* | | | | | | | | | | | | | | | | Merge "Add unit test to update server metadata"Jenkins2013-01-041-0/+16
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | Add unit test to update server metadataMauro S. M. Rodrigues2013-01-021-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug: 1088878 Partially implements blueprint nova-v2-api-audit The bug description points to 4 situations to be tested, although those situations are already tested on create metadata tests which use the same method as update_all to operate over metadata (_update_instance_metadata) so no more tests are needed. Change-Id: Ifae3c69c0e00c361ae36f5a8dede835c96568d11
* | | | | | | | | | | | | | | | | | Merge "Add unit test to revert resize server action"Jenkins2013-01-041-0/+8
|\| | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | Add unit test to revert resize server actionMauro S. M. Rodrigues2013-01-021-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a unit test to check the revert resize to an invaid server Fix bug: 1088874 Partially implements blueprint nova-v2-api-audit Change-Id: I19128ac1a99e85721e7911176fa6d6c78bd5c256
* | | | | | | | | | | | | | | | | | Merge "Fix spelling mistakes in nova.virt"Jenkins2013-01-045-5/+5
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | Fix spelling mistakes in nova.virtJoe Gordon2013-01-025-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ic5ff80af956726b768610290e4fae7dd46549711
* | | | | | | | | | | | | | | | | | | Merge "Remove inconsistent usage of variable from hyperv."Jenkins2013-01-041-27/+22
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | Remove inconsistent usage of variable from hyperv.Sean M. Collins2012-12-211-27/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Portions of the code use instance_name, while others use instance['name']. In some cases, within the same function. See lines 123, 153, and 156. Change-Id: I69763e62a1db79b4c5a4468104fd26ad9b3aad5a
* | | | | | | | | | | | | | | | | | | | Merge "Added sample tests to FlavorDisabled API."Jenkins2013-01-049-1/+311
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | Added sample tests to FlavorDisabled API.Giampaolo Lauria2012-12-179-1/+311
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Partially implements blueprint nova-api-samples Fixes bug 1070161 Change-Id: I889a5b2e85d31fda1e9a8a82043ba7e9ddf1b7a6
* | | | | | | | | | | | | | | | | | | | | Merge "fix resize of unpartitioned images with libguestfs"Jenkins2013-01-042-13/+20
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|_|_|_|_|/ / / / / / / |/| | | | | | | | | | | | | | | | / / / / | | |_|_|_|_|_|_|_|_|_|_|_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | fix resize of unpartitioned images with libguestfsPádraig Brady2013-01-032-13/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Following on from I9c974e138ff90e8b7a5a40f5b31dcdb25a59622d Ensure that the libguestfs path also throws a NovaException, handled by can_resize_fs(). * nova/virt/disk/vfs/guestfs.py (setup): Move the debug message and guestfs handle outside the exception handler as we don't want to map failure of those to a NovaException that indicates an issue with the image as opposed to the nova code or libguestfs installation. Change to a more explicit exception thrown by guestfs, so as to avoid masking other issues. (teardown): Remove redundant calls to str(). * nova/virt/disk/api.py (can_resize_fs): Cleanup the debug messages, and use the newer vfs API directly, rather than the slightly hacky call to inject_data() with no data to inject. Fixes bug: 1094373 Change-Id: I3e1305cf6bb64278a8caf37e4c5005cb9683f632
* | | | | | | | | | | | | | | | | | | | Merge "Enable nova exception format checking in tests."Jenkins2013-01-0428-93/+106
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | Enable nova exception format checking in tests.Dan Prince2012-12-2128-93/+106
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updates the Nova test runner so that it enables 'fatal_exception_format_errors' for testing. Fixes a slew of issues in the tests including: -adds a whole bunch of missing kwarg's to exception constructors -removing an extra print statement in ec2/test_cloud (this was causing failures...) -removing a duplicate (and misnamed) flavor test case: test_remove_tenant_access_with_bad_access -removing an odd test case which tested throwing an exception with and without kwargs explicitly (the part testing without kwargs was removed) -explicitly update test/test_misc.py's test_exceptions_raise to disable formatting checks for now. This allows it to continue to pass. Change-Id: Ibb5083c93f1a5de6edadb1b613292ff9eb26ddf0
* | | | | | | | | | | | | | | | | | | | | Merge "xenapi: Avoid hotplugging volumes on resize."Jenkins2013-01-046-28/+119
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | | xenapi: Avoid hotplugging volumes on resize.Cory Stone2013-01-036-28/+119
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In finish_migration, instead of starting up the vm and then hotplugging in the volumes to it, just attach the volumes before starting the vm. This avoids the issue where the guest OS hasn't loaded the PV drivers for the volume yet. Fixes bug 1094351 Change-Id: I51d754f8f82f1d22bc123b39777449b58b03e389
* | | | | | | | | | | | | | | | | | | | | | Merge "Imported Translations from Transifex"OpenStack Jenkins2013-01-031-337/+435
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | | | Imported Translations from TransifexOpenStack Jenkins2013-01-031-337/+435
| | |_|_|_|_|_|_|_|_|_|_|_|/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ifb57826e0cfe1c94511c86641b6392f042c4b866
* | | | | | | | | | | | | | | | | | | | | | Merge "Removes unused imports"Jenkins2013-01-037-18/+3
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | | Removes unused importsZhongyue Luo2013-01-037-18/+3
| |/ / / / / / / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I76d243329f810cdcd6f8a5459eb66fddd5871e33
* | | | | | | | | | | | | | | | | | | | | Merge "Clarify & fix docs for nova-novncproxy"Jenkins2013-01-032-7/+7
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | | Clarify & fix docs for nova-novncproxyDaniel P. Berrange2013-01-032-7/+7
| | |_|_|_|_|_|_|_|_|_|_|_|_|_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Clarify docs for nova-novncproxy to indicate that it only serves noVNC web content. Also remove duplicated link from manpage Change-Id: Ia1f4aff21591132a5c41810a371d248ea0001e1f
* | | | | | | | | | | | | | | | | | | | | Merge "Add remaining get_backdoor_port() rpc calls to coverage."Jenkins2013-01-031-0/+15
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | Add remaining get_backdoor_port() rpc calls to coverage.Matthew Treinish2013-01-021-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds the get_backdoor_port() rpc calls for cert, conductor, console, consoleauth, and scheduler so that the coverage extension can now telnet into all of those services. Change-Id: I5de654a11ac224b1192a3b1601855c302b771efd
* | | | | | | | | | | | | | | | | | | | | Merge "Provide a configdrive helper which uses contextlib."Jenkins2013-01-036-33/+42
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | Provide a configdrive helper which uses contextlib.Michael Still2012-12-216-33/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As suggested on review of Ib23d117ad4cd5dc92298a0812eb468f7d557417c. Resolves bug 1092248. Change-Id: I2829ac60732b86b9853983b03ef6f30f5c5a3283
* | | | | | | | | | | | | | | | | | | | | Merge "Improve baremetal driver error handling"Jenkins2013-01-028-107/+160
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | | Improve baremetal driver error handlingDevananda van der Veen2013-01-028-107/+160
| | |_|_|_|_|_|_|_|_|_|_|_|_|_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Several improvements to baremetal driver are implemented in this patch. There is now significantly more error handling during spawn(). It also includes an addition to nova/tests/utils.py to provide additional sample information from get_test_network_info(). blueprint general-bare-metal-provisioning-framework Change-Id: I65d93051d7fcfd79f4d24d4ddb62fb1a55bee646
* | | | | | | | | | | | | | | | | | | | | Merge "Remove unused imports"Jenkins2013-01-028-10/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | | Remove unused importsJoe Gordon2012-12-278-10/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | And one unused variable Change-Id: I81edaf7ddedd7723c2601b30951ca82bd2cd5256
* | | | | | | | | | | | | | | | | | | | | | Merge "Check for image_meta in libvirt.driver.spawn"Jenkins2013-01-022-1/+35
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | | Check for image_meta in libvirt.driver.spawnKravchenko Pavel2012-12-272-1/+35
| |/ / / / / / / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds check that image_meta exists in libvirt driver spawn method. In case it doesn't the image shouldn't be created. part of libvirt driver support for blueprint rebuild-for-ha DocImpact Change-Id: I097c7cbfde96bcb21b4f06af4e5bac00440849c2 Co-authored-by: Oshrit Feder <oshritf@il.ibm.com>
* | | | | | | | | | | | | | | | | | | | | Merge "Remove session.flush() and session.query() monkey patching"Jenkins2013-01-022-9/+12
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | Remove session.flush() and session.query() monkey patchingBoris Pavlovic2012-12-282-9/+12
| | |_|_|_|_|_|_|_|_|_|_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use custom Session class to avoid monkey patching of SqlAlchemy Session instance. blueprint db-session-cleanup Change-Id: I7d40c20e585782403556f3587141805c197bfa64
* | | | | | | | | | | | | | | | | | | | Merge "Remove improper NotFound except block in list servers"Jenkins2013-01-022-6/+24
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | Remove improper NotFound except block in list serversJay Pipes2012-12-262-6/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added a couple tests to verify that NotFound was actually never raised when no results are returned from list servers or list servers details, and then removed the except blocks from the controller. Change-Id: I36856066b9e4c4e3f7b85ddf8facb35204034838 fixes: lp bug #1089385
* | | | | | | | | | | | | | | | | | | | | Merge "Raise BadRequest when updating 'personality'"Jenkins2013-01-022-0/+18
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | | Raise BadRequest when updating 'personality'Janis Gengeris2013-01-012-0/+18
| | |_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updating booted server instance through server PUT API call with 'personality' property set is not returning 'Bad Request', although the property is not allowed to be updated once the instance is booted. This fixes the described problem. Fixes bug #1032546 Change-Id: Ia086f5ea5176640a9916a3ba42c79d4d5944ce76
* | | | | | | | | | | | | | | | | | | | | Merge "Add unit test for too long metadata for server rebuild action"Jenkins2013-01-021-0/+15
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |_|_|_|_|_|_|_|_|_|_|_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | Add unit test for too long metadata for server rebuild actionMauro S. M. Rodrigues2013-01-021-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes bug: 1088686 Partially implements blueprint nova-v2-api-audit Change-Id: Ibf2b468cb95e0b1be9f47c0ff01ae55ce56d421d
* | | | | | | | | | | | | | | | | | | | | Merge "Adds os-volume_attachments 'volume_id' validation"Jenkins2013-01-022-0/+30
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|_|_|_|/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | Adds os-volume_attachments 'volume_id' validationJanis Gengeris2013-01-022-0/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If you POST to os-volume_attachments but give the name of the volume instead of a UUID, it returns a 404 error; this is invalid HTTP. 404 means that the Request-URI has not been found. In fact, the Request-URI has been found, but a child attribute has been incorrectly specified. Some other error could be used. This fixes the problem by returning 'BadRequest' in place of 'NotFound'. The 'volumeId' is checked to be a valid UUID string before moving further. Fixes bug #1062494 Change-Id: Icc5dbc7ac94051514709997457cafb16e870bea9
* | | | | | | | | | | | | | | | | | | | | Merge "Report failures to mount in localfs correctly."Jenkins2013-01-024-8/+101
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
generated by cgit (git 2.34.1) at 2026-02-16 20:05:18 +0000