summaryrefslogtreecommitdiff
path: root/openstackclient/volume/v1
Commit message (Collapse)AuthorAgeFilesLines
* 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
* Merge "Add volume type show for volume v1"Jenkins2015-07-191-0/+24
|\
| * Add volume type show for volume v1jiaxi2015-07-191-0/+24
| | | | | | | | | | | | | | volume type show is missing for volume V1 API. Closes-Bug: #1475879 Change-Id: Ic8a0845ecec04146d536412463175f57ef6511ae
* | --property should be required in `os unset` commandsjiaxi2015-07-193-0/+3
|/ | | | | | | | | | | | | | The three commands below openstack volume unset openstack snapshot unset openstack volume type unset Should have --property as a required argument, not optional. The reason is the command will not work without --property. Closes-Bug: #1475872 Change-Id: Ib9e29392472db38982cc2817af2dd5055f5a01ca
* Support multiple volume qos deleteTerryHowe2015-07-171-38/+37
| | | | | | | Also make sure that we are using the singular qos_spec when we should and the plural qos_specs otherwise. Change-Id: If4bbeb6fc245d7d80dc3d7dccfe9f949e802653c
* No need for get_parser on QoS listSteve Martinelli2015-06-281-4/+0
| | | | | | | There are no arguments so there's no need to define the get_parser function. Change-Id: Icfa8accf6dbb7f8d1a0472926403b405da3cc611
* Alphabetize QoS specsSteve Martinelli2015-06-281-104/+104
| | | | | | | | setup.cfg and the implementation had some functions that were not in alphabetical order. Since the rest of OSC is alphabetized, let's stick to that. Change-Id: Ief5d4694c7b6bc20a0898437b96305885104d45c
* Add support for volume API v1 QoS commandsDavid Moreau Simard2015-06-261-0/+307
| | | | | | | | | | | | | | | This commit adds the following commands: volume qos associate volume qos create volume qos delete volume qos disassociate volume qos list volume qos set volume qos show volume qos unset Change-Id: I72ea1b9a4d0bd0e35eda03071ea438b75439fce9 Partial-Bug: #1467967
* Adds support for container selection for backupDag Stenstad2015-04-211-1/+1
| | | | | | | | | | | | Changed argument from parsed_args.volume to parsed_args.container as per what is expected in python-cinderclient. If not defined, defaults to "backup_swift_container" in cinder.conf. Stops there being a container for every volume you take backups off. Change-Id: I4a34894222f71b0f207d007c32427040589406aa Closes-Bug: 1446751
* Change volume create --snapshot-id to --snapshotlin-hua-cheng2015-02-061-3/+12
| | | | | | | Keeping the option --snapshot-id hidden for backward compatibility. Change-Id: Iefa0aed9be255a5626a9bbb7db77911ef5ca8595 Closes-Bug: #1418742
* Check volume status before extending sizelin-hua-cheng2015-02-061-0/+5
| | | | | | | Added test for Volume Set as well. Change-Id: I8e6794f67b160ca328fcafa8f1d67138b4f3becd Closes-Bug: #1415182
* Add the ability to extend volumes in `osc volume set`Steve Martinelli2015-01-271-1/+14
| | | | | | | | Create a --size option for volume set, so the user may extend a volume's size. Change-Id: Ic8d3396d788a14ea1e10bf1da73edfd7f5d85070 Closes-Bug: #1413954
* Merge "Command doc: snapshot"Jenkins2015-01-091-11/+11
|\
| * Command doc: snapshotSteve Martinelli2015-01-091-11/+11
| | | | | | | | Change-Id: Ibe5cd0a8422788762e0c52b702b7bd54e6a46813
* | Merge "Fix up snapshot command"Jenkins2015-01-091-14/+115
|\ \ | |/
| * Fix up snapshot commandSteve Martinelli2015-01-091-14/+115
| | | | | | | | | | | | | | | | | | | | | | Several issues with the current snapshot command were resolved: * --long for list was added to include volume id/name, and properties * changed output from metadata to properties * added new option to set properties with 'snapshot set' * added new command to unset properties with 'snapshot unset' Change-Id: I5902cfe876cefada701d4d658a50a4282ff300d6
* | Merge "Command doc: backup"Jenkins2015-01-081-11/+11
|\ \ | |/
| * Command doc: backupSteve Martinelli2015-01-081-11/+11
| | | | | | | | Change-Id: Iecd4dbddea637bd6540d94b37253a9ba434c9db3