summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
* Prevent "server migrate --wait" from hangingJulie Pichon2018-06-111-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: 19620 (cherry picked from commit 814ffb2a6f22aed640ba6adad62c1288f917b09a)
* Remove hard-coded policy choices for creating a server groupMatt Riedemann2017-11-292-4/+5
| | | | | | | | | | | | 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 (cherry picked from commit d0917cd14b94ce13fd92237490efb5722761eda4)
* Merge "Neutron agent delete: remove the wrong argument" into stable/pikeZuul2017-11-152-18/+8
|\
| * Neutron agent delete: remove the wrong argumentnpraveen352017-11-072-18/+8
| | | | | | | | | | | | | | | | | | | | | | The take_action() function was calling get_agent() with the wrong attribute, causing agent deletion to fail. It turns out calling get_agent() isn't necessary; this removes the call entirely and moves the 'ignore_missing' argument to the delete_agent() function. Change-Id: Iaa3754a3be0765112f396495fa5fb2e32e6eae4e Closes-Bug: #1711301 (cherry picked from commit 6c8e5177bc54ba215aad310094b1eefd6ce709cf)
* | Merge "Backport Zuul v3 tests" into stable/pikeZuul2017-11-143-90/+16
|\ \ | |/ |/|
| * Backport Zuul v3 testsDean Troyer2017-11-063-90/+16
| | | | | | | | | | | | | | Do a single copy of the current test state from master Depends-On: I6ba35ea9e2080012f098569efcd65e1145829d91 Change-Id: I428516388cee564da86be69ad0591108217f82f6
* | Fix 'project purge' deletes ALL images problemlihaijing2017-11-024-13/+22
|/ | | | | | Closes-Bug: #1717130 Change-Id: I33c6fc7897dfee85d1c197a1267bde4abfa5bbd9 (cherry picked from commit 254dbf3294c0f1edc4a2a469f556b3c4b3123a00)
* Merge "wrong values in openstack quota show command"3.12.0Jenkins2017-07-251-5/+21
|\
| * wrong values in openstack quota show commandnidhimittalhada2017-07-241-5/+21
| | | | | | | | | | | | | | | | | | | | | | "openstack quota show" shows wrong value in field project and project_id. project UUID is shown in field project. and project is coming as None. Corrected it. Change-Id: I237e49858feab14ec5e5bc6d8cb79c9f6f5ea9c0 Closes-Bug: #1679906
* | Merge "Add optional parameter "user_id" and "type" to list credentials"Jenkins2017-07-252-4/+63
|\ \
| * | Add optional parameter "user_id" and "type" to list credentialszhanghongtao2017-07-252-4/+63
| | | | | | | | | | | | | | | | | | | | | | | | In keystone version 3.5, "type" optional attribute has been added to list credentials. This patch add "user_id" and "type" optional parameter in openstack client. Change-Id: Ia09ee7c39204fdff2dfd7b9b606d888d007caac5
* | | network tag UT: avoid using mix-in test classAkihiro Motoki2017-07-256-359/+628
|/ / | | | | | | | | | | | | | | | | | | This is a follow-up patch on https://review.openstack.org/#/c/461195/ There is a suggestion to keep test code more straight-forward and avoid using mix-in to reduce the code complexity. This commit moves all logic implemented in _test_tag.py into individual network tests. Change-Id: I0a9f8c6cd758db9035b0fd60ce4b9bfc791b6cbd
* | Network tag supportAkihiro Motoki2017-07-2320-102/+891
|/ | | | | | | | | | | | | Neutron tag mechanism now supports network, subnet, port, subnetpool and router. Tag support for more resources is planned. This commit introduces a common mixin class to implement tag operation and individual resource consumes it. To support tag remove, network unset command is added. Implements blueprint neutron-client-tag Change-Id: Iad59d052f46896d27d73c22d6d4bb3df889f2352
* Merge "Start using 'cliff.sphinxext'"Jenkins2017-07-211-9/+23
|\
| * Start using 'cliff.sphinxext'Stephen Finucane2017-07-211-9/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 'cliff', the command line library used by 'osc_lib' (and, thus, 'python-openstackclient') recently gained a Sphinx extension to automatically document cliff commands. This allows us to use the documentation we already have in code instead of duplicating it in the documentation. Introduce the use of this, starting with the 'server' commands. This requires extending the descriptions for two commands to ensure no information is lost. Change-Id: If701af8d5a3f78f4b173ceb476dd0c163be4b6ca
* | Add domain parameter to Identity ProviderKristi Nikolla2017-07-213-4/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Identity providers are now associated with domains. This change allows a user to specify a domain by ID or by name when creating an identity provider. [0] This also adds the column for Domain ID in listing. Updating a domain for an identity provider is not supported, so that isn't changed. [0]. Id18b8b2fe853b97631bc990df8188ed64a6e1275 Closes-Bug: 1698390 Change-Id: Icc408e2fe88f257d5863bd3df716a777d52befcc
* | Merge "Clean up the changes of os.environ in functional tests"Jenkins2017-07-2137-228/+336
|\ \
| * | Clean up the changes of os.environ in functional testsRui Chen2017-07-2037-228/+336
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use fixtures to restore the API version changes of os.environ in each functional tests, aims to avoid the following test cases failing in unexpected context. And make sure setUpClass/tearDownClass call super class's corresponding methods first. Change-Id: Ie248fe9d3a9e25f1b076c9f2c363200f29a83817 Closes-Bug: #1696080
* | | Merge "Now OSC server create check keys in --nic"Jenkins2017-07-202-6/+70
|\ \ \
| * | | Now OSC server create check keys in --nicM V P Nitesh2017-06-232-6/+70
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now OSC command to create server will check all the keys in --nic and throws an exception if the key is invalid key. Change-Id: I5482da0ae63d6d4298aa614e4d09bb0547da9ec3 Closes-Bug: #1681411
* | | | Merge "Enable some off-by-default checks"Jenkins2017-07-202-6/+2
|\ \ \ \
| * | | | Enable some off-by-default checksblue552017-06-232-6/+2
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | Some of the available checks are disabled by default, like: [H203] Use assertIs(Not)None to check for None Change-Id: I59dafb62cedc5217b6e5eb6efb997a9ee3c29bbb
* | | | Merge "Add support for virtio-forwarder VNIC type"Jenkins2017-07-201-2/+3
|\ \ \ \ | |_|/ / |/| | |
| * | | Add support for virtio-forwarder VNIC typeJan Gutter2017-07-191-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * This patch adds support for the virtio-forwarder VNIC type. * The virtio-forwarder VNIC type has been added as another option for setting the --vnic-type property on the "port set" and "port create" commands. This requests a low-latency virtio port inside the instance, likely backed by hardware acceleration. Currently the Agilio OVS external plugin provides support for this, with support from other vendors following soon. * Corresponding neutron-lib change: https://review.openstack.org/#/c/483530/ * Nova spec for Agilio OVS enablement: https://specs.openstack.org/openstack/nova-specs/specs/pike/approved/netronome-smartnic-enablement.html Change-Id: Idbc8071afe95f8594b40e2f93e5411e7185f946f Signed-off-by: Jan Gutter <jan.gutter@netronome.com>
* | | | Merge "Network L3 Router Commands for OSC"Jenkins2017-07-206-50/+396
|\ \ \ \ | |_|_|/ |/| | |
| * | | Network L3 Router Commands for OSCAnkur Gupta2017-07-116-50/+396
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements: blueprint network-l3-commands Co-Authored-By: Akihiro Motoki <amotoki@gmail.com> Change-Id: Ia24d76227e164062e89a74c1621b8acb830b26cf
* | | | Merge "Fix 'domain' filter not work well in some commands"Jenkins2017-07-205-5/+96
|\ \ \ \
| * | | | Fix 'domain' filter not work well in some commandsjiangpch2017-07-185-5/+96
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The 'domain' filter not work well in commands 'project show', 'user show' and 'user set'. Depends-On: I490900d6249f01654d4cba43bddd3e7af7928a84 Closes-Bug: #1704097 Change-Id: Ib4f47cbaba27eb56c4a41d187fee74a995e62dc7
* | | | Merge "Fix column names for server list --no-name-lookup"Jenkins2017-07-202-25/+27
|\ \ \ \
| * | | | Fix column names for server list --no-name-lookupDean Troyer2017-07-172-25/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When --long is not present change the 'Image Name' column to 'Image' and add the 'Flavor' column. These columns will contain Names unless --no-name-lookup is specified when they will contain IDs. Change-Id: I92cfb22136aee32616894e60e9227b4da185da99
* | | | | Merge "Fix dynamic names in network functional tests"Jenkins2017-07-2011-25/+36
|\ \ \ \ \
| * | | | | Fix dynamic names in network functional testsDean Troyer2017-07-1911-25/+36
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move all of the dynamic resource naming in Network functional tests into setUpClass() methods (if they exist) rather than assigning those names at load-time. Change-Id: Ic550ff7d40c2b3ca5128cacccbe331790d6ae340
* | | | | Merge "Add server list -n and --no-name-lookup arguments"Jenkins2017-07-202-16/+79
|\ \ \ \ \ | |/ / / / |/| / / / | |/ / /
| * | | Add server list -n and --no-name-lookup argumentsBoris Pavlovic2017-07-102-16/+79
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove translation of Image ID and Flavor ID to Image and Flavor names In large environments amount of images can be very large (thousands) Which requires ~hundreds of requests to Glance to get all images (by default client request only 20 images) As a result listing even few servers is going to take minutes This patch allows to avoid these queries by not doing translation, which allows one to get information about servers in seconds. Change-Id: I4ae00e6324a41c4c79bf5b620179dae99aea5431
* | | | Merge "Add 'data_plane_status' option to Port classes"Jenkins2017-07-133-0/+78
|\ \ \ \
| * | | | Add 'data_plane_status' option to Port classesCarlos Goncalves2017-07-123-0/+78
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds 'data_plane_status' option to SetPort and UnsetPort classes. Closes-Bug: #1684989 Change-Id: I26e23b551afb8c37e6babdea1655efb7c5c6873b
* | | | | image-list should support filters 'name','status'nidhimittalhada2017-07-123-0/+62
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | nova api support parameters like 'name', 'server', 'status', etc in image-list(). So openstackclient should support this too. DocImpact Closes-Bug: #1698742 Change-Id: Ice66b409f989e6785aa3b2d42f2fdbf6e23fa0aa
* | | | Fix unit test failures related to new os-client-config and osc-libAkihiro Motoki2017-07-122-18/+16
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [breakage related to os-client-config 1.28.0] os-client-config 1.28.0 add a check if filebased and envvars are both used. This check causes OSC unit test failure. OSC now instantiates OpenStackConfig twice as a workaround. The unit test mocks _load_config_file() and it returns a config dict, but os-client-config OpenStackConfig.__init__ updates the dict returned. As a result, when OpenStackConfig is instantiated second time, the mock of _load_config_file returns a modified version of the config dict. This hits the new check in os-client-config 1.28.0. This commit changes the mock to use side_effect rather than return_value to ensure the original dict is used. [breakage related to osc-lib 1.7.0] The change in osc-lib 1.7.0 added "if" logic to avoid calling get() twice. In tests.unit.volume.test_find_resource, kwargs is empty dict in find_resource(), so the second call to get() is NOT called now. Removing the second elements of side_effect addresses the unit failure. Co-Authored-By: Rui Chen <chenrui.momo@gmail.com> Change-Id: Ib9d14661b2755bbd6619e15c0d9023fbc9d27d70 Closes-Bug: #1703782 Closes-Bug: #1703783
* | | Merge "Allow objects to be streamed to stdout"Jenkins2017-07-065-6/+63
|\ \ \
| * | | Allow objects to be streamed to stdoutHonza Pokorny2017-07-055-6/+63
| |/ / | | | | | | | | | Change-Id: Icd8de6b2122fe77926d93da9bda08f56c3672a7a
* | | Add new parameter "is_default" to Network QoS policy.Rodolfo Alonso Hernandez2017-07-054-13/+87
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a set of exclusive parameters to the Network QoS policy: --default: makes this policy the default policy for the project to which the qos policy belongs. --no-default: unset the property. Closes-Bug: #1639220 Depends-On: If5ff2b00fa828f93aa089e275ddbd1ff542b79d4 Depends-On: Ibe7b7881cb190bfd5582f35b6de51a8bc21135de Change-Id: I0269b837dc29bbd8ee2089d847cadb72d800fa30
* | | Merge "Added 'openstack image set --visibility'"Jenkins2017-07-011-4/+30
|\ \ \
| * | | Added 'openstack image set --visibility'M V P Nitesh2017-06-301-4/+30
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | This feature will allow image owners to share images across multiple tenants/projects without explicitly creating members individually through the glance API V2. “Community images” will not appear iu user's default image listings Change-Id: Ic02bf44cca5d6d793d68a8737d3cfe3f44407d88 Closes-Bug: #1626837
* | | Merge "Show neutron tags in OSC network show"Jenkins2017-06-303-0/+10
|\ \ \ | |/ / |/| |
| * | Show neutron tags in OSC network showM V P Nitesh2017-06-223-0/+10
| |/ | | | | | | | | | | | | | | | | Now tag information of the network is displayed when the user tries to see the network details using the command openstack network show <net-id> Change-Id: I587d2bca37b8dbef4400db3d8ace3c81d87e2db3 Closes-Bug: #1695783
* | Merge "When creating a trust, send role_ids instead or role_names"Jenkins2017-06-222-7/+7
|\ \
| * | When creating a trust, send role_ids instead or role_namesKristi Nikolla2017-06-222-7/+7
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changes create a trust to use ids instead of names because of the possibility of roles sharing a name. Even if the user uniquely identified a role by inputting the id, the request sent to the identity service would used the name, therefore the command would fail in the case that two roles share a name. This does not change how trusts are displayed during trust list or trust show, a name will still be shown instead of an id. Depends-On: I38e0ac35946ee6e53128babac3ea759a380572e0 Change-Id: I5bdf89f1e288954a7f5c2704231f270bc7d196f5 Closes-Bug: 1696111
* | Merge "Add direction field to QoS bandwidth limit."Jenkins2017-06-224-34/+85
|\ \
| * | Add direction field to QoS bandwidth limit.Rodolfo Alonso Hernandez2017-06-224-34/+85
| |/ | | | | | | | | | | | | | | | | | | This patch enables the direction ('ingress'/'egress') field on the QoS bandwidth limit rule object and CRUD commands. Closes-Bug: #1614121 Depends-On: Ia13568879c2b6f80fb190ccafe7e19ca05b0c6a8 Depends-On: I90c412a5c8757b3ffe8abfc1165a70bdb8744702 Change-Id: Ic6981474f22efbf294ac11c2e0304b04494a1bbe
* | Merge "Add project purge command to osc"Jenkins2017-06-222-0/+482
|\ \