From 2d4a7371601290abca1878f07b952dc8d9deff02 Mon Sep 17 00:00:00 2001
From: TerryHowe <terrylhowe@gmail.com>
Date: Wed, 8 Jul 2015 11:21:41 -0600
Subject: Remove the --dhcp option to network list

The --dhcp option lists agents, not networks.  This does not make
a lot of sense.  Another command should be created to list agents.

BackwardsIncompatibleImpact
Closes-Bug: #1472613

Change-Id: I5ecfe3fc046a07eb64a4dabd41dbd99de7c2215f
---
 openstackclient/network/v2/network.py | 64 +++++++++++++++--------------------
 1 file changed, 27 insertions(+), 37 deletions(-)

(limited to 'openstackclient/network')

diff --git a/openstackclient/network/v2/network.py b/openstackclient/network/v2/network.py
index 1fa05462..336b3086 100644
--- a/openstackclient/network/v2/network.py
+++ b/openstackclient/network/v2/network.py
@@ -156,10 +156,6 @@ class ListNetwork(lister.Lister):
             default=False,
             help='List external networks',
         )
-        parser.add_argument(
-            '--dhcp',
-            metavar='<dhcp-id>',
-            help='DHCP agent ID')
         parser.add_argument(
             '--long',
             action='store_true',
@@ -172,40 +168,34 @@ class ListNetwork(lister.Lister):
         self.log.debug('take_action(%s)' % parsed_args)
         client = self.app.client_manager.network
 
-        if parsed_args.dhcp:
-            data = client.api.dhcp_agent_list(dhcp_id=parsed_args.dhcp)
-
-            columns = ('ID',)
-            column_headers = columns
+        data = client.api.network_list(external=parsed_args.external)
+
+        if parsed_args.long:
+            columns = (
+                'ID',
+                'Name',
+                'Status',
+                'project_id',
+                'state',
+                'Shared',
+                'Subnets',
+                'provider:network_type',
+                'router_type',
+            )
+            column_headers = (
+                'ID',
+                'Name',
+                'Status',
+                'Project',
+                'State',
+                'Shared',
+                'Subnets',
+                'Network Type',
+                'Router Type',
+            )
         else:
-            data = client.api.network_list(external=parsed_args.external)
-
-            if parsed_args.long:
-                columns = (
-                    'ID',
-                    'Name',
-                    'Status',
-                    'project_id',
-                    'state',
-                    'Shared',
-                    'Subnets',
-                    'provider:network_type',
-                    'router_type',
-                )
-                column_headers = (
-                    'ID',
-                    'Name',
-                    'Status',
-                    'Project',
-                    'State',
-                    'Shared',
-                    'Subnets',
-                    'Network Type',
-                    'Router Type',
-                )
-            else:
-                columns = ('ID', 'Name', 'Subnets')
-                column_headers = columns
+            columns = ('ID', 'Name', 'Subnets')
+            column_headers = columns
 
         for d in data:
             d = _prep_network_detail(d)
-- 
cgit v1.2.1