summaryrefslogtreecommitdiff
path: root/openstackclient/volume
Commit message (Collapse)AuthorAgeFilesLines
* log take_action parameters in a single placeAkihiro Motoki2016-02-0210-265/+41
| | | | | | | | | | | | 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-1810-42/+0
| | | Change-Id: I3526ecd202d0908d91305a066ad72d03cee794b5
* Merge "Fix exception when doing volume set operation"Jenkins2015-12-031-1/+1
|\
| * Fix exception when doing volume set operationXi Yang2015-12-031-1/+1
| | | | | | | | | | | | | | | | | | | | The v2 SetVolume extends from show.ShowOne and returns None after setting volume operation. It will raise an exception. This patch is going to fix the issue by changing the parent class of SetVolume to command.Command. Change-Id: Iefa453fe4adad06f2a0601a052c01e74004be5b7 Closes-bug: 1521896
* | Consistency of the --all argument for snapshotsJean-Philippe Evrard2015-11-302-2/+24
| | | | | | | | | | | | | | | | | | This change is inspired by the volume.py. It allow the user to use openstack snapshot list --all. Closes-Bug: #1517386 Change-Id: I72a53fcd0c5c5af539cd88b37e71b4331fa67473
* | Merge "Use is_public to set access of volume type"Jenkins2015-11-301-2/+2
|\ \ | |/ |/|
| * Use is_public to set access of volume typeXi Yang2015-11-261-2/+2
| | | | | | | | | | | | | | | | Currently the 'public' and 'private' keys does not work when creating volume type, 'is_public' should be used. Change-Id: If34a66053ea6c192882a1b9d8bbb1d3666be3f83 Closes-bug: 1520115
* | Fix a bug of "openstack volume delete"xiexs2015-11-271-4/+4
|/ | | | | | | | | While multi volumes specified for the osc "openstack volume delete", only the last volume is deleted. This patch tries to fix it. Change-Id: I171b2869f85b29c88fda16eaf5bf163c55795df0 Closes-Bug: #1520541
* Trivial: Fix wrong param name in comment.Tang Chen2015-11-131-1/+1
| | | | | | "volume" should be "attachments". Change-Id: Id1e9a733e18db595d8981b5b3a7735313a346787
* Trivial: Fix wrong param name in comment.Tang Chen2015-11-121-1/+1
| | | | | | "volume" should be "attachments". Change-Id: I41c797f0c7cccb2727e9e6cd6424ea2f740624b7
* Change method to get the user_idxiexs2015-11-081-3/+3
| | | | | | | | Instead of "find_project", using "find_user" to get the user_id while the option --user is specified for "openstack volume list" Change-Id: Iea8472b7b8e709a8792a56575e00003a9cbdaa39 Closes-Bug: #1514145
* Follow-on for volume list - add tests, clean helpDean Troyer2015-10-162-13/+12
| | | | | | | | | | * 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
* Add filtering by project/user for 'openstack volume list'kafka2015-10-151-5/+34
| | | | | | | | | | added project, user and domain options for filtering results, also cleaned up the order to match the docs. Co-Authored-By: Steve Martinelli <stevemar@ca.ibm.com> Closed-bug: #1483976 Change-Id: I9d955094d31d4a28e215d24f7521a11c62bee8db
* Add one parenthesisAtsushi SAKAI2015-10-021-1/+1
| | | | | | | | | | | | | | | | | | | In the following help message, last parenthesis is missing. $ openstack --os-volume-api-version 2 help volume delete usage: openstack volume delete [-h] [--force] <volume> [<volume> ...] Delete volume(s) positional arguments: <volume> Volume(s) to delete (name or ID) optional arguments: -h, --help show this help message and exit --force Attempt forced removal of volume(s), regardless of state (defaults to False Change-Id: I45c4030abf076cba14450019c379d333eb6530d6
* Merge "Use a common decorator to log 'take_action' activation"Jenkins2015-09-119-48/+48
|\
| * Use a common decorator to log 'take_action' activationJoshua Harlow2015-09-019-48/+48
| | | | | | | | | | | | | | | | | | 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
* | Volume v2 list does not show server nameTerryHowe2015-08-281-1/+1
|/ | | | | | | | The volume v2 list was using the volume id rather than the server id. Change-Id: Ibe03d34b5b503af2d00202dabd640f796449cf9a Closes-Bug: #1489954
* default OS_VOLUME_API_VERSION to v2Steve Martinelli2015-08-241-1/+1
| | | | | | | | | Cinder is trying to deprecate/remove support for v1, so we should, as a client library default to v2 and keep support for v1. Related-Bug: 1467589 Change-Id: I732448a57fc3fd06a8d82ec0f0d2ace671036ca2
* Merge "Add set feature to volume type v2"Jenkins2015-08-071-0/+94
|\
| * Add set feature to volume type v2heha2015-08-061-0/+94
| | | | | | | | | | | | | | | | | | | | "volume type set" and "volume type unset" is not in the v2. Co-Authored-By: Lin Hua Cheng <os.lcheng@gmail.com> implements bp: volume-v2 Change-Id: Ia804787d76d2029726c030b43c61eac3b411f66a
* | Add list feature to volume v2heha2015-08-061-0/+109
| | | | | | | | | | | | | | | | | | | | "volume list" is not in the v2. Co-Authored-By: Lin Hua Cheng <os.lcheng@gmail.com> implements bp: volume-v2 Change-Id: I9f4585202f5f9ec5f4c091278fc6c4036efb1290
* | Merge "Add support for volume v2 commands"Jenkins2015-07-251-0/+235
|\ \
| * | Add support for volume v2 commandsAmey Bhide2015-07-241-0/+235
| |/ | | | | | | | | | | | | | | | | | | Adds the following commands: openstack volume create openstack volume set openstack volume unset Implements: blueprint volume-v2 Change-Id: Icb7404815763aa88550112fb42f5200ce05c2486
* | Merge "Do not set default versions in parsed args"Jenkins2015-07-241-5/+3
|\ \ | |/ |/|
| * Do not set default versions in parsed argsTerryHowe2015-07-181-5/+3
| | | | | | | | | | | | | | | | | | | | | | Setting default versions in parsed args makes it so OCC cannot tell if the argument was parsed, an environment variable was set or it is just defaulted. In order to set api versions from OCC, it will have to be defaulted after processing OCC. Closes-Bug: #1453229 Change-Id: I4d065919397b783f3bdd4022c986c0234a7a16e6
* | Merge "Add create and list for volume type v2"Jenkins2015-07-201-0/+97
|\ \
| * | Add create and list for volume type v2chengkunye2015-07-201-0/+97
| | | | | | | | | | | | | | | | | | | | | | | | Volume API v2 is missing create and list features. implements bp: volume-v2 Change-Id: I34a1ae440e9620b1c65546f4f43b369c8661250d
* | | Merge "Format volume type properties when showing"Jenkins2015-07-201-0/+2
|\ \ \ | |/ / |/| |
| * | Format volume type properties when showingSteve Martinelli2015-07-191-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | Currently, the properties of a volume type are unformatted. Use the formatter to keep things consistent with the way properties are represented in OSC. Change-Id: I81c6bd3fdbc30568c269e501c740473b2a1ffb4e
* | | 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-172-76/+74
| | | | | | | Also make sure that we are using the singular qos_spec when we should and the plural qos_specs otherwise. Change-Id: If4bbeb6fc245d7d80dc3d7dccfe9f949e802653c
* Rename endpoint type to interfaceTerryHowe2015-07-151-3/+2
| | | | | Change-Id: I4e21d09bc747e8210f4f79a1d6c4c7ccf2f25d1c Closes-Bug: #1454392
* Merge "Add --os-endpoint-type cli optional argument"Jenkins2015-07-031-0/+5
|\
| * Add --os-endpoint-type cli optional argumentRoxana Gherle2015-07-021-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | User should be able to specify the endpoint type through a CLI optional argument/ENV variable setting. We will name this new optional argument: --os-endpoint-type (Env: OS_ENDPOINT_TYPE) and based on the value given, the service API will use that specific endpoint type. Possible values: public, admin, internal. DocImpact Closes-Bug: #1454392 Change-Id: Ife3d4e46b44c0ddcd712b1130e27e362545a9a29
* | Merge "Add support for volume API v2 QoS commands"Jenkins2015-06-301-0/+303
|\ \
| * | Add support for volume API v2 QoS commandsDavid Moreau Simard2015-06-291-0/+303
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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: If3c679557ac9abb0dfc75d290b96fb9c8d46c7b7 Partial-Bug: #1467967
* | | 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
* | Merge "Add support for volume backup v2 command"Jenkins2015-06-181-0/+132
|\ \
| * | Add support for volume backup v2 commandAmey Bhide2015-06-041-0/+132
| |/ | | | | | | | | | | | | | | | | openstack backup create openstack backup list openstack backup restore Implements: blueprint volume-v2 Change-Id: I77965730065dd44f256c46bcc43c1e6a03b63145
* | Add support for volume snapshot v2 commandAmey Bhide2015-06-041-2/+203
|/ | | | | | | | | | openstack snapshot create openstack snapshot set openstack snapshot unset openstack snapshot list Implements: blueprint volume-v2 Change-Id: Ia1d9f4426baa0099281a9931f4eec99ebe1969b1
* Add support for volume v2 APIAmey Bhide2015-05-306-1/+294
| | | | | | | | | | | | | | | Added following commands for volume V2 API: volume show volume delete volume type show volume type delete snapshot show snapshot delete backup show backup delete Implements: blueprint volume-v2 Change-Id: I68bd303c194f304ad15f899d335b72a8bf3ebe10
* 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
* Defer client importsDean Troyer2015-04-151-9/+11
| | | | | | | | | | | | So we really weren't deferring the loading of client libs dadgummit, do that for real where possible. This shaves a couple of tenths off the static import times. Also defer as much import-time procesing as possible. This is a little ugly in api.auth but this also eliminates import of the auth plugins until they are needed. Change-Id: Ia11d4b9cf98231d37449103fc29101dc17afb009
* 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