summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | take_action() method from command.Command shouldn't returnMohan Muppidi2016-02-2916-43/+18
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | command.Command and command.Showone are base classes implemented in cliff framework. Showone extends Command to allow take_action() to return data to be formatted using a user-selectable formatter. Most of the classes which are extended from Command in openstackclient/identity/v3/ in some cases return data or return nothing where it is not necessary, this commit fixes most of them. Change-Id: I84c72ea4d6680f8bdbef5449316dd9a8af8c8286 Closes-Bug: 1550892
* | | | Merge "Make SetSecurityGroup inherit from cliff.Command"Jenkins2016-02-294-17/+27
|\ \ \ \ | | |/ / | |/| |
| * | | Make SetSecurityGroup inherit from cliff.CommandTang Chen2016-02-274-17/+27
| | | | | | | | | | | | | | | | | | | | | | | | set/unset comamnd classes should inherit from cliff.Command class. Change-Id: Ie28711ac8823dc9eb13cf83877864ca436b928bc Partial-Bug: 1546065
* | | | Merge "Make SetFlavor and UnsetFlavor inherit from cliff.Command"Jenkins2016-02-295-32/+39
|\ \ \ \ | |/ / /
| * | | Make SetFlavor and UnsetFlavor inherit from cliff.CommandTang Chen2016-02-275-32/+39
| | | | | | | | | | | | | | | | | | | | | | | | set/unset comamnd classes should inherit from cliff.Command class. Change-Id: I54e5608ac0768d7d94b7f7d516ea1948daefdc1b Partial-Bug: 1546065
* | | | Merge "[Compute] Check return value is None in compute unit tests."Jenkins2016-02-293-45/+33
|\ \ \ \
| * | | | [Compute] Check return value is None in compute unit tests.Tang Chen2016-02-273-45/+33
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | take_action() in commands inheriting from Command returns nothing. So we should assert the return is None in the unit tests of these commands. Change-Id: I953480ecff3b5beb12255d866d0e1df45f130efd Partial-Bug: #1550636
* | | | Router: Add --route and --clear-routes options to "router set" commandTang Chen2016-02-273-4/+95
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | --route option is used to set routes to the router. It is used like this: --route destination=subnet,gateway=ip-address destination: destination subnet CIDR gateway: nexthop IP address --clear-routes is used to clear all routes on the router. Change-Id: I97ce4871113c684b29c98cdad4dec9cc80ed20f7 Implements: blueprint neutron-client Partial-bug: #1519503
* | | | Add MultiKeyValueAction to custom parser actionTang Chen2016-02-272-0/+210
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Class MultiKeyValueAction will be used to parse arguments like this: --route destination=xxx,gateway=xxx --route destination=yyy,gateway=yyy The result is a list like this: [{destination:xxx, gateway:xxx}, {destination:yyy, gateway:yyy}] This action also contain validation of the parameters. Change-Id: Ie3aa8635c6a13fc2e429fe6922acd681dc7244cf
* | | Merge "Add shell --profile option to trigger osprofiler from CLI"Jenkins2016-02-265-0/+76
|\ \ \ | |/ / |/| |
| * | Add shell --profile option to trigger osprofiler from CLIDina Belova2016-02-255-0/+76
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will allow to trigger profiling of various services that allow it currently and which APIs support is added to openstackclient. Cinder and Glance have osprofiler support already, Nova and Keystone are in progress. To use this functionality osprofiler (and its storage backend) needs to be installed in the environment. If so, you will be able to trigger profiling via the following command, for example: $ openstack --profile SECRET_KEY user list At the end of output there will be message with <trace_id>, and to plot nice HTML graphs the following command should be used: $ osprofiler trace show <trace_id> --html --out result.html Related Keystone change: https://review.openstack.org/#/c/103368/ Related Nova change: https://review.openstack.org/#/c/254703/ The similar change to the keystoneclient (https://review.openstack.org/#/c/255308/) was abandoned as new CLI extenstions are not more accepted to python-keystoneclient. Change-Id: I3d6ac613e5da70619d0a4781e5d066fde073b407
* | | Merge "Add missing command/configuration object"Jenkins2016-02-265-2/+89
|\ \ \
| * | | Add missing command/configuration objectRui Chen2016-02-265-2/+89
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add missing 'command' and 'configuration' object into command object list, add command, module, host commands describe in command-objects, and add 'mask' and 'unmask' options in configuration command details, fix some format issues by the way. Change-Id: Iea80c7b1e413e02e49b9090b9d3cb9c59aab4c38
* | | | Updated from global requirementsOpenStack Proposal Bot2016-02-261-1/+1
|/ / / | | | | | | | | | Change-Id: I582644850642651c61d259e4a53f670b421dce95
* | | Merge "Defaults are ignored with flake8"Jenkins2016-02-265-6/+10
|\ \ \
| * | | Defaults are ignored with flake8Tom Cocozzello2016-02-235-6/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If “ignore” is not set under flake8 in the tox.ini file there there are defaults set to be ignored. The depended patch fixes many of the problems. Change-Id: Ieed2fe1c4654e201d3fe6d40ef93e247ee736f8b Doc: http://flake8.readthedocs.org/en/latest/config.html#default Depends-On: I935ab48e7c5bac5f88ecdb3a05f73fb44fc9f41d Closes-Bug: #1548910
* | | | Merge "Floating IP: Neutron support for "ip floating show" command"Jenkins2016-02-257-2/+175
|\ \ \ \ | |_|_|/ |/| | |
| * | | Floating IP: Neutron support for "ip floating show" commandTang Chen2016-02-247-2/+175
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I30350076621c83c758927444e5f8bcc2b7d0fc74 Partial-Bug: 1519502 Related-to: blueprint neutron-client
* | | | Merge "Fixed a bunch of spacing"Jenkins2016-02-2547-76/+183
|\ \ \ \ | | |/ / | |/| |
| * | | Fixed a bunch of spacingBrandon Palm2016-02-2347-76/+183
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Nothing too complicated here. I fixed a bunch of spacing issues that I saw in OSC. Change-Id: I935ab48e7c5bac5f88ecdb3a05f73fb44fc9f41d
* | | | Merge "Add python-searchlightclient to list of adopters."Jenkins2016-02-251-0/+1
|\ \ \ \
| * | | | Add python-searchlightclient to list of adopters.Travis Tripp2016-01-211-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds python-searchlightclient to the list of adopters. Change-Id: Ifb6f1931c00fc735a0b0fcc830979a4d83c0527d
* | | | | Merge "update heat object and command doc"Jenkins2016-02-253-2/+16
|\ \ \ \ \
| * | | | | update heat object and command docMark Vanderwiel2016-02-253-2/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Many OpenStack client heat command patches are in progress, seems like good time to also update some of the highlevel doc. Added new objects for orchestration and software openstack orchestration resource type list/show openstack software config create/delete/show/list Added new action for restore openstack stack snapshot create/delete/list/show/restore Added link to CLI reference for complete plugin syntax and details Removed deprecated tasker client Change-Id: I2c94a5981954edcba95f364a0f909c799f1c403b Blueprint: heat-support-python-openstackclient
* | | | | | Merge "Add some test cases for "server list" command"Jenkins2016-02-252-27/+78
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Add some test cases for "server list" commandting.wang2016-02-242-27/+78
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add some test cases that test 'server list' command when specifying flavor or image. Because I add some attribution to fake.py, I have to change some code in create server test. Despite all this, I think it's good for testing. Change-Id: I714deac1f6f940b790a3c20af5f7ffa724ac44d1
* | | | | Merge "Improve tox to show coverage report on same window"Jenkins2016-02-241-1/+3
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Improve tox to show coverage report on same windowting.wang2016-02-241-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With this patch coverage report will be shown with the results when tox -e cover will be run. Change-Id: I96713a8dd5d82019631c9e48c1abd7a94e201569
* | | | | Merge "Refactor network AZ exception handling"Jenkins2016-02-241-3/+2
|\ \ \ \ \
| * | | | | Refactor network AZ exception handlingRichard Theis2016-02-101-3/+2
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Exceptions that occur while getting network availability zones should not be masked as if the extension does not exist. Change-Id: I07213ec6c4d83e97261b58bf8d42417c1cdfae6a Related-Bug: #1534202
* | | | | Merge "Py3 replace dict.iteritems with six.iteritems"Jenkins2016-02-241-1/+1
|\ \ \ \ \
| * | | | | Py3 replace dict.iteritems with six.iteritemsting.wang2016-02-241-1/+1
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All dict.iteritems in osc are replaced with six.iteritems except this one. So fix it to add py3 compatibility. Change-Id: I1aa51399a36e650d262d839ce2b4ec04d3f91db2
* | | | | Updated from global requirementsOpenStack Proposal Bot2016-02-241-1/+1
|/ / / / | | | | | | | | | | | | Change-Id: Ib681d0b07afdde821ab4f9a17b1333bd12739eec
* | | | Merge "Use update_parser_common() in ShowNetwork"Jenkins2016-02-231-2/+1
|\ \ \ \
| * | | | Use update_parser_common() in ShowNetworkTang Chen2016-02-231-2/+1
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | ShowNetwork inherits from NetworkAndComputeCommand. So we should use update_parser_common() in it, not overwrite parent's get_parser(). Change-Id: I21bb1407962344b9800fd31caee4b2582674fe24
* | | | Merge "Add "security group rule show" command"Jenkins2016-02-237-4/+236
|\ \ \ \
| * | | | Add "security group rule show" commandRichard Theis2016-02-237-4/+236
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add the "os security group rule show" command which will use the SDK when neutron is enabled, and use the nova client when nova network is enabled. Change-Id: I41efaa4468ec15e4e86d74144cc72edc25a29024 Partial-Bug: #1519512 Implements: blueprint neutron-client
* | | | | Fix wrong return value in TestDeleteFloatingIPNetworkTang Chen2016-02-231-1/+1
| |/ / / |/| | | | | | | | | | | | | | | | | | | delete_ip() should return None, not the fake floating IP. Change-Id: I1476189a09a94c76c90f9a3986e3ae57dc66d796
* | | | Merge "Use instanceof instead of type"Jenkins2016-02-233-6/+6
|\ \ \ \
| * | | | Use instanceof instead of typeBrandon Palm2016-02-223-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adjusted conditional statements to use instanceof when comparing variables. Instanceof supports inheritance type checking better than type. Change-Id: I4ee0004934dc2322d43ef07e797a6811e39a812c Closes-Bug: 1548530
* | | | | Merge "Add functional tests for "volume" commands v2"Jenkins2016-02-234-2/+116
|\ \ \ \ \
| * | | | | Add functional tests for "volume" commands v2Tang Chen2016-02-214-2/+116
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The tests for v2 "volume" commands are quite similar to v1. This patch also map 'metadata' to 'properties', 'volume_type' to 'type' to align to the v1 output. Change-Id: Icf2c5463b186fc78c890ccd96453090c4a2c2eb6 Partial-bug: #1519503
* | | | | | Merge "Add unit tests for 'hypervisor stats' command"Jenkins2016-02-232-0/+140
|\ \ \ \ \ \
| * | | | | | Add unit tests for 'hypervisor stats' commandting.wang2016-02-222-0/+140
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'hypervisor stats show' command isn't covered by unit tests, so add unit tests to test it. Change-Id: Ic355230cbdd596e848191b599803dca7f27c2ffb
* | | | | | | Merge "Add "os subnet show" command using SDK"Jenkins2016-02-236-2/+133
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | |
| * | | | | | Add "os subnet show" command using SDKBrad Behle2016-02-226-2/+133
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement the openstack client subnet show command using SDK calls. This shows the details of a specific subnet. Co-Authored-By: Terry Howe <terrylhowe@gmail.com> Partially implements: blueprint neutron-client Closes-Bug: #1542359 Change-Id: Iaf18b9e44af35ca0cd61033b468e0c60cd3b05d6
* | | | | | Initialize _keys in __init__() in FakeFlavorResourceTang Chen2016-02-231-2/+5
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | _keys is defined as a class attribute in FakeFlavorResource. So when we call set_keys() to update it, it changes. And this change may bring trouble to the other tests afterward. So define and initialize it in __init__() as an object attribute. Change-Id: Ib18c03877b67e1b7c2e107f598076b928a58e4fb Closes-bug: #1548378
* | | | | Merge "Add unit test for "flavor show" command"Jenkins2016-02-222-10/+89
|\ \ \ \ \
| * | | | | Add unit test for "flavor show" commandTang Chen2016-02-202-10/+89
| |/ / / / | | | | | | | | | | | | | | | Change-Id: I1591649e5b97a885707042fcccad3335ee8c7aec
* | | | | Merge "gitignore .idea"Jenkins2016-02-221-0/+1
|\ \ \ \ \
View Lorry Controller Status