Merge "Add provider attribute to lb-pool-create command"

Author: Jenkins

neutronclient/neutron/v2_0/lb/pool.py

@@ -22,13 +22,18 @@
 from neutronclient.neutron import v2_0 as neutronV20
 
 
+def _format_provider(pool):
+    return pool.get('provider') or 'N/A'
+
+
 class ListPool(neutronV20.ListCommand):
     """List pools that belong to a given tenant."""
 
     resource = 'pool'
     log = logging.getLogger(__name__ + '.ListPool')
-    list_columns = ['id', 'name', 'lb_method', 'protocol',
+    list_columns = ['id', 'name', 'provider', 'lb_method', 'protocol',
                     'admin_state_up', 'status']
+    _formatters = {'provider': _format_provider}
     pagination_support = True
     sorting_support = True
 
@@ -73,6 +78,9 @@ def add_known_arguments(self, parser):
             '--subnet-id', metavar='SUBNET',
             required=True,
             help='the subnet on which the members of the pool will be located')
+        parser.add_argument(
+            '--provider',
+            help='provider name of loadbalancer service')
 
     def args2body(self, parsed_args):
         _subnet_id = neutronV20.find_resourceid_by_name_or_id(
@@ -85,7 +93,7 @@ def args2body(self, parsed_args):
         }
         neutronV20.update_dict(parsed_args, body[self.resource],
                                ['description', 'lb_method', 'name',
-                                'protocol', 'tenant_id'])
+                                'protocol', 'tenant_id', 'provider'])
         return body
 
 

neutronclient/tests/unit/lb/test_cli20_pool.py

@@ -60,17 +60,19 @@ def test_create_pool_with_all_params(self):
         subnet_id = 'subnet-id'
         tenant_id = 'my-tenant'
         my_id = 'my-id'
+        provider = 'lbaas'
         args = ['--admin-state-down',
                 '--description', description,
                 '--lb-method', lb_method,
                 '--name', name,
                 '--protocol', protocol,
                 '--subnet-id', subnet_id,
-                '--tenant-id', tenant_id]
+                '--tenant-id', tenant_id,
+                '--provider', provider]
         position_names = ['admin_state_up', 'description', 'lb_method', 'name',
-                          'protocol', 'subnet_id', 'tenant_id']
+                          'protocol', 'subnet_id', 'tenant_id', 'provider']
         position_values = [False, description, lb_method, name,
-                           protocol, subnet_id, tenant_id]
+                           protocol, subnet_id, tenant_id, provider]
         self._test_create_resource(resource, cmd, name, my_id, args,
                                    position_names, position_values)