From af7129cda33dcd4ac784097ddb4f119e145c40d5 Mon Sep 17 00:00:00 2001
From: guiyanxing <guiyanxing@cmss.chinamobile.com>
Date: Thu, 8 Dec 2016 15:17:53 +0800
Subject: Add '--type'and other options to network rbac list

This patch adds '--type','--action','--long' filtering options
to network rbac list command

Change-Id: I21846820ab223bb7832e89eb2d7658bbec271aec
Closes-Bug: #1648307
Partially-Implements: blueprint network-commands-options
---
 openstackclient/network/v2/network_rbac.py | 36 +++++++++++++++++++++++++++++-
 1 file changed, 35 insertions(+), 1 deletion(-)

(limited to 'openstackclient/network')

diff --git a/openstackclient/network/v2/network_rbac.py b/openstackclient/network/v2/network_rbac.py
index e837af3a..90754737 100644
--- a/openstackclient/network/v2/network_rbac.py
+++ b/openstackclient/network/v2/network_rbac.py
@@ -164,6 +164,30 @@ class DeleteNetworkRBAC(command.Command):
 class ListNetworkRBAC(command.Lister):
     _description = _("List network RBAC policies")
 
+    def get_parser(self, prog_name):
+        parser = super(ListNetworkRBAC, self).get_parser(prog_name)
+        parser.add_argument(
+            '--type',
+            metavar='<type>',
+            choices=['qos_policy', 'network'],
+            help=_('List network RBAC policies according to '
+                   'given object type ("qos_policy" or "network")')
+        )
+        parser.add_argument(
+            '--action',
+            metavar='<action>',
+            choices=['access_as_external', 'access_as_shared'],
+            help=_('List network RBAC policies according to given '
+                   'action ("access_as_external" or "access_as_shared")')
+        )
+        parser.add_argument(
+            '--long',
+            action='store_true',
+            default=False,
+            help=_("List additional fields in output")
+        )
+        return parser
+
     def take_action(self, parsed_args):
         client = self.app.client_manager.network
 
@@ -178,7 +202,17 @@ class ListNetworkRBAC(command.Lister):
             'Object ID',
         )
 
-        data = client.rbac_policies()
+        query = {}
+        if parsed_args.long:
+            columns += ('action',)
+            column_headers += ('Action',)
+        if parsed_args.type is not None:
+            query['object_type'] = parsed_args.type
+        if parsed_args.action is not None:
+            query['action'] = parsed_args.action
+
+        data = client.rbac_policies(**query)
+
         return (column_headers,
                 (utils.get_item_properties(
                     s, columns,
-- 
cgit v1.2.1