summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | Add geneve provider network typeRichard Theis2016-06-101-4/+4
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add the "geneve" choice to the "os network create" command's "--provider-network-type" option. Change-Id: I7573232ec3594ec4acbfae43a8456b8c3fcd1a83 Implements: blueprint neutron-client
* | | | | | Merge "Standardize logger usage in volume"Jenkins2016-06-154-30/+42
|\ \ \ \ \ \
| * | | | | | Standardize logger usage in volumeTang Chen2016-06-144-30/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | self.app.log is the logger in class OpenStackShell, which should be used to record logs that have nothing to do with any specific command. So, use the file logger instead. This patch also fixes some usage that doesn't follow rules in: http://docs.openstack.org/developer/oslo.i18n/guidelines.html 1. add variables to logger as an argument 2. do not wrap variables with str() Change-Id: I248861a38a4de0412a080046aa7a6f6473c3e082 Implements: blueprint log-usage
* | | | | | | Merge "Fix foundation copyrights"Jenkins2016-06-147-7/+7
|\ \ \ \ \ \ \
| * | | | | | | 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
|\ \ \ \
View Lorry Controller Status