summaryrefslogtreecommitdiff
path: root/openstackclient/tests/unit
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Fix subnet host_routes error"Zuul2018-06-241-0/+30
|\
| * Fix subnet host_routes errorHuang Cheng2018-06-141-0/+30
| | | | | | | | | | | | | | When updating subnet with "no-host-route" option, set host_routes to an empty list as neutron_lib.api.validators expected. Change-Id: I6fe039793d813758429c7a104fd40172b4f8122b Closes-Bug: #1747101
* | Fix the `role implies list` command.Sami MAKKI2018-06-242-12/+22
|/ | | | | | | | | | The code was calling an unexisting function which never existed. The module refers now to the correct `InferenceRuleManager`. It also allows the compatibility with the future python-keystoneclient in which the compatibility method will be removed from the RoleManager. Change-Id: I08f785dc9e840da2e16915683eecfe49189c44b3
* Merge "Fix server show for microversion 2.47"Zuul2018-06-121-0/+27
|\
| * Fix server show for microversion 2.47Matt Riedemann2018-06-081-0/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Compute API version 2.47 embeds the server's internal flavor in the response. The original flavor id is not preserved since it could have changed if the flavor was deleted and re-created after the server was created, which was the dreaded Horizon "Edit Flavor" issue. So the flavor dict in the server response is a dict of information about the flavor representing the server "right now" excluding the id. The original flavor name is shown though along with the ram/disk/vcpu etc information. The server list command has a similar issue which will be fixed in a follow up change. Change-Id: I1a92999758006d02567c542b6be8902a049899cc Task: 13864 Story: 1751104
* | Merge "Optimize _prep_server_detail to avoid redundant find_resource"Zuul2018-06-121-4/+4
|\ \ | |/
| * Optimize _prep_server_detail to avoid redundant find_resourceMatt Riedemann2018-06-081-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When showing a server or doing a rebuild, we already have the latest version of the server so _prep_server_detail getting the server again is an unnecessary performance hit. ShowServer is pretty obvious here. For RebuildServer, the compute API actually refreshes the server before returning it in the response, so the client already gets the latest when the rebuild call returns. The only other usage of _prep_server_detail that does require a refresh is CreateServer since the POST /servers response is a minimal version of the server object. This adds a new refresh kwarg, backward compatible by default, to _prep_server_detail but changes ShowServer and RebuildServer to no longer refresh. Change-Id: Ib1c9c424ed1cafc2dfd8be90af8de8a774bdfbf0
* | Merge "Add system role functionality"Zuul2018-06-091-1/+41
|\ \ | |/ |/|
| * Add system role functionalityLance Bragstad2018-06-041-1/+41
| | | | | | | | | | | | | | | | | | | | This commit adds the necessary bits to expose system role assignments to openstackclient via python-keystoneclient. bp system-scope Depends-On: Iecbcbf020a15f2bec777334c648d4477f89f3b2c Change-Id: I261e84700b51e8715eaebdc3f8f8bc46b68542c2
* | Allow setting network-segment on subnet updateHarald Jensas2018-06-061-0/+23
| | | | | | | | | | | | | | | | | | To enable the possibility to migrate a non-routed network to a routed network allow updating the segment_id of a subnet. Change-Id: I3ebae2ff28d5d4e5373ebd1f52194f8c52071b88 Partial-Bug: bug/1692490 Depends-On: I1aee29dfb59e9769ec0f1cb1f5d2933bc5dc0dc5
* | Make max_burst_kbps option as optional for bw limit QoS ruleSławek Kapłoński2018-05-251-6/+49
|/ | | | | | | | | | Attribute max_burst_kbps of QoS bandwidth limit rule in Neutron's is optional in API so it should be also optional on client's side. Change-Id: Ie085b73fa885ff12f9ac080666cf3ca6a09b632a Related-Bug:#1770622 Task: 19658 Story: 2002017
* Merge "Fix limits show command without Nova and Cinder"Zuul2018-04-234-0/+343
|\
| * Fix limits show command without Nova and CinderJude Cross2018-04-034-0/+343
| | | | | | | | | | | | | | | | | | This patch implements an endpoint lookup when showing limits. This addresses the issue when showing limits without both Nova and Cinder and will display limits if one is missing. Change-Id: I2214b281e0206f8fe117aae52de2bf4c4e2c6525 Closes-bug: #1707960
* | Merge "Network: Add tag support for floating ip"Zuul2018-04-232-3/+134
|\ \
| * | Network: Add tag support for floating ipDongcan Ye2018-04-172-3/+134
| | | | | | | | | | | | | | | Change-Id: I7a500a4ff6cec2442b4050df26c0b017d9f71903 Closes-Bug: #1750985
* | | Merge "Add support to list image members"Zuul2018-04-021-0/+46
|\ \ \
| * | | Add support to list image membersMohammed Naser2018-03-211-0/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The OpenStack client presently has support to add or remove members from an image, but no way to list image members. This patch addreses this issue. Change-Id: Ie85c5de23c6beb21fd6b4c04c83ddf2a116606ef
* | | | Merge "Add --image-property parameter in 'server create'"Zuul2018-03-311-0/+158
|\ \ \ \ | |_|_|/ |/| | |
| * | | Add --image-property parameter in 'server create'Chen Hanxiao2018-03-281-0/+158
| | | | | | | | | | | | | | | | | | | | | | | | | | | | add --image-property option, just like --image-with of novaclient did. Change-Id: Ic1a8976559255529a8785b1b301a0307812433cb Signed-off-by: Chen Hanxiao <chenhx@certusnet.com.cn>
* | | | Merge "Fix additional output encoding issues"Zuul2018-03-291-8/+8
|\ \ \ \ | |/ / / |/| | |
| * | | Fix additional output encoding issuesDean Troyer2018-03-201-8/+8
| | |/ | |/| | | | | | | | | | | | | | | | | | | This is a followup to https://review.openstack.org/#/c/541609/ that changes most outstanding direct uses of sys.stdout to use the encoded stdout set up by cliff. Change-Id: I07cfc418385fc787d3b7d3c32d39676cf81bb91f
* | | Merge "neutron: add --mtu for create/set network"Zuul2018-03-232-0/+10
|\ \ \
| * | | neutron: add --mtu for create/set networkEmilien Macchi2018-03-152-0/+10
| | |/ | |/| | | | | | | | | | | | | | | | Support Neutron network mtu configuration with a new argument, --mtu that allows CLI users to set MTU for Neutron networks. Change-Id: I93d23581c7e8c84eaf9bb3b293360036f60f456b
* | | Merge "Remove duplicated network attributes"Zuul2018-03-231-1/+0
|\ \ \ | |/ / |/| |
| * | Remove duplicated network attributesDongcan Ye2018-03-141-1/+0
| |/ | | | | | | Change-Id: If77609d06e7d80c8da5111b8a30036b58fbfa187
* | Merge "Add CRUD support for application credentials"Zuul2018-03-142-0/+341
|\ \
| * | Add CRUD support for application credentialsColleen Murphy2018-01-302-0/+341
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add support for creating, retrieving, and deleting application credentials. Application credentials do not support updates. In order to provide a positive user experience for the `--role` option, this patch also includes an improvement to the `identity.common._get_token_resource()` function that allows it to introspect the roles list within a token. This way there is no need to make a request to keystone to retrieve a role object, which would fail most of the time anyway due to keystone's default policy prohibiting unprivileged users from retrieving roles. bp application-credentials Change-Id: I29e03b72acd931305cbdac5a9ff666854d05c6d7
* | | Re-implement novaclient bits removed in 10.0Dean Troyer2018-03-134-80/+410
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a) /os-floating-ips was removed in Compute API 2.36 and from novaclient's Python API in 10.0 Add to api.computev2: floating_ip_add() floating_ip_remove() Convert add floating IP command to nova-net/neutron split: "server add floating ip" "server remove floating ip" b) /os-hosts was removed in Compute API 2.43 and from novaclient's Python API in 10.0. Add to api.computev2: host_list() host_set() host_show() Convert host commands to use intenal api: "host list" "host set" "host show" c) The introduction of the Network-style commands into the server group broke cliff's autoprogram directive as it executes the get_parser() methods without fully initializing the Command object. NOTE: This is really three reviews squashed to get through the gate in one pass. Depends-on: Id6de87211d6c4ea8fd14aa9203d8d5b17e9e2f04 Change-Id: I5116086f9a9e4b2b31a744bf8f4558c79f0bfe59
* | | Merge "Add support for endpoint group commands"Zuul2018-02-251-0/+16
|\ \ \
| * | | Add support for endpoint group commandsJose Castro Leon2018-02-231-0/+16
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | Implements the commands for endpoint group filter management. Includes the CRUD management of the endpoint groups and the association management between them and the projects that are using this method. Implements: blueprint keystone-endpoint-filter Change-Id: I4265f7f8598d028191e90d76781b7b6ece6fef64
* | | Merge "Add project tags functionality"Zuul2018-02-253-4/+111
|\ \ \
| * | | Add project tags functionalityGage Hugo2018-02-233-4/+111
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change adds tags functionality for projects in keystone. A user can add a single tag with "--tag", chain "--tag" to add multiple tags, or clear tags with "--no-tag". Change-Id: I31cfef3e76dcefe299dacb00c11bb1a10a252628 Partially-Implements: bp project-tags
* | | | Merge "Add support for "--dns-domain" argument"Zuul2018-02-242-0/+6
|\ \ \ \ | |_|/ / |/| | |
| * | | Add support for "--dns-domain" argumentTytus Kurek2018-02-232-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patchset implements support for "--dns-domain" argument to the following commands: "openstack port create" / "openstack port set". Change-Id: I4bb001054b00a969b74db3bb310e567033bf589b Depends-On: https://review.openstack.org/#/c/500660/ Closes-Bug: #1714878 Partial-Bug: #1704769
* | | | Use find_ip from openstacksdkMonty Taylor2018-01-291-83/+25
| |_|/ |/| | | | | | | | | | | | | | | | | The find_ip from openstacksdk started being usable by OSC back in 0.9.15 but the local method never got replaced. Change-Id: I18a334280e5f384f8bb96198cdad79c612a02290
* | | Merge "Check that Glance returns image data before processing it"Zuul2018-01-221-0/+51
|\ \ \
| * | | Check that Glance returns image data before processing itMike Fedosin2018-01-051-0/+51
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now if Glance v2 cannot find image data it returns an empty response with 204 status code, instead of raising an error. Glance client handles this situation and wraps the response with a RequestIdProxy object, whose 'wrapped' attribute is None. But when openstack client tries to parse this object using glanceclient's save_image util function, it fails with "NoneType object is not iterable" message, for the object doesn't contain any data. This patch adds additional check to prevent such behaviour and raises SystemExit exception if no data was returned from the server. Glance v1 is not affected, because it raises an error if can't find an image data. Change-Id: I016a60462ba586f9fa7585c2cfafffd7be38de7b Closes-Bug: #1741223
* | | Merge "Add floating IP qos_policy actions"Zuul2018-01-112-0/+130
|\ \ \
| * | | Add floating IP qos_policy actionsLIU Yulong2018-01-112-0/+130
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now we can associate a qos policy to the floating IP, and dissociate it. The commands are: $ openstack floating ip create --qos-policy ... $ openstack floating ip set --qos-policy ... $ openstack floating ip set --no-qos-policy ... $ openstack floating ip unset --qos-policy These commands are based on the neutron change: I4efe9e49d268dffeb3df4de4ea1780152218633b Partially-Implements blueprint: floating-ip-rate-limit Change-Id: I932b32f78cc5a2b53926feaec1a0b392cf7e8b57
* | | Allow ports filtering with device_idHongbin Lu2018-01-031-0/+19
|/ / | | | | | | | | | | | | | | | | | | Right now, if a neutron port is owned by a container powered by Kuryr, there is no way to list and filter those ports because OSC assumed a neutron port is owned by either a server or router. This patch adds support for that by introducing an option '--device-id' to the 'port list' command. Change-Id: Ib1fd27e8d843a99fb02ccabd8a12a24ac27cec9c
* | Merge "Send 'changes-since' instead of 'changes_since' query parameter"3.13.0Zuul2017-12-131-2/+2
|\ \
| * | Send 'changes-since' instead of 'changes_since' query parameterDaniel Speichert2017-12-111-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Per API reference, only 'changes-since' is accepted and the variant with underscore is ignored, making the CLI functionality broken. [dtroyer] added release note and fixed unit tests. Change-Id: I0c596531a8af03da17d5ce39d75b12e941403aa5 Closes-Bug: 1732216
* | | Merge "Add support for endpoing filter commands"Zuul2017-12-132-0/+166
|\ \ \
| * | | Add support for endpoing filter commandsJose Castro Leon2017-11-212-0/+166
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | Implements the commands that allow to link and endpoint to a project for endpoint filter management. Implements: blueprint keystone-endpoint-filter Change-Id: Iecf61495664fb8413d35ef69f07ea929d190d002
* | | Merge "Fix credentials in create, update and list."Zuul2017-12-111-14/+0
|\ \ \
| * | | Fix credentials in create, update and list.yangweiwei2017-11-171-14/+0
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now, keystone has supported serverl auth method, like 'totp'. Before we use this method, we should create the credential first. And we need create it with type 'totp'. But now we cannot create credential with this method. Also, I think the type should not have constrains. We can create any type in keystone project. So, we should do these actions too. The type would be more which We cannot control. Change-Id: Ie0482da3133fb515e4bb8e45f8c54f509589cc5e Closes-bug: #1731848
* | | Fix SDK Connection creation alternative to ProfileDean Troyer2017-12-071-4/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | Do a dummy import to determine which SDK is installed (Pre/post merge). This solves the DevStack error "Cloud defaults was not found" in -tips jobs. Depends-On: Ia111f127fbdceac2afe20fd9d1fe032145cdd72c Change-Id: I60c2d418dd5a393eee2cc2a5c2fdebfffdabf2d3
* | | Remove a bunch of things we promised to remove in 2H 2017Dean Troyer2017-11-301-1/+1
| | | | | | | | | | | | Change-Id: I060559fe13e354fe87551cd9dd82774bddb54640
* | | Remove hard-coded policy choices for creating a server groupMatt Riedemann2017-11-191-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The hard-coded choices for the server group policy make it impossible to create a server group with the soft-affinity or soft-anti-affinity policy rules which were added in compute API microversion 2.15. This removes the hard-coded choices so that the policy is restricted on the server side rather than the client side. Change-Id: Ib3dc39422ac1015872d56ae2fdeddf0f29613494 Closes-Bug: #1732938
* | | Merge "Network: Add interfaces info in router show"Zuul2017-11-161-4/+19
|\ \ \
View Lorry Controller Status