summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Batch up minor cleanups for release"3.19.0Zuul2019-06-065-7/+7
|\
| * Batch up minor cleanups for releaseDean Troyer2019-05-205-7/+7
| | | | | | | | | | Change-Id: Id45788e17c5388cee54e79cab1c120cfcc8f9f62 Signed-off-by: Dean Troyer <dtroyer@gmail.com>
* | Merge "Deprecate openstack server migrate --host option"Zuul2019-06-063-8/+255
|\ \
| * | Deprecate openstack server migrate --host optionMatt Riedemann2019-05-243-8/+255
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Per the discussion at the Train Forum [1] this deprecates the problematic --live option on the server migrate command which, depending on the compute API version used, forcefully bypasses the scheduler and also does not allow you to live migrate a server and let the scheduler pick a host. The --live option is replaced here with two new options: * --live-migration: this simply tells the command you want to perform a live rather than cold migration; if specified with --live the --live-migration option takes priority. * --host: when specified, this will request a target host for the live migration and will be validated by the scheduler; if not specified, the scheduler will pick a host. This option is mutually exclusive with --live. We can build on the --host option by supporting cold migrations with a specified host when using compute API version 2.56 or greater but that will come in a separate change. If the --live option is ever used we log a warning. Note there are several related changes for this issue: - https://review.openstack.org/#/c/628334/ - https://review.openstack.org/#/c/626949/ - https://review.openstack.org/#/c/627801/ - https://review.openstack.org/#/c/589012/ - https://review.openstack.org/#/c/460059/ This change allows us to deprecate the --live option and provide a replacement which is backward compatible without having to use something potentially error-prone like nargs='?'. Closes-Bug: #1411190 [1] https://etherpad.openstack.org/p/DEN-osc-compute-api-gaps Change-Id: I95d3d588e4abeb6848bdccf6915f7b5da40b5d4f
* | | Merge "Add changes-before attribute to server list"Zuul2019-06-064-2/+175
|\ \ \
| * | | Add changes-before attribute to server listzhangbailin2019-05-314-2/+175
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes-Bug: #1827844 Part of bp support-to-query-nova-resources-filter-by-changes-before Change-Id: I4f28168188973730247bcbcb70ba0e70eb81e3be
* | | | Merge "Compute: Add description support for server"Zuul2019-06-054-1/+288
|\ \ \ \ | |/ / / |/| | |
| * | | Compute: Add description support for serverChen2019-05-174-1/+288
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds functionality to configure server's description with: 1 server create 2 server set 3 server unset 4 server rebuild Change-Id: Ic06d97b29e51828b29d7ac5172645c288e4ada9e Story: 2002005 Task: 19640
* | | | Merge "Remove code migrated to osc-lib long ago"Zuul2019-05-1810-377/+5
|\ \ \ \
| * | | | Remove code migrated to osc-lib long agoDean Troyer2019-05-1610-377/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Remove openstackclient.api.utils and use osc_lib.api.utils * Remove openstackclient.common.clientmanager.ClientManager.auth_ref * Remove openstackclient.common.commandmanager Change-Id: I67e1dbc53cc0b37967c0011bcb2fc09bdef62d94 Signed-off-by: Dean Troyer <dtroyer@gmail.com>
* | | | | Merge "Serialize more aggregate functional tests"Zuul2019-05-182-98/+52
|\ \ \ \ \
| * | | | | Serialize more aggregate functional testsDean Troyer2019-05-172-98/+52
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These tests are showing signs of problems running in parallel so serialse the create/delete/list/set/unset tests. They all used two aggregates each anyway... Change-Id: Iba4b52c179e6914eaeefea1da0f7eaefcdcf1f87 Signed-off-by: Dean Troyer <dtroyer@gmail.com>
* | | | | Merge "Fix RuntimeError when showing project which has extra properties"Zuul2019-05-172-1/+6
|\ \ \ \ \
| * | | | | Fix RuntimeError when showing project which has extra propertiesYang Youseok2018-06-092-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If you use python3, items() returns iterator which is not allowed to remove item during iteration. Fix to iterate by copied list. Change-Id: I64c037d04e2b127d8f19f56cab65122af89a7200 Closes-Bug: 1740232
* | | | | | Merge "Fix bug in endpoint group deletion"Zuul2019-05-174-3/+565
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Fix bug in endpoint group deletionJose Castro Leon2019-05-084-3/+565
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is a typo in the endpoint group deletion, due to this you can't remove endpoint groups once assigned. I am adding also the unit tests to avoid this kind of issues in the future Task: 30640 Story: 2005521 Change-Id: Ie938f2c9894bb39b4c0ed1f7aa3a6a751a303058
* | | | | | Merge "Aggregate functional test tweak"Zuul2019-05-171-2/+3
|\ \ \ \ \ \
| * | | | | | Aggregate functional test tweakDean Troyer2019-05-161-2/+3
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This seems to still be racy, lengthen the timeout to wait for agregate creation. Change-Id: I3601c5baee03745ae21714b9dff0e278ad016877 Signed-off-by: Dean Troyer <dtroyer@gmail.com>
* | | | | | Merge "Update sphinx requirement."Zuul2019-05-161-1/+2
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | |
| * | | | | Update sphinx requirement.Natal Ngétal2019-05-151-1/+2
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | Sphinx 2.0 no longer works on python 2.7, start cappingit there as well. Change-Id: I8a7d227b2f925066fc8213aa62b5756927ee263b
* | | | | Stop leaving temp files after unit test runsBrian Haley2019-05-102-19/+37
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test_shell.CLOUD_2 is using an absolute path for a temp file, so leaves /tmp/test_log_file around after the unit tests are run. Use a fixture instead so it's cleaned automatically, which also removes the possibility of two tests using the same file and interfering with each other. Change-Id: If722b860be4010b91635c6d46f634da980e17152
* | | | Merge "document the --timing option"Zuul2019-05-101-0/+4
|\ \ \ \
| * | | | document the --timing optionGuang Yee2019-05-081-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I2d13088ea026ac7288213fe808874c4a3a81313a Story: #2005315 Task: #30863
* | | | | Blacklist Bandit 1.6.0 due to directory exclusion bugDean Troyer2019-05-091-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bandit 1.6.0 introduces a regression[0] with the -x option, a fix is expected to be included in 1.6.1 soon. [0] https://github.com/PyCQA/bandit/issues/488 [1] https://github.com/PyCQA/bandit/pull/489 Change-Id: I110829ef960e3ee146f47871ef076491244bf4fa Signed-off-by: Dean Troyer <dtroyer@gmail.com>
* | | | | Merge "Fix --limit option in image list sub-command"Zuul2019-05-083-4/+15
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Fix --limit option in image list sub-commandRadoslaw Smigielski2019-02-043-4/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Client site fix of --limit option. This bugfix makes client "image list" command working again with "--limit" option. This option was ignored and even if user specified it, still list of all available images was returned. Story: 2004314 Change-Id: I30a78d65a644c9b7d23706a6637ce77bca2c2386 Depends-On: https://review.openstack.org/#/c/634776/
* | | | | Merge "Dropping the py35 testing"Zuul2019-05-033-7/+5
|\ \ \ \ \
| * | | | | Dropping the py35 testingGhanshyam Mann2019-04-223-7/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All the integration testing has been moved to Bionic now[1] and py3.5 is not tested runtime for Train or stable/stein[2]. As per below ML thread, we are good to drop the py35 testing now: http://lists.openstack.org/pipermail/openstack-discuss/2019-April/005097.html [1] http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004647.html [2] https://governance.openstack.org/tc/reference/runtimes/stein.html https://governance.openstack.org/tc/reference/runtimes/train.html Change-Id: Ie7bcc327fd588a1ff6b2556d49017df56bc55bf8
* | | | | | Merge "Document that server dump create requires 2.17"Zuul2019-05-031-0/+2
|\ \ \ \ \ \
| * | | | | | Document that server dump create requires 2.17Matt Riedemann2019-05-011-0/+2
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is no indication to the user in the command help that they have to use 2.17 or greater [1] to run the "openstack server dump create" command. This mentions that requirement in the help of the command. [1] https://developer.openstack.org/api-ref/compute/#trigger-crash-dump-in-server Change-Id: I02c06e10a26eb38ddecb70f970cfcbfad962201c
* | | | | | Fix link to new opendev repoAdam Spiers2019-05-011-1/+1
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | The switch to an opendev URL wasn't quite right, resulting in a 404. Change-Id: I652f093384a584a56290a9b080913392873efd9f
* | | | | Merge "Fix docs bug link to go to storyboard rather than launchpad"Zuul2019-04-243-5/+3
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Fix docs bug link to go to storyboard rather than launchpadMatt Riedemann2019-04-163-5/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes the docs bug link generation for the normal docs and release notes docs. The requirement on openstackdocstheme is bumped to 1.23.2 to pick up fix I2ed164b9b0badade702c50543ac1a5eea4d1867b. Change-Id: I89711a391ee0fb7e40c1fbf83f950e2b582358d9 Story: #2005467 Task: #30546
* | | | | Followup opendev cleanup and test jobsDean Troyer2019-04-193-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * upper-constraints references need s/plain/raw/ Change-Id: I04368dc42f1a62a048ac9d11497747ef6f600515 Signed-off-by: Dean Troyer <dtroyer@gmail.com>
* | | | | OpenDev Migration PatchOpenDev Sysadmins2019-04-192-6/+6
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit was bulk generated and pushed by the OpenDev sysadmins as a part of the Git hosting and code review systems migration detailed in these mailing list posts: http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.html http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004920.html Attempts have been made to correct repository namespaces and hostnames based on simple pattern matching, but it's possible some were updated incorrectly or missed entirely. Please reach out to us via the contact information listed at https://opendev.org/ with any questions you may have.
* | | | Merge "Tweak network segment range fiunction tests"Zuul2019-04-151-8/+8
|\ \ \ \
| * | | | Tweak network segment range fiunction testsDean Troyer2019-04-121-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We seem to be having occasional overlaps in the ranges, as they were identical in all tests, change each test to not overlap the others so running in parallel is not racy. Change-Id: I7ea467a3aa2e4a4b4a334c10ea6ba21409c46af0 Signed-off-by: Dean Troyer <dtroyer@gmail.com>
* | | | | Merge "Before writing object data to stdout, re-open it in binary mode"Zuul2019-04-133-4/+29
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Before writing object data to stdout, re-open it in binary modeTim Burke2019-04-103-4/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Otherwise, you can hit TypeErrors on Python3. Change-Id: I9a891508886feddac3982ce593bd95130392e035 Closes-Bug: 1775482
* | | | | Merge "Volume backup functional test tweak"Zuul2019-04-051-2/+3
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Volume backup functional test tweakDean Troyer2019-04-021-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Waiting for status in all the wrong places... Change-Id: I531ee6e0c00b623c6fd30d40df1f1f36bf86233f Signed-off-by: Dean Troyer <dtroyer@gmail.com>
* | | | | Ignore case in security group rule --ethertypeJim Rollenhagen2019-04-012-0/+33
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, this only allows 'IPv4' or 'IPv6', but one can imagine a user frequently typing e.g. 'ipv6' and getting frustrated. Allow any case, while still keeping correct case for the choices and the value sent to Neutron. Change-Id: I70ce1f43d32aad01b174437d03c984a5b608b161
* | | | Update master for stable/steinOpenStack Release Bot2019-03-222-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add file to the reno documentation build to show release notes for stable/stein. Use pbr instruction to increment the minor version number automatically so that master versions are higher than the versions on stable/stein. Change-Id: I3edfae7c1c5f8268186455efc7add28dc38810fb Sem-Ver: feature
* | | | Merge "Fix: incorrect check when no shared/private input"Zuul2019-03-162-2/+51
|\ \ \ \
| * | | | Fix: incorrect check when no shared/private inputKailun Qin2019-03-142-2/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When neither of "--shared" and "--private" is input, we should not allow to specify "--project". Defaulting the created network segment range to shared is expected. Therefore, "project_id" attr should only be populated on a private range creation. Change-Id: Iab345e1651dd8b7904ff64a20633f194d719bb84 Story: 2005206 Task: 29980
* | | | | Merge "Fix service discovery in functional tests"Zuul2019-03-159-41/+41
|\ \ \ \ \
| * | | | | Fix service discovery in functional testsGlenn Van de Water2019-03-139-41/+41
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a required service is not enabled then we skip the test. The discovery is done by tests/functional/base.py:is_service_enabled but this method is broken, credentials are not passed to the 'openstack service show' command so every call will fail and every test that relies on it will be skipped. This commit fixed that method and the issues that popped up when re-enabling tests. Network segment range: - issue where we assumed network-segment-range extension is always present - issue where we compare integers and string representations of numbers Subnet: - issue where we try to deepcopy an uncopyable object in UnsetSubnet Change-Id: Id3cc907c1ed2a25b49cf6f4a7233e0401a02383a Story: 2005169 Task: 29908
* | | | | Merge "Fix: set invalid None project_id on range creation"Zuul2019-03-152-8/+36
|\ \ \ \ \ | |/ / / /
| * | | | Fix: set invalid None project_id on range creationKailun Qin2019-03-132-8/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "project_id" attribute should not be set to None on shared network segment range creation since it is not a valid string type which is required for the API. Change-Id: Ia2bab12e39b4bb7e05ff2acfffb851252c100651 Story: 2005205 Task: 29975
* | | | | Merge "Delete the LB object quotas set command in openstackclient"Zuul2019-03-133-15/+0
|\ \ \ \ \ | |/ / / / |/| | | |