Changeset 204

Timestamp:

08/03/12 00:12:23 (12 years ago)

Author:

atzm

Message:

• ctl interface minor change

File:

• etherws/trunk/etherws.py (modified) (2 diffs)

etherws/trunk/etherws.py

@@ -817 +817 @@
     def handle_ctl_addport(args):
         result = request(args, 'addPort', {
-            'type':    args.type,
+            'type':    args.iftype,
             'target':  args.target,
             'options': {
-                'insecure': args.insecure,
-                'cacerts':  args.cacerts,
-                'user':     args.user,
-                'passwd':   args.passwd,
+                'user':     getattr(args, 'user', None),
+                'passwd':   getattr(args, 'passwd', None),
+                'cacerts':  getattr(args, 'cacerts', None),
+                'insecure': getattr(args, 'insecure', None),
             },
         })
@@ -880 +880 @@
     subcommand = parser.add_subparsers(dest='subcommand')
 
-    # -- sw command parser
-    parser_s = subcommand.add_parser('sw')
-
-    parser_s.add_argument('--debug', action='store_true', default=False)
-    parser_s.add_argument('--foreground', action='store_true', default=False)
-    parser_s.add_argument('--ageout', type=int, default=300)
-
-    parser_s.add_argument('--path', default='/')
-    parser_s.add_argument('--host', default='')
-    parser_s.add_argument('--port', type=int)
-    parser_s.add_argument('--htpasswd')
-    parser_s.add_argument('--sslkey')
-    parser_s.add_argument('--sslcert')
-
-    parser_s.add_argument('--ctlpath', default='/ctl')
-    parser_s.add_argument('--ctlhost', default='')
-    parser_s.add_argument('--ctlport', type=int)
-    parser_s.add_argument('--ctlhtpasswd')
-    parser_s.add_argument('--ctlsslkey')
-    parser_s.add_argument('--ctlsslcert')
-
-    # -- ctl command parser
-    parser_c = subcommand.add_parser('ctl')
-    parser_c.add_argument('--ctlurl', default='http://localhost/ctl')
-    parser_c.add_argument('--ctluser')
-    parser_c.add_argument('--ctlpasswd')
-
-    control_method = parser_c.add_subparsers(dest='control_method')
-
-    parser_c_ap = control_method.add_parser('addport')
-    parser_c_ap.add_argument(
-        'type', choices=EtherWebSocketControlHandler.IFTYPES.keys())
-    parser_c_ap.add_argument('target')
-    parser_c_ap.add_argument('--insecure', action='store_true', default=False)
-    parser_c_ap.add_argument('--cacerts')
-    parser_c_ap.add_argument('--user')
-    parser_c_ap.add_argument('--passwd')
-
-    parser_c_sp = control_method.add_parser('shutport')
-    parser_c_sp.add_argument('port', type=int)
-    parser_c_sp.add_argument('--no', action='store_false', default=True)
-
-    parser_c_dp = control_method.add_parser('delport')
-    parser_c_dp.add_argument('port', type=int)
-
-    parser_c_lp = control_method.add_parser('listport')
-
-    parser_c_lf = control_method.add_parser('listfdb')
+    # - sw
+    parser_sw = subcommand.add_parser('sw')
+
+    parser_sw.add_argument('--debug', action='store_true', default=False)
+    parser_sw.add_argument('--foreground', action='store_true', default=False)
+    parser_sw.add_argument('--ageout', type=int, default=300)
+
+    parser_sw.add_argument('--path', default='/')
+    parser_sw.add_argument('--host', default='')
+    parser_sw.add_argument('--port', type=int)
+    parser_sw.add_argument('--htpasswd')
+    parser_sw.add_argument('--sslkey')
+    parser_sw.add_argument('--sslcert')
+
+    parser_sw.add_argument('--ctlpath', default='/ctl')
+    parser_sw.add_argument('--ctlhost', default='')
+    parser_sw.add_argument('--ctlport', type=int)
+    parser_sw.add_argument('--ctlhtpasswd')
+    parser_sw.add_argument('--ctlsslkey')
+    parser_sw.add_argument('--ctlsslcert')
+
+    # - ctl
+    parser_ctl = subcommand.add_parser('ctl')
+    parser_ctl.add_argument('--ctlurl', default='http://localhost/ctl')
+    parser_ctl.add_argument('--ctluser')
+    parser_ctl.add_argument('--ctlpasswd')
+
+    control_method = parser_ctl.add_subparsers(dest='control_method')
+
+    # -- ctl addport
+    parser_ctl_addport = control_method.add_parser('addport')
+    iftype = parser_ctl_addport.add_subparsers(dest='iftype')
+
+    # --- ctl addport tap
+    parser_ctl_addport_tap = iftype.add_parser(TapHandler.IFTYPE)
+    parser_ctl_addport_tap.add_argument('target')
+
+    # --- ctl addport client
+    parser_ctl_addport_client = iftype.add_parser(EtherWebSocketClient.IFTYPE)
+    parser_ctl_addport_client.add_argument('target')
+    parser_ctl_addport_client.add_argument('--user')
+    parser_ctl_addport_client.add_argument('--passwd')
+    parser_ctl_addport_client.add_argument('--cacerts')
+    parser_ctl_addport_client.add_argument(
+        '--insecure', action='store_true', default=False)
+
+    # -- ctl shutport
+    parser_ctl_shutport = control_method.add_parser('shutport')
+    parser_ctl_shutport.add_argument('port', type=int)
+    parser_ctl_shutport.add_argument(
+        '--no', action='store_false', default=True)
+
+    # -- ctl delport
+    parser_ctl_delport = control_method.add_parser('delport')
+    parser_ctl_delport.add_argument('port', type=int)
+
+    # -- ctl listport
+    parser_ctl_listport = control_method.add_parser('listport')
+
+    # -- ctl listfdb
+    parser_ctl_listfdb = control_method.add_parser('listfdb')
 
     # -- go