summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | Fix foundation copyrightsDean Troyer2016-06-147-7/+7
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The OpenStack LLC copyrights date from before OpenStack was transferred to the foundation. These appear to be getting copy-pasted to new files so at least should reflect the correct entity. Change-Id: I02953d752cb24ead6aa4ad8bfe257a48317c9f13
* | | | | | | Merge "Remove blank line in release notes"Jenkins2016-06-141-1/+0
|\ \ \ \ \ \ \
| * | | | | | | Remove blank line in release notesSamYaple2016-06-141-1/+0
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The blank line was causing yamllint to fail in other projects. [1] Related-Id: I400131bf84a4cf48a83952efa7dfac8e4954b587 [1] http://logs.openstack.org/64/329564/1/check/gate-heat-templates-pep8/e538974/console.html#_2016-06-14_17_39_02_316 Change-Id: Icfd902b6c9408ccb9a710d10d52d65efe753b77f
* | | | | | | Merge "Fix errors in ``set/unset flavor`` unit tests"Jenkins2016-06-141-9/+20
|\ \ \ \ \ \ \
| * | | | | | | Fix errors in ``set/unset flavor`` unit testsTang Chen2016-06-141-9/+20
| | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. We should assert flavor.set_keys() is called correctly or not called in ``set flavor`` unit tests. 2. Commit c7e6973ff50ff84af9ad55f7bdaeeea83ae40f0b from me removed test_flavor_unset_no_project which used to test if --project was specified as '', and assert CommandError was raised, which is incorrect. So I removed it. But after looking into the code, I think we should not remove it, but specify nothing after --project, and assert ParserException was raised. So in this patch, we fix it as so. ('--project', '') --> ('--project') assert CommandError --> assert ParserException Change-Id: Ifd33c72d5b7581aaabffb09e9b5e38ecc67e18c0
* | | | | | | Fix missing i18n supports in api/ and shell.pyHuanxuan Ao2016-06-144-37/+52
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | Change-Id: I28d79d7f44b27d2b600dedad2a3601180650ad83 Partial-bug: #1574965
* | | | | | Merge "Ensure endpoint type is used for network commands"Jenkins2016-06-141-0/+1
|\ \ \ \ \ \
| * | | | | | Ensure endpoint type is used for network commandsStuart McLaren2016-06-141-0/+1
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently OS_ENDPOINT_TYPE and --os-interface are being ignored for network commands. This means the public URL is always used. Make sure that these are picked up correctly so we hit the correct endpoint (internal/admin/etc.) for commands such as: $ openstack --os-interface internal network list Change-Id: Iac05204e3056e386d84d3644b5da1a2bb322bb0a Closes-bug: 1592368
* | | | | | Merge "osc-lib: timing"Jenkins2016-06-143-119/+9
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | osc-lib: timingDean Troyer2016-06-133-119/+9
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I3fe27d98efa5090e084c676f7f8e6dad0157ed21
* | | | | | Merge "Fix image delete multiple arguments error"Jenkins2016-06-142-5/+50
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Fix image delete multiple arguments errorsunyajing2016-06-132-5/+50
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix image delete command, support processing multiple arguments delete error. Fix doc/source/command-errors.rst, make the msg format correct. Change-Id: Icbe347fe077bc148bf71ea8f7399b0e934b7cdf9 Partially-Implements: blueprint multi-argument-image
* | | | | Merge "osc-lib: command"Jenkins2016-06-1481-88/+82
|\ \ \ \ \ | | |/ / / | |/| | |
| * | | | osc-lib: commandDean Troyer2016-06-1381-88/+82
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Leave command.py and test_command.py as a sanity check during the deprecation period. Change-Id: I24e1b755cbfbcbcaeb5273ec0c9706b82384fc85
* | | | | Merge "osc-lib: parseractions"Jenkins2016-06-1426-174/+36
|\ \ \ \ \ | |/ / / /
| * | | | osc-lib: parseractionsDean Troyer2016-06-1326-174/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Leave parseractions.py and test_parseractions.py as a sanity check during the deprecation period. Change-Id: I1a7469b6d872284e0276502a1a287bc0b87f8f83
* | | | | Merge "osc-lib: logs"Jenkins2016-06-143-185/+18
|\ \ \ \ \ | |/ / / /
| * | | | osc-lib: logsDean Troyer2016-06-133-185/+18
| | | | | | | | | | | | | | | | | | | | Change-Id: I2a4d40cd72cc22e97a600751ae29c2309ebed28b
* | | | | Merge "Fix i18n problems for common files in identity"Jenkins2016-06-142-20/+22
|\ \ \ \ \
| * | | | | Fix i18n problems for common files in identityTang Chen2016-06-142-20/+22
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some missing parts in identity. Change-Id: I8777b845613d7d7df36ac3c198da552e11aaad1b Partial-bug: #1574965
* | | | | Merge "Fix errors in flavor unit tests"Jenkins2016-06-141-24/+9
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Fix errors in flavor unit testsTang Chen2016-06-131-24/+9
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes the problems below: 1. flavor.unset_keys() is not checked in flavor unit tests. So check them in the tests. 2. test_flavor_unset_no_project makes no sense. It is OK to specify ``--project ''`` without raising any exception. It can pass because in the test, we set nither project nor property. So remove this test. Change-Id: I04e537349936343b6d8c85b06bc6d0ba6bd41d6f
* | | | osc-lib: utilsDean Troyer2016-06-13112-546/+183
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use osc-lib directly for utils. Leave openstackclient.common.utils for deprecation period. Change-Id: I5bd9579abc4e07f45219ccd0565626e6667472f7
* | | | osc-lib: exceptionsDean Troyer2016-06-1351-61/+93
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use osc-lib directly for exceptions. Leave openstackclient.common.exceptions for deprecation period. Change-Id: Iea3e862302372e1b31ccd27f69db59b4953ca828
* | | | Merge "Moving authentication from keystoneclient to keystoneauth"Jenkins2016-06-1317-145/+329
|\ \ \ \
| * | | | Moving authentication from keystoneclient to keystoneauthNavid Pustchi2016-06-0917-145/+329
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | Use osc-lib and set up deprecation warningsDean Troyer2016-06-102-520/+16
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The initial use of osc-lib is behind the compatibility/deprecation modules that we will leave in place for a time for plugins to catch up. * openstackclient.common.exceptions * openstackclient.common.utils Module-level warnings are emitted directly on stderr since logging has not been configured yet. Change-Id: I79e57ce9523a20366bccaf9b949ab5906792ea0d
* | | Merge "Make set/unset commands in network return normally when nothing ↵Jenkins2016-06-0912-38/+50
|\ \ \ | | | | | | | | | | | | specified"
| * | | Make set/unset commands in network return normally when nothing specifiedTang Chen2016-06-0812-38/+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
* | | | Merge "Fix network"Jenkins2016-06-094-7/+7
|\ \ \ \
| * | | | Fix networksunyajing2016-06-084-7/+7
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | modify ``columns, data`` to ``(columns, data)`` in network, to keep code consistence. Change-Id: I0522c499d1651b4ea948bebccbdfd520934e8e6a
* | | | Merge "Remove duplicate file logger formatter setting"Jenkins2016-06-081-2/+0
|\ \ \ \
| * | | | Remove duplicate file logger formatter settingRui Chen2016-06-081-2/+0
| |/ / / | | | | | | | | | | | | Change-Id: Ia7851a4266fa61658c48cc894163c6afb7252750
* | | | 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 "Fix compute service set command"Jenkins2016-06-081-1/+1
|\ \ \
| * | | Fix compute service set commandsunyajing2016-06-081-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | add ``Binary name`` to help msg of compute service set command. Change-Id: I5ed3824ec5bede250ce41f187835e8901b5e11fd
* | | | Merge "Error handling for KeyValueAction class."Jenkins2016-06-082-11/+10
|\ \ \ \
| * | | | Error handling for KeyValueAction class.Huanxuan Ao2016-06-082-11/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "Fix i18n supports in commom"Jenkins2016-06-078-50/+83
|\ \ \ \
| * | | | Fix i18n supports in commomHuanxuan Ao2016-06-078-50/+83
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | I checked all the files in openstackclient/common and fixed the missing i18n supprots. Change-Id: Id7f76a24aae663f5832ef9bcf1bd5a6b7081af24 Partial-bug: #1574965
* | | | Merge "Support error handling for "port delete" command"Jenkins2016-06-072-9/+76
|\ \ \ \ | |/ / / |/| | |
| * | | Support error handling for "port delete" commandHuanxuan Ao2016-06-072-9/+76
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "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
* | | | Fix i18n support problems in identityTang Chen2016-06-0725-319/+364
| | | | | | | | | | | | | | | | | | | | Change-Id: I3b48d17850343051239b5b69e8b890dba32d3ac8 Partial-bug: #1574965
* | | | Merge "Fix errors for "volume type unset" command"Jenkins2016-06-072-2/+5
|\ \ \ \
| * | | | Fix errors for "volume type unset" commandHuanxuan Ao2016-06-062-2/+5
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | 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-062-46/+49
|\ \ \ \
| * | | | Refactor SetService --enable/disable optionTang Chen2016-06-042-46/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-069-47/+0
|\ \ \ \ \