summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | Add unit tests for "server start" command.Tang Chen2015-11-271-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I19060419a17b8e081e8f51c08959f24b84e7e570 Implements: blueprint osc-unit-test-framework-improvement
* | | | | Merge "Add unit tests for "server resume" command."Jenkins2015-11-261-0/+20
|\ \ \ \ \ | |/ / / /
| * | | | Add unit tests for "server resume" command.Tang Chen2015-11-271-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I8961e72b6901a52612cf62f52c02b6c2ff3dcd94 Implements: blueprint osc-unit-test-framework-improvement
* | | | | Merge "Add unit tests for "server suspend" command."Jenkins2015-11-261-0/+20
|\ \ \ \ \ | |/ / / /
| * | | | Add unit tests for "server suspend" command.Tang Chen2015-11-271-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Idfdd98fd27d97e1216abe9b14d3dea7c8f2a5a68 Implements: blueprint osc-unit-test-framework-improvement
* | | | | Merge "Add unit tests for "server unlock" command."Jenkins2015-11-261-0/+20
|\ \ \ \ \ | |/ / / /
| * | | | Add unit tests for "server unlock" command.Tang Chen2015-11-271-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I4fbf115f925a6ffdeaec3d856d22224fa7730729 Implements: blueprint osc-unit-test-framework-improvement
* | | | | Merge "Add unit tests for "server lock" command."Jenkins2015-11-261-0/+20
|\ \ \ \ \ | |/ / / /
| * | | | Add unit tests for "server lock" command.Tang Chen2015-11-271-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I0396fc4836ee918298bbe3860b7c1f42d3b97e33 Implements: blueprint osc-unit-test-framework-improvement
* | | | | Merge "Add unit tests for "server unpause" command."Jenkins2015-11-261-0/+20
|\ \ \ \ \ | |/ / / /
| * | | | Add unit tests for "server unpause" command.Tang Chen2015-11-271-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ia74d9875b8aa413b84d0077d22adb75b866a5701 Implements: blueprint osc-unit-test-framework-improvement
* | | | | Merge "Abstract a helper function for server.xxx() tests."Jenkins2015-11-261-59/+24
|\ \ \ \ \ | |/ / / /
| * | | | Abstract a helper function for server.xxx() tests.Tang Chen2015-11-271-59/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The test cases for server.xxx() are all the same, with one or more faked servers. So use a helper function to reduce code duplicate. Change-Id: I660c7731e2de8bf4d815b414a621d8d9ca6d5a8b Implements: blueprint osc-unit-test-framework-improvement
* | | | | Merge "Add multiple servers test case to TestServerDelete."Jenkins2015-11-261-0/+22
|\ \ \ \ \ | |/ / / /
| * | | | Add multiple servers test case to TestServerDelete.Tang Chen2015-11-271-0/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds a multiple servers test case to class TestServerDelete. Change-Id: I8b0c37ec2a8e7b23889e25c9004f867f2907f630 Implements: blueprint osc-unit-test-framework-improvement
* | | | | Merge "Use setup_servers_mock() in the base class in TestServerDelete."Jenkins2015-11-261-16/+17
|\ \ \ \ \ | |/ / / /
| * | | | Use setup_servers_mock() in the base class in TestServerDelete.Tang Chen2015-11-271-16/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use setup_servers_mock() in class TestServerDelete to coordinate the test class format. Change-Id: I1901d6b781d97820667984241f4d68764d045854 Implements: blueprint osc-unit-test-framework-improvement
* | | | | Merge "Move setup_servers_mock() to class TestServer."Jenkins2015-11-261-23/+13
|\ \ \ \ \ | |/ / / /
| * | | | Move setup_servers_mock() to class TestServer.Tang Chen2015-11-271-23/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This function will be used in almost every test case. So move it to the base class to avoid define it each time a test case is added. Change-Id: I060b54f0935b42a85042ad217d851fea649ec8d9 Implements: blueprint osc-unit-test-framework-improvement
* | | | | Trivial: Fix typo in find() in network.Tang Chen2015-11-271-2/+4
| |/ / / |/| | | | | | | | | | | Change-Id: Ic54fbb9160adefe3d025d537125e125128f75ee6
* | | | Trivial: Fix a typo.Tang Chen2015-11-271-1/+1
|/ / / | | | | | | | | | Change-Id: I236b4f53ee23cc97900e6244ab709404cc44a4ca
* | | Merge "Move FakeServer to tests.common.v2.compute.fakes"Jenkins2015-11-263-80/+87
|\ \ \
| * | | Move FakeServer to tests.common.v2.compute.fakeslin-hua-cheng2015-11-253-80/+87
| | | | | | | | | | | | | | | | | | | | | | | | | | | | FakeServer should not be in tests.fakes since this should be just for generic re-usable classes. Change-Id: I19209952de69626dfa3caadc5d1cc69b7feadeba
* | | | Refactor: Order of security group class namesRichard Theis2015-11-251-123/+123
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | Per comment in [1], refactor the security group class names to be in alphabetical order. [1] https://review.openstack.org/#/c/249223 Change-Id: If28a153cdab57c0659ff5c78b276766d4043467f
* | | Merge "Trivial: Add missing doc for parameter in wait_for_delete()."Jenkins2015-11-251-0/+1
|\ \ \
| * | | Trivial: Add missing doc for parameter in wait_for_delete().Tang Chen2015-11-261-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | The doc of parameter manager is missing. Change-Id: I4e99c06ab713532d73615670ada0a61462285d76
* | | | Merge "Remove the old fake server data."Jenkins2015-11-251-9/+0
|\ \ \ \
| * | | | Remove the old fake server data.Tang Chen2015-11-211-9/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The old fake server framework has be replaced with the new class FakeServer, which is stronger in: 1. faking multiple servers with random names and ids 2. faking methods in a server 3. easier to use So remove the old fake server data, and use class FakeServer from now on. Change-Id: Ife8ee37a7ce14d9a3201104bce8075a918a97613 Implements: blueprint osc-unit-test-framework-improvement
* | | | | Merge "Use class FakeServer in TestServerResize."Jenkins2015-11-251-21/+18
|\ \ \ \ \ | |/ / / /
| * | | | Use class FakeServer in TestServerResize.Tang Chen2015-11-211-21/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are the same problems with TestServerCreate in TestServerResize. Use the new class FakeServer to fix them. Change-Id: Ibde3e68a7bc55bbbf8357ba98be2559a6d0d41b6 Implements: blueprint osc-unit-test-framework-improvement
* | | | | Merge "Use class FakeServer in TestServerImageCreate."Jenkins2015-11-251-10/+8
|\ \ \ \ \ | |/ / / /
| * | | | Use class FakeServer in TestServerImageCreate.Tang Chen2015-11-211-10/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are the same problems with TestServerCreate in TestServerImageCreate. Use the new class FakeServer to fix them. Change-Id: Ie723fa95620549f09a81ef72953f46877ef9252a Implements: blueprint osc-unit-test-framework-improvement
* | | | | Merge "Use class FakeServer in TestServerDelete."Jenkins2015-11-251-16/+15
|\ \ \ \ \ | |/ / / / | | / / / | |/ / / |/| | |
| * | | Use class FakeServer in TestServerDelete.Tang Chen2015-11-211-16/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are the same problems with TestServerCreate in TestServerDelete. Use the new class FakeServer to fix them. Change-Id: Icdcc90cc93ed1080187fb0edca885b0db56ab35d Implements: blueprint osc-unit-test-framework-improvement
* | | | Merge "Use class FakeServer in TestServerCreate."Jenkins2015-11-251-38/+68
|\ \ \ \ | |/ / /
| * | | Use class FakeServer in TestServerCreate.Tang Chen2015-11-211-38/+68
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the current TestServerCreate, there are several problems: 1. The fake create() returns a server with no 'networks' field. The new_server is used to fake the created server which is supposed to be returned by create(), but it has a 'networks' field. They have the same name and id, but they are actually not the same server. As a result, when checking the return value from create(), 'networks' is not checked. 2. The fake server is not accessable in the test functions. So each time a test function wants to get the server name or id, it has to use the constants defined in compute_fakes. This is not good. We should make the fake server accessable in all test functions to ensure they actually get the same server. This patch fix them both by using the new class FakeServer to fake a server. Change-Id: I8ffc8e233f8710034329ed33fccb2c734898ec2d Implements: blueprint osc-unit-test-framework-improvement
* | | | Merge "Unable to set some compute quotas"Jenkins2015-11-253-0/+43
|\ \ \ \
| * | | | Unable to set some compute quotasRichard Theis2015-11-203-0/+43
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The OpenStackClient mapping of 'openstack quota set' arguments isn't correct for compute quota items that have to different names. For example, the --injected-file-size argument is mapped to injected_file_size, but the compute quotas item is actually injected_file_content_bytes. This incorrect mapping prevented the impacted compute quota items from being set. The problem impacts the following 'openstack quota set' arguments: --injected-file-size --injected-path-size --properties --secgroup-rules --secgroups This patch set also expands the compute quota unit tests to verify all compute quota items that can be set. Change-Id: I0a2f241e425f4811e4ae55be183ac0c8b0805c2a Closes-Bug: #1475831
* | | | Merge "Add "openstack server shelve" into OSC"Jenkins2015-11-242-0/+79
|\ \ \ \
| * | | | Add "openstack server shelve" into OSCxiexs2015-11-242-0/+79
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, the shelve operation is not supported by OSC. So, this patch attempts to add it into OSC. Change-Id: I92545300bef006a069338168d2de800e8a58af69 Implements: blueprint introduce-shelve-into-osc
* | | | | Merge "Doc: Update and add IP address"Jenkins2015-11-243-16/+16
|\ \ \ \ \
| * | | | | Doc: Update and add IP addressRichard Theis2015-11-243-16/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the 'ip floating' command documentation file name. Add missing command list documentation for the 'ip fixed' and 'ip floating pool' commands. In addition, update the command description and argument help to fix minor issues and use consistent terminology. And finally, cleanup the Network command object formatting. Change-Id: I0168891bc674c9f0d4df4062f7e93f497dc8fe6f
* | | | | | Merge "Doc: Add security group and security group rule"Jenkins2015-11-241-10/+10
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Doc: Add security group and security group ruleRichard Theis2015-11-241-10/+10
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add missing command list documentation for the 'security group' and 'security group rule' commands. In addition, update the command description and argument help to fix minor issues and use consistent terminology. Change-Id: I9f4a3fbac5637289f19511874e16391d3fe27132
* | | | | Enable "openstack server resume" command to take multiple servers.Tang Chen2015-11-241-6/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Current "openstack server resume" command could only resume one server. Improve it to be able to handle more than one servers. Also improve the doc to reflect the new feature. Change-Id: I726eb86bfa3df3a9911f45770e6641264dbc1e0b Implements: blueprint cmd-with-multi-servers
* | | | | Enable "openstack server suspend" command to take multiple servers.Tang Chen2015-11-241-6/+8
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Current "openstack server suspend" command could only suspend one server. Improve it to be able to handle more than one servers. Also improve the doc to reflect the new feature. Change-Id: Ic0417ee28f46c9198a35744c0180342e61966b26 Implements: blueprint cmd-with-multi-servers
* | | | Merge "Trivial: Fix wrong doc for wait_for_status()."Jenkins2015-11-231-1/+2
|\ \ \ \
| * | | | Trivial: Fix wrong doc for wait_for_status().Tang Chen2015-11-241-1/+2
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Two trivial fixes: 1. docs for parameters are not sorted correctly 2. missing doc for a parameter Change-Id: I0cfb65e0f897c391b9b6e7225251e88855b07a56
* | | | Merge "Remove deprecated 'project usage list' command"Jenkins2015-11-201-17/+0
|\ \ \ \
| * | | | Remove deprecated 'project usage list' commandDean Troyer2015-11-181-17/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove the 'project usage list' command that was deprecated in version 1.0.2 in review Ie08d4f88d71a660fca1862405351109cd0aa86b6. Note that the removed command class is a good example of how to wrap a command and show a deprecation message. Change-Id: I6c750730963615895f5d9953487d2d5a905885a8