| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
| |
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
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| | |
volume type show is missing for volume V1 API.
Closes-Bug: #1475879
Change-Id: Ic8a0845ecec04146d536412463175f57ef6511ae
|
| |/
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
| |
Also make sure that we are using the singular qos_spec when
we should and the plural qos_specs otherwise.
Change-Id: If4bbeb6fc245d7d80dc3d7dccfe9f949e802653c
|
| |
|
|
|
|
|
| |
There are no arguments so there's no need to define the
get_parser function.
Change-Id: Icfa8accf6dbb7f8d1a0472926403b405da3cc611
|
| |
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
| |
Keeping the option --snapshot-id hidden for backward compatibility.
Change-Id: Iefa0aed9be255a5626a9bbb7db77911ef5ca8595
Closes-Bug: #1418742
|
| |
|
|
|
|
|
| |
Added test for Volume Set as well.
Change-Id: I8e6794f67b160ca328fcafa8f1d67138b4f3becd
Closes-Bug: #1415182
|
| |
|
|
|
|
|
|
| |
Create a --size option for volume set, so the user may extend
a volume's size.
Change-Id: Ic8d3396d788a14ea1e10bf1da73edfd7f5d85070
Closes-Bug: #1413954
|
| |\ |
|
| | |
| |
| |
| | |
Change-Id: Ibe5cd0a8422788762e0c52b702b7bd54e6a46813
|
| |\ \
| |/ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |\ \
| |/ |
|
| | |
| |
| |
| | |
Change-Id: Iecd4dbddea637bd6540d94b37253a9ba434c9db3
|
| |\ \
| |/ |
|
| | |
| |
| |
| |
| |
| |
| |
| | |
Name and Description were not appearing at all, and we didn't have
a --long alternative, which had a bunch of useful information.
Closes-Bug: #1408585
Change-Id: I7ca42a8d23ad60f6b9cc862799cb08a3e491b6e8
|
| | |
| |
| |
| | |
Change-Id: Id1e500d5fb19ffdeb0d1bde9e22c3143c0873d0c
|
| |/
|
|
| |
Change-Id: I7e36daa027639d6a782043d4181c1b328335975a
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
Previously, no data was being returned for the 'attached to' field
when listing volumes. Dig into the the returned array to format
the column.
Change-Id: Iebd79e5ddcb4a335703d9b2675aa7128995de918
Closes-Bug: #1404931
|
| |/
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is part1, add support for these objects:
compute.server
imagev1.image
imagev2.image
network.network
volume.volume
volume.backup
volume.snapshot
Closes-Bug: #1400597
Change-Id: Ice21fee85203a8a55417e0ead8b509b8fd6705c1
|
| |
|
|
|
|
|
| |
'volume create --image' should allow an image name to be used.
Closes-Bug: #1383333
Change-Id: I996d46db321eef2d75c3d19b480319f8a78c09b3
|
| |
|
|
| |
Change-Id: I2b21bc904e35c1cc50da369d148e607fe3e8cf90
|
| |
|
|
|
|
|
|
| |
There are files containing string format arguments inside
logging messages. Using logging function parameters should
be preferred.
Change-Id: Ic749ac9eb55564ed631d57055a5a4dfc3aebd169
|
| |
|
|
|
|
|
| |
This makes it consistent with the other --type options in OSC. Also
add a few more volume_create tests.
Change-Id: I50ef927932cabf157ecdfd6c4faa1914b4fdf413
|
| |
|
|
|
|
|
|
|
|
| |
The volume type create command should properly output the
properties. The code was doing a create on the volume type and
then setting the properties, but it was printing out the volume
object from the create.
Change-Id: I23c8a0182e77bb71903ad87c1b01ba2b62405f3b
Closes-Bug: #1303978
|
| |
|
|
|
|
|
|
|
|
| |
This fixes some errors and inconsistencies I found reviewing the
help strings:
* Capitalize help strings
* Add missing space between words (in multi-line strings)
* Improve wording
Change-Id: I2fb31ab4191c330146e31c1a9651115a6657769a
|
| |
|
|
|
|
|
|
|
| |
According to policy change in HACKING:
http://docs.openstack.org/developer/hacking/#openstack-licensing
empty files should no longer contain copyright notices.
Change-Id: Iba09a00f24dfbd1cd03c1c9f70ea216788e64d93
Closes-Bug: #1262424
|
| |
|
|
|
|
|
|
|
| |
This required https://review.openstack.org/26323 in keystoneclient,
merged long ago...
Also adds some tests for 'volume create'
Change-Id: I55bededbc20b5dcf2833c59eb2b6b069703d8a9a
|
| |
|
|
|
|
|
| |
Mostly options and help strings:
* image, server, project usage, volume
Change-Id: I788b0660f8c2daacde53c20a72dd1afc60cf5159
|
| |
|
|
|
|
|
|
|
|
|
| |
* use six.iteritems()
* replace basestring with six.string_types
* convert print statements to functions (they're all debugging and should
be removed eventually anyway)
* clean up OpenStack copyright: LLC -> Foundation
Change-Id: Icb14212bcb408e63816bfec3922a697bc1a6c946
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| | |
In the unset method in volume_type, it was calling the volume
manager, instead of the volume_type.
Bug: 1203561
Change-Id: Iea1a9214db90f15815a456955040c0c5a795ff3d
|
| |\ \
| |/
|/| |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* Reformat default dict output to key='value' using utils.format_dict()
* Changes utils.get_item_properties() to pass the specific field to
the formatter function rather than the entire resource object, this
allows the formatter to handle multiple attributes.
* Updates server, volume, volume type commands
Change-Id: I90eebf6b84ae200532f09cd925f371598ea54a64
|
| |/
|
|
|
|
|
|
|
|
|
|
|
| |
* Add quota set and quota show commands; these work on both
the compute and volume APIs
* Add the --class variation on the above commands
Note: this replaces the existing volume-only quota commands and eliminates quota list
Blueprint: cinder-client
Bug: 1172064
Change-Id: I766d40e410e48f05e36e17e567a4f01a9411b40e
|
| |
|
|
|
|
|
|
|
|
|
| |
As discussed in https://review.openstack.org/#/c/36352/ for each
command, we were setting api = identity or volume... etc,
this was for an old way of calling commands that are is no longer
used.
Also removed openstackclient/common/command.py
Change-Id: I2705f35d343f2ae729dc22d6aed0b852b2f8ca19
|
| |
|
|
|
| |
Change-Id: Iedccd329ff6fb3155eb29649cd0bc84cfc5ebedf
Implements: blueprint volume-backup
|
| |
|
|
|
|
|
|
|
|
|
| |
Basic cleanups:
* change metadata to property
* add new KeyValueAction to parse the property options
* multiple properties can be set using multiple --property args
* consistent formatting
* do lookups for volume args
Change-Id: Ib6c43f01ad46b395aee8c61e886f42e2a5f5573e
|
| |
|
|
| |
Change-Id: Ifdc4322b699f2bd91a6900e55695acd3d736568e
|
| |
|
|
| |
Change-Id: I454cbe685dc5afa0a09ecc976a90d6eb6bc57d14
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Now able to pass metadata to the create method,
as well as update it with set, and remove it with unset.
I'm currently passing it as an optional param like the following:
--meta-data=key1=value1
which seems weird, what about
--meta-data=key1:value1
I'd have to update type too if that is the case.
Change-Id: I0fc8398768da015c1f53c5c564c68a2cfb53e778
|
| |
|
|
|
|
|
|
|
|
|
| |
changed to volume-type
changed command to --long, added a formatter for key=value
Just noticed there is an option in cinder for a command called
extra-specs-list, only relates to volume-types, to list the
meta-data key/value pair. Added that functionality.
Also made minor changes so delete/set/unset can be called by name.
Change-Id: If534ccd1d8a3abc6d235c60ec9a964b88e3fa66d
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| | |
rebase again, and change util to look for display_name too
minor changes and rebase
add create/delete/list/set/show support for snapshot
Change-Id: I80261653fa919555a44ddda07b0a827ccd16e5e0
|