summaryrefslogtreecommitdiff
path: root/openstackclient/tests
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | osc-lib: utilsDean Troyer2016-06-1324-434/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use osc-lib directly for utils. Leave openstackclient.common.utils for deprecation period. Change-Id: I5bd9579abc4e07f45219ccd0565626e6667472f7
* | | | osc-lib: exceptionsDean Troyer2016-06-1323-26/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use osc-lib directly for exceptions. Leave openstackclient.common.exceptions for deprecation period. Change-Id: Iea3e862302372e1b31ccd27f69db59b4953ca828
* | | | Moving authentication from keystoneclient to keystoneauthNavid Pustchi2016-06-099-69/+237
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently OpenStackClient uses keystoneclient for authentication. This change will update OpenStackClient to use keystoneauth for authentication. All dependant test have been updated. Updating how auth_ref is set in the tests to use KSA fixtures had some racy side-effects. The user_role_list tests failed when they picked up an auth_ref that was a fixture. This exposed a weakness in ListUserRole that needed to be fixed at the same time re handling of unscoped tokens and options. Change-Id: I4ddb2dbbb3bf2ab37494468eaf65cef9213a6e00 Closes-Bug: 1533369
* | | Merge "Make set/unset commands in network return normally when nothing ↵Jenkins2016-06-096-15/+50
|\ \ \ | | | | | | | | | | | | specified"
| * | | Make set/unset commands in network return normally when nothing specifiedTang Chen2016-06-086-15/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | set/unset commands should ends up normally instead of raising an exception when nothing is specified to modify. The main reason is: When nothing is specified, the command sets/unsets nothing, which is a normal behavior, and ends up normally. No API call fails. No error happens. This patch also adds a releasenote for both network, and volume commands that fix patch has been merged. Change-Id: I78c348066078decd350417a431f3b8bea8fcf9ef Partial-bug: #1588588
* | | | Clean up fakes.py in volumev2Huanxuan Ao2016-06-081-188/+0
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | All of volumev2 fake classes are completed. Now the unit tests of volumev2 are using fake classes so that the old codes are useless and can be removed. Change-Id: I502a8a0b0404975744d3b208af3e95c8692e6c47
* | | Merge "Error handling for KeyValueAction class."Jenkins2016-06-081-10/+7
|\ \ \
| * | | Error handling for KeyValueAction class.Huanxuan Ao2016-06-081-10/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The set --property command requires that the input match the "key=value" type, but if the type don't match, the return value will be None, and the command still can be implemented successfully, this may confuse the users. I think we should raise exception if the argument type don't match "key=value". So I make some changes in KeyValueAction class in this patch. Change-Id: I14e64922faa7e083bc8b5e7e1cac41ef8117c224 Closes-Bug: #1589935
* | | | Merge "Update unit test test_extension with fake class"Jenkins2016-06-085-159/+222
|\ \ \ \ | |_|_|/ |/| | |
| * | | Update unit test test_extension with fake classHuanxuan Ao2016-06-075-159/+222
| |/ / | | | | | | | | | | | | | | | | | | Add FakeExtension class in networkv2, computev2, volumev2, identityv2_0 and update unit test test/common/test_extension.py Change-Id: I94815de7801860edb7fa91a7d146455cab946652
* | | Merge "Support error handling for "port delete" command"Jenkins2016-06-071-7/+61
|\ \ \ | |/ / |/| |
| * | Support error handling for "port delete" commandHuanxuan Ao2016-06-071-7/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "Port delete" command supported deleting multi ports before but didn't support error handing, This patch add the error handling following the rules in doc/source/command-errors.rst. Change-Id: I4ea69f2279763626d6a27cad1ca0ee99822d016d Partially-Implements: blueprint multi-argument-network
* | | Merge "Fix errors for "volume type unset" command"Jenkins2016-06-071-2/+4
|\ \ \
| * | | Fix errors for "volume type unset" commandHuanxuan Ao2016-06-061-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Normally, we can unset multi properties but the "volume type unset" command could not, because the action "append" was missed. So I add it and also fix the unit test and doc in this patch. Change-Id: I20470f2b7bb2a8d7f292cea498826669c3418c77
* | | | Merge "Refactor SetService --enable/disable option"Jenkins2016-06-061-31/+26
|\ \ \ \
| * | | | Refactor SetService --enable/disable optionTang Chen2016-06-041-31/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch changes the following: 1. --enable/disable option should follow the rules in the doc below: http://docs.openstack.org/developer/python-openstackclient/command-options.html#boolean-options 2. "--disable-resion" is specified but not "--disable", an exception is raised instead of igoring "--disable-reason" option. Change-Id: I92e9234111e661bfe7119a8e19389a87c874ab0c
* | | | | Merge "Make set/unset commands in volume return normally when nothing specified"Jenkins2016-06-061-3/+0
|\ \ \ \ \
| * | | | | Make set/unset commands in volume return normally when nothing specifiedTang Chen2016-06-041-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | set/unset commands should ends up normally instead of logging an error when nothing is specified to modify. The main reason is: When nothing is specified, the command sets/unsets nothing, which is a normal behavior, and ends up normally. No API call fails. No error happens. Change-Id: Ib03a512650e5da90aa1ef38019772448383d0d33 Partial-bug: #1588588
* | | | | | Merge "Add server set/unset unit test cases"Jenkins2016-06-061-0/+87
|\ \ \ \ \ \
| * | | | | | Add server set/unset unit test casesRui Chen2016-06-061-0/+87
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unit test cases don't cover compute "server set/unset" commands, the patch add some test cases for them. Change-Id: I440c32968bd41b948352a9764a37c9af3e68803d
* | | | | | Merge "Add network availability for osc"Jenkins2016-06-062-0/+224
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Add network availability for oscManjeet Singh Bhatia2016-06-032-0/+224
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch implements openstack client for network ip availability. Implements: blueprint neutron-ip-capacity Depends-On: I3b40d8edea87c068c4e8133e436511765064d5f8 Change-Id: Iffaa2e20ff495fbd205d3397e027e8141d04385e
* | | | | | Merge "Support multiple argument for compute agent delete command"Jenkins2016-06-051-5/+56
|\ \ \ \ \ \
| * | | | | | Support multiple argument for compute agent delete commandsunyajing2016-06-031-5/+56
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | Change-Id: I3b19e4914d475b86d7e8aa8d76e62a2ac811272f Partially-Implements: blueprint multi-argument-compute
* | | | | | Merge "Add support for volume transfer request list"Jenkins2016-06-044-4/+334
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | |
| * | | | | Add support for volume transfer request listSheel Rana2016-06-044-4/+334
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | OSC does not support to list volume transfer from one user to other user. This patch will provide support for listning volume transfer requests. Closes-Bug:#1554886 Implements: bp cinder-command-support Change-Id: Ie659bc39cb1d1e931cca7a40b2f126a067ee484c
* | | | | | Merge "Fix wrong test in flavor unit tests"Jenkins2016-06-041-5/+3
|\ \ \ \ \ \
| * | | | | | Fix wrong test in flavor unit testsTang Chen2016-06-031-5/+3
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In test_flavor_set_no_project(), we aimed to test a situation like this: User specifies "--project" option, but didn't specifies the project name or ID. But in the source code, it becomes "--project ''". The test could past because if project and property are both None, the command will raise an exception. Change-Id: I39567306debb901e8bad420fa2492f1b207efddc
* | | | | | Merge "Check port name in set port tests"Jenkins2016-06-041-1/+5
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | |
| * | | | | Check port name in set port testszhouqi2016-06-031-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I1bf11245b107f82fedee70dacc37c4c6dc5210ea
* | | | | | Merge "Fix --enable options on commands"Jenkins2016-06-043-4/+19
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Fix --enable options on commandsRichard Theis2016-06-023-4/+19
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The --enable option on commands is ignored when the arguments are parsed. This is related to the --enable-beta-commands option. Renaming the option to --os-beta-command fixes the problem. There's no need to handle backwards compatibility for the option name change because there hasn't been an OSC release yet with beta commands. Change-Id: I0327ba8a2058858a83e9a42e231470ed733cc834 Closes-Bug: #1588384
* | | | | Merge "Add support for removing flavor-access"Jenkins2016-06-021-2/+86
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Add support for removing flavor-accessHuanxuan Ao2016-06-021-2/+86
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add "--project" and "--project-domain" options in "flavor unset" command to remove flavor access. Change-Id: Ia5c5a80d9890d5af066b75b4e202647c18c7d915 Partial-Bug: #1575461
* | | | | Merge "Add FakeQos class and update unit test for qos_specs in VolumeV2"Jenkins2016-06-022-204/+207
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Add FakeQos class and update unit test for qos_specs in VolumeV2Huanxuan Ao2016-05-312-204/+207
| | | | | | | | | | | | | | | | | | | | Change-Id: I108c8d343678f3d19d1ca1f93d998c46c7c32eff
* | | | | Merge "Fix unit test for volume commands in volumev2"Jenkins2016-05-311-27/+21
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Fix unit test for volume commands in volumev2Huanxuan Ao2016-05-311-27/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are some issues in test_volume.py in volumev2, I make three changes in this patch: 1.modified some codes that not used FakeVolume class. 2.added a fake image by FakeImage class for this test. 3.added a fake snapshot by FakeSnapshot class for this test. Change-Id: I02ba73d3aaee95624b0e2307b255e0e485b0c3a3
* | | | | Merge "add unit test for compute agent command"Jenkins2016-05-312-0/+255
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | add unit test for compute agent commandsunyajing2016-05-312-0/+255
| |/ / / | | | | | | | | | | | | Change-Id: I966d5a3a307fcd7f4efb1267aa2896efd53be50d
* | | | Merge "Add support for setting flavor-access"Jenkins2016-05-302-2/+100
|\ \ \ \
| * | | | Add support for setting flavor-accessHuanxuan Ao2016-05-302-2/+100
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds "--project" option in "flavor set" command to support for setting flavor access. Change-Id: I75b473600080d8ab1dd6ad01561c4f989ed3c3bd Partial-Bug: #1575461
* | | | Merge "Add network segment command object"Jenkins2016-05-304-0/+257
|\ \ \ \
| * | | | Add network segment command objectRichard Theis2016-05-274-0/+257
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add network segment command object in support of routed networks. This patch set includes documentation, unit tests and functional tests (currently skipped until segments enabled in neutron by default) for the following new commands: - "os network segment list" - "os network segment show" These new commands are currently marked as beta commands. Change-Id: I1a79b48dc6820fe2a39fcceb11c8cae3bda413a0 Partially-Implements: blueprint routed-networks
* | | | Merge "Add "image unset" command"Jenkins2016-05-302-0/+95
|\ \ \ \
| * | | | Add "image unset" commandsunyajing2016-05-282-0/+95
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch add a command that supports unsetting image tags and properties Change-Id: I6f2cf45a61ff89da6664f3a34ae49fdd85d8c986 Closes-Bug:#1582968
* | | | Merge "[compute] Add server backup function"Jenkins2016-05-302-0/+273
|\ \ \ \
| * | | | [compute] Add server backup functionjichenjc2016-05-272-0/+273
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add server backup function There is no return value for this command per following doc http://developer.openstack.org/api-ref-compute-v2.1.html#createBackup, also novaclient can't be updated now due to backward compatible issue http://lists.openstack.org/pipermail/openstack-dev/2016-March/089376.html, so we have to get the information ourselves. The Image tests were not using warlock images, so that needed to be fixed before we could completely test things like --wait. Change-Id: I30159518c4d3fdec89f15963bda641a0b03962d1
* | | | Merge "Support to set server state"Jenkins2016-05-301-0/+61
|\ \ \ \
| * | | | Support to set server stateRui Chen2016-05-281-0/+61
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a server is unexpected state, OSC don't support reset the server to active or error state, that's supported by novaclient, and it's an important command for operators, the patch implement this function. Change-Id: I3e7800feb192832b0719ef9a353945beb6bfd509 Implements: blueprint server-reset-state