summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | Merge "Support "--long" option in ListService"Jenkins2016-03-143-10/+54
|\ \ \ \ \
| * | | | | Support "--long" option in ListServiceRui Chen2016-03-143-10/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add "--long" option in ListService so that compute service disabled reason can be showed. Change-Id: I1ace8f1c4e4efe0a1a8f6710425d73eb5db9e5e1 Closes-Bug: #1556815
* | | | | | Merge "Add "os subnet create" command using SDK"Jenkins2016-03-143-7/+557
|\ \ \ \ \ \
| * | | | | | Add "os subnet create" command using SDKBrad Behle2016-03-103-7/+557
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement the openstack client subnet create command using SDK calls. Co-Authored-By: Terry Howe <terrylhowe@gmail.com> Partially implements: blueprint neutron-client Closes-Bug: #1542364 Change-Id: Ia6120b8dccf2ee83dc89b3f496f7180d4dc5199a
* | | | | | | Merge "Use assertItemsEqual() instead of assertListEqual()"Jenkins2016-03-141-10/+2
|\ \ \ \ \ \ \
| * | | | | | | Use assertItemsEqual() instead of assertListEqual()Tang Chen2016-03-141-10/+2
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | assertListEqual() is order sensitive. So we need to sort the lists before we compare them. Use assertItemsEqual() instead is better. Change-Id: I9eaa98716c7401f5b099b007438acc916dae619b
* | | | | | | Fix dict.keys() compatibility for python 3Tang Chen2016-03-1211-11/+11
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In Python 2, dict.keys() will return a list. But in Python 3, it will return an iterator. So we need to fix all the places that assuming dict.keys() is a list. Change-Id: I8d1cc536377b3e5c644cfaa0892e40d0bd7c11b1 Closes-Bug: #1556350
* | | | | | Merge "Fixed command list"Jenkins2016-03-112-8/+28
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Fixed command listBrandon Palm2016-03-092-8/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The cliff module expects an array of tuples however the array that this function was returning was an array of tuples that was also containing an array of values for the commands attached to each group and the cliff module wasn't liking it. The output now comes out looking like: | openstack.common | limits show | | | extension list | | openstack.baremetal.v1 | baremetal set | Change-Id: Ifa1c149cb5c66ba27dc72bf72d7c8f2f50e42f73 Closes-Bug: 1545609
* | | | | | Merge "Trivial: Reorder classes in identity v3 in alphabetical order"Jenkins2016-03-116-217/+217
|\ \ \ \ \ \
| * | | | | | Trivial: Reorder classes in identity v3 in alphabetical orderTang Chen2016-03-116-217/+217
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Iaf2e336c2415416ec69f6b974743c26509bba561
* | | | | | | Merge "Enhance list extension unit test"Jenkins2016-03-112-40/+137
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | Enhance list extension unit testFang Zhen2016-03-112-40/+137
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Extension list involves identity, compute, volume and network. Current test covers only identity and network. This patch added test against compute and volum. Also refactored current implentation. Change-Id: If9b36cba24c50a817a17f685801e418fb898596a
* | | | | | Merge "Refactor security group create to use SDK"Jenkins2016-03-114-165/+203
|\ \ \ \ \ \
| * | | | | | Refactor security group create to use SDKRichard Theis2016-03-104-165/+203
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Refactored the 'os security group create' command to use the SDK when neutron is enabled, but continue to use the nova client when nova network is enabled. Added a release note for the change in security group rules output due to Network v2. The tenant_id column name was fixed to align with the 'os security group show' command. Change-Id: Ib29df42edcddcc73a123fff6a64743a6bfcb7fbf Partial-Bug: #1519511 Implements: blueprint neutron-client
* | | | | | | Merge "Refactor security group show to use SDK"Jenkins2016-03-117-92/+303
|\ \ \ \ \ \ \ | |/ / / / / / | | / / / / / | |/ / / / / |/| | | | |
| * | | | | Refactor security group show to use SDKRichard Theis2016-03-107-92/+303
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Refactored the 'os security group show' command to use the SDK when neutron is enabled, but continue to use the nova client when nova network is enabled. Added a release note for the change in security group rules output due to Network v2. The column names remain unchanged to maintain backwards compatibility. Change-Id: I25233ddb8115d18b8b88affb3de13346084a339d Partial-Bug: #1519511 Implements: blueprint neutron-client
* | | | | Merge "[Subnet pool] Add 'subnet pool create' command support"Jenkins2016-03-103-4/+138
|\ \ \ \ \
| * | | | | [Subnet pool] Add 'subnet pool create' command supportTang Chen2016-03-083-4/+138
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch supports creating a new subnet pool, with pool prefixes, default prefix length, minimum prefix length, and maximum prefix length specified. Change-Id: I9150797c8cfa794d5264ad07965aa967d9a8f5bc Partial-Bug: #1544586 Related-to: blueprint neutron-client
* | | | | | Merge "[Subnet pool] Add 'subnet pool set' command support"Jenkins2016-03-102-6/+180
|\ \ \ \ \ \ | |/ / / / / | | / / / / | |/ / / / |/| | | |
| * | | | [Subnet pool] Add 'subnet pool set' command supportTang Chen2016-03-082-6/+180
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch supports setting a new name, pool prefix, default prefix length, minimum prefix length, and maximum prefix length for a subnet pool. Change-Id: I65bd71e0f54f2f65acefbc542df67a1b1ec26397 Partial-Bug: #1544591 Related-to: blueprint neutron-client
* | | | | Merge "Trivial: Use 'SSH' rather than 'Ssh'"Jenkins2016-03-091-1/+1
|\ \ \ \ \
| * | | | | Trivial: Use 'SSH' rather than 'Ssh'Stephen Finucane2016-03-091-1/+1
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | This is reflected in the '--help' screen. Change-Id: Ic22a65ff6a56b069b37a0ea8365cce2b3f93621c
* | | | | Merge "Remove FakeFlavorResource class"Jenkins2016-03-091-29/+16
|\ \ \ \ \
| * | | | | Remove FakeFlavorResource classTang Chen2016-03-091-29/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In unit tests, all real methods are faked. They should not do any real operations in the tests. So, FakeFlavorResource is not necessary. Just fake get_keys(), set_keys and unset_keys() in FakeResource would be enough. Change-Id: Icc3473ba9c77f4817d0edddb7ff3e1bd2946fac7
* | | | | | Merge "remove py26 workaround in osc"Jenkins2016-03-091-24/+0
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | remove py26 workaround in oscSteve Martinelli2016-03-081-24/+0
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | we don't support py2.6, so let's remove the workarounds we have. Change-Id: Id9c8fda065d4aceba3192b044b5c5f2124ee204f
* | | | | Merge "[Identity] Check return value is None in identity v3 unit tests"Jenkins2016-03-096-15/+28
|\ \ \ \ \
| * | | | | [Identity] Check return value is None in identity v3 unit testsTang Chen2016-03-096-15/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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: I53eeb88316b2c20882fed97149d55cb04bcb2b2e Closes-Bug: #1550636
* | | | | | Merge "Add --reason for disable service"Jenkins2016-03-092-6/+42
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Add --reason for disable servicejichenjc2016-03-092-6/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | disbale service allow a reason to be input as disable reason. This patch add support for it. Change-Id: I59622c3970e055ebd46bf03c33c864b6d064db28
* | | | | | Add 'port set' commandJas2016-03-082-4/+127
| |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add CLI support for the 'port set' command Change-Id: I2bea508e11290284aa64b1ab548a0bb61e7290d3 Partial-bug: #1519909 Partially-implements: blueprint neutron-client
* | | | | Merge "Trivial: Fix incorrect comments in compute fakes.py"Jenkins2016-03-081-3/+3
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Trivial: Fix incorrect comments in compute fakes.pyTang Chen2016-03-081-3/+3
| | |_|/ | |/| | | | | | | | | | Change-Id: I18b1720af13b444527dda1ecab52e3cc8d8d9376
* | | | [Identity] Check return value is None in identity v3 unit testsTang Chen2016-03-089-30/+64
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | 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: I02af06b3d476aac2d93a23ef2111cdc7fa0892ec Partial-Bug: #1550636
* | | Merge "Add port list command"Jenkins2016-03-082-0/+68
|\ \ \
| * | | Add port list commandJas2016-03-072-0/+68
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds the ability to list all created ports Change-Id: Ie1a48c203cabc96346a4950f21b83493d58a66a5 Partial-bug: #1519909 Partially-implements: blueprint neutron-client
* | | | Merge "Use _get_columns() to obtain columns in network.py"Jenkins2016-03-071-4/+4
|\ \ \ \
| * | | | Use _get_columns() to obtain columns in network.pyTang Chen2016-03-061-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Objects returned by Network v2 and Compute v2 are different. When getting columns to display, Network v2 uses obj.keys(), while Compute v2 uses obj._info.keys(). But both of them could obtain the keys of the object by _get_columns(). Change-Id: I347815f2d28822a95bd6f57d429b84b7ca96e0ee
* | | | | Merge "Test take_action() instead of run() in unit tests"Jenkins2016-03-0711-101/+107
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Test take_action() instead of run() in unit testsTang Chen2016-03-0511-101/+107
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some of the unit tests test run(), but not take_action(). For example, in openstackclient/tests/volume/v1/test_volume.py, there is: def test_volume_set_size_smaller(self): ...... result = self.cmd.run(parsed_args) self.assertEqual(0, result) ...... run() is defined in class Command in cliff. We don't need to test it in OSC unit tests. On the contrary, we should test take_action(), which is overwritten in each command classes in OSC. Change-Id: If07e89953d40ac530f08cbb1ec05f5805171364b Closes-bug: #1553468
* | | | | Trivial: Remove useless returnTang Chen2016-03-079-13/+0
| |_|_|/ |/| | | | | | | | | | | | | | | | | | | If a function returns nothing, do not add return in the end. Change-Id: I298b8717462f68d3076a1619d674775be2a94c42
* | | | Merge "" openstack server image create " doesn't print proper info"Jenkins2016-03-061-5/+19
|\ \ \ \ | |_|/ / |/| | |
| * | | " openstack server image create " doesn't print proper infoMohan Muppidi2016-03-031-5/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After creating a snapshot of a running instance, a print out similar to server create is expected, but it prints out something like "_info" which is nothing related to created image. _prep_image_detail method is added to /compute/v2/server.py to enable the priting, while running the test properly. Change-Id: I4b06be959768bcdaafd9aa8df497490958bee649 Closes-Bug:1551586
* | | | Merge "Refactor security group set to use SDK"Jenkins2016-03-053-41/+189
|\ \ \ \ | |_|/ / |/| | |
| * | | Refactor security group set to use SDKRichard Theis2016-03-033-41/+189
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Refactored the 'os security group set' command to use the SDK when neutron is enabled, but continue to use the nova client when nova network is enabled. This patch set also fixes a compute bug which ignores name and description when set to an empty value. Change-Id: I4225179dca4aedf799e1656ec49236bdedc5e9bd Partial-Bug: #1519511 Implements: blueprint neutron-client
* | | Merge "[compute] Support restore server"Jenkins2016-03-042-0/+42
|\ \ \
| * | | [compute] Support restore serverjichenjc2016-02-232-0/+42
| | | | | | | | | | | | | | | | | | | | | | | | Server in soft-delete state can be restored, add this command. Change-Id: Id9d7246f89ae65273505f36dcb664996534ae986
* | | | Merge "Add 'port create' command"Jenkins2016-03-042-53/+316
|\ \ \ \
| * | | | Add 'port create' commandJas2016-03-042-53/+316
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds usage of 'port create' in CLI Change-Id: I888af50784c3b6c7ec30552ade79f05a5e974711 Partial-bug: #1519909 Partially-implements: blueprint neutron-client
View Lorry Controller Status