diff options
| author | Huanxuan Ao <huanxuan.ao@easystack.cn> | 2016-09-25 15:44:03 +0800 |
|---|---|---|
| committer | Huanxuan Ao <huanxuan.ao@easystack.cn> | 2016-09-26 02:42:47 +0000 |
| commit | 3ebc7520a6ecdf6260757daf7ad913a1bff4e342 (patch) | |
| tree | 8166244d969003e1ab483bbbbc164cdf86cafbb5 /openstackclient | |
| parent | 559215d59d40a35c3a741af99116878c6024078f (diff) | |
| download | python-openstackclient-3ebc7520a6ecdf6260757daf7ad913a1bff4e342.tar.gz | |
Add warning message for --state option of set command in volume
There are some set commands can set object state, it maybe
a danger behavor for users, so add explanation and warning
in the help message of the "--state" option to talk users
be caution when using (cinderclient have done this too)
Change-Id: I6a902887ea98879999c9972f36b1b7ef332173c3
Diffstat (limited to 'openstackclient')
| -rw-r--r-- | openstackclient/volume/v2/backup.py | 5 | ||||
| -rw-r--r-- | openstackclient/volume/v2/snapshot.py | 7 | ||||
| -rw-r--r-- | openstackclient/volume/v2/volume.py | 5 |
3 files changed, 13 insertions, 4 deletions
diff --git a/openstackclient/volume/v2/backup.py b/openstackclient/volume/v2/backup.py index 4a133d36..2ca35b24 100644 --- a/openstackclient/volume/v2/backup.py +++ b/openstackclient/volume/v2/backup.py @@ -305,7 +305,10 @@ class SetVolumeBackup(command.Command): '--state', metavar='<state>', choices=['available', 'error'], - help=_('New backup state ("available" or "error") (admin only)'), + help=_('New backup state ("available" or "error") (admin only) ' + '(This option simply changes the state of the backup ' + 'in the database with no regard to actual status, ' + 'exercise caution when using)'), ) return parser diff --git a/openstackclient/volume/v2/snapshot.py b/openstackclient/volume/v2/snapshot.py index 0d826551..4994e0da 100644 --- a/openstackclient/volume/v2/snapshot.py +++ b/openstackclient/volume/v2/snapshot.py @@ -230,8 +230,11 @@ class SetSnapshot(command.Command): metavar='<state>', choices=['available', 'error', 'creating', 'deleting', 'error-deleting'], - help=_('New snapshot state. Valid values are available, ' - 'error, creating, deleting, and error-deleting.'), + help=_('New snapshot state. ("available", "error", "creating", ' + '"deleting", or "error_deleting") (admin only) ' + '(This option simply changes the state of the snapshot ' + 'in the database with no regard to actual status, ' + 'exercise caution when using)'), ) return parser diff --git a/openstackclient/volume/v2/volume.py b/openstackclient/volume/v2/volume.py index 0805b2be..b0067189 100644 --- a/openstackclient/volume/v2/volume.py +++ b/openstackclient/volume/v2/volume.py @@ -402,7 +402,10 @@ class SetVolume(command.Command): 'maintenance'], help=_('New volume state ("available", "error", "creating", ' '"deleting", "in-use", "attaching", "detaching", ' - '"error_deleting" or "maintenance")'), + '"error_deleting" or "maintenance") (admin only) ' + '(This option simply changes the state of the volume ' + 'in the database with no regard to actual status, ' + 'exercise caution when using)'), ) bootable_group = parser.add_mutually_exclusive_group() bootable_group.add_argument( |