summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Remove white space between print and ()yuyafei2016-08-023-6/+6
| | | | | | TrivialFix Change-Id: Ice05cc9848bc0b647ee20cfad20c23b47f8fedbd
* Merge "Updated from global requirements"Jenkins2016-08-011-1/+1
|\
| * Updated from global requirementsOpenStack Proposal Bot2016-08-011-1/+1
| | | | | | | | Change-Id: I2de707122245ed7933b4a53095f3fbc17448220e
* | Merge "Fix typo in DeviceDetachFailed exception message"Jenkins2016-08-011-1/+1
|\ \
| * | Fix typo in DeviceDetachFailed exception messageMatt Riedemann2016-08-011-1/+1
| |/ | | | | | | | | | | The trailing ) makes the error message confusing. Change-Id: I73cfe1127c6b8acc5d438acf7c11699a11a5be69
* | Merge "Initialise oslo.privsep early in main"Jenkins2016-08-012-0/+4
|\ \
| * | Initialise oslo.privsep early in mainAngus Lees2016-08-012-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Any process using oslo.privsep should now initialise the library before first use with things like the rootwrap command to use. This should be done near the top of main() in any command that expects to make privileged calls via oslo.privsep (eg: nova-compute, and not nova-api). See I3ea73e16b07a870629e7d69e897f2524d7068ae8 for the corresponding change in oslo.privsep. Change-Id: I3a52f762deb176fe9201b2a0f0da363057f8aaec Depends-On: I52259e2023e277e8fd62be5df4fd7f799e9b36d7 Closes-Bug: #1592043
* | | Merge "Add the 'min' param to IntOpts where applicable"Jenkins2016-08-011-1/+8
|\ \ \
| * | | Add the 'min' param to IntOpts where applicableEdLeafe2016-07-261-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was pointed out [0] that any IntOpt with a logical minimum value should have that specified by including the 'min=' parameter in the option definition. As that patch was concerned with the help text for options, it was felt that a separate follow-up patch would be better for adding these parameters. [0] https://review.openstack.org/#/c/281901/7/nova/conf/network.py@259 Blueprint centralize-config-options-newton Change-Id: I0dbef37736f72401268f3036f77f35a4bbf2b5b7
* | | | Merge "Mention proxy API deprecation microversion in api-ref"Jenkins2016-08-0114-12/+15
|\ \ \ \
| * | | | Mention proxy API deprecation microversion in api-refghanshyam2016-08-0114-12/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Proxy APIs are deprecated in microversion 2.36 which means all proxy APIs will return 404 after 2.35 version. It will be nice to mention that explicitly in api-ref too. Partially implements blueprint deprecate-api-proxies Change-Id: Iad210d215fb98bc247f049b23b97d8d7d4265c82
* | | | | Make Aggregate.save work with the API dbMark Doffman2016-08-013-8/+132
| |_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Aggregate.save now updates aggregates if they are located in the api db. blueprint cells-aggregate-api-db Co-Authored-By: Dan Smith <dansmith@redhat.com> Change-Id: I6066857a421bfe489ae70f0e22f338c434195bf9
* | | | Merge "Fixed typos in nova, nova/api, nova/cells directory"Jenkins2016-08-015-5/+5
|\ \ \ \
| * | | | Fixed typos in nova, nova/api, nova/cells directoryAlex Szarka2016-08-015-5/+5
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patchset fixes the typo errors in nova root(non recursively), nova/api, nova/cells and nova/cmd directory. TrivialFix Change-Id: Id4e5dbeafb6fdb63bb733eef203a676f94143f32
* | | | Merge "Add initial framing of placement API"Jenkins2016-08-0120-0/+825
|\ \ \ \
| * | | | Add initial framing of placement APIChris Dent2016-07-2920-0/+825
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The placement API will initially host resource provider information and then will grow to provide a full placement service. Eventually it will be fully extracted from Nova. To facilitate that extraction, this service is being developed with few ties to existing nova.wsgi structures. Instead it uses relatively plain WSGI apps that are: * uses the Routes library with declarative mapping defined in ROUTE_DECLARATIONS * basic wsgi apps, with webob and the request class, for each handler define as functions * does not use a paste-ini file to compose middleware, instead code which minimally inspects the oslo config to know how to adjust middleware (in the initial case choosing an auth_strategy) Many of these "features" will be demonstrated in commits that follow this one that enable specific behaviors for resource providers and their inventories. In order to ensure that this change can be merged in an atomic fashion it includes microversion support that was previously in its own commit. The microversion_parse library is used in a WSGI middleware to parse incoming microversion headers and place the determined value into the WSGI environment at the 'placement.microversion' key. Response headers are adjusted to add the required outgoing headers as described in: http://specs.openstack.org/openstack/api-wg/guidelines/microversion_specification.html If a client requests an invalid microversion, they will receive a 400 response. If the microversion is of a valid form but not available, they will received a 406 response. The format of that response is dependent on the accept header of the request. If it is JSON, the min and max available microversions are presented. A request to '/' will return version discovery information. Thus far nothing else is done with the microversion information. It is there for when we need it in the future. For now everything is version 1.0. The groundwork for using gabbi to test the API is in place in nova/tests/functional/api/openstack/placement. The gabbi tests are included in the functional target. To run just the placement tests it is possible to run 'tox -efunctional placement'. Change-Id: Icb9c8f7a1fa8a9eac66c2d72f4b7e4efd4e1944f Partially-Implements: blueprint generic-resource-pools
* | | | | Trivial: add 'DEPRECATED' for os-certificates API refghanshyam2016-08-011-3/+3
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | os-certificates API has been deprecated in I0ffa9200d8b4f52da4751e731aecffe09fd3d4be This patch just add 'DEPRECATED' in its heading in api-ref to make it more clear. Change-Id: I5ef30330e033eeb0576d3b4d31441d5481cc7d2e
* | | | Merge "Option Consistency for availability_zone.py"Jenkins2016-08-011-5/+0
|\ \ \ \
| * | | | Option Consistency for availability_zone.pystewie9252016-07-271-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was already partially completed in a previous commit de2df72cd1f97f38e6c20f5a7b099b16454e9162. blueprint centralize-config-options-newton Change-Id: If74607ccc90e373608007907b5e81d6ff5eba70c
* | | | | Merge "Deprecate barbican options"Jenkins2016-07-303-26/+59
|\ \ \ \ \
| * | | | | Deprecate barbican optionsSarafraj Singh2016-07-293-26/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. Deprecating barbican options as these are moved to Castellan library. 2. Added new formatting Blueprint centralize-config-options-newton Change-Id: Ic8bd86e2652b7702c039ea1d2e15a7bf5a2a9586
* | | | | | Merge "Free new pci_devices on revert-resize"Jenkins2016-07-302-5/+68
|\ \ \ \ \ \
| * | | | | | Free new pci_devices on revert-resizeMoshe Levi2016-07-292-5/+68
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On confirm-resize the drop_move_claim is called with prefix="old_" which means it should drop the old flavor allocations. On revert-resize the drop_move_claim is called with prefix="new_" which means it should drop the new flavor allocations. This patch update that the drop_move_claim will drop pci devices according to the prefix Co-Authored-By: Ludovic Beliveau <ludovic.beliveau@windriver.com> Closes-Bug: #1607219 Change-Id: I4601f89d8ab23db19b7515d0ad9779638ec66b49
* | | | | | | Merge "numa: remove the redundant check for hw_cpu/hw_mem list"Jenkins2016-07-291-6/+2
|\ \ \ \ \ \ \
| * | | | | | | numa: remove the redundant check for hw_cpu/hw_mem listChen Fan2016-07-281-6/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | we use a hw_numa_mem/cpu_set to check the hw_numa_mem/cpu list is empty or not, actually, we only need to use 'if' to check the list if empty. don't need the redundant flag. so remove it. Change-Id: If81c842a05eecd5257c59c9324c3341be803e3d8
* | | | | | | | Merge "Add support for oslo.context 2.6.0"Jenkins2016-07-292-0/+40
|\ \ \ \ \ \ \ \
| * | | | | | | | Add support for oslo.context 2.6.0Tony Breeds2016-07-272-0/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The 2.6.0 release of oslo.context adds a new attribute 'is_admin_project'. Always include that attribute when serializing the object, and expect it in our tests. Change-Id: I563cf810385e84ab30d49ef079b75df279006f0d Related-Bug: 1602081
* | | | | | | | | Merge "Add missing ComputeHostNotFound exception in live-migration"Jenkins2016-07-295-1/+39
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Add missing ComputeHostNotFound exception in live-migrationPawel Koniszewski2016-07-295-1/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit c824982e6a3d6660697e503f7236377cc8202d41 did not update expected exceptions. Therefore we end up with 500 internal server error when triggering targeted live migration to non-existing compute node. This patch adds ComputeHostNotFound to expected exception list in both, conductor and API. Change-Id: If515a90217a8e329d932dcacb357b78081c505c1 Related-bug: 1538837
* | | | | | | | | | Merge "Fixed typos in api-ref and releasenotes directory"Jenkins2016-07-296-16/+26
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Fixed typos in api-ref and releasenotes directoryAlex Szarka2016-07-206-16/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patchset fixes the typo errors in api-ref and releasenotes directories. TrivialFix Change-Id: Ib72e5c4ac7539c1b08adc858b9b3a0071cb3c866
* | | | | | | | | | | Merge "Update RequestSpec nested flavor when a resize comes in"Jenkins2016-07-294-10/+78
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Update RequestSpec nested flavor when a resize comes inSylvain Bauza2016-07-264-10/+78
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was a regression (because of a PEBKAC) when we implemented calling the original RequestSpec because we weren't updating the flavor so the scheduler was verifying the original flavor instead of the new one. Change-Id: I8abdf58a6537dd5e15a012ea37a7b48abd726579 Closes-Bug: #1606269
* | | | | | | | | | | | Merge "Updated from global requirements"Jenkins2016-07-291-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Updated from global requirementsOpenStack Proposal Bot2016-07-291-1/+1
| | |_|_|_|_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I004a05132eb9a7cd7ccccfbd4fdda8f5ee43c518
* | | | | | | | | | | | Merge "libvirt: Fix invalid test data"Jenkins2016-07-291-49/+47
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | libvirt: Fix invalid test dataMatthew Booth2016-07-271-49/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix up a couple of classes of invalid test data: * Invalid backing file names: filenames which don't correspond to their instance. * Invalid uuids. This change is not intended to be comprehensive: it only touches tests which are affected by the storage refactor series. Change-Id: I08daffdfd6eedfe84c7f1b010ab0fa204074ffec
* | | | | | | | | | | | | Merge "libvirt: Fix fake _disk_info data in LibvirtDriverTestCase"Jenkins2016-07-291-70/+132
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / /
| * | | | | | | | | | | | libvirt: Fix fake _disk_info data in LibvirtDriverTestCaseMatthew Booth2016-07-271-70/+132
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fake was returning dicts which were missing backing_file and over_committed_disk_size. This change adds the missing fields with their correct default values. We also move it out of LibvirtDriverTestCase to the top level so tests in other suites can use it in future patches. The data returned was also not related to the instance being tested. This meant that _test_finish_migration was explicitly testing for the wrong behaviour, as it was expecting a swap disk to be created for an instance which did not have one. We also update LibvirtDriverTestCase._create_instance to use fake_instance.fake_instance_obj, which helps avoid some accidental DB access in tests. Change-Id: If2ebba557b05330387141647049cf9a5cbe6aa34
* | | | | | | | | | | | | Merge "Add docs about microversion testing in Tempest"Jenkins2016-07-292-4/+14
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|/ / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Add docs about microversion testing in TempestMatt Riedemann2016-07-282-4/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As discussed at the newton midcycle and in the dev mailing list: http://lists.openstack.org/pipermail/openstack-dev/2016-July/099860.html We should add a Tempest test for any microversion that changes the response schema so we have coverage in Tempest and also so we don't have to fill large gaps in coverage in Tempest when adding other unrelated tests. Change-Id: Ie7cfe7ee857caf630d4380cf673ae208842fbc00
* | | | | | | | | | | | | Merge "Remove the useless version calculation for proxy api deprecated version"Jenkins2016-07-293-29/+11
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Remove the useless version calculation for proxy api deprecated versionHe Jie Xu2016-07-283-29/+11
| | |_|_|_|_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch use constant instead of the calculation for the proxy API deprecated version. This is more readable. Partially implements blueprint deprecate-api-proxies Change-Id: I5e9972643a7dd2e2559f7f9593b85618df2647c3
* | | | | | | | | | | | Merge "Config options consistency for configdrive.py"Jenkins2016-07-291-7/+0
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Config options consistency for configdrive.pyAnusha Unnam2016-07-261-7/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Checked the file for fix_opt_description, check_deprecation_status, check_opt_group_and_type consistency and removed the tags according to wiki. https://wiki.openstack.org/w/index.php?title=ConfigOptionsConsistency Change-Id: I0bded86f6de43742cbcc2aefb4328b120b949327
* | | | | | | | | | | | Merge "Make Aggregate metadata functions work with API db"Jenkins2016-07-293-8/+221
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Make Aggregate metadata functions work with API dbMark Doffman2016-07-263-8/+221
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds the ability to update aggregate metadata where the aggregate is contained in the API db. Modifications will be made to the 'aggregate_metadata' table in the API db if that is where the aggregate is located or the cell database otherwise. blueprint cells-aggregate-api-db Change-Id: I456beb753298f4aabbed6d20bea7cbc537842e50
* | | | | | | | | | | | | Merge "Fixes compute API unit tests for python3"Jenkins2016-07-282-12/+17
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Fixes compute API unit tests for python3Scott Wilson2016-06-302-12/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Use response.text (string) rather than response.body (bytes) - Look directly at the Content-Type header, avoiding response.content_type property which assumes string header values. Change-Id: I8fe1a6d454a0c45ba7ef58b9fa1367639650a297 Partially Implements: blueprint nova-python3-newton
* | | | | | | | | | | | | | Merge "Enable python34 tests for nova.tests.unit.pci.test_manager and ↵Jenkins2016-07-286-26/+31
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test_stats"