summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Add unit tests for "host list" and "host show" commands"Jenkins2016-07-212-1/+111
|\
| * Add unit tests for "host list" and "host show" commandsHuanxuan Ao2016-06-272-1/+111
| | | | | | | | | | | | | | Missing unit tests for "host list" and "host show" commands in computev2, this patch add them. Change-Id: Ib157920fa2267ec96d206cdf46213563a105501b
* | Merge "Implement rbac list and show command"Jenkins2016-07-203-0/+248
|\ \
| * | Implement rbac list and show commandting wang2016-06-273-0/+248
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. implement "openstack network rbac list" 2. implement "openstack network rbac show" 3. also add FakeRBACPolicy to test "network rbac xxx" command The unit test class similar to FakeRouter, which is able to fake one or more rbac policies. It will be used by the rbac CRUD patches. Change-Id: I6c97bc8819698546895fd530464a2cbb347bf77d Co-Authored-By: Huanxuan Ao <huanxuan.ao@easystack.cn> Partially-Implements: blueprint neutron-client-rbac Depends-On: I88f409a24947b67146c0f93ec8480834cef56d2f
* | | Merge "Transfer "ip floating CRUD" to "floating ip CRUD""Jenkins2016-07-201-5/+104
|\ \ \
| * | | Transfer "ip floating CRUD" to "floating ip CRUD"Tang Chen2016-07-201-5/+104
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch does the following things to transfer "ip floating xxx" to "floating ip xxx": * Add new command "floating ip create/delete/list/show", and doc. * Deprecate "ip floating create/delete/list/show" command. Change-Id: Ib071acaac81988431244e858bddafa7f93403df5 Implements: blueprint rework-ip-commands Closes-bug: 1555990 Co-Authored-By: Dean Troyer <dtroyer@gmail.com>
* | | | Merge "Add port security option to network commands"Jenkins2016-07-203-0/+51
|\ \ \ \ | |/ / / |/| | |
| * | | Add port security option to network commandsRichard Theis2016-06-293-0/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add the "--enable-port-security" and "--disable-port-security" options to the "network create" and "network set" commands. This supports setting the default port security for ports created on a network. Change-Id: I1deb505bd77cef2e4bc3c2dbbb0c450665136f47 Implements: blueprint neutron-client
* | | | Merge "Exchange the check order for the dhcp and no-dhcp"Jenkins2016-07-192-4/+1
|\ \ \ \
| * | | | Exchange the check order for the dhcp and no-dhcpqtang2016-07-192-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The dhcp is setting with True by default and progress always jump into the first if check. So the no-dhcp option always ignored there. Check the no-dhcp option first and then the dhcp option value to avoid this. Change-Id: Ide640e2cab3936d419ca62105304ff5d4a8a2074 Closes-Bug: #1602588
* | | | | Merge "Modify some help and error messages in ec2creds identityv2"Jenkins2016-07-191-2/+2
|\ \ \ \ \
| * | | | | Modify some help and error messages in ec2creds identityv2Huanxuan Ao2016-07-121-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Usually we use "(s)" to show about multi deletion in help message. In addition, I think "EC2 credentials" is better than "EC2 keys" in the error message. Change-Id: I6a6461291542701d87a55d9ea0ea1fda6db04601
* | | | | | Merge "Update doc for credential in indentityv3"Jenkins2016-07-191-10/+10
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Update doc for credential in indentityv3Huanxuan Ao2016-07-151-10/+10
| |/ / / / | | | | | | | | | | | | | | | Change-Id: I5f49c038a75ea67674b6d9279a6e60d6ded8d12f
* | | | | Support bulk deletion for delete commands in identityv3Huanxuan Ao2016-07-1921-58/+282
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support bulk deletion for delete commands in the list below identity/v3/consumer identity/v3/credential identity/v3/domain identity/v3/ec2creds identity/v3/endpoint identity/v3/federation_protocol identity/v3/identity_provider identity/v3/mapping identity/v3/policy identity/v3/region identity/v3/service_provider identity/v3/service The unit test in identityv3 need to be refactored, so I add some functional tests instead. I will add all unit tests at one time after the refactor completed. Change-Id: I82367570f59817b47c87b6c7bfeae95ccfe5c50e Closes-Bug: #1592906
* | | | | Merge "Add create_one_image_member() in FakeImage class and update test"Jenkins2016-07-182-13/+32
|\ \ \ \ \
| * | | | | Add create_one_image_member() in FakeImage class and update testHuanxuan Ao2016-07-152-13/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Usually we use a fake object in unit test. To match the other test, add create_one_image_member() in FakeImage class in imagev2 to create a fake image member, delete the old data and update the unit test of image. Change-Id: I062a362b15db7e8bc89ec48c540d310199fd6c0b
* | | | | | Merge "Support error handling for delete commands in volumev2"Jenkins2016-07-189-40/+364
|\ \ \ \ \ \
| * | | | | | Support error handling for delete commands in volumev2Huanxuan Ao2016-07-149-40/+364
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some delete conmmands in volumev2 did not support error handling, this patch add them and also add the unit tests for bulk deletion Change-Id: I56ade6f9c7396c78fb989547476c4d94ccd76eae
* | | | | | | Merge "Make set/unset commands pass normally when nothing specified in ↵Jenkins2016-07-1819-94/+32
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | identityv3"
| * | | | | | | Make set/unset commands pass normally when nothing specified in identityv3Huanxuan Ao2016-07-1419-94/+32
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I554b41969f96b62a2c6d37024caa56b1441d5ed1 Partial-bug: #1588588
* | | | | | | Merge "Add "--project" option to "volume type create" command"Jenkins2016-07-182-1/+46
|\ \ \ \ \ \ \
| * | | | | | | Add "--project" option to "volume type create" commandHuanxuan Ao2016-07-122-1/+46
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add "--project" and "--project-domain" options to "volume type create" command. We can use these options to add the type access to a given project when we create the volume type. Change-Id: I483a6b61dae137682c3d1f7527531b40e508ba92 Closes-Bug: #1602169
* | | | | | | Merge "image list: Add Checksum column"Jenkins2016-07-184-0/+8
|\ \ \ \ \ \ \
| * | | | | | | image list: Add Checksum columnMarc Abramowitz2016-07-114-0/+8
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The checksum could potentially be useful for spotting duplicated images or checking if images with the same name are identical or different. Closes-Bug: #1602073 Change-Id: Ia0c41970c846d550de14297e18bc738e847e5a3b
* | | | | | | Merge "Pass security group id to novaclient"Jenkins2016-07-182-1/+54
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | |
| * | | | | | Pass security group id to novaclientRajasi Kulkarni2016-07-112-1/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In RemoveServerSecurityGroup we currently pass the entire security group object, which results in TypeError in novaclient. Added unit test case to test command 'openstack server remove security group -h <server> <group>' Change-Id: I6d486403a83804c3a30d6f89d2cf7f64f09797c6 Closes-Bug: 1590883
* | | | | | | Merge "Change to plural form of object in multi delete error message in ↵Jenkins2016-07-144-7/+7
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | networkv2"
| * | | | | | | Change to plural form of object in multi delete error message in networkv2Huanxuan Ao2016-07-144-7/+7
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Usually, the error message of multi delete is: "'result' of 'total' 'objects' failed to delete" the objects is a plural form. To match the other multi delete error messages in OSC, change the object in delete error message in networkv2 to a plural form. Just add a 's' in the message. Change-Id: I17e0735d025bb61014db709d2639813565015b3d
* | | | | | | Merge "Modify compute agent set command"Jenkins2016-07-142-17/+92
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | Modify compute agent set commandsunyajing2016-07-112-17/+92
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Migrate ``compute agent set`` arguments: version, url, md5hash to be optional. BackwardsIncompatibleImpact Change-Id: I092b7ed24274bafa548f0537c4586504be3a2825 Co-Authored-By: Huanxuan Ao <huanxuan.ao@easystack.cn>
* | | | | | | Merge "Use FakeProject and FakeDomain classes in unit tests of networkv2"Jenkins2016-07-147-121/+77
|\ \ \ \ \ \ \
| * | | | | | | Use FakeProject and FakeDomain classes in unit tests of networkv2Huanxuan Ao2016-07-117-121/+77
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | FakeProject and FakeDomain classes have been completed in identityv2_0 and v3, Now we can use these classes instead of old test data in networkv2. Change-Id: I2f698e54ff35e24ffbdcaba29da6f96eb263cc0a Partially-Implements: blueprint use-fake-project
* | | | | | | | Merge "Change the wrong import order"Jenkins2016-07-141-1/+2
|\ \ \ \ \ \ \ \
| * | | | | | | | Change the wrong import orderHuanxuan Ao2016-07-121-1/+2
| | |_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | Change-Id: Ia48f0aae0250a5d29c0f6fa46386465e118f760f
* | | | | | | | Merge "Remove useless dest of option in volume v1&v2"Jenkins2016-07-143-4/+0
|\ \ \ \ \ \ \ \
| * | | | | | | | Remove useless dest of option in volume v1&v2Huanxuan Ao2016-07-113-4/+0
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | some "dest" argument is useless because they are the same as the option name. So remove them. Change-Id: Idd37e8816cf8b0833c94a5e741fdfe56602551f9
* | | | | | | | Temp work around for missing select_auth_plugin()Dean Troyer2016-07-131-2/+74
| |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These were removed prematurely from osc-lib (by me) but the real fix in https://review.openstack.org/329189 is having racy functional test issues that may be related to osc-lib, so let's clear this up while we fix that... Change-Id: I8f67466967751fdf6fd24ae1b16ccee2aec52323
* | | | | | | Merge "Add "--incremental" option to "backup create" command in volume v2"Jenkins2016-07-112-0/+11
|\ \ \ \ \ \ \
| * | | | | | | Add "--incremental" option to "backup create" command in volume v2Paul Bourke2016-07-112-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cinder V2 API supports creating volume backup with "--incremental" option. However, OSC doesn't support this argument. So this patch adds the "--incremental" option to allow users to create incremental style backups. Change-Id: Iefac5f1a6c9ef006ad9c22d4250ae6df50504781 Closes-Bug: 1600196
* | | | | | | | Merge "Unskip tests caused by bug 1599333"Jenkins2016-07-114-18/+24
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
| * | | | | | | Unskip tests caused by bug 1599333Steve Martinelli2016-07-074-18/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is now a second .get() call in osc_lib.utils.find_resources. These tests were failing because they only mocked a single access call to .get(). Ensure there are two calls to .get(), with the first one raising an exception. Change-Id: Idd2ad4a27a6db5bee633cc37a1042dbb0a57aa71 Closes-Bug: #1599333
* | | | | | | | Remove FakeService.get_serviceszheng yin2016-07-091-20/+0
| |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | FakeService.get_services is useless in other test files, so remove it Change-Id: If90b8742be97697f285750a05896b574c45f3504
* | | | | | | Merge "Add notes, modify notes in fakes docstring"Jenkins2016-07-111-6/+17
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | |
| * | | | | | Add notes, modify notes in fakes docstringzheng yin2016-07-081-6/+17
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some code has no notes, then add them Some code notes are not perfect, then improve them Some code notes are incorrect， then modify them Change-Id: I43f9a8663ae138bdd494bc234c1fac00c2dd6c95
* | | | | | remove unused LOGji-xuepeng2016-07-091-4/+0
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | This is to remove unused LOG to keep code clean. Change-Id: Ic544157b19cefb4120381f3314334c60a9020baf
* | | | | Merge "Correct reraising of exception"Jenkins2016-07-0712-29/+29
|\ \ \ \ \
| * | | | | Correct reraising of exceptionqinchunhua2016-07-0712-29/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When an exception was caught and rethrown, it should call 'raise' without any arguments because it shows the place where an exception occured initially instead of place where the exception re-raised. Change-Id: I5fb6dea5da7fb6e1e2b339a713c7d37f8c99e407
* | | | | | Merge "osc-lib: api.auth"Jenkins2016-07-074-226/+12
|\ \ \ \ \ \
| * | | | | | osc-lib: api.authDean Troyer2016-06-304-226/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move auth plugin checking to osc-lib. Change-Id: I673d9c2d6e8bbf724c3000459a729e831d747814
View Lorry Controller Status