summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | Added a note on how to test changesRyan Selden2016-07-071-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's confusing for new contributors to test the client. Added a quick note to the README explaining how to do so. Change-Id: I2a185f04bbec7ab01fdd2ab436a5f413f33819e7
* | | | | | | | | Merge "Unskip the tests affected by warlock 1.3.0"Jenkins2016-07-181-3/+0
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Unskip the tests affected by warlock 1.3.0Steve Martinelli2016-07-171-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1) This reverts commit 044a46ed5f040020cf80adc1ace987802344e87d which was broken by warlock 1.3.0 Change-Id: Ia286d9394586884018c4e62cf581e39a809f5545
* | | | | | | | | | Merge "Follow upper constraints for all tox targets"Jenkins2016-07-182-1/+61
|\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / /
| * | | | | | | | | Follow upper constraints for all tox targetsSteve Martinelli2016-07-172-1/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With the exception of releasenotes and cover, we should follow upper constraints. The tox_install file was copied over from python-neutronclient [1]. [1] http://git.openstack.org/cgit/openstack/python-neutronclient/tree/tools/tox_install.sh Change-Id: I633fa149820efafd7b2acec0388fa8bc8d06c988
* | | | | | | | | | 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 "Allow format selection in get_opts"Jenkins2016-07-181-2/+3
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Allow format selection in get_optsKyrylo Romanenko2016-07-141-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Default is "value". Change-Id: I244253f6cd53104a57ef12f6b14e1653c89d38bc
* | | | | | | | | | | 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-184-1/+67
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Add "--project" option to "volume type create" commandHuanxuan Ao2016-07-124-1/+67
| |/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-185-0/+12
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | image list: Add Checksum columnMarc Abramowitz2016-07-115-0/+12
| |/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "Add network-topolopy support"Jenkins2016-07-181-0/+44
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|_|/ / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Add network-topolopy supportdongwenshuai2016-07-081-0/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a new api about network-topolopy in openstackclient It is the intention for OSC to collect data from existing REST APIs Change-Id: I23709ee906555d773c3efeb094aef50587732a1b Related-Bug: #1586584 Implements: bp network-topology-support
* | | | | | | | | | | | | Updated from global requirementsOpenStack Proposal Bot2016-07-171-1/+1
| |_|_|_|_|/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I74744755891f65da2c1d7b7b475230d395a3a8ee
* | | | | | | | | | | | Merge "Add python-neutronclient to OSC plugins"Jenkins2016-07-153-3/+11
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Add python-neutronclient to OSC pluginsRichard Theis2016-07-053-3/+11
| | |_|_|_|_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Neutron is transitioning its CLI to OSC. Some CLIs will reside within python-openstackclient while others will be OSC plugins within python-neutronclient. Depends-On: I9a20bc7a3d8aa7b631fb0fd534fc5705c23326ce Change-Id: I2d8d369739d1df1597201368dd2158e07c8d2baf Related-Bug: #1521291
* | | | | | | | | | | | Updated from global requirementsOpenStack Proposal Bot2016-07-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I51ce0df359d992153cbe0588c4f67d829a360032
* | | | | | | | | | | | 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-146-30/+125
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Modify compute agent set commandsunyajing2016-07-116-30/+125
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "Add Python3.5 to setup.cfg tox.ini"Jenkins2016-07-142-1/+2
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Add Python3.5 to setup.cfg tox.inizheng yin2016-07-122-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now that there is a passing gate job, we can claim support for Python 3.5 in the classifier. This patch also adds the convenience py35 venv. Change-Id: If93af96739741584c87913ba140d0c6cee2aa10d
* | | | | | | | | | | | 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-114-0/+24
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Add "--incremental" option to "backup create" command in volume v2Paul Bourke2016-07-114-0/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | | | | | | | Merge "Make the doc build reproducible"Jenkins2016-07-111-1/+1
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Make the doc build reproducibleThomas Goirand2016-07-111-1/+1
| | |_|_|_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes Debian bug: https://bugs.debian.org/826676 Change-Id: I638488448a4d6e644ab0e6deaebd65c92fa7904a
* | | | | | | | | | | Merge "Add missing "Volume version 2 only" message in backup.rst"Jenkins2016-07-111-0/+5
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Add missing "Volume version 2 only" message in backup.rstHuanxuan Ao2016-07-111-0/+5
| | |_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Former patchs add some volume v2 only options for backup but miss "Volume version 2 only" message. This patch adds them. Change-Id: Id50db25617776b6ee0d0ad6b4b82bd443b940343
* | | | | | | | | | | 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
* | | | | | | | | Merge "Updated from global requirements"Jenkins2016-07-101-1/+1
|\ \ \ \ \ \ \ \ \