summaryrefslogtreecommitdiff
path: root/openstackclient/tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Add command to unset information from Subnet-pools"Jenkins2016-07-011-0/+39
|\
| * Add command to unset information from Subnet-poolsreedip2016-06-301-0/+39
| | | | | | | | | | | | | | | | This patch introduces the ``subnet pool unset`` command to clear the pool prefix information from the subnet-pools. Change-Id: I84b7259d6e26e695343d41cea6d807396faaf69a Implements: blueprint network-property-unset
* | Merge "Add "--snapshot" option to "backup create" command in volumev2"Jenkins2016-07-012-1/+14
|\ \
| * | Add "--snapshot" option to "backup create" command in volumev2Huanxuan Ao2016-07-012-1/+14
| |/ | | | | | | | | | | | | | | Add "--snapshot" option to "backup create" command to support backing up a snapshot. Change-Id: Ibecbf1902599875f422d372d529835f73211d3ec Closes-Bug: #1597184
* | Merge "Add "--property" option to "flavor create" command"Jenkins2016-06-302-4/+17
|\ \ | |/ |/|
| * Add "--property" option to "flavor create" commandHuanxuan Ao2016-06-292-4/+17
| | | | | | | | | | | | | | | | Add "--property" option to "flavor create" command to support adding properties to a new falvor. Change-Id: I4f06b364375d5a81584fe41122d48e9568fa712a Closes-Bug: #1596798
* | Merge "Add "--device-owner" option to "port list""Jenkins2016-06-291-0/+41
|\ \
| * | Add "--device-owner" option to "port list"Richard Theis2016-06-201-0/+41
| | | | | | | | | | | | | | | | | | | | | | | | Add "--device-owner" option to the "port list" command to enable listing ports based on device owner. Change-Id: I0a538ec41800b9f842e86dceb6ca4180ef239c95 Implements: blueprint neutron-client
* | | Merge "Add command to unset information from Subnets"Jenkins2016-06-291-0/+106
|\ \ \
| * | | Add command to unset information from Subnetsreedip2016-06-291-0/+106
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch introduces the ``subnet unset`` command to clear the host-routes, allocation-pools and dns-nameservers from subnets. Implements: blueprint network-property-unset Change-Id: I31324a2423f6d2315eed27445dfdcfe863e0b550
* | | | Merge "Add command to unset information from Routers"Jenkins2016-06-281-0/+51
|\ \ \ \
| * | | | Add command to unset information from Routersreedip2016-06-271-0/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch introduces the ``router unset`` command to clear the routing information from the routers. Implements: blueprint network-property-unset Change-Id: Iac8d32ca42fb28878805b4b58ab411b67fa6555b
* | | | | Merge "Add "--force" option to "volume qos delete" command"Jenkins2016-06-282-3/+35
|\ \ \ \ \
| * | | | | Add "--force" option to "volume qos delete" commandHuanxuan Ao2016-06-282-3/+35
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add ``--force`` option to ``volume qos delete`` command in volume v1 and v2 to allow users to delete in-use QoS specification(s). Change-Id: I46036e5f55ced8b8a1be54c521f2a5c242b89160 Closes-Bug: #1596821
* | | | | Merge "Add "--force" option to "backup create" command in volumev2"Jenkins2016-06-281-2/+6
|\ \ \ \ \
| * | | | | Add "--force" option to "backup create" command in volumev2Huanxuan Ao2016-06-271-2/+6
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cinder V2 API supports creating volume backup with "--force" option. However, OSC doesn't support this argument. So this patch add the "--force" option to allow users to back up a in-use volume. Change-Id: I326f8d6172b2830da4cf1317348af50142cc5490 Closes-Bug: #1596443
* | | | | Merge "Support bulk deletion for delete commands in computev2"Jenkins2016-06-283-11/+125
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Support bulk deletion for delete commands in computev2Huanxuan Ao2016-06-273-11/+125
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support bulk deletion and error handling for "keypair delete" and "service delete" commands in computev2. Up to now, all the delete commands in computev2 support bulk deletion. Change-Id: I6d5c960e9716188e56615514d0921618a15a88ec Partially-Implements: blueprint multi-argument-compute Partial-Bug: #1592906
* | | | Update Fakes.py and unit tests for commands in identity V2.0sunyajing2016-06-276-383/+319
|/ / / | | | | | | | | | | | | | | | | | | Update remaining commands:role, service, user, token. Change-Id: I06eed60dd2f312bad6076c78b53cd07bcd4cd55c Partially-Implements: blueprint refactor-identity-unit-test
* | | Support multi-delete for commands in identity V2sunyajing2016-06-232-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Commands are "ec2 credentials delete", "service delete", "endpoint delete". Also update their unit tests and functional tests. Partial-Bug: #1592906 Change-Id: I1a0b7160b803a523646d09d030e6f112c81c4c24
* | | Merge "Support JSON data for port binding profile"Jenkins2016-06-231-0/+95
|\ \ \
| * | | Support JSON data for port binding profileRichard Theis2016-06-201-0/+95
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update the "--binding-profile" option on the "port create" and "port set" commands to support both <key>=<value> and JSON input for the port custom binding profile data. The JSON input is sometimes needed to maintain the value type (e.g. integer) for more advanced data. The port custom binding profile data is unique across neutron so a custom argparse.Action class was created instead of writting a generic class in osc-lib. Change-Id: I82ac6d4f95afdc866f5282fc00d390f850f54d21 Implements: blueprint neutron-client
* | | Merge "Add FakeObject classes to fakes.py, update unit tests in identity V2."Jenkins2016-06-235-275/+422
|\ \ \
| * | | Add FakeObject classes to fakes.py, update unit tests in identity V2.sunyajing2016-06-225-275/+422
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Clean up fakes.py , use FakeCatalog, FakeProject, FakeService, FakeEndpoint, FakeRole classes instead, also update their unit tests. Change-Id: I510d175ec194165b0595ebd430e8cc596d363587 Partially-Implements: blueprint refactor-identity-unit-test
* | | | Merge "Improve masking of secrets in configuration show"Jenkins2016-06-231-3/+14
|\ \ \ \
| * | | | Improve masking of secrets in configuration showAlvaro Lopez Garcia2016-06-211-3/+14
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The command "configuration show" tries to redact some of the secrets that are shown on the screen. However, this failed redacting options that were marked as secrete by the auth plugins (if any) and it redacted other options that were not redacted at all. For example, when using the OpenID Connect plugins, it redacted the "access_token_endpoint" as the word "token" appears there, but it failed to redact "client_secret" even when this option is marked as secret in the corresponding plugin. Change-Id: Idfad4fbbe5ddcff5e729e1dcd756d0379ad31dee
* | | | Merge "support multi-delete for volume-type"Jenkins2016-06-231-2/+2
|\ \ \ \
| * | | | support multi-delete for volume-typeSteve Martinelli2016-06-201-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added the ability to delete multiple volume types at once. Note there are no unit tests exist for v1 volume-types, so instead a functional test was created. Partial-Bug: #1592906 Change-Id: I99f3f22901ab35252b91a3072b14de7d19cb17ca
* | | | | Merge "Use resource id when name given for identity show"Jenkins2016-06-234-0/+62
|\ \ \ \ \
| * | | | | Use resource id when name given for identity showDavid Rosales2016-06-224-0/+62
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently a user is allowed to specify either a resource ID or name when running openstack identity comands. In some cases, when a name is specified instead of an ID, the command will return as not able to find the resource when it in fact does exist. The changes here are to check the client against the token on such requests and to extract the ID of the resource specified if enough information exists between the two. We then use the ID associated with the resource to complete the user requests. Change-Id: I40713b0ded42063b786dc21247e854224b9d2fe2 Closes-Bug: #1561599
* | | | | Merge "Fix errors for "host set" command"Jenkins2016-06-232-8/+7
|\ \ \ \ \
| * | | | | Fix errors for "host set" commandHuanxuan Ao2016-06-222-8/+7
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "Host set" command cannot work. Because: 1.Host has no 'ID' attribute, so 'ID' attribute cannot be found in "host set" command. 2.value "True" and "Flase" are invalid in updata() method of host. 3.Some update functionalities is not supported in host API now. This patch solves the problems 1 and 2 in OSC. But the problem 3 is a API problem and can't be solved in OSC, only XenServer driver support to set enable/disable and maintenance host, it is a normal problem. After this patch the output of "host set" command is: The requested functionality is not supported. (HTTP 501) (Request-ID: req-14031fce-8c90-48a0-8492-dc8e3dd349f3) Just the same as the "host-update" command in novaclient. Change-Id: Ibe94c4d3d492d3d63355de803810edb988e1b4e9 Closes-Bug: #1594689
* | | | | Merge "Refactor unit test of "compute service list" command"Jenkins2016-06-232-7/+40
|\ \ \ \ \
| * | | | | Refactor unit test of "compute service list" commandHuanxuan Ao2016-06-222-7/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The unit test of "compute service list" only checked the "Disabled Reason" columns and its data. It is not enough. This patch change the test to check all datas in the list. Also, this patch modify the "Id" to "ID" in this command. Change-Id: I988fd6365f2652185dd96d9417f294eba9c31cd9
* | | | | | Merge "Make set/unset commands in compute/image/common return normally when ↵Jenkins2016-06-222-6/+13
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | nothing specified"
| * | | | | Make set/unset commands in compute/image/common return normally when nothing ↵Tang Chen2016-06-212-6/+13
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | specified After this patch, all set/unset commands will return normally when nothing specified. Change-Id: Id94d0329faa1a674006a9aae901f834b41917317 Close-bug: #1588588
* | | | | Merge "Support bulk deletion for "flavor/aggregate delete""Jenkins2016-06-223-21/+140
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | Support bulk deletion for "flavor/aggregate delete"Huanxuan Ao2016-06-213-21/+140
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support bulk deletion and error handling for "aggregate delete" and "flavor delete" commands. Change-Id: I3f6105cbeeab1c9f8cd571c63ce0e7ac3d4252b3 Partially-Implements: blueprint multi-argument-compute Partial-Bug: #1592906
* | | | Add "--project" option to the "flavor create" command.Huanxuan Ao2016-06-211-1/+27
| |/ / |/| | | | | | | | | | | | | | | | | | | | Add ``--project`` and ``--project-domain`` options to the ``flavor create`` command. We can use these options to add the flavor access to a givin project when we create the flavor. Change-Id: Ic1907272c1d1ae526f9c9e86f32ba06c6da147c0
* | | Merge "Error handling of "router delete" command"Jenkins2016-06-201-7/+59
|\ \ \
| * | | Error handling of "router delete" commandHuanxuan Ao2016-06-201-7/+59
| |/ / | | | | | | | | | | | | | | | | | | | | | "Router delete" command supports multi deletion but no error handling. This patch add the error handling follow the rule in doc/source/command-error.rst Change-Id: I3376d957b4dc28d8282599dc909ecc5ed2b5f46a
* | | Merge "Add "--network-segment" option to "subnet create""Jenkins2016-06-202-15/+84
|\ \ \ | |_|/ |/| |
| * | Add "--network-segment" option to "subnet create"Richard Theis2016-06-132-15/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add "--network-segment" option to the "subnet create" command. This is a beta command option and subject to change. Use global option "--os-beta-command" to enable this option. This patch set also provides a devref update for beta command options. Change-Id: I4d0fbe079b2a873307364c41c22ce9ba88e632e6 Partially-Implements: blueprint routed-networks
* | | Merge "Standardize logger usage"Jenkins2016-06-201-9/+6
|\ \ \
| * | | Standardize logger usageTang Chen2016-06-201-9/+6
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use file logger for all command specific logs. This patch also fixes some usage that doesn't follow rules in: http://docs.openstack.org/developer/oslo.i18n/guidelines.html After this patch, all self.log and self.app.log will be standardized to LOG(). NOTE: In shell.py, we got the log in class OpenStackShell, which is also known as self.app.log in other classes. This logger is used to record non-command-specific logs. So we leave it as-is. Change-Id: I114f73ee6c7e84593d71e724bc1ad00d343c1896 Implements: blueprint log-usage
* | | Merge "Fix console url show command broken in microversion case"Jenkins2016-06-201-0/+149
|\ \ \
| * | | Fix console url show command broken in microversion caseRui Chen2016-06-161-0/+149
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The response data of nova get_xxx_console API is changed from "console" to "remote_console" in microversion 2.6, and nova server side API schema verify the spice vnc type to "spice-html5", update OSC code to apply these change so that OSC can work in different nova microversion cases. Change-Id: I3bb4fe057e656209d00d2bb308ac3f7f837cb03f Closes-Bug: #1590318
* | | | Use osc_lib in server_image.pyTang Chen2016-06-201-2/+3
| |/ / |/| | | | | | | | | | | | | | | | | server_image.py is a newly created file. So I think we forgot to use osc_lib in it. Change-Id: Ieda13438662ea55b03f549108aac63c18b9af913
* | | Merge "Do not prompt for scope options with default scoped tokens"Jenkins2016-06-171-5/+5
|\ \ \
| * | | Do not prompt for scope options with default scoped tokensDolph Mathews2016-06-171-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changes the scope validation to occur after a token has already been created. Previous flow: 1. Validate authentication options. 2. Validate authorization options if the command requires a scope. 3. Create a token (using authentication + authorization options) 4. Run command. This means that scope was being checked, even if a default scope was applied in step 3 by Keystone. New flow: 1. Validate authentication options. 2. Create token (using authentication + authorization options) 3 Validate authorization options if the command requires a scope and the token is not scoped. 4. Run command. Change-Id: Idae368a11249f425b14b891fc68b4176e2b3e981 Closes-Bug: 1592062