diff options
| author | Tang Chen <chen.tang@easystack.cn> | 2016-06-04 21:29:44 +0800 |
|---|---|---|
| committer | Tang Chen <chen.tang@easystack.cn> | 2016-06-04 21:29:44 +0800 |
| commit | 2672e37ee211bf08a4b6078668bd36a08c9c606a (patch) | |
| tree | e62996ffa79fd3ac584ada99503dcfe52fbb8631 /openstackclient/volume/v2/snapshot.py | |
| parent | 55a7ba890d9c3a53b7728efe41053b79ba1e60d3 (diff) | |
| download | python-openstackclient-2672e37ee211bf08a4b6078668bd36a08c9c606a.tar.gz | |
Make set/unset commands in volume return normally when nothing specified
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
Diffstat (limited to 'openstackclient/volume/v2/snapshot.py')
| -rw-r--r-- | openstackclient/volume/v2/snapshot.py | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/openstackclient/volume/v2/snapshot.py b/openstackclient/volume/v2/snapshot.py index 2886aaef..d9017080 100644 --- a/openstackclient/volume/v2/snapshot.py +++ b/openstackclient/volume/v2/snapshot.py @@ -205,11 +205,6 @@ class SetSnapshot(command.Command): if parsed_args.description: kwargs['description'] = parsed_args.description - if (not kwargs and not parsed_args.property and not - parsed_args.state): - self.app.log.error(_("No changes requested\n")) - return - if parsed_args.property: volume_client.volume_snapshots.set_metadata(snapshot.id, parsed_args.property) @@ -271,5 +266,3 @@ class UnsetSnapshot(command.Command): snapshot.id, parsed_args.property, ) - else: - self.app.log.error(_("No changes requested\n")) |