Skip to content

Commit a739265

Browse files
Jenkinsopenstack-gerrit
authored andcommitted
Merge "Fix subnet pool prefix length option"
2 parents e666a8a + 9174bc0 commit a739265

2 files changed

Lines changed: 9 additions & 9 deletions

File tree

openstackclient/network/v2/subnet_pool.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,11 +39,11 @@ def _get_attrs(parsed_args):
3939
if parsed_args.prefixes is not None:
4040
attrs['prefixes'] = parsed_args.prefixes
4141
if parsed_args.default_prefix_length is not None:
42-
attrs['default_prefix_length'] = parsed_args.default_prefix_length
42+
attrs['default_prefixlen'] = parsed_args.default_prefix_length
4343
if parsed_args.min_prefix_length is not None:
44-
attrs['min_prefix_length'] = parsed_args.min_prefix_length
44+
attrs['min_prefixlen'] = parsed_args.min_prefix_length
4545
if parsed_args.max_prefix_length is not None:
46-
attrs['max_prefix_length'] = parsed_args.max_prefix_length
46+
attrs['max_prefixlen'] = parsed_args.max_prefix_length
4747

4848
return attrs
4949

openstackclient/tests/network/v2/test_subnet_pool.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -119,9 +119,9 @@ def test_create_prefixlen_options(self):
119119
columns, data = (self.cmd.take_action(parsed_args))
120120

121121
self.network.create_subnet_pool.assert_called_once_with(**{
122-
'default_prefix_length': self._subnet_pool.default_prefixlen,
123-
'max_prefix_length': self._subnet_pool.max_prefixlen,
124-
'min_prefix_length': self._subnet_pool.min_prefixlen,
122+
'default_prefixlen': self._subnet_pool.default_prefixlen,
123+
'max_prefixlen': self._subnet_pool.max_prefixlen,
124+
'min_prefixlen': self._subnet_pool.min_prefixlen,
125125
'name': self._subnet_pool.name,
126126
})
127127
self.assertEqual(self.columns, columns)
@@ -278,8 +278,8 @@ def test_set_this(self):
278278

279279
attrs = {
280280
'name': 'noob',
281-
'default_prefix_length': '8',
282-
'min_prefix_length': '8',
281+
'default_prefixlen': '8',
282+
'min_prefixlen': '8',
283283
}
284284
self.network.update_subnet_pool.assert_called_once_with(
285285
self._subnet_pool, **attrs)
@@ -305,7 +305,7 @@ def test_set_that(self):
305305
prefixes.extend(self._subnet_pool.prefixes)
306306
attrs = {
307307
'prefixes': prefixes,
308-
'max_prefix_length': '16',
308+
'max_prefixlen': '16',
309309
}
310310
self.network.update_subnet_pool.assert_called_once_with(
311311
self._subnet_pool, **attrs)

0 commit comments

Comments
 (0)