summaryrefslogtreecommitdiff
path: root/openstackclient/tests
Commit message (Collapse)AuthorAgeFilesLines
* Remove hard-coded policy choices for creating a server groupMatt Riedemann2017-11-291-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 (cherry picked from commit d0917cd14b94ce13fd92237490efb5722761eda4)
* Merge "Neutron agent delete: remove the wrong argument" into stable/pikeZuul2017-11-151-16/+7
|\
| * Neutron agent delete: remove the wrong argumentnpraveen352017-11-071-16/+7
| | | | | | | | | | | | | | | | | | | | | | 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-023-12/+14
|/ | | | | | Closes-Bug: #1717130 Change-Id: I33c6fc7897dfee85d1c197a1267bde4abfa5bbd9 (cherry picked from commit 254dbf3294c0f1edc4a2a469f556b3c4b3123a00)
* Merge "Add optional parameter "user_id" and "type" to list credentials"Jenkins2017-07-251-1/+29
|\
| * Add optional parameter "user_id" and "type" to list credentialszhanghongtao2017-07-251-1/+29
| | | | | | | | | | | | | | | | 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-2314-90/+641
| | | | | | | | | | | | | 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
* Add domain parameter to Identity ProviderKristi Nikolla2017-07-212-3/+81
| | | | | | | | | | | | | | | | 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-201-0/+60
|\ \
| * | Now OSC server create check keys in --nicM V P Nitesh2017-06-231-0/+60
| | | | | | | | | | | | | | | | | | | | | | | | 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 "Network L3 Router Commands for OSC"Jenkins2017-07-204-24/+269
|\ \
| * | Network L3 Router Commands for OSCAnkur Gupta2017-07-114-24/+269
| | | | | | | | | | | | | | | | | | | | | 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-202-1/+83
|\ \ \
| * | | Fix 'domain' filter not work well in some commandsjiangpch2017-07-182-1/+83
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-201-11/+6
|\ \ \ \
| * | | | Fix column names for server list --no-name-lookupDean Troyer2017-07-171-11/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-201-0/+50
|\ \ \ \ \ | |/ / / / |/| / / / | |/ / /
| * | | Add server list -n and --no-name-lookup argumentsBoris Pavlovic2017-07-101-0/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-132-0/+60
|\ \ \ \
| * | | | Add 'data_plane_status' option to Port classesCarlos Goncalves2017-07-122-0/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-122-0/+46
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-063-0/+51
|\ \ \
| * | | Allow objects to be streamed to stdoutHonza Pokorny2017-07-053-0/+51
| |/ / | | | | | | | | | Change-Id: Icd8de6b2122fe77926d93da9bda08f56c3672a7a
* | | Add new parameter "is_default" to Network QoS policy.Rodolfo Alonso Hernandez2017-07-053-3/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "Show neutron tags in OSC network show"Jenkins2017-06-302-0/+8
|\ \ \ | |/ / |/| |
| * | Show neutron tags in OSC network showM V P Nitesh2017-06-222-0/+8
| |/ | | | | | | | | | | | | | | | | 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-221-1/+1
|\ \
| * | When creating a trust, send role_ids instead or role_namesKristi Nikolla2017-06-221-1/+1
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-223-21/+66
|\ \
| * | Add direction field to QoS bandwidth limit.Rodolfo Alonso Hernandez2017-06-223-21/+66
| |/ | | | | | | | | | | | | | | | | | | 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-221-0/+314
|\ \
| * | Add project purge command to oscSteve Martinelli2017-06-131-0/+314
| |/ | | | | | | | | | | | | | | | | See the initial implementation: https://github.com/openstack/ospurge/blob/master/ospurge/client.py Partial-Bug: 1584596 Change-Id: I3aa86af7c85e7ca3b7f04b43e8e07125f7d956d1
* | Merge "Don't show hint about vlan transparent in network set"Jenkins2017-06-221-3/+0
|\ \ | |/ |/|
| * Don't show hint about vlan transparent in network setHong Hui Xiao2017-06-121-3/+0
| | | | | | | | | | | | | | Update this attribute is not allowed in neutron. Change-Id: I38010b26e116246c13dbb6cc6a777d2f22f6dc30 Closes-Bug: #1691776
* | Add default-quota to subnet pool commandsReedip2017-06-122-5/+76
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Add --default-quota option to subnet pool create and set commands. Setting default-quota back to None may break the current Neutron behavior, therefore support for Unset command is not provided in this patch. Neutron API: https://github.com/openstack/neutron/blob/a0e0e8b6686b847a4963a6aa6a3224b5768544e6/neutron/api/v2/attributes.py#L239 Closes-Bug: #1667294 Change-Id: Ia4e7c23a49e91a090133c729353cdb8e62bc5674
* | volume functest: ensure snapshots deleted when volume deleteAkihiro Motoki2017-06-0610-129/+130
| | | | | | | | | | | | | | | | | | | | | | | | | | Deleting snapshot may take time. The current volume API does not allow to delete volumes with snapshots, so if deleting snapshot may take time, a delete request for a parent volume will fail. This sometimes causes functional test failures in slow environments. wait_for_status() checks whether volume status is in error statuses but previously the expected error status was wrong. Cinder API uses lower case as volume status, so it did not work expectedly. Change-Id: I095894ba39f23bf81d71351818d24dbb5ca459fb
* | Merge "Replace "Display Name" by "Name" in volume list"Jenkins2017-06-055-26/+208
|\ \
| * | Replace "Display Name" by "Name" in volume listRui Chen2017-05-265-26/+208
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Current "volume list --name" command use "display_name" as search_opts to send to cinder API, and show the result table with "Display Name" column title in osc, cinder list API support "name" as search opts too, and there is "name" attribute in volume response body, so we can replace all "Display Name" by "Name" in order to keep "volume list" command consistent with other commands, like: server list, network list and so on, only use "Name" attribute for all objects. Support a mapping for volume list -c "Display Name" (Volume v1 and v2) and volume create/show -c "display_name" (Volume v1) for minimal backward compatibility until R release. Change-Id: I120be0118e7bb30093b4237c5eeb69a9eedef077 Closes-Bug: #1657956 Depends-On: I1fb62219b092346ea380099811cbd082cae5bafe
* | | Merge "To display image size in human friendly format"Jenkins2017-06-014-4/+60
|\ \ \
| * | | To display image size in human friendly formatBadhmapriya Boopalan2017-01-114-4/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Include option '--human-readable' to 'image show' command. This option displays image size in human readable format (such as K, M, G, T,..) Related Commit: I0ef74c2ec978483fe49156c88acf5c369a8fa5c2 Closes-Bug: #1640086 Change-Id: I28cd5702925d51303d0607ed8dccf12c56434682
View Lorry Controller Status