summaryrefslogtreecommitdiff
path: root/openstackclient
diff options
context:
space:
mode:
authorreedip <reedip.banerjee@nectechnologies.in>2016-03-31 17:33:32 +0900
committerReedip <reedip.banerjee@nectechnologies.in>2016-04-01 06:51:07 +0000
commit7027d915b5d00e719ec2035b34ea9a3702b2f3fc (patch)
treea336c1ee49b9c2ff013621a984b4a82e0a2da261 /openstackclient
parenta1a470693e264e991b1dc9497512769bb8d510d8 (diff)
downloadpython-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.py3
-rw-r--r--openstackclient/tests/network/v2/test_subnet_pool.py1
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)