summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* 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
* 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
* 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
* 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
* 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 "Add unit tests for "server pause" command."Jenkins2015-11-191-0/+57
|\
| * Add unit tests for "server pause" command.Tang Chen2015-11-201-0/+57
| | | | | | | | | | | | | | | | This patch adds unit tests for "server pause" command, including one and multiple servers. Change-Id: If5551e77d7dd4f7f48c6ee4a7f80f8313817f492 Implements: blueprint osc-unit-test-framework-improvement
* | Merge "Introduce random server faking mechanism."Jenkins2015-11-191-0/+70
|\ \ | |/
| * Introduce random server faking mechanism.Tang Chen2015-11-191-0/+70
| | | | | | | | | | | | | | | | | | | | | | This patch introduces a new server faking mechanism to support multiple servers faking. Server names and ids can be generated randomly, and use APIs in class FakeServer to get one or more servers. Change-Id: Ic54f3bf7c77294dc7dfb9acdbf4a721eb5eef6af Implements: blueprint osc-unit-test-framework-improvement
* | Merge "Enable FakeResource to fake methods."Jenkins2015-11-191-1/+26
|\ \ | |/
| * Enable FakeResource to fake methods.Tang Chen2015-11-191-1/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use MagicMock to fake a method in FakeResource. A new function: add_method(name, return_value) is added to FakeResource. The caller specifies method @name and @return_value, the function will add an attribute with @name, which is a callable MagicMock object whose return value is @return_value. When user access the attribute with a (), @return_value will be returned by MagicMock, which looks like a function call. Change-Id: I12eb876cbebab064773df7b5dd612de69bbf3f01 Implements: blueprint osc-unit-test-framework-improvement
* | Allow error status to be specifiedMark Vanderwiel2015-11-182-1/+42
|/ | | | | | | | For some apis, heat, the error status is "failed". This patch changes the wait_for_status method to allow for the error status to be passed in the same way as the success status. Change-Id: I20db4051d3f5611a4b13fe23ea8798b82a40da81
* Merge "Add release notes for 1.9.0"1.9.0Jenkins2015-11-171-0/+58
|\
| * Add release notes for 1.9.0Steve Martinelli2015-11-161-0/+58
| | | | | | | | | | | | Sum up the latest bug fixes for the latest release. Change-Id: I4f6e9e9d5062e5ccb94f088abdc270fc40373d16
* | Merge "Enable "openstack server unlock" command to take multiple servers."Jenkins2015-11-173-10/+12
|\ \ | |/ |/|
| * Enable "openstack server unlock" command to take multiple servers.Tang Chen2015-11-133-10/+12
| | | | | | | | | | | | | | | | | | Current "openstack server unlock" command could only unlock one server. Improve it to be able to handle more than one servers. Also improve the doc to reflect the new feature. Change-Id: Ibf57b2021a504da950a491d63139a438087aed0b Implements: blueprint cmd-with-multi-servers
* | Merge "Enable "openstack server lock" command to take multiple servers."Jenkins2015-11-163-10/+12
|\ \ | |/
| * Enable "openstack server lock" command to take multiple servers.Tang Chen2015-11-133-10/+12
| | | | | | | | | | | | | | | | | | Current "openstack server lock" command could only lock one server. Improve it to be able to handle more than one servers. Also improve the doc to reflect the new feature. Change-Id: Ifcf103b1c32e6c547ac09f688b887b1c03f92b09 Implements: blueprint cmd-with-multi-servers
* | Merge "Enable "openstack server unpause" command to take multiple servers."Jenkins2015-11-163-10/+12
|\ \ | |/
| * Enable "openstack server unpause" command to take multiple servers.Tang Chen2015-11-133-10/+12
| | | | | | | | | | | | | | | | | | Current "openstack server unpause" command could only unpause one server. Improve it to be able to handle more than one servers. Also improve the doc to reflect the new feature. Change-Id: I069ebdd6dcd121f6e55c2bf40d42197f93830e0c Implements: blueprint cmd-with-multi-servers
* | Merge "better format remote IDs for identity providers"Jenkins2015-11-153-10/+15
|\ \
| * | better format remote IDs for identity providersSteve Martinelli2015-11-123-10/+15
| |/ | | | | | | | | | | | | | | | | remote-ids are a list, and we should format these values as such, rather than python representations of lists/arrays. Closes-Bug: 1478995 Change-Id: Ia6ced0fab2435b8cb486822c676c0dee32613abe
* | Merge "Improve "server list" command to have the same output as "nova list"."Jenkins2015-11-131-2/+43
|\ \
| * | Improve "server list" command to have the same output as "nova list".Tang Chen2015-11-141-2/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "nova list" will also output "Task State" and "Power State" by default. This patch improves "server list" command to have the same columns, but not by default. These two columns will be output if --long is added. The power state is an int, so also adds a formatter helper function to translate it to human readable string, just as "Networks" does. Change-Id: I0530a910bec03835839a5ba7687c66d5643338f3
* | | Merge "Split the vol_id from a dev mapping"Jenkins2015-11-133-4/+94
|\ \ \
| * | | Split the vol_id from a dev mappingxiexs2015-11-113-4/+94
| | |/ | |/| | | | | | | | | | | | | | | | Add a split into the dev mapping to get the right vol_id. Change-Id: I1a7bf6351491b1321c5ca0fa4a27f29825400eaf Closes-Bug: #1514394
* | | Add capability to update description of an IdPlin-hua-cheng2015-11-123-2/+64
| |/ |/| | | | | | | Change-Id: I854067642bbfde6fdf84b22b9cc1de8afc7767c0 Closes-Bug: #1515815
* | Merge "validate non-ascii values for swift properties"Jenkins2015-11-122-0/+16
|\ \
| * | validate non-ascii values for swift propertiesSteve Martinelli2015-11-122-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | skip properties that are non-ascii values, but proceed with properties that work. log these failed values back to the user. Change-Id: Iaca8909f4465a01c8aebfd290b1a322823702359 Closes-Bug: 1503898
* | | Merge "Trivial: Fix wrong param name in comment."Jenkins2015-11-121-1/+1
|\ \ \
| * | | Trivial: Fix wrong param name in comment.Tang Chen2015-11-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | "volume" should be "attachments". Change-Id: Id1e9a733e18db595d8981b5b3a7735313a346787
* | | | Merge "Add project-name/-id validation for the OSC "openstack quota set""Jenkins2015-11-122-10/+31
|\ \ \ \ | |/ / / |/| | |
| * | | Add project-name/-id validation for the OSC "openstack quota set"xiexs2015-11-042-10/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The quota info would be set into DB, even though the project actually does not exist. This patch tried to add a validation to forbid this undesirable behavior. Change-Id: Ia2d8c96527820e25b074e6486d3f39c5ad7eae60 Closes-Bug: #1512638
* | | | Merge "Fix the bug of "openstack console log show""Jenkins2015-11-123-0/+72
|\ \ \ \ | |_|_|/ |/| | |
| * | | Fix the bug of "openstack console log show"xiexs2015-11-093-0/+72
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The behaviors are inconsistent while different negative line numbers specified. Change-Id: I2573f3e789f5603c896758971830ffc0b94c5e2b Closes-Bug: #1512263
* | | | Merge "Set default network api to 2.0 instead of 2"Jenkins2015-11-121-1/+3
|\ \ \ \
| * | | | Set default network api to 2.0 instead of 2Monty Taylor2015-11-091-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | neutronclient expects 2.0 as the version if you go through the discovery constructor. For that reason, 2.0 is the 'correct' version to set in config files or environment variables for if you're using things that are not OSC. However, if you do that, OSC prints a warning that 2.0 is not in the supported network version list. Let's support both so that users don't get a confuse. Change-Id: I7412519693f75fcd29f5621ce9e5a2df2da92684
* | | | | Merge "Add --owner to `image create`"Jenkins2015-11-123-7/+11
|\ \ \ \ \
| * | | | | Add --owner to `image create`NiallBunting2015-11-063-7/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds --owner to `image create`. This is backwards compatable with v1. Change-Id: I9e79cf880c91a1386419db729818d23dfe632179 Depends-On: I8d572a070bbb04dccdd051b8e0ad199c5754746e
* | | | | | Merge "Fix the bug of "openstack usage show""Jenkins2015-11-111-4/+12
|\ \ \ \ \ \
| * | | | | | Fix the bug of "openstack usage show"xiexs2015-11-021-4/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When there is no resouce usage associated with the project, an odd output will be displayed. This patch tried to fix this issue. Change-Id: I6f254c6ba37fbb760ada08e640c4938668d560dc Closes-Bug: #1512220
* | | | | | | Merge "Fix issue when displaying image_member"Jenkins2015-11-113-4/+10
|\ \ \ \ \ \ \
| * | | | | | | Fix issue when displaying image_memberlin-hua-cheng2015-10-233-4/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | image_member doesn't have a _info attribute, glanceclient returns warlock object instead of a Resource object. Change-Id: If6e7c4bd404454bd6cbe8c111879c1afa1380211 Closes-Bug: #1509054
* | | | | | | | Merge "Add Command Options guideline doc"Jenkins2015-11-112-0/+136
|\ \ \ \ \ \ \ \
| * | | | | | | | Add Command Options guideline docDean Troyer2015-10-212-0/+136
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a developer guideline for command options to define the options used across multiple commands and make them behave consistently. Change-Id: I1dbbafe8061e10b271cd55cac056731508c52204
* | | | | | | | | Merge "Add compute service delete"Jenkins2015-11-116-0/+254
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Add compute service deleteAlex Schultz2015-10-166-0/+254
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change adds 'compute service delete <service>' to the openstack client. This is the equivalent of 'nova service-delete <id>' Change-Id: I69ef1cac72cbe125c2114f8e958e22350a70f367 Closes-Bug: #1503510
* | | | | | | | | | Trivial: Fix wrong param name in comment.Tang Chen2015-11-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "volume" should be "attachments". Change-Id: I41c797f0c7cccb2727e9e6cd6424ea2f740624b7
* | | | | | | | | | Merge "Trivial cleanup: Use plural format for "server delete" doc."Jenkins2015-11-102-2/+2
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / |/| | | | | | | | |
| * | | | | | | | | Trivial cleanup: Use plural format for "server delete" doc.Tang Chen2015-11-102-2/+2
| | |_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I4dca25f1933d5925db443eca75f602a39a833413