openstack
/
python-troveclient
Code Issues Proposed changes

Merge "Throw correct error on creation of size 0"

This commit is contained in:
Jenkins 2016-10-04 16:25:55 +00:00 committed by Gerrit Code Review
parent fb3b74e580 ca9be53870
commit 4ac8124801
2 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
troveclient/tests/test_v1_shell.py
View File

@ -274,6 +274,10 @@ class ShellTest(utils.TestCase):
self.run_command('cluster-force-delete cls-1234')
self.assert_called('DELETE', '/clusters/cls-1234')
def test_boot_fail_with_size_0(self):
self.assertRaises(exceptions.ValidationError, self.run_command,
'create test-member-1 1 --size 0 --volume_type lvm')
def test_boot(self):
self.run_command('create test-member-1 1 --size 1 --volume_type lvm')
self.assert_called_anytime(

6
troveclient/v1/shell.py
View File

@ -521,7 +521,11 @@ def do_create(cs, args):
"""Creates a new instance."""
flavor_id = _find_flavor(cs, args.flavor).id
volume = None
if args.size:
if args.size is not None and args.size <= 0:
raise exceptions.ValidationError(
"Volume size '%s' must be an integer and greater than 0."
% args.size)
elif args.size:
volume = {"size": args.size,
"type": args.volume_type}
restore_point = None