summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Refactor: Abstract columns and datalist out in volume test cases"Jenkins2016-01-227-603/+226
|\
| * Refactor: Abstract columns and datalist out in volume test casesSaiKiran2016-01-207-603/+226
| | | | | | | | | | | | | | | | columns and datalist has been set in each test case in volume, which is not necessary. This patch abstract it out and remove all redundant code. Change-Id: I3a09d5d2db86da986bdcfbf2310978ced181017d
* | Merge "Initialize activation status"Jenkins2016-01-211-0/+1
|\ \
| * | Initialize activation statusTerry Howe2016-01-191-0/+1
| | | | | | | | | | | | | | | | | | | | | activation_status throws an exception if the image set fails because it is not initialized. Change-Id: Iff6d5a8844eed954fb1c0bcea96118b99ac6bcf4
* | | Merge "Change --owner to --project in image commands"Jenkins2016-01-214-40/+181
|\ \ \
| * | | Change --owner to --project in image commandsDean Troyer2016-01-154-40/+181
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * image create and image set now use --project to specify an alternate project to own the image * --owner is still silently accepted but deprecated, add warning messages * --project and --owner are mutually exclusive to prevent precedence issues Closes Bug: 1527833 Change-Id: Iccb1a1d9175ef9b5edcd79d294607db12641c1f0
* | | | Merge "log_method: get logger from decorated method if unspecified"Jenkins2016-01-212-99/+44
|\ \ \ \
| * | | | log_method: get logger from decorated method if unspecifiedAkihiro Motoki2016-01-202-99/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit makes 'log' optional. 'log' attribute of each command class does not exist when the class is defined because 'log' is now setup dynamically when a class is instantiated. Instead log_method looks for a logger from a decorating method. compute.v2.server is changed in this commit as an example. Change-Id: Ic4d128f8e027d3b8e6f884f31369e9085c0f0871 Partial-Bug: #1532294
* | | | | Refactor abstract columns and datalist out in compute test casesSaiKiran2016-01-213-167/+83
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | columns and datalist has been set in each test case in compute, which is not necessary. This patch abstract it out and remove all redundant code. Change-Id: I5e8423722416ea31fdced4c932ed141de90028ab Closes-Bug: #1531816
* | | | | Merge "Set up logger of each command by metaclass"Jenkins2016-01-213-22/+80
|\ \ \ \ \ | |/ / / /
| * | | | Set up logger of each command by metaclassAkihiro Motoki2016-01-203-22/+80
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | compute.v2.flavor is changed in this commit as an initial example. Partial-Bug: #1532294 Change-Id: I262af6ade0ae03fbe1cd2ad198faf4ebb4ecf7ce
* | | | Add support to delete the portsSaiKiran2016-01-203-0/+172
|/ / / | | | | | | | | | | | | | | | | | | | | | This patch adds "port delete" command to osc. Change-Id: I5c92b2f573249df4e6551506584ccafb4ff290b2 Implements: blueprint neutron-client Partial-Bug: #1519909
* | | Merge "Make --image parameter optional in "server rebuild""Jenkins2016-01-152-5/+76
|\ \ \ | |/ / |/| |
| * | Make --image parameter optional in "server rebuild"David Moreau Simard2015-12-112-5/+76
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The command will now default to the image currently in-use by the server, effectively making the --image parameter optional. This commit also adds basic tests for ServerRebuild since there wasn't any. Will add more full tests for it. Change-Id: I733fd3ad5a825f06563c72aa430122e1a0e3b3b0 Closes-bug: #1524406 Co-Authored-By: David Moreau Simard <dms@redhat.com> Co-Authored-By: Tang Chen <tangchen@cn.fujitsu.com>
* | | Merge "Changed the abstract columns and datalists from test cases of common ↵Jenkins2016-01-1419-814/+562
|\ \ \ | | |/ | |/| | | | and Identity"
| * | Changed the abstract columns and datalists from test cases of common and ↵SaiKiran2016-01-1219-816/+564
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Identity columns and datalist has been set in each test case in compute, which is not necessary. This patch abstract it out and remove all redundant code. Change-Id: I45a3582088161f12e60e7c933da8e7dcc0ae8e7b Closes-Bug: #1532384
* | | Merge "Refactor network endpoint enablement checking"Jenkins2016-01-144-9/+68
|\ \ \
| * | | Refactor network endpoint enablement checkingRichard Theis2016-01-044-9/+68
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move the network endpoint enablement checking from the 'server create' command to the common client manager. This allows future network commands to use either nova or neutron networking based on the cloud environment. This patch set also includes related unit test enhancements to the common client manager to trigger authentication on the tests. Change-Id: Ia37e81d4fb05a1e2fceb3e5d367bda769ab8e64b Related-Bug: #1519511 Related-to: blueprint neutron-client
* | | | Merge "Support non-interactive user password update"Jenkins2016-01-112-2/+48
|\ \ \ \
| * | | | Support non-interactive user password updateGuang Yee2016-01-112-2/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently user password update require interactive prompting of user's original password. This is problematic because we can't support non-interactive applications and therefore hinders automation. This patch make it possible by optionally accepting an '--original-password' argument. If specified, we would use it instead of prompting. DocImpact Change-Id: I2d994e8c2be949f7ae616ac1d1594fb94e1a27cd Closes-Bug: 1531360
* | | | | Merge "Trivial: Remove useless string_to_bool()"Jenkins2016-01-081-4/+0
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Trivial: Remove useless string_to_bool()Tang Chen2015-12-291-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | string_to_bool() is not used by anyone. Furthermore, it is not well designed. It tries to convirt 't', '1' to True, which could be confused. So remove it. If we need something similar, let's make a better one. Change-Id: Ic1f63480c806bf7bcc9f541fc806eed297ddf718
* | | | | Merge "Further improve output for "os security group show""Jenkins2016-01-071-1/+19
|\ \ \ \ \
| * | | | | Further improve output for "os security group show"Richard Theis2016-01-061-1/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve the security group rules output when running the "os security group show" command. Empty and duplicate information for each security group rule is now removed. This will ensure that the rules remain readable when direction and ethertype information is returned as part of the transition to neutron networking. Change-Id: Ib49c27a9d7f4d5d38ceb2b0d785ddf94d88b2d89 Partial-Bug: #1519511 Related-To: blueprint neutron-client
* | | | | | Merge "Use assertTrue/False instead of assertEqual(T/F)"Jenkins2016-01-071-3/+3
|\ \ \ \ \ \
| * | | | | | Use assertTrue/False instead of assertEqual(T/F)Swapnil Kulkarni (coolsvap)2016-01-071-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The usage of assertEqual(True/False, ***) should be changed to a meaningful format of assertTrue/False(***). Change-Id: I3437634329fc8ecef25082b43b5fc0e1030cdbda Closes-Bug:#1512207
* | | | | | | Replace assertEqual(*, None) with assertIsNone in testsEinst Crazy2016-01-071-2/+2
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replace assertEqual(*, None) with assertIsNone in tests to have more clear messages in case of failure. Change-Id: I8964a10ae3529e978bfab1d8140f95da4b56615c
* | | | | | Delete the unused LOG configure codezhurong2016-01-052-8/+0
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | Delete the unused LOG configure code and import code Change-Id: I1fb0cacfe44b6a2fd4e4b3f504b6d1dec055c5c4
* | | | | Merge "Add support to list all security group rules"Jenkins2016-01-042-15/+103
|\ \ \ \ \
| * | | | | Add support to list all security group rulesRichard Theis2015-12-172-15/+103
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Both nova and neutron allow security group rules to be listed without specifying the owning security group. This patch set makes the group argument on 'os security group rule list' optional. Behavior is unchanged when the argument is specified. When the argument is not specified then all accessible security group rules will be listed. The listing will include the owning security group for each rule. Change-Id: I6914baecf70a65354e1e82dad92c6afbd32b4973 Related-Bug: #1519512
* | | | | | Merge "Implementation for project unset cmd for python-openstackclient."Jenkins2016-01-042-1/+103
|\ \ \ \ \ \
| * | | | | | Implementation for project unset cmd for python-openstackclient.Jude Job2016-01-042-1/+103
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch introduces a unit test class TestProjectUnset for testing unset cmd. Co-Authored-By: Steve Martinelli <stevemar@ca.ibm.com> Change-Id: Ib4a414d2313e3d37e48d1cb3639f064231aec508 Closes-Bug: #1486597
* | | | | | | Merge "Enabling domain lookup for project set v3 command"Jenkins2016-01-042-11/+2
|\ \ \ \ \ \ \
| * | | | | | | Enabling domain lookup for project set v3 commandJude Job2015-12-312-11/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently the domain option for `project set` attempts to set a new domain that owns the project. This is actually an action that is denied by keystone server. Instead, the domain option should be used as a lookup, to find projects that exist in the non-default domain. Co-Authored-By: Steve Martinelli <stevemar@ca.ibm.com> Closes-Bug: #1524456 Change-Id: I30a3812184fe262667e09baa106d2275c2cbb981
* | | | | | | | Merge "Use Block Storage instead of Volume"Jenkins2016-01-042-3/+3
|\ \ \ \ \ \ \ \ | |_|_|_|_|/ / / |/| | | | | | |
| * | | | | | | Use Block Storage instead of VolumeXi Yang2015-11-272-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Volume is better to be replaced by Block Storage in the doc. Change-Id: I736669ee01c7385b6e701cb20f4334eff1c49286
* | | | | | | | Merge "Map some of the SDK field names"Jenkins2016-01-013-12/+26
|\ \ \ \ \ \ \ \
| * | | | | | | | Map some of the SDK field namesTerry Howe2015-12-103-12/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The keys() method returns the keys returned from Neutron, but the SDK maps some things like tenant_id to project_id. This makes the output a little prettier. Change-Id: Ibd8c890b61ffc94021f93fc1051fcf5dabd1e9ea
* | | | | | | | | Docstring should say 'default' if option is defaultRushi Agrawal2015-12-311-1/+1
| |_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | Change-Id: I59d83e7a0a590fa537f509f7f19940640f555679
* | | | | | | | Merge "Improve output for "os security group show""Jenkins2015-12-283-4/+12
|\ \ \ \ \ \ \ \
| * | | | | | | | Improve output for "os security group show"Richard Theis2015-12-233-4/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve the security group rules output when running the "os security group show" command. Each security group rule is now displayed on a separate line. Current output example: $ openstack security group show default +-------------+------------------------- ... ---+ | Field | Value ... | +-------------+------------------------- ... ---+ | description | Default security group ... | | id | 048a5fc3-3be1-407d-ae47-9... | | name | default ... | | project_id | 3b96bb2020c1459da76963f9e... | | rules | [u"id='5d812367-9829-4340...t"] | +-------------+------------------------- ... ---+ New output example: +-------------+------------------------- ... ---+ | Field | Value ... | +-------------+------------------------- ... ---+ | description | Default security group ... | | id | 048a5fc3-3be1-407d-ae47-9... | | name | default ... | | project_id | 3b96bb2020c1459da76963f9e... | | rules | id='5d812367-9829-4340-95...lt' | | | id='ee451d1c-ade3-4975-8e...lt' | +-------------+------------------------- ... ---+ Change-Id: I1386075310896c58a2b776e2bbec3603bd00eff1 Partial-Bug: #1519511 Related-To: blueprint neutron-client
* | | | | | | | | Merge "Trivial: Remove useless return from files in image and volume"Jenkins2015-12-2811-44/+0
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / |/| | | | | | | |
| * | | | | | | | Trivial: Remove useless return from files in image and volumeTang Chen2015-12-1811-44/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I3526ecd202d0908d91305a066ad72d03cee794b5
* | | | | | | | | Refactor: Initialize parser in setUp() in TestNonNegativeActionTang Chen2015-12-281-26/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I12846acc4450d31d19897bbdfc6846bde8c8f2ce
* | | | | | | | | Refactor: Initialize parser in setUp() in TestKeyValueActionTang Chen2015-12-281-52/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | No need to initialize parser in each test case. Do it in setUp(). Also remove the test_default_values case because it could be tested in the test_good_values case. Change-Id: Ia2ed7c9e46bf6baabbd62b9d50511c5e8103e5e2
* | | | | | | | | Replace assertEqual(None, *) with assertIsNone(*)Xi Yang2015-12-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch is going to replace assertEqual(None, *) with assertIsNone(*) in unit test code to have more clear messages in case of failure. Change-Id: I6f85498347e8fc7cad5ea7afb832b9acda7daafc
* | | | | | | | | Add all regions to cloud configurationRichard Theis2015-12-231-1/+1
| |/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | OCC now requires all regions to be listed in the cloud configuration. Update the shell unit tests to list all of the regions being tested. Change-Id: Ic7300d1f708339701b5daadbf2c4769b239a2adb Closes-Bug: #1528926
* | | | | | | | Merge "Add owner validation for "openstack image create/set""Jenkins2015-12-222-4/+104
|\ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / |/| | | | | | |
| * | | | | | | Add owner validation for "openstack image create/set"xiexs2015-12-212-4/+104
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Owner validation is necessary if a new image owner will be created/set. Change-Id: I621774e02866bfa98a31b613deff5d7b6a962737 Closes-Bug: #1517134
* | | | | | | | Merge "TestServerGeneral: Add test for _prep_server_detail()"Jenkins2015-12-221-2/+44
|\ \ \ \ \ \ \ \
View Lorry Controller Status