secgroup-create description is not optional

The description field to the Nova API for secgroup
create is not optional. This enforces the supplying
of the description field in novaclient as the Nova
call will fail if it is not supplied.

v3 version does not need to be updated as it was never
changed.

Change-Id: I6fc823990190b935a8e1214efc8c3ac3a0941abf
Closes-Bug: 1367121
This commit is contained in:
Chris Yeoh 2014-09-16 11:25:18 +09:30
parent 915e9161a6
commit bd65fde64c
2 changed files with 1 additions and 8 deletions

View File

@ -1901,13 +1901,6 @@ class ShellTest(utils.TestCase):
{'name': 'test',
'description': 'FAKE_SECURITY_GROUP'}})
def test_security_group_create_without_description(self):
self.run_command('secgroup-create test')
self.assert_called('POST', '/os-security-groups',
{'security_group':
{'name': 'test',
'description': None}})
def test_security_group_update(self):
self.run_command('secgroup-update test te FAKE_SECURITY_GROUP')
self.assert_called('PUT', '/os-security-groups/1',

View File

@ -2399,7 +2399,7 @@ def do_secgroup_delete_rule(cs, args):
@utils.arg('name', metavar='<name>', help=_('Name of security group.'))
@utils.arg('description', metavar='<description>', nargs='?',
@utils.arg('description', metavar='<description>',
help=_('Description of security group.'))
def do_secgroup_create(cs, args):
"""Create a security group."""