summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | Merge "Prevent "server migrate --wait" from hanging"Zuul2018-06-011-0/+1
|\ \ \ \ \ \ \
| * | | | | | | Prevent "server migrate --wait" from hangingJulie Pichon2018-05-091-0/+1
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Migrate uses the same mechanism in the backend than Resize and so the steps and step names are similar. Currently when using the --wait option with 'migrate', we wait forever because the status won't get to active until the user performs an action. This makes it return on verify_resize status just like 'resize' does, so that the user can perform the next manual step. Change-Id: Ie1aeac52506bc8801f88fd6a6eb4f6094cf20050 Story: 2001994 Task: 19621
* | | | | | | Make max_burst_kbps option as optional for bw limit QoS ruleSławek Kapłoński2018-05-253-11/+53
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | | Update command test for volume.v3Monty Taylor2018-05-181-1/+1
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The default cinder version in devstack changed to v3 in https://review.openstack.org/#/c/566747/which breaks this test. Change the test to test what's going to happen. Change-Id: Iff4d8b47812a86d21bf5dbdddbd642b9d63ff8fe
* | | | | Merge "Fix limits show command without Nova and Cinder"Zuul2018-04-236-11/+383
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Fix limits show command without Nova and CinderJude Cross2018-04-036-11/+383
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-233-4/+156
|\ \ \ \ \
| * | | | | Network: Add tag support for floating ipDongcan Ye2018-04-173-4/+156
| | |/ / / | |/| | | | | | | | | | | | | | | | | | Change-Id: I7a500a4ff6cec2442b4050df26c0b017d9f71903 Closes-Bug: #1750985
* | | | | Fix functional job failedDongcan Ye2018-04-201-3/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After Neutron patch I3c93818002c2d7753454547231ba08544b6fa1c0 merged, the default value of segment description is an empty string. This patch will determine whether the Neuron shim extension standard-attr-segment supported. Change-Id: I68a3c018f03e5bb53bd637844ac9d7742a765db0
* | | | | Clean up W503 and E402 pep8 errorsSean McGinnis2018-04-1513-255/+232
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | pycodestyle 2.40 and later enforce these rules that were not previously enforced. Rather than just skipping them, this cleans up the trivial instances of these violations. This does also include some other updates that were not triggering errors in an attempt to keep some of the style consistent. Change-Id: Id7c0a6b8f1f835e69d844b000e3ed751852ada63 Closes-bug: #1762803
* | | | | Merge "Add support to list image members"Zuul2018-04-022-0/+79
|\ \ \ \ \
| * | | | | Add support to list image membersMohammed Naser2018-03-212-0/+79
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-312-0/+203
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Add --image-property parameter in 'server create'Chen Hanxiao2018-03-282-0/+203
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add --image-property option, just like --image-with of novaclient did. Change-Id: Ic1a8976559255529a8785b1b301a0307812433cb Signed-off-by: Chen Hanxiao <chenhx@certusnet.com.cn>
* | | | | Merge "Update help text for encryption provider"Zuul2018-03-292-8/+8
|\ \ \ \ \
| * | | | | Update help text for encryption providerBrianna Poulos2018-02-232-8/+8
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The volume encryption provider no longer uses class names. Instead, 'luks' and 'plain' are used. This patch updates the help text for the volume encryption provider to use the new encryption provider format constants. Change-Id: I2911098505a99658e04ac4008e5f3e857db81f95
* | | | | Merge "Fix additional output encoding issues"Zuul2018-03-297-54/+78
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Fix additional output encoding issuesDean Troyer2018-03-207-54/+78
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "Fix crashing "console log show""Zuul2018-03-231-3/+4
|\ \ \ \ \ | |/ / / /
| * | | | Fix crashing "console log show"Thomas Goirand2018-03-161-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because of encoding issue, the "openstack console show log" is prone to a stack dump, as explained in the bug report. Use the stdout handle that has already been set up by cliff's App class with a sane default encoding. Change-Id: I4d8b0df7f16ee0463e638bb11276220e5b92023b Closes-Bug: 1747862
* | | | | Merge "neutron: add --mtu for create/set network"Zuul2018-03-233-0/+24
|\ \ \ \ \
| * | | | | neutron: add --mtu for create/set networkEmilien Macchi2018-03-153-0/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | Fix typo in 'floating ip associate' command and docBrian Haley2018-03-151-1/+1
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | Assocaite -> Associate Trivialfix Change-Id: I432e35ddcd80ef77c865bee9a54ac2777f5b6386
* | | | Merge "Add CRUD support for application credentials"Zuul2018-03-145-0/+711
|\ \ \ \
| * | | | Add CRUD support for application credentialsColleen Murphy2018-01-305-0/+711
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | Merge "Cleanup error messages on failure"Zuul2018-03-141-2/+6
|\ \ \ \ \
| * | | | | Cleanup error messages on failureMatthew Treinish2018-02-281-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When test_server_commands_main_help() fails it dumps a ton of unformatted text on an exception message. This commit attempts to clean it up to make it easier to read. Change-Id: I793e6337728a22302a5a87938dbec60d7f2320d8
* | | | | | Re-implement novaclient bits removed in 10.0Dean Troyer2018-03-138-110/+618
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | | Imported Translations from ZanataOpenStack Proposal Bot2018-03-011-46/+4
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: Ie76221736bb0fda2d7f4ee114787e5b11fc734ac
* | | | | Merge "Add support for endpoint group commands"Zuul2018-02-252-0/+340
|\ \ \ \ \
| * | | | | Add support for endpoint group commandsJose Castro Leon2018-02-232-0/+340
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-255-5/+235
|\ \ \ \ \
| * | | | | Add project tags functionalityGage Hugo2018-02-235-5/+235
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-243-0/+14
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Add support for "--dns-domain" argumentTytus Kurek2018-02-233-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | Make osc-functional-devstack-tips actually use tipsMonty Taylor2018-02-051-1/+1
| |_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The base job has tox_install_siblings: false - which we want. But that means we need tox_install_siblings: true on the tips job. While we're at it - add fetch-tox-output so that we have tox log files in the fetched build output for easier verification of what wound up installed. Don't look for ResourceNotFound string in test The string ResourceNotFound is not in the error string anymore. Look for the text that is. Depends-On: https://review.openstack.org/541033 Change-Id: Id6de1485bcafb41f238f3e74277094ce64a6acf4
* | | | Use find_ip from openstacksdkMonty Taylor2018-01-292-143/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | Fix use of new openstacksdk connectionMonty Taylor2018-01-231-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | We store the created conn on the instance, but we never pull it back off if there is already one present. Change-Id: I2d890dd206d4ddf67fa42d798e6fd2c652799785
* | | | Rework Network client config for new SDK ConnectionDean Troyer2018-01-221-19/+26
| |_|/ |/| | | | | | | | | | | | | | | | | | | | network.client.make_client() has always put a copy of it's SDK Connection directly into ClientManager, the new-style Connection create will move into osc-lib ClientManager, do it here too until then. Change-Id: I1edfd19c9e73320768fb9640931fafe857c980b4
* | | Merge "Replace assert with condition"Zuul2018-01-221-3/+1
|\ \ \
| * | | Replace assert with conditionPavlo Shchelokovskyy2018-01-221-3/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | a piece of code in image client has some business logic behind assert, which can be lost when running python in optimized mode (-O). Change-Id: I2179970df495e1215d691915c51cebe5cb4541a7
* | | | Merge "Check that Glance returns image data before processing it"Zuul2018-01-222-0/+58
|\ \ \ \ | |/ / / |/| | |
| * | | Check that Glance returns image data before processing itMike Fedosin2018-01-052-0/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | Corrected spelling mistakeDavid Rabel2018-01-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | in quotas -> in quotes Change-Id: I3adb1ccd8f3a9c495f0b9cf688aee5c4c1e63507
* | | | Merge "Add floating IP qos_policy actions"Zuul2018-01-113-3/+169
|\ \ \ \
| * | | | Add floating IP qos_policy actionsLIU Yulong2018-01-113-3/+169
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-032-0/+26
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "Allow port list to shown undefined attributes"Zuul2017-12-312-2/+14
|\ \ \