add comments

Signed-off-by: Guohan Lu <[email protected]>

Author: lguohan

config/main.py

@@ -18,8 +18,9 @@
 from swsssdk import ConfigDBConnector, SonicV2Connector, SonicDBConfig
 from utilities_common.db import Db
 from utilities_common.intf_filter import parse_interface_in_filter
-import utilities_common.cli as clicommon # lgtm [py/import-and-import-from]
-from utilities_common.cli import cfglog as log
+import utilities_common.cli as clicommon
+from .utils import log
+
 
 import aaa
 import mlnx

config/utils.py

@@ -0,0 +1,6 @@
+from sonic_py_common import logger
+
+SYSLOG_IDENTIFIER = "config"
+
+# Global logger instance
+log = logger.Logger(SYSLOG_IDENTIFIER)

config/vlan.py

@@ -1,6 +1,7 @@
 import click
 
 import utilities_common.cli as clicommon
+from .utils import log
 
 #
 # 'vlan' group ('config vlan ...')
@@ -33,7 +34,7 @@ def add_vlan(db, vid):
 def del_vlan(db, vid):
     """Delete VLAN"""
 
-    clicommon.cfglog.log_info("'vlan del {}' executing...".format(vid))
+    log.log_info("'vlan del {}' executing...".format(vid))
 
     ctx = click.get_current_context()
 
@@ -66,7 +67,7 @@ def add_vlan_member(db, vid, port, untagged):
 
     ctx = click.get_current_context()
 
-    clicommon.cfglog.log_info("'vlan member add {} {}' executing...".format(vid, port))
+    log.log_info("'vlan member add {} {}' executing...".format(vid, port))
 
     if not clicommon.is_vlanid_in_range(vid):
         ctx.fail("Invalid VLAN ID {} (1-4094)".format(vid))
@@ -110,7 +111,7 @@ def del_vlan_member(db, vid, port):
 
     ctx = click.get_current_context()
 
-    clicommon.cfglog.log_info("'vlan member del {} {}' executing...".format(vid, port))
+    log.log_info("'vlan member del {} {}' executing...".format(vid, port))
 
     if not clicommon.is_vlanid_in_range(vid):
         ctx.fail("Invalid VLAN ID {} (1-4094)".format(vid))

utilities_common/cli.py

@@ -7,17 +7,12 @@
 
 from utilities_common.db import Db
 
-from sonic_py_common import logger
 from swsssdk import ConfigDBConnector
 
-SYSLOG_IDENTIFIER_FOR_CONFIG = "config"
 VLAN_SUB_INTERFACE_SEPARATOR = '.'
 
 pass_db = click.make_pass_decorator(Db, ensure=True)
 
-# Global logger instance
-cfglog = logger.Logger(SYSLOG_IDENTIFIER_FOR_CONFIG)
-
 class AbbreviationGroup(click.Group):
     """This subclass of click.Group supports abbreviated subgroup/subcommand names
     """