summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Refactor TestVolumeCreate to use FakeVolume"2.1.0Jenkins2016-02-041-67/+66
|\
| * Refactor TestVolumeCreate to use FakeVolumexiexs2016-02-021-67/+66
| | | | | | | | | | | | | | | | Class FakeVolume should be used in volume tests. Change-Id: Idf7d3e2a0654cd7d7993f169c4743b1d38902f1b Implements: blueprint improve-volume-unittest-framework Co-Authored-By: Tang Chen <chen.tang@easystack.cn>
* | Merge "Minor typo in help text"Jenkins2016-02-041-1/+1
|\ \
| * | Minor typo in help textJas2016-02-031-1/+1
| | | | | | | | | | | | | | | | | | fix misspelled 'project' in router-create help text Change-Id: I4874563c0dd7ca35ef153cf24b99e357c0c076b9
* | | Merge "Support listing network availability zones"Jenkins2016-02-046-12/+188
|\ \ \
| * | | Support listing network availability zonesRichard Theis2016-02-026-12/+188
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update the "os availability zone list" command to support listing network availability zones along with the currently listed compute and volume availability zones. This adds the --network option to the command in order to only list network availability zones. By default, all availability zones are listed. The --long option was also updated to include a "Zone Resource" column which is applicable to network availability zones. Example zone resources include "network" and "router". If the Network API does not support listing availability zones then a warning message will be issued when the --network option is specified. This support requires an updated release of the SDK in order to pull in [1]. [1] https://bugs.launchpad.net/python-openstacksdk/+bug/1532274 Change-Id: I78811d659b793d9d2111ea54665d5fe7e4887264 Closes-Bug: #1534202
* | | Merge "Allow wait_for_delete to work for all clients"Jenkins2016-02-042-2/+28
|\ \ \
| * | | Allow wait_for_delete to work for all clientsMark Vanderwiel2016-01-282-2/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow the exception and error status strings to be passed in such that other plugins can make use of this function. There is a comment in find_resource: The exception to catch here is dependent on which client library the manager passed in belongs to. Eventually this should be pulled from a common set of client exceptions. Since I think that is a long ways off, this change will work now and also work when a common exception is defined and used. Change-Id: Iab56cd1166028caed4f1e657e0b1ee81af3f48d8
* | | | Merge "Fix wrong type of volume attachments in FakeVolume"Jenkins2016-02-031-2/+3
|\ \ \ \
| * | | | Fix wrong type of volume attachments in FakeVolumeTang Chen2016-02-021-2/+3
| | |/ / | |/| | | | | | | | | | | | | | | | | | The volume attachments should be a list of dict, not a single dict. Change-Id: I3cec62bcb3953e4f38f9d3dd23f3eb6ef984464c
* | | | Merge "Refactor security group delete to use SDK"Jenkins2016-02-039-25/+385
|\ \ \ \
| * | | | Refactor security group delete to use SDKRichard Theis2016-02-029-25/+385
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Refactored the 'os security group delete' command to use the SDK when neutron is enabled, but continue to use the nova client when nova network is enabled. This patch set introduces a new NetworkAndComputeCommand class to be used for commands that must support neutron and nova network. The new class allows both the parser and actions to be unique. The current DeleteSecurityGroup class is now a subclass of this new class and has moved under the network v2 commands. This patch set also introduces a new FakeSecurityGroup class for testing security groups. And finally, this patch set updates the command documentation for security group and security group rule to indicate that Network v2 is also used. Change-Id: Ic21376b86b40cc6d97f360f3760ba5beed154537 Partial-Bug: #1519511 Related-to: blueprint neutron-client
* | | | Merge "Network: Abstract get_body() out to be a private helper."Jenkins2016-02-033-39/+46
|\ \ \ \
| * | | | Network: Abstract get_body() out to be a private helper.Tang Chen2016-02-023-39/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | get_body() is needed in each network files to construct a dict to pass to sdk proxy. And it is also used by several functions in each file. So define it as a file level private helper function. The unified prototype should be: def _get_attrs(client_manager, parsed_args): 1. The name, in sdk, the parameter passed to proxy is named "attrs". And it is a private method. So let's call it _get_attrs(). 2. The parameters, besides parsed_args, when we deal with project and project_domain, we have to make use of identity_client. So let's pass in the client manager. Change-Id: Ib044ebd4ddedbcd805f46334a7fe99e4ebb5b249
* | | | | Merge "Add "os port show" command"Jenkins2016-02-038-4/+222
|\ \ \ \ \
| * | | | | Add "os port show" commandRichard Theis2016-02-028-4/+222
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add "os port show" command. Change-Id: Id87c81640e74c60ae8f247c722c64fdadff022a2 Partial-Bug: #1519909 Partially-Implements: blueprint neutron-client
* | | | | Merge "Add availability zone support for router commands"Jenkins2016-02-034-0/+53
|\ \ \ \ \
| * | | | | Add availability zone support for router commandsJas2016-01-294-0/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch allows the adding of availability_zone_hints during router create. Also allows for the display of availability_zones during list and and show commands. Change-Id: Ifbc5c218bc7103d28076d726212ce25321bcf7f1 Partial-bug: #1519503 Partially-implements: blueprint neutron-client
* | | | | | Merge "Fix a spell typos"Jenkins2016-02-032-3/+3
|\ \ \ \ \ \
| * | | | | | Fix a spell typosvenkatamahesh2016-02-032-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I422fc8086af83efa4c04fd1951b4738404c1a1b7
* | | | | | | Merge "Add --marker option to "image list" command"Jenkins2016-02-034-0/+41
|\ \ \ \ \ \ \
| * | | | | | | Add --marker option to "image list" commandTang Chen2016-02-034-0/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Users could specify the last image (name or ID) of the previous page with --marker option to control the start image of the output. Change-Id: Idca0235ee83b1226b00c89cf3d38500fa898b7d0 Closes-Bug: #1540988
* | | | | | | | Merge "Add limit option to "image list" command"Jenkins2016-02-034-0/+36
|\ \ \ \ \ \ \ \ | |/ / / / / / /
| * | | | | | | Add limit option to "image list" commandxiexs2016-02-034-0/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This option is quite useful if there are too many images. Change-Id: If6a901c27c5da2d1f4412e8fa9ba3bed3b72fdd9 Co-Authored-By: Tang Chen <chen.tang@easystack.cn> Partial-Bug: #1540988
* | | | | | | | Merge "Remove marker and loop from "image list" command"Jenkins2016-02-032-25/+5
|\ \ \ \ \ \ \ \ | |/ / / / / / / | | / / / / / / | |/ / / / / / |/| | | | | |
| * | | | | | Remove marker and loop from "image list" commandTang Chen2016-02-032-25/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since --page-size has never worked, there is no paginate logic needs to be implemented in "image list" command. So remove the unnecessary loop. And also, the marker is not necessary because --marker option has not been implemented. Will add it back when implementing --marker option. Change-Id: I71fea1502f92f447a49697edb52e8e82f336772f Partial-Bug: #1540988
* | | | | | | Trivial: Reorder test class in test_volume.py into alphabetical orderTang Chen2016-02-031-41/+41
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | TestVolumeDelete should be after TestVolumeCreate. Change-Id: I764543a0e0723633aec6b18c2d50a01931465e6b
* | | | | | Merge "Drop log_method decorator"Jenkins2016-02-021-27/+0
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Drop log_method decoratorAkihiro Motoki2016-02-021-27/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As a result of the recent logging refactoring, log_method decorator is no longer required. oslo.log provides a similar decorator oslo_log.helpers.log_method_call. If a similar feature is needed, we can use the decorator from oslo_log. searchlightclient is the only OSC external plugin which uses this decorator. The depending patch removes it, so we can safely drop the decorator. Change-Id: If3df09cf6aa0a401d9f89e8924adce851d0c6dec Depends-On: Ib94e7ba77262a9a8cbfce71f3083c47cb1973364
* | | | | | Trivial: Fix wrong comment in test_image.pyTang Chen2016-02-021-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Code in test_image.py has nothing to do with server. Change-Id: Ia73d7b99effb394c5db9635fee6da350b0b1086b
* | | | | | Merge "Consume openstackclient.common.command in subnet/port"Jenkins2016-02-022-4/+3
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | |
| * | | | | Consume openstackclient.common.command in subnet/portAkihiro Motoki2016-02-022-4/+3
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Follow-up patch of https://review.openstack.org/#/c/269613/ network/v2/subnet and port still use cliff classes directly. This patch fixes it. Change-Id: If9d90e5151ece7f4cf1e0d6fd2f32919865f2f2e
* | | | | Merge "Updated from global requirements"Jenkins2016-02-021-1/+1
|\ \ \ \ \
| * | | | | Updated from global requirementsOpenStack Proposal Bot2016-02-021-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I2f00e8ff1609d76c2ab20226f2b4503231fdb003
* | | | | | Merge "Fix showing network quotas for a project"Jenkins2016-02-023-11/+37
|\ \ \ \ \ \
| * | | | | | Fix showing network quotas for a projectRichard Theis2016-02-023-11/+37
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The OpenStack SDK is now used for the network client. However, the 'openstack quota show' command wasn't updated for the client change. As a result, the command will fail to show network quotas when a project name is specified. For example: $ openstack quota show admin 'Proxy' object has no attribute 'show_quota' This patch set fixes the command by using the OpenStack SDK to get network quotas for a project. Change-Id: I59a7b6780a7b80cd09e79d40d214751b25d3016e Related-To: blueprint neutron-client Closes-Bug: #1528249
* | | | | | Merge "Add missing release notes"Jenkins2016-02-0210-0/+49
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Add missing release notesDean Troyer2016-02-0210-0/+49
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add release notes for the user-visible features added since the 2.0.0 release Change-Id: I753a894f1a3f6eef3eef50661ee66c729a2567f4
* | | | | log take_action parameters in a single placeAkihiro Motoki2016-02-0271-1391/+242
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously each command logs take_action parameters explicitly by using @utils.log_method decorator or log.debug(). Some commands have no logging. This commit calls a logger in the base class and drops all logging definition from individual commands. Closes-Bug: #1532294 Change-Id: I43cd0290a4353c68c075bade9571c940733da1be
* | | | Merge "Revert "Skip identity v2 functional tests""Jenkins2016-02-029-24/+9
|\ \ \ \
| * | | | Revert "Skip identity v2 functional tests"Richard Theis2016-02-019-24/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 5032dbc8074d5133c6b71610cd57d3c8da07c9b9 based on [1]. [1] https://review.openstack.org/#/c/274703/ Change-Id: Ic10ef7c37d71e452fbc4bd36e28be79d669b4e3f
* | | | | Merge "Add availability zone support for network commands"Jenkins2016-01-314-3/+51
|\ \ \ \ \
| * | | | | Add availability zone support for network commandsBrad Behle2016-01-284-3/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add --availability-zone-hint parm to network create. Also add availability_zones and availability_zone_hints to the network list and network show commands Change-Id: Ib4dc2e3e7897939be7bef6b25a095c8222b885bc Partially-implements: blueprint neutron-client
* | | | | | Merge "Update translation setup"Jenkins2016-01-315-7/+7
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Update translation setupAkihiro Motoki2016-01-295-7/+7
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Follow new infra setup for translations, see spec http://specs.openstack.org/openstack-infra/infra-specs/specs/translation_setup.html for full details. This basically renames python-openstackclient/locale/python-openstackclient.pot to openstackclient/locale/openstackclient.pot. For this we need to update setup.cfg. Update also domain name in i18n.py. Change-Id: I89fad12f20775c8b7cd228348ff82a77488e6ab2
* | | | | Skip identity v2 functional testsRichard Theis2016-01-299-9/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | DevStack now uses identity v3 by default thus causing OSC to only load openstack.identity.v3 commands. This prevents running functional tests on openstack.identity.v2 commands. As a result, this patch set skips all identity v2 functional tests to unblock the gate. Change-Id: I066187318be71792a966fa21226fab0d406c3758 Partial-Bug: #1539780
* | | | | Updated from global requirementsOpenStack Proposal Bot2016-01-281-1/+1
|/ / / / | | | | | | | | | | | | Change-Id: If070122805497e0943918900c6d1f6a54faef37a
* | | | Merge "Remove the Tuskar client"Jenkins2016-01-283-4/+0
|\ \ \ \
| * | | | Remove the Tuskar clientDougal Matthews2016-01-273-4/+0
| | |_|/ | |/| | | | | | | | | | | | | | | | | | The Tuskar project is now inactive and has been archived. Change-Id: I4c7cb50560b40ba1bc130be5bdc9446a46b09c24
* | | | Merge "Use correct terminology for subnets"Jenkins2016-01-273-5/+5
|\ \ \ \ | |/ / / |/| | |