summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Add --attached / --detached parameter to volume set"Zuul2019-03-064-0/+95
|\
| * Add --attached / --detached parameter to volume setDavid Rabel2019-02-284-0/+95
| | | | | | | | | | | | | | | | | | As to reflect cinder reset-state --attach-status functionality, this patch adds --attached / --detached parameter to OSC's volume set command. Change-Id: Ic8ee928c9ab0e579512cfb7608f63bfcc2993c7b Closes-Bug: #1745699
* | Merge "Fix: Restore output 'VolumeBackupsRestore' object is not iterable"Zuul2019-02-284-5/+67
|\ \
| * | Fix: Restore output 'VolumeBackupsRestore' object is not iterablewhoami-rajat2019-02-274-5/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | VolumeBackupsRetore object has '_info' attribute which contains the output data of the restore command which should be returned instead of the 'VolumeBackupsRestore' object. Change-Id: I64b75649c1ac9c24e05a197f7280975564b4d386 Story: 2004740 Task: 28811
* | | Merge "Partially Revert "Add command to unset information from Subnet-pools""Zuul2019-02-273-63/+1
|\ \ \
| * | | Partially Revert "Add command to unset information from Subnet-pools"Bernard Cafarelli2018-09-053-63/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We do not support removing a prefix from a subnet pool, only updating with a larger prefix (which is handled by the set command) This reverts commit 063c722a110031883e9615064092644de6df8da2. Change-Id: I11224fbdb94dc1caef42a8a64cbcebaf1dc542fe Story: #1670230 Task: #13697
* | | | Merge "This fix removes an erroneous underscore found within the function ↵Zuul2019-02-272-2/+2
|\ \ \ \ | | | | | | | | | | | | | | | named test_snapshot_delete within test_snapshot.py found in both volume v1 and v2 of python-openstackclient."
| * | | | This fix removes an erroneous underscore found within the function namedJeremy Houser2019-02-112-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test_snapshot_delete within test_snapshot.py found in both volume v1 and v2 of python-openstackclient. Story: 2004977 Change-Id: Iae29ba7992dcf8596f4fb4333d8bcf1889ecd7e6
* | | | | Merge "Remove str() when setting network objects names"Zuul2019-02-277-8/+8
|\ \ \ \ \
| * | | | | Remove str() when setting network objects namesBernard Cafarelli2019-02-137-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Most network commands use str() on name argument, which fails on python 2 with Unicode characters. This comes from parsed arguments so does not actually need this call. Sample command failing with current code: openstack network create test_unicode™ Change-Id: Ie10b67864c912ee5c33e90b10c3d9705ee8307e7 Story: 2004356 Task: 27955
* | | | | | Merge "Fix help message of image add project"Zuul2019-02-271-1/+1
|\ \ \ \ \ \
| * | | | | | Fix help message of image add projectDavid Rabel2019-02-211-1/+1
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Only with the admin role you can use the project name with 'image add project'. With the normal member role you have to use the project id instead. If you try to use the name, you don't receive an error, but it won't work. Change-Id: I61d402b39558320502dc08905b8c3a146e5e740a Task: 29543 Story: 2002535
* | | | | | Merge "add python 3.7 unit test job"Zuul2019-02-271-0/+1
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | |
| * | | | | add python 3.7 unit test jobZhongShengping2019-02-191-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a mechanically generated patch to add a unit test job running under Python 3.7. See ML discussion here [1] for context. [1] http://lists.openstack.org/pipermail/openstack-dev/2018-October/135626.html Change-Id: Ic1d05a80d286ff95777eb30d66086ef2b57bdb7f Story: #2004073 Task: #27438
* | | | | | Disabling c-backup service for osc-functional-devstack-tips jobwhoami-rajat2019-02-261-0/+5
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since swift isn't compatible with py3 currently and disabled for this gate job, c-backup service will always fail configuring. The backup related tests can be handled by other jobs having swift enabled. The c-backup service can be enabled along with swift services once swift is compatible with py3. This patch disables the the c-backup service for osc-functional-devstack-tips gate job. Change-Id: Ifd3a4e1a15f1365107a2a1367513e2ef79bd13cc
* | | | | Fix help message of image add projectDavid Rabel2019-02-191-1/+1
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Only with the admin role you can use the project name with 'image add project'. With the normal member role you have to use the project id instead. If you try to use the name, you don't receive an error, but it won't work. Change-Id: I2d11c07a256917d12c46a7c302c5a5e8752a1df0 Task: 29543 Story: 2002535
* | | | Add py36 envwhoami-rajat2019-02-131-1/+1
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While running `tox` command in binoic env, we see the following message, ERROR: InterpreterNotFound: python3.5 It is because the default py3 version for binoic is py36. This patch adds the env in OSC also maintaining consistency with setup.cfg Change-Id: I8e5cf72901cba34ad44f2b356609f85b3b0c431f
* | | More state handling in volume transfer requests functional testsDean Troyer2019-02-061-8/+19
|/ / | | | | | | | | | | | | | | | | | | | | | | Using addCleanup() for removing the pending volume transfer request has no way to wait for the volume status to become available before cleaning up the volume and gets racy when the tests are run with slow performance in the volume backend. So we pause at the end of the test after either accepting the transfer request or explicitly deleting it so the cleanup can delete the volume. Change-Id: I04862069cab28bc76eeafd60ba32be646f478d86 Signed-off-by: Dean Troyer <dtroyer@gmail.com>
* | Merge "fix multiple server delete produce multiple new lines"Zuul2019-01-161-7/+3
|\ \
| * | fix multiple server delete produce multiple new linesNoam Angel2019-01-161-7/+3
| | | | | | | | | | | | | | | Closes-Bug: #1809874 Change-Id: Ib988b189b41af03d3d871b660bb5b5cc090c3f30
* | | Merge "Add floating IP filter to floating IP list command"Zuul2019-01-153-0/+31
|\ \ \
| * | | Add floating IP filter to floating IP list commandJohannes Kulik2019-01-143-0/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a parameter ``--floating-ip-address`` to ``floating ip list`` because it's supported by the API and also more efficient than the current ``floating ip show``. This also works as a work-around for pagination issues ``floating ip show`` might run into with an IP parameter. Change-Id: I113e3fa2495e1e86bb553c55c44f71a3f9f49d23
* | | | Use os-cloud instead of OS env vars for functional testsMonty Taylor2019-01-106-51/+87
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In order to support switching auth contexts, such as for registered_limits which take a system scoped token, switch the functional tests to using the --os-cloud command line parameter. However, honor the OS_CLOUD env var as a way that someone can select a different cloud, including 'envvars', to use. Use devstack-system-admin cloud for limit tests Keystone requires these to have system scope now. Change-Id: Ia81eebd3e00ae986cf3ba7e3d98f3e8a1647b622
* | | Merge "trivial: remove commented-out code"Zuul2018-12-301-2/+0
|\ \ \ | |/ / |/| |
| * | trivial: remove commented-out codeChen2018-09-281-2/+0
| | | | | | | | | | | | | | | | | | | | | These comments have existed from the beginning. But they seem to be meaningless. Change-Id: Ic38272ecfb321d77219d477634e9e29b968e7f00
* | | Merge "Support enable/disable uplink status propagation"Zuul2018-12-245-0/+74
|\ \ \
| * | | Support enable/disable uplink status propagationHongbin Lu2018-12-055-0/+74
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add options to enable/disable uplink status propagation on creating a neutron port. Related patches: * neutron: https://review.openstack.org/#/c/571899/ * openstacksdk: https://review.openstack.org/#/c/586687/ Depends-On: https://review.openstack.org/#/c/586687/ Change-Id: I095a98fc5f5aee62d979a16b3cd79d91ec3b9ddb Related-Bug: #1722720
* | | | Merge "More volume functional test fixes"Zuul2018-12-211-43/+48
|\ \ \ \
| * | | | More volume functional test fixesDean Troyer2018-12-181-43/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove the use of class setup/teardown from volume transfer functional tests as that just doesn't work too well here. Also wait for volume status before attempting transfer request operations, some test nodes take a while to create the volumes. Change-Id: Ib9378ab5c973deb2aa86c9b9ed31408f3a05115a Signed-off-by: Dean Troyer <dtroyer@gmail.com>
* | | | | Merge "Remove testr.conf as it's been replaced by stestr"Zuul2018-12-211-9/+0
|\ \ \ \ \
| * | | | | Remove testr.conf as it's been replaced by stestrRui Yuan Dou2018-12-201-9/+0
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ia596789a916b9271933c5b10f00399d83f18d44a
* | | | | | Merge "Update the URL in doc"Zuul2018-12-211-1/+1
|\ \ \ \ \ \
| * | | | | | Update the URL in docmelissaml2018-09-231-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I1c0c58dada83a930fafc7141d5446c754ca5c80c
* | | | | | | Add osc repo to the base job definitionJens Harbott2018-12-201-0/+2
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The osc-functional-devstack job is to be run against devstack changes in order to verify that the devstack-tox-functional job, which our job is based upon, doesn't introduce a regression[0]. In order to prevent this from failing, include our own repo as required-project. [0] https://review.openstack.org/526115 Change-Id: I782cd51dda1477c2e3a067cd902a3d3f29490083
* | | | | | Merge "Modify the help message for 'registered limit set'"Zuul2018-12-181-4/+13
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Modify the help message for 'registered limit set'Vishakha Agarwal2018-11-091-4/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Regsitered limit set CLI takes --service, --region and --resource-name as param which can be updated along with --default limit for existing registered limit. Default limit can be updated with same value and CLI return the success. But --service, --region and --resource- name cannot be same as existing one. CLI return 409 for this case. Which is valid behaviour because more than one limit with same service and same resource cannot exist. But help message of --service, --region and --resource-name are not much clear to tell that they cannot be passed with same value. This patch clarifies the help message for resigtered limit set CLI. Reference Scenario: * openstack registered limit set --default-limit 91 64c2e97fbe904b888544ffdcab21989b limit updated sucessfully Updating limit with exsiting service and resource-name: * openstack registered limit set --default-limit 92 --resource-name snapshot 64c2e97fbe904b888544ffdcab21989b Conflict occurred attempting to store registered_limit - Duplicate entry. (HTTP 409) *openstack registered limit set --default-limit 93 --service compute 64c2e97fbe904b888544ffdcab21989b Conflict occurred attempting to store registered_limit - Duplicate entry. (HTTP 409) *openstack registered limit set --default-limit 91 --resource-name snapshot --service glance 64c2e97fbe904b888544ffdcab21989b Conflict occurred attempting to store registered_limit - Duplicate entry. (HTTP 409) Change-Id: I9e78a6250567cd981adde96946818bb016760a49
* | | | | | Merge "Use devstack functional base job"Zuul2018-12-124-28/+1
|\ \ \ \ \ \
| * | | | | | Use devstack functional base jobMonty Taylor2018-11-224-28/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We extracted some of our functional base job to the devstack repo. Consume it. Change-Id: I11e6f9dab935e4b2cd16228f031f7e0adb3a6c89 Depends-On: I84de60181cb88574e341ff83cd4857cce241f2dd
* | | | | | | Merge "Supports router gateway IP QoS"Zuul2018-12-123-1/+217
|\ \ \ \ \ \ \
| * | | | | | | Supports router gateway IP QoSLIU Yulong2018-12-103-1/+217
| | |_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds --qos-policy and --no-qos-policy to `openstack router set`: --qos-policy <qos-policy> Attach QoS policy to router gateway IPs --no-qos-policy Remove QoS policy from router gateway IPs Adds --qos-policy to `openstack router unset`: --qos-policy Remove QoS policy from router gateway IPs Partially-Implements blueprint: router-gateway-ip-qos Closes-Bug: #1757044 Change-Id: Ifec3b2cf9bdb59513c8bcd7bd60305506a071192
* | | | | | | Merge "Replace assertEqual(True/False, expr) with assertTrue/assertFalse"Zuul2018-12-128-136/+41
|\ \ \ \ \ \ \
| * | | | | | | Replace assertEqual(True/False, expr) with assertTrue/assertFalseliuyamin2018-11-148-136/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In some cases, If the result of expr is a boolen value, we shoud use assertTrue/assertFalse to instead. Because it is clear and simple. Change-Id: I53b345fc3915a7b0e737e9dd4d58fe09c746d61c
* | | | | | | | Merge "Don't display router's is_ha and is_distributed attributes always"Zuul2018-12-102-7/+60
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | |
| * | | | | | | Don't display router's is_ha and is_distributed attributes alwaysSławek Kapłoński2018-05-252-7/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In case when is_ha or is_distributed attribute of Neutron's router is set to None, it means that it wasn't returned from server and should not be displayed. Otherwise it might be confusing for user is making openstack router show <router_name> call as an admin will return e.g. is_ha=True but same call done as regular user will return False or None. It might happen like that because returning of those attributes is forbidden for regular users in Neutron's policy.json Depends-On: https://review.openstack.org/567606/ Change-Id: I626b5193d9ecb308baad7b27939f9673c32b4182 Closes-Bug: #1689510 Task: 19789 Story: 2002110
* | | | | | | | Change openstack-dev to openstack-discusssunjia2018-12-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Mailinglists have been updated. Openstack-discuss replaces openstack-dev. Change-Id: I7c7fdfb71348f77aa6063ec99b3c4a90bde21d06
* | | | | | | | Add Python 3.6 classifier to setup.cfgqingszhao2018-11-301-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Id723040f895f06527d46755dd408dec16200c001
* | | | | | | | Merge "Updated the take_actions for unified limits"Zuul2018-11-293-12/+84
|\ \ \ \ \ \ \ \ | |_|_|/ / / / / |/| | | | | | |
| * | | | | | | Updated the take_actions for unified limitsVishakha Agarwal2018-11-093-12/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When user passes --region None, the find_resource of osc_lib calls get() of region. The get API of region ignores the name param returning all the regions in result. As the find_resource checks many cases against the result returned by get API. The output comes greater than 1, thus returning "More than one region ID exist" which is incorrect. However in case of region which cannot be filtered by name we do not require to check these many cases. The solution is to directly call the get method of APIs and returning No resource name exist with the xyz" on passing invaid parameter. And returning all in case of None. Thus created a new function get_resource which can be used in future too by these types of API's. Change-Id: Ib3f881d34a82af97199ce51bfbefc6f3f08599f1 Closes-bug: #1799153
* | | | | | | | Fix i18n issueliuyamin2018-11-131-1/+1
| |_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fix some i18n issues in the files vapi/compute_v2.py. Change-Id: Ic4da472ca585a35ce64512cf0e72e2fe9d4c9d6e
* | | | | | | Add volume backup import/export commandsSean McGinnis2018-11-108-197/+250
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds commands to import and export volume backup records so they can be imported and restored on other Cinder instances or to the original instance if the service or database has been lost and had to be rebuilt. I know this is a commonly used process by some users, so it would be good to have this functionality in osc so they do not have to switch clients. More details about the export and import process can be found here: https://docs.openstack.org/cinder/latest/admin/blockstorage-volume-backups-export-import.html Change-Id: Ic95f87b36a416a2b50cb2193fd5759ab59336975 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
View Lorry Controller Status