summaryrefslogtreecommitdiff
path: root/openstackclient/tests/unit
Commit message (Collapse)AuthorAgeFilesLines
* Funcional tests: quota listDean Troyer2017-05-021-254/+428
| | | | | | | | | | | | The quota list tests have a race in them where occasionally a project is deleted in another test between the time that quota list gets a list of all projects and it gets the quota for the projects from the service; the get quota call fails on the non-existant project. The quota list functional tests have been substantially re-written to properly test the exception handling. Change-Id: I71e6bbb5d46fcea4718a5a870f9a66a2c20fff0f
* Merge "Fix volume qos spec list"Jenkins2017-04-292-17/+76
|\
| * Fix volume qos spec listDean Troyer2017-04-282-17/+76
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This has been sporadically failing in functional tests due to the way the volume qos spec list command calls get_associations() for each spec. When tests run in parallel occasionally a spec from another test is present in the list returned and is deleted before the get_associations() call is made, causing a NotFound exception. We should just keep going when this occurs. * make v1 match v2 * add tests to ensure the exception is being caught and handled Closes-Bug: #1687083 Change-Id: If2d17c1deb53d293fc2c7f0c527a4e4ef6f69976
* | Fix Nova-net netowrk commandsDean Troyer2017-04-272-15/+33
|/ | | | | | | | | | | | In cleaning up functional tests for nova-net, I discovered some problems in network create: * --subnet option is required in network create command * Switch API to use /os-networks rather than /os-tenant-networks as this is what we were actually using via novaclient * Fix functional tests for nova-net * Normalize some private function names in network/v2/network.py Change-Id: I426b864406756d58d140575a3a45ee9aee67ce84
* Merge "Fix NoneType error for volume snapshot create command"Jenkins2017-04-262-33/+18
|\
| * Fix NoneType error for volume snapshot create commandHuanxuan Ao2017-04-262-33/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | In volume snapshot command, <volume> is the same as <snapshot-name> when --volume is not specified, but <volume> cannot be None, so when <snapshot-name> is not specified (<snapshot-name> is None), a NoneType error appears. So make <snapshot-name> no longer optional, it should be always present. Change-Id: I3d9f10753a8ef601e70816421c160598e2cc811f Closes-bug: #1659894
* | Merge "SDK refactor: Set "is_admin_state_up" for network agent"Jenkins2017-04-261-2/+2
|\ \ | |/ |/|
| * SDK refactor: Set "is_admin_state_up" for network agentHuanxuan Ao2017-02-021-2/+2
| | | | | | | | | | | | | | | | OpenStackSDK is >=0.9.13 now in requirement, so we can update "is_admin_state_up" as well for the SDK refactor Change-Id: I02de0ebc752ce602032bbe9d73256ed376993e78
* | Merge "Make test_server.py more elegant and simple"Jenkins2017-04-251-14/+7
|\ \
| * | Make test_server.py more elegant and simpleDavid Rabel2017-04-211-14/+7
| | | | | | | | | | | | | | | | | | | | | | | | Use only one line for mocking network resources like ports and networks in test_server.py. Change-Id: I451a504c3afdd365e64d66079516ec6308c206db Depends-On: I624b1bc557a195bdf8a7c5a32dc0e72a6fa8b075
* | | Merge "Add --network and --port to server create"Jenkins2017-04-251-10/+34
|\ \ \ | |/ /
| * | Add --network and --port to server createDavid Rabel2017-04-071-10/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | --nic option is quite unhandy. It is better to have two seperate options --network and --port to add a network to a new server. Change-Id: I523abdc83ca2dd4c5dd3871f8f109c2bf57c2e02 Closes-Bug: #1612898
* | | Improve no-auth pathDean Troyer2017-04-241-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The commands that do not require authentication sometimes still need to call ClientManager.is_network_endpoint_enabled() to see if Neutron is available. Optimize the paths a bit to skip auth when it is not necessary; the upshot is Neutron will be assumed in these cases now. This gets a LOT cleaner when it appears is a future osc-lib. Change-Id: Ifaddc57dfa192bde04d0482e2cdcce111313a22a
* | | Low-level Compute v2 API: floating ip poolDean Troyer2017-04-184-26/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | api.compute.APIv2 floating ip pool function. novaclient 8.0 is now released without support for the previously deprecated nova-net functions, so include a new low-level REST implementation of the removed APIs. Also includes a handful of cleanups that the previous security group and floating IP reviews missed. Change-Id: I20116ec4fc1113857d8d917bfb30fa3170d05b9f
* | | Low-level Compute v2 API: networkDean Troyer2017-04-174-167/+317
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | api.compute.APIv2 network functions. novaclient 8.0 is now released without support for the previously deprecated nova-net functions, so include a new low-level REST implementation of the removed APIs. Change-Id: If230f128e91cda44461fe93c976cac2aecec2252
* | | Split network testsDean Troyer2017-04-172-372/+392
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Split up the network unit tests between compute and network APIs into separate files in preparation for reworking the compute (nova-net) implementations to deal with the removal of deprecated nova-net support in novaclient 8.0.0. No code changes are intended here, just splitting two files into four. Change-Id: I2d001118af436f95025d2851341f8ca802e78830
* | | Merge "Low-level Compute v2 API: floating ip"Jenkins2017-04-173-75/+186
|\ \ \
| * | | Low-level Compute v2 API: floating ipDean Troyer2017-04-113-75/+186
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | api.compute.APIv2 floating ip functions. novaclient 8.0 is now released without support for the previously deprecated nova-net functions, so include a new low-level REST implementation of the removed APIs. Change-Id: Ic461b8d15e072e0534dcd73fff6857581d83c89b
* | | | Merge "Low-level Compute v2 API: security group rules"Jenkins2017-04-174-109/+195
|\ \ \ \
| * | | | Low-level Compute v2 API: security group rulesDean Troyer2017-04-114-109/+195
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | api.compute.APIv2 security group rule functions. novaclient 8.0 is now released without support for the previously deprecated nova-net functions, so include a new low-level REST implementation of the removed APIs. Change-Id: Ieabd61113bc6d3562738686f52bb06aa84fca765
* | | | Merge "Low-level Compute v2 API: security group"Jenkins2017-04-176-208/+455
|\ \ \ \ | |/ / /
| * | | Low-level Compute v2 API: security groupDean Troyer2017-04-116-208/+455
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | api.compute.APIv2 starts with security group functions. novaclient 8.0 is now released without support for the previously deprecated nova-net functions, so include a new low-level REST implementation of the removed APIs. Change-Id: Id007535f0598226a8202716232313e37fe6247f9
* | | | Merge "Allow override of distributed router flag"Jenkins2017-04-171-0/+29
|\ \ \ \ | |/ / / |/| | |
| * | | Allow override of distributed router flagvenkata anil2017-04-061-0/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When router_distributed=True is set in neutron.conf, user can't override this and create a centralized router through openstack client. Openstack client allows modifying routers from distributed to centralized after creation but not supporting centralized flag during creation. We allow centralized and distributed flags during router creation with this change. Closes-bug: #1664255 Change-Id: I75f72ca695338ad8c381cfa89fbb9d8e61ee7bc5
* | | | Split floating IP testsDean Troyer2017-04-104-282/+316
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Split up the floating IP unit tests between compute and network APIs into separate files in preparation for reworking the compute (nova-net) implementations to deal with the removal of deprecated nova-net support in novaclient 8.0.0. No code changes are intended here, just splitting two files into four. Change-Id: Id62148bb21e913116f9f2084c5761cfa24e8d34c
* | | | Merge "Split security group tests"Jenkins2017-04-074-917/+959
|\ \ \ \
| * | | | Split security group testsDean Troyer2017-04-064-917/+959
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Split up the security group tests between compute and network APIs into separate files in preparation for reworking the compute (nova-net) implementations to deal with the upcoming removal of deprecated nova-net support in novaclient 8.0.0. No code changes are intended here, just splitting two files into four. Change-Id: I3fbdde45f593a30de545ddd11e319a4d6f900b18
* | | | | Merge ""floating ip set/unset port" for OSC"Jenkins2017-04-061-25/+161
|\ \ \ \ \
| * | | | | "floating ip set/unset port" for OSCSindhu Devale2017-04-051-25/+161
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements Neutron feature of floating ip associate/disassociate into OpenStack Client. Previously, network.find_ip() function only supported to search floating ip by UUID. Hence, _find_floating_ip() function is used in floating_ip.py, to search fip both by UUID and ip_address. [1] adds the ability to find fip object using both UUID and ip_address. This functionality however, won't be available until the SDK is released. Hence, we continue to use _find_floating_ip() method, which was cleaned up by [2] to remove the use of ip_cache. Once, the SDK is released, we will remove all the usage of _find_floating_ip() method and instead only use network.find_ip(). [1] https://review.openstack.org/#/c/449879/2 [2] https://review.openstack.org/#/c/447938/ Change-Id: I6c5222287c46ca42365917d2deae70bdb626347 Co-Authored-By: Reedip<reedip.banerjee@nectechnologies.in> Co-Authored-By: RuiChen<chenrui.momo@gmail.com> Closes-Bug: #1560297
* | | | | Merge "Add "qos-policy" option to "port create" & "port set""Jenkins2017-04-062-1/+61
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Add "qos-policy" option to "port create" & "port set"Ha Van Tu2017-04-052-1/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds "qos-policy" option to "port create" command, and "qos-policy", "no-qos-policy" options to "port set" command and "qos-policy" option to "port unset". Change-Id: I78072e1ff0dd30a2e23a0fb833ce6ab5cf246016 Co-Authored-By: Nguyen Phuong An <AnNP@vn.fujitsu.com> Co-Authored-By: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com> Partial-Bug: #1612136 Partially-Implements: blueprint network-commands-options
* | | | | Merge "Enable to create legacy router"Jenkins2017-04-051-4/+11
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Enable to create legacy routerCedric Brandily2017-03-311-4/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some deployments create by default HA routers, this change enables to force the creation of a legacy router using: openstack router create --no-ha ... Closes-Bug: #1675514 Change-Id: I78f7dc3640a2acfdaf085e0e387b30373e8415f1
* | | | | Merge "Enable to specify which fixed-ip to add to a vm."Jenkins2017-04-051-6/+14
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Enable to specify which fixed-ip to add to a vm.Cedric Brandily2017-04-041-6/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change enables to specify which fixed-ip will be added to a vm using: openstack server add fixed ip <vm> <network> --fixed-ip-address <ip> This change uses interface_attach instead of add_fixed_ip[1] which is less flexible and uses a deprecated API. [1] https://review.openstack.org/384261 Closes-Bug: #1678140 Change-Id: I7fe4621439ef0d8dca080551ffaeb614c5a91174
* | | | | Structure FindFloatingIP() to work without ip_cacheReedip2017-04-041-12/+6
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently we have an ip_cache returned from _find_floating_ip() which is generally ignored as it is not always required. This patch removes the need of ip_cache in _find_floating_ip(). Co-Authored-By: Sindhu Devale<sindhu.devale@intel.com> Change-Id: I8b92271185f82f275fa73adad03e9dad70be70e4
* | | | Merge "Introduce neutron flavor associate, disassociate to OSC"Jenkins2017-04-041-0/+84
|\ \ \ \
| * | | | Introduce neutron flavor associate, disassociate to OSCShashank Kumar Shankar2017-04-031-0/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch introduces network flavor associate and disassociate to OSC. The following neutron equivalent commands are implemented in OSC: - neutron flavor-associate - neutron flavor-disassociate Change-Id: Icba4dbf7300a36353142586359059cd6784049dc
* | | | | Merge "Enable to add/remove port to/from a server"Jenkins2017-04-031-0/+96
|\ \ \ \ \
| * | | | | Enable to add/remove port to/from a serverCedric Brandily2017-04-031-0/+96
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change enables to add/remove a specific port to/from a server using the new commands: openstack server add port <vm> <port> openstack server remove port <vm> <port> Closes-Bug: #1678137 Change-Id: I6ee57df089235ccc1fb9d38316bd484956b1134d
* | | | | | Merge "Fix block-device-mapping when volume_size is empty"Jenkins2017-04-031-2/+1
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Fix block-device-mapping when volume_size is emptyJens Rosenboom2017-04-031-2/+1
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Nova API responds with an validation error when a bdm is submitted containing an empty volume_size. So instead omit that attribute when it is empty. Change-Id: Iba905fca8c440a03e828c20922f3b813bba3fa3a Closes-Bug: 1677236
* | | | | Merge "doc: Correct Sphinx warnings"Jenkins2017-04-012-5/+5
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | doc: Correct Sphinx warningsStephen Finucane2017-03-282-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Fix option-describe typos - Correct option and envvar markup, for commands that are using the reference form instead of the definition form or are marking up option arguments as options - Avoid duplicate commands - Fix some invalid docstrings - Fix some invalid indentation - Disable the murano plugin, which has invalid docs - Correct issues with- and track the network-topology spec - Include API modules in docs Change-Id: I3d5ed5e872540fe13f3e4bd5e9335829dc9a5226
* | | | | Merge "Fix volume transfers request commands"Jenkins2017-03-282-18/+94
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Fix volume transfers request commandsDean Troyer2017-03-272-18/+94
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fix volume transfer request accept to actually not crash when trying to call Volume API. * Fix volume transfer request accept syntax to have only one positional argument, which is the ID of the resource in the command * Change the output column order in volume transfer request list to have ID followed by Name then the remaining columns. Closes-bug: 1633582 Change-Id: I5cc005f039d171cc70859f60e7fe649b09ead229
* | | | Merge "Simplify logic around option lists in port set"Jenkins2017-03-271-76/+60
|\ \ \ \ | |/ / / |/| | |
| * | | Simplify logic around option lists in port setDean Troyer2017-03-201-76/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use a common pattern to handle option pairs --XYZ and --no-XYZ for managing lists of attributes. This pattern looks at the presence of the option in parsed_args first and branches as necessary. Some specific steps are included for the SDK Network resources to reliably set the 'dirty' flag for changed attributes via one or both of the following: * iterate over lists of original resource attributes to force the creation of a new list object * use [].extend() rather than += to add to the existing list (substitute {}.update() for dicts) Change-Id: I0c3f9a52ffe1ae2b5b230cb13d6376dd9131aaf9
* | | | Merge "OSC Extension Show"Jenkins2017-03-231-0/+58
|\ \ \ \
| * | | | OSC Extension ShowSindhu Devale2017-03-221-0/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement Neutron feature of Extension Show into OpenStack Client. Change-Id: Ifecb794838cb3bf8c2466d178345349db3cd4003 Implements: blueprint extension-show
View Lorry Controller Status