diff options
| author | reedip <reedip.banerjee@nectechnologies.in> | 2016-03-31 17:33:32 +0900 |
|---|---|---|
| committer | Reedip <reedip.banerjee@nectechnologies.in> | 2016-04-01 06:51:07 +0000 |
| commit | 7027d915b5d00e719ec2035b34ea9a3702b2f3fc (patch) | |
| tree | a336c1ee49b9c2ff013621a984b4a82e0a2da261 /openstackclient | |
| parent | a1a470693e264e991b1dc9497512769bb8d510d8 (diff) | |
| download | python-openstackclient-7027d915b5d00e719ec2035b34ea9a3702b2f3fc.tar.gz | |
Add default value to pool-prefix in Subnet-pool
If user tries to create a subnet pool without --pool-prefix,
it fails at the NeutronServer [1].
This patch tries to add a default value to the --pool-prefix
to try and resolve it.
Closes-Bug: #1564271
[1] : http://paste.openstack.org/show/492537/
Change-Id: I6cf324a5a8037048602e59c0bbfc93b40e73a74e
Diffstat (limited to 'openstackclient')
| -rw-r--r-- | openstackclient/network/v2/subnet_pool.py | 3 | ||||
| -rw-r--r-- | openstackclient/tests/network/v2/test_subnet_pool.py | 1 |
2 files changed, 4 insertions, 0 deletions
diff --git a/openstackclient/network/v2/subnet_pool.py b/openstackclient/network/v2/subnet_pool.py index 8c90b955..6b6fc090 100644 --- a/openstackclient/network/v2/subnet_pool.py +++ b/openstackclient/network/v2/subnet_pool.py @@ -111,6 +111,9 @@ class CreateSubnetPool(command.ShowOne): def take_action(self, parsed_args): client = self.app.client_manager.network attrs = _get_attrs(self.app.client_manager, parsed_args) + # NeutronServer expects prefixes to be a List + if "prefixes" not in attrs: + attrs['prefixes'] = [] obj = client.create_subnet_pool(**attrs) columns = _get_columns(obj) data = utils.get_item_properties(obj, columns, formatters=_formatters) diff --git a/openstackclient/tests/network/v2/test_subnet_pool.py b/openstackclient/tests/network/v2/test_subnet_pool.py index c79b9179..093e26c6 100644 --- a/openstackclient/tests/network/v2/test_subnet_pool.py +++ b/openstackclient/tests/network/v2/test_subnet_pool.py @@ -149,6 +149,7 @@ class TestCreateSubnetPool(TestSubnetPool): 'default_prefixlen': self._subnet_pool.default_prefixlen, 'max_prefixlen': self._subnet_pool.max_prefixlen, 'min_prefixlen': self._subnet_pool.min_prefixlen, + 'prefixes': [], 'name': self._subnet_pool.name, }) self.assertEqual(self.columns, columns) |
