summaryrefslogtreecommitdiff
path: root/openstackclient/volume/v1
Commit message (Collapse)AuthorAgeFilesLines
* Add and modify options for "volume create" commandHuanxuan Ao2016-09-281-8/+22
| | | | | | | | | | | | | | 1.Add mutually exclusive options into a mutually exclusive group. 2.Add "--source-replicated", "--consistency-group", "--hint" and "multi-attach" options 3.Make --size option to be optional under some cases Closes-Bug: #1568005 Closes-Bug: #1627913 Implements: bp implement-cinder-features Co-Authored-By: Roman Vasilets <rvasilets@mirantis.com> Change-Id: I2c4c3073195d33774e477f4d7f22e383b14b41dd
* Fix unset commands to pass normally when nothing specifiedHuanxuan Ao2016-09-233-5/+0
| | | | | | | | | | | | | | | | After I found this problem appear in "volume unset", I checked all the volume command and also found some same problems. This patch fix them all. The main reason of we ignored this problem before is there was not any tests for it. So I add tests for "nothing unset" for them all to test and aviod this problem. Also, I add unit tests for all snapshot commands in volume v1 by the way in this patch. We will need more tests to avoid some ignored problem. Change-Id: I46775f24643d715e168b30785b8b531c0431a55b Partial-bug: #1588588
* Implement "volume transfer request show/accept" commandHuanxuan Ao2016-09-221-0/+49
| | | | | | | | | | | | Add "volume transfer request show" and "volume transfer accept" commands in volume v1 and v2. Also add the unit tests, docs, release note and functional tests Implements: bp cinder-command-support Co-Authored-By: Sheel Rana <ranasheel2000@gmail.com> Change-Id: I5787fc486b3401307125caa316f517b9c96a95a5
* Fix "volume unset" command pass normally when nothing specifiedHuanxuan Ao2016-09-211-2/+0
| | | | | | | | | | | | When nothing specified in "volume unset" command, there will be an error message says that the "--properties" option is required, it is unusual behaviour, this patch fix it and also add unit test for it. Also, this patch add unit test for "volume show" command by the way. Change-Id: I5b5d587670acf0af4262b8521292455bf9f60fe5 Partial-bug: #1588588
* Merge "Support mark volume as bootable in volume set"Jenkins2016-09-141-1/+17
|\
| * Support mark volume as bootable in volume setqtang2016-09-141-1/+17
| | | | | | | | | | | | | | | | Add --bootable | --non-bootable option in volume set to mark or unmark volume as bootable. Change-Id: Ifa6c2dd1642202f55b6d50e3b8614d3513d488f6 Closes-Bug:#1535704
* | Merge "Do not show "os-volume-type-access:is_public" property of volume type"Jenkins2016-09-141-3/+3
|\ \
| * | Do not show "os-volume-type-access:is_public" property of volume typeHuanxuan Ao2016-09-071-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "os-volume-type-access:is_public" property is the same as "is_public" property in volume type object. So stop showing "os-volume-type-access:is_public" property and leave "is_public" property only. Change-Id: Ic78a9ee69b0ab356edff18cdb4c46fc24b495d2b Closes-Bug: #1620922
* | | Trivial: Rename ListTransferRequests to ListTransferRequestTang Chen2016-09-121-2/+2
| |/ |/| | | | | | | | | The class name should use plural. Change-Id: I2de2489ff9aa60c2d1bf12743cbd41f2091739ca
* | Implement "volume transfer request delete" commandHuanxuan Ao2016-09-111-0/+41
| | | | | | | | | | | | | | | | | | | | Add "volume transfer request delete" command in volume v1 and v2. Also add the unit tests, docs, release note and functional tests Change-Id: Ic3d375bc8df3312fac53c1800d75f48376b8c91c Implements: bp cinder-command-support Co-Authored-By: Sheel Rana <ranasheel2000@gmail.com>
* | Implement "volume transfer request create" commandHuanxuan Ao2016-09-111-1/+31
| | | | | | | | | | | | | | | | | | | | Add "volume transfer request create" command in volume v1 and v2. Also add the unit tests, docs, release note and functional tests Change-Id: If362df1acf214efdf6ba129cd917d33eb54e1030 Implements: bp cinder-command-support Co-Authored-By: Sheel Rana <ranasheel2000@gmail.com>
* | Merge "Error handling of multi REST API calls for "snapshot set" command"Jenkins2016-09-111-3/+17
|\ \
| * | Error handling of multi REST API calls for "snapshot set" commandHuanxuan Ao2016-09-111-4/+23
| |/ | | | | | | | | | | | | | | | | Support multi REST API calls error handling for "snapshot set" command follow the rule in doc/source/command-errors.rst. Also add a unit test for testing the error handling Change-Id: I0c6214271bc54a25b051c0a62438c3344c8b51d7
* | Merge "Add "--limit" and "--marker" options to "volume list" command"Jenkins2016-09-111-1/+11
|\ \
| * | Add "--limit" and "--marker" options to "volume list" commandHuanxuan Ao2016-09-111-1/+11
| |/ | | | | | | | | | | | | | | | | Add ``--limit`` option to ``volume list`` command in volume v1, add ``--limit`` and ``--marker`` options to ``volume list`` command in volume v2. Change-Id: I327a252aa83ed84166da99cf6aa80334e0e6dd44 Partial-Bug: #1612484
* | Support error handling for delete commands in volume v1Huanxuan Ao2016-09-114-18/+90
|/ | | | | | | | | Some delete commands in volume v1 support multi delete but do not support error handling, this patch fixes them, and this patch also refactor (or add new) unit tests for some delete commands in volume v1. Change-Id: Ia8177698f8733cfe75ea0ff00eee8fdc0820f62e
* Add "volume service set" commandHuanxuan Ao2016-08-171-0/+56
| | | | | | | | Add "volume service set" command in volume v1 and v2 (v1 is the same as v2) to disable or enable volume service. Change-Id: Ibb2db7e93b24cb2e0d2a7c28b6fd8bcc851b8d2f Closes-Bug: #1613597
* Rename backup commands in volume v1 and v2Huanxuan Ao2016-08-111-15/+101
| | | | | | | | | | | | | | | | | | | | | Backup commands are used only in volume service now, but "backup" is too generic, users may not know the commands are used for volume from the commands name. By seeing the command name, users can only see the "backup" but do not know which object the backup commands work for. It may confuse users. I think rename "backup" to "volume backup" can depict resource relation and will be helpful for users to know the commands clearly. So add new commands ``volume backup create/delete/ list/show/restore`` to replace the old commands ``backup create/delete/list/show/restore``. And also deprecate old commands. Change-Id: I4f844d9bc48573eb4d17288ce6b8a90cea00d16a Implements: bp backup-snapshot-renamed-for-volume-resource Co-Authored-By: Sheel Rana <ranasheel2000@gmail.com>
* Remove useless dest of option in volume v1&v2Huanxuan Ao2016-07-111-1/+0
| | | | | | | some "dest" argument is useless because they are the same as the option name. So remove them. Change-Id: Idd37e8816cf8b0833c94a5e741fdfe56602551f9
* Merge "Add "--force" option to "volume qos delete" command"Jenkins2016-06-281-1/+7
|\
| * Add "--force" option to "volume qos delete" commandHuanxuan Ao2016-06-281-1/+7
| | | | | | | | | | | | | | | | Add ``--force`` option to ``volume qos delete`` command in volume v1 and v2 to allow users to delete in-use QoS specification(s). Change-Id: I46036e5f55ced8b8a1be54c521f2a5c242b89160 Closes-Bug: #1596821
* | Change "ID only" to "name or ID" for backup commandsHuanxuan Ao2016-06-271-3/+3
|/ | | | | | | | Tested "find_resulce()" can find a backup by name or ID. But some help messages and doc were "ID only", this patch modify them. Change-Id: I22dcc3a0b40f2bafabb6d26498158e2468037312
* support multi-delete for volume-typeSteve Martinelli2016-06-201-6/+29
| | | | | | | | | Added the ability to delete multiple volume types at once. Note there are no unit tests exist for v1 volume-types, so instead a functional test was created. Partial-Bug: #1592906 Change-Id: I99f3f22901ab35252b91a3072b14de7d19cb17ca
* Merge "Standardize logger usage in volume"Jenkins2016-06-151-5/+8
|\
| * Standardize logger usage in volumeTang Chen2016-06-141-5/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | self.app.log is the logger in class OpenStackShell, which should be used to record logs that have nothing to do with any specific command. So, use the file logger instead. This patch also fixes some usage that doesn't follow rules in: http://docs.openstack.org/developer/oslo.i18n/guidelines.html 1. add variables to logger as an argument 2. do not wrap variables with str() Change-Id: I248861a38a4de0412a080046aa7a6f6473c3e082 Implements: blueprint log-usage
* | Fix foundation copyrightsDean Troyer2016-06-141-1/+1
|/ | | | | | | | The OpenStack LLC copyrights date from before OpenStack was transferred to the foundation. These appear to be getting copy-pasted to new files so at least should reflect the correct entity. Change-Id: I02953d752cb24ead6aa4ad8bfe257a48317c9f13
* osc-lib: commandDean Troyer2016-06-137-7/+7
| | | | | | | Leave command.py and test_command.py as a sanity check during the deprecation period. Change-Id: I24e1b755cbfbcbcaeb5273ec0c9706b82384fc85
* osc-lib: parseractionsDean Troyer2016-06-134-4/+4
| | | | | | | Leave parseractions.py and test_parseractions.py as a sanity check during the deprecation period. Change-Id: I1a7469b6d872284e0276502a1a287bc0b87f8f83
* osc-lib: utilsDean Troyer2016-06-137-7/+11
| | | | | | | | Use osc-lib directly for utils. Leave openstackclient.common.utils for deprecation period. Change-Id: I5bd9579abc4e07f45219ccd0565626e6667472f7
* Merge "Make set/unset commands in volume return normally when nothing specified"Jenkins2016-06-064-17/+0
|\
| * Make set/unset commands in volume return normally when nothing specifiedTang Chen2016-06-044-17/+0
| | | | | | | | | | | | | | | | | | | | | | set/unset commands should ends up normally instead of logging an error when nothing is specified to modify. The main reason is: When nothing is specified, the command sets/unsets nothing, which is a normal behavior, and ends up normally. No API call fails. No error happens. Change-Id: Ib03a512650e5da90aa1ef38019772448383d0d33 Partial-bug: #1588588
* | Merge "Add support for volume transfer request list"Jenkins2016-06-041-0/+51
|\ \ | |/ |/|
| * Add support for volume transfer request listSheel Rana2016-06-041-0/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | OSC does not support to list volume transfer from one user to other user. This patch will provide support for listning volume transfer requests. Closes-Bug:#1554886 Implements: bp cinder-command-support Change-Id: Ie659bc39cb1d1e931cca7a40b2f126a067ee484c
* | i18n support for help and error messages in cinderSheel Rana2016-05-236-97/+115
|/ | | | Change-Id: I98fbc959034fe0530966291643b381855801de20
* Merge "Support for volume service list"Jenkins2016-04-281-0/+70
|\
| * Support for volume service listSheel Rana2016-04-251-0/+70
| | | | | | | | | | | | | | | | | | | | | | | | OSC does not support to list volume services. This patch will provide support for adding volume service related support. Closes-bug:#1550999 Implements: bp cinder-command-support Change-Id: I50ac14aeb96c4b8ddbf7b33e519feea0d126f752
* | Trivial: Fix incorrect comment textTang Chen2016-04-141-1/+1
| | | | | | | | Change-Id: Ia6ec15f11a535a8de5769569d75e81094caed171
* | Doc: Unify repeatable option commentsTang Chen2016-04-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | There are lots of "this option can be repeated" comments in the doc, which are not consistent to other similar docs. This patch changes them to the following format: "repeat option to do something" Change-Id: I54e01053091c428bf87bb36bb95f73a0b80ab6e7
* | Merge "Make snapshot and backup name optional"Jenkins2016-04-072-2/+0
|\ \ | |/ |/|
| * Make snapshot and backup name optionalIvan Kolodyazhny2016-04-062-2/+0
| | | | | | | | | | | | | | Cinder does not require snapshot and backup name. These arguments are optional. Change-Id: I05d59efc9642205a25684bf0b77758328296d959
* | Merge "Docs cleanup: volume type"Jenkins2016-03-291-25/+25
|\ \
| * | Docs cleanup: volume typeDean Troyer2016-03-281-25/+25
| |/ | | | | | | | | | | | | Clean up volume type command help, add volume type show to doc, sort command classes in v1/volume_type.py. Change-Id: I1f8e5c047d9c08f5704fc23cfb694f23d32e3caf
* | Docs cleanup: volume command helpDean Troyer2016-03-281-23/+23
|/ | | | | | | | | This formats the volume command help text consistent with the rest of OSC, adds some reference targets and some explanation text to some commands. No functional changes have been made, only cosmetic/help output. Change-Id: Ib86ec3ca58bdea5f33078ced3ec3583b2be0e89a
* log take_action parameters in a single placeAkihiro Motoki2016-02-025-133/+20
| | | | | | | | | | | | 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
* Trivial: Remove useless return from files in image and volumeTang Chen2015-12-185-21/+0
| | | Change-Id: I3526ecd202d0908d91305a066ad72d03cee794b5
* Consistency of the --all argument for snapshotsJean-Philippe Evrard2015-11-301-1/+12
| | | | | | | | | This change is inspired by the volume.py. It allow the user to use openstack snapshot list --all. Closes-Bug: #1517386 Change-Id: I72a53fcd0c5c5af539cd88b37e71b4331fa67473
* Trivial: Fix wrong param name in comment.Tang Chen2015-11-121-1/+1
| | | | | | "volume" should be "attachments". Change-Id: I41c797f0c7cccb2727e9e6cd6424ea2f740624b7
* Follow-on for volume list - add tests, clean helpDean Troyer2015-10-161-5/+5
| | | | | | | | | | * Add volume list tests for v1 (a copy of the v2 tests) * Converts volume v2 tests to use Identity v3 so domains can be tested * Add volume list (v2) tests for new options * Re-orders volume list options (both v1 and v2) to match * MArks the new volume list (v2) options as v2-only in doc Change-Id: I2181b2c48cfde2147d7d0ef135322df8a81e7ce8
* Use a common decorator to log 'take_action' activationJoshua Harlow2015-09-015-30/+30
| | | | | | | | | Instead of duplicating the same log statement throughout the code, the same logic can be provided by a shared decorator that abstracts away the logging capability and unifies it behind a common function instead. Change-Id: Icc63bced7347c8bbf0299a4c5821425a10892a79
* Rename type.py to volume_type.pySteve Martinelli2015-07-191-0/+0
| | | | | | | | For Volume V1, we have a type.py file which should be renamed to volume_type.py (as it's named for V2). Change-Id: If860bbafe4a801d8b4fa06938eef20658c4fcc2c Closes-Bug: 1475958
View Lorry Controller Status