summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Add Python 3.6 classifier to setup.cfgqingszhao2018-11-301-0/+1
| | | | Change-Id: Id723040f895f06527d46755dd408dec16200c001
* Merge "Updated the take_actions for unified limits"Zuul2018-11-293-12/+84
|\
| * Updated the take_actions for unified limitsVishakha Agarwal2018-11-093-12/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When user passes --region None, the find_resource of osc_lib calls get() of region. The get API of region ignores the name param returning all the regions in result. As the find_resource checks many cases against the result returned by get API. The output comes greater than 1, thus returning "More than one region ID exist" which is incorrect. However in case of region which cannot be filtered by name we do not require to check these many cases. The solution is to directly call the get method of APIs and returning No resource name exist with the xyz" on passing invaid parameter. And returning all in case of None. Thus created a new function get_resource which can be used in future too by these types of API's. Change-Id: Ib3f881d34a82af97199ce51bfbefc6f3f08599f1 Closes-bug: #1799153
* | Fix i18n issueliuyamin2018-11-131-1/+1
| | | | | | | | | | | | This patch fix some i18n issues in the files vapi/compute_v2.py. Change-Id: Ic4da472ca585a35ce64512cf0e72e2fe9d4c9d6e
* | Add volume backup import/export commandsSean McGinnis2018-11-108-197/+250
|/ | | | | | | | | | | | | | | | | | This adds commands to import and export volume backup records so they can be imported and restored on other Cinder instances or to the original instance if the service or database has been lost and had to be rebuilt. I know this is a commonly used process by some users, so it would be good to have this functionality in osc so they do not have to switch clients. More details about the export and import process can be found here: https://docs.openstack.org/cinder/latest/admin/blockstorage-volume-backups-export-import.html Change-Id: Ic95f87b36a416a2b50cb2193fd5759ab59336975 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* Merge "Add project param in LimitList parser"Zuul2018-11-084-2/+23
|\
| * Add project param in LimitList parser3.17.0Vishakha Agarwal2018-11-084-2/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | when doing openstack limit list --project xyz_id, CLI raising error unrecognized arguments, whereas in api-ref document [1], user can pass project_id as query param.This addresses the above issue, by adding param --project in parser of LimitList. [1] https://developer.openstack.org/api-ref/identity/v3/index.html Change-Id: If4644cc99a3803f61f4a688b828aeb73977fc0dd Closes-Bug: #1798744
* | Merge "Remove invalid 'unlock-volume' migration arg"Zuul2018-11-084-41/+7
|\ \
| * | Remove invalid 'unlock-volume' migration argSean McGinnis2018-10-264-41/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is an optional flag that can be passed in to a volume migration to tell Cinder to 'lock' a volume so no other process can abort the migration. This is reflected correctly with the --lock-volume argument flag to `openstack volume migrate`, but there is another --unlock-volume flag that is shown in the help text for this command that does not do anything and is not used anywhere. Since there is no action to "unlock" a volume, this just causes confusion - including for Cinder developers that know this API. To avoid confusion, this invalid flag should just be removed from the command. Change-Id: I5f111ed58803a1bf5d34e828341d735099247108
* | | Merge "Add --name-lookup-one-by-one option to server list"Zuul2018-11-083-23/+91
|\ \ \
| * | | Add --name-lookup-one-by-one option to server listPavlo Shchelokovskyy2018-11-023-23/+91
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | usually in a big cloud there are many images and flavors, while each given project might use only some of those. This patch introduces '--name-lookup-one-by-one' argument to server list command (mutually exclusive with '--no-name-lookup') When provided (or either '--image' or '--flavor' is specified) to the `server list` command, name resolving for corresponding entity is now using targeted GET commands instead of full entities list. In some situations this can significantly speedup the execution of the `server list` command by reducing the number of API requests performed. Change-Id: I59cbf3f75c55e5d3747654edcc9be86ad954cf40 Story: #2002039 Task: #19682
* | | | Merge "image/v2: support multiple property filters"Zuul2018-11-082-9/+15
|\ \ \ \
| * | | | image/v2: support multiple property filtersSven Wegener2018-11-072-9/+15
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I8ba40cb8ca647ec24b80c2824bb64e84430535d4 Signed-off-by: Sven Wegener <sven.wegener@inovex.de>
* | | | | Handle multiple ports in AddFloatingIPmelanie witt2018-11-063-15/+164
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | AddFloatingIP refers to an old nova proxy API to neutron that was deprecated in nova. The neutron API for floating IP associate requires a port to be specified. Currently, the code is selecting the first port if the server has multiple ports. But, an attempt to associate the first port with a floating IP can fail if the first port is not on a network that is attached to an external gateway. In order to make the command work better for users who have a server with multiple ports, we can: 1. Select the port corresponding to the fixed_ip_address, if one was specified 2. Try to associate the floating IP with each port until one of the attempts succeeds, else re-raise the last exception. (404 ExternalGatewayForFloatingIPNotFound from neutron) This also fixes incorrect FakeFloatingIP attributes that were being set in the TestServerAddFloatingIPNetwork unit tests, which were causing the tests to use None as parsed args for ip-address and --fixed-ip-address and thus bypassing code in the 'if parsed_args.fixed_ip_address:' block. Task: 27800 Story: 2004263 Change-Id: I11fbcebf6b00f12a030b000c84dcf1d6b5e86250
* | | | Merge "Add --key-name and --key-unset option for server rebuild API."Zuul2018-11-025-2/+124
|\ \ \ \
| * | | | Add --key-name and --key-unset option for server rebuild API.Fan Zhang2018-10-095-2/+124
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I6d4793a8e961080ea1d6d414cef8d6bbed0c53e7 Story: 2002609 Task: 22228 Signed-off-by: Fan Zhang <zh.f@outlook.com>
* | | | | Merge "Make use of keystoneauth service-type filtering for versions"Zuul2018-11-023-16/+4
|\ \ \ \ \
| * | | | | Make use of keystoneauth service-type filtering for versionsMonty Taylor2018-10-193-16/+4
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The first version of the versions show command does client-side service-type filtering, which while functional, causes many more API calls than needed. Now that keystoneauth supports the filtering at the source, use it. Change-Id: I57c49e67f9cb285a5f5bc19ec53a42d10de9f0da
* | | | | Merge "Address issues from volume backend commands"Zuul2018-11-023-38/+39
|\ \ \ \ \
| * | | | | Address issues from volume backend commandsSean McGinnis2018-10-123-38/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes some minor issues in release notes and the command list for the new volume backend commands. Also sorts the fakes used for volume unit tests to allow for multiple command update patches to hopefully reduce the odds of merge conflicts. Change-Id: Ic6e40f4c639368338cf085c68c17038f81da5361 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | | | | | trivial: modify spelling error of projectlvxianguo2018-11-011-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I6cf68325bc58cfd073c5fca4eb2773d108735399
* | | | | | Merge "Improve document 'openstack complete'"Zuul2018-10-281-2/+5
|\ \ \ \ \ \
| * | | | | | Improve document 'openstack complete'Robin Cernin2018-10-191-2/+5
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The openstack complete command requires bash-completion package Without the relevant package the bash completion doesn't work. Change-Id: I47c77f3e7efe112417d2b96936fcd0cafeb9442e Closes-Bug: #1798493
* | | | | | Merge "Update release note version reference table"Zuul2018-10-281-0/+2
|\ \ \ \ \ \
| * | | | | | Update release note version reference tableSean McGinnis2018-10-231-0/+2
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The table indicating the openstackclient release at the time of the overall OpenStack release had not been updated since Pike. This adds release versions for Queens and Rocky. Change-Id: Icd9a72c2460fae3d1b714b6da473564e6c709ef6 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | | | | | Update the Neutron CLI decoder documentBrian Haley2018-10-261-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The floatingip-disassociate mapping had 'port' instead of '--port', also fixed a few places where 'unset' was missing from some of the mappings. Trivialfix Change-Id: I3b01db28dda674e9988176d496154fbd26e4449f
* | | | | | Merge "Default --nic to 'auto' if creating a server with >= 2.37"Zuul2018-10-264-7/+66
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Default --nic to 'auto' if creating a server with >= 2.37Matt Riedemann2018-03-214-7/+68
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Compute API version >= 2.37 requires a 'networks' value in the server create request. The novaclient CLI defaults this to 'auto' if not specified, but the novaclient ServerManager.create python API binding code does not, as it wants clients to be explicit. For the purposes of the OSC CLI, we should follow suit and if the user is requesting OS_COMPUTE_API_VERSION>=2.37 without specific nics, we should just default to 'auto'. Change-Id: Ib760c55e31209223338a4086ff1f4fee88dc6959 Closes-Bug: #1750395
* | | | | | Merge "Handle not having cinderclient.v1 available"Zuul2018-10-154-21/+19
|\ \ \ \ \ \
| * | | | | | Handle not having cinderclient.v1 availableSean McGinnis2018-10-124-21/+19
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Cinder v1 API was deprecated several years ago and may be removed from python-cinderclient in the near future. To handle the case where v1 is no longer present, this updates cinderclient initialization to work without it and give an appropriate error if v1 is requested with a version where it is no longer available. Change-Id: I277d7b48b8ad4cce383ec3722f8117938378f615 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | | | | | Merge "Allow endpoint filtering on both project and project-domain"Zuul2018-10-122-1/+42
|\ \ \ \ \ \
| * | | | | | Allow endpoint filtering on both project and project-domainJulie Pichon2018-10-092-1/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The --project and --project-domain flags are currently mutually exclusive for listing endpoints, however the --project-domain argument is supposed to help with filtering projects with colliding names. They should be allowed together. Story: 2004018 Task: 27007 Change-Id: I7340e01f509e3515f07cb46f175fb603f1ce8b67
* | | | | | | Merge "Add volume backend pool list command"Zuul2018-10-126-2/+191
|\ \ \ \ \ \ \ | | |_|_|/ / / | |/| | | | |
| * | | | | | Add volume backend pool list commandSean McGinnis2018-10-096-2/+191
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds an equivalent for "cinder get-pools" with "volume backend pool list" and "cinder get-pools --detail" with "volume backend pool list --long". Story: 1655624 Task: 136949 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com> Change-Id: I826c9946ffe11340d44ad57914f72fc2a72b6938
* | | | | | | Merge "Add volume backend capability show command"Zuul2018-10-128-1/+214
|\ \ \ \ \ \ \ | |/ / / / / / | | | / / / / | |_|/ / / / |/| | | | |
| * | | | | Add volume backend capability show commandSean McGinnis2018-10-098-1/+214
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds and equivalend for "cinder get-capabilities" command to show the capabilities supported by a Cinder backend. Story: 1655624 Task: 26947 Change-Id: I38686a26cd503e45ce0102705a6632994ef10274 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | | | | Add --property option to 'server rebuild' commandRuby Loo2018-10-053-1/+39
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add '--property' option to the 'server rebuild' command, to provide the ability to specify properties of the rebuilt instance. This is equivalent to the '--meta' option of the compute's 'nova rebuild' command. Change-Id: I25ea6622e970416090109316e1e28fab8b0b3f07 Story: #2003979 Task: #26922
* | | | Merge "Add metavar for name parameter in subnet create"Zuul2018-10-031-0/+1
|\ \ \ \
| * | | | Add metavar for name parameter in subnet createDavid Rabel2018-09-121-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I2511677006687fff3166441c51e91191492962f1 Closes-Bug: #1747731
* | | | | Merge "fix tox python3 overrides"Zuul2018-10-021-0/+5
|\ \ \ \ \
| * | | | | fix tox python3 overridesDoug Hellmann2018-09-261-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We want to default to running all tox environments under python 3, so set the basepython value in each environment. We do not want to specify a minor version number, because we do not want to have to update the file every time we upgrade python. We do not want to set the override once in testenv, because that breaks the more specific versions used in default environments like py35 and py36. Change-Id: I051f1c18b719a27372b626d483e47327085dd3b7 Signed-off-by: Doug Hellmann <doug@doughellmann.com>
* | | | | | Remove python-ceilometerclientWitold Bedyk2018-09-281-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Project is retired [1]. [1] https://github.com/openstack/python-ceilometerclient Change-Id: Idc3a340c99c9fd79d80c90e45fd9a1c44c205218
* | | | | | Add monascaclient to `not plugins` listWitold Bedyk2018-09-281-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I377a018a4e3c7b6fa44ce9323d1ee2edbd0e4728
* | | | | | Fix some spaces in help messagesBrian Haley2018-09-272-12/+12
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix some missing and not necessary trailing spaces in the network v2 API files. Also fixed one block indent that was different from all its friends. Trivialfix Change-Id: Ic6491203c2fb9085543d69f0bb5f38e5a96039da
* | | | | Fix help message for subnetpool default-quota valueWenran Xiao2018-09-192-8/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Default pre-project quota is the number of IP addresses that can be allocated from the subnet pool. For example, with a quota of 128, I might get a 64 addresses subnet1, and 16 addresses subnet2, and still have room to allocate 48 more addresses in the future. Change-Id: Ia0ba827790b190647aed990e47347560fc9e9f0c
* | | | | Fix 'project purge' deleting wrong project's servers and volumesJulie Pichon2018-09-142-14/+20
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Project purge would delete the servers and volumes for the project the user is currently authenticated for, regardless of the --project flag. Note: This change means that no server at all will be deleted if the logged in user doesn't have the get_all_tenants permission set in the Nova policy (default: admin_api). This doesn't appear to be an issue with Cinder as the default rule appears to be admin_or_owner. Change-Id: If1c54e24e1482438b81c3c32fd5fc9fdd7a7be04 Story: 1747988 Task: 13854
* | | | Merge "osc-included image signing (using openstacksdk)"Zuul2018-09-084-2/+71
|\ \ \ \
| * | | | osc-included image signing (using openstacksdk)Josephine Seifert2018-09-084-2/+71
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This extension adds image signing functionality to the "image create" command. Therefore, new CLI options --sign-key-path and --sign-cert-id have been added. This patch uses openstacksdk as the signing backend library instead of cursive. Therefore, requirements and lower_constraints have been updated to use the openstacksdk version 0.17 or higher. Depends-On: Idc15b9a12d408bd4b2e096da8402c374be56f9fa Change-Id: Ia20bc02a49c3fbeb9222e485e3396395f4ab817a Story: 2002128 Co-Authored-By: Markus Hentsch <markus.hentsch@secustack.com>
* | | | | Use templates for cover and lower-constraintsAndreas Jaeger2018-09-071-5/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use openstack-tox-cover template, this runs the cover job in the check queue only. Use openstack-lower-constraints-jobs template Remove jobs that are part of the templates. Change-Id: Id210b2f15dcba9dcf9ad75e0436995e7d5c1b62b
* | | | | add lib-forward-testing-python3 test jobqingszhao2018-08-211-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a mechanically generated patch to add a functional test job running under Python 3 as part of the python3-first goal. See the python3-first goal document for details: https://governance.openstack.org/tc/goals/stein/python3-first.html Change-Id: I1d186a6759bd3e633661c75e62545d0d56700244 Story: #2002586 Task: #24320
View Lorry Controller Status