diff options
author | A. Wilcox <AWilcox@Wilcox-Tech.com> | 2020-12-08 22:26:13 -0600 |
---|---|---|
committer | A. Wilcox <AWilcox@Wilcox-Tech.com> | 2020-12-08 22:26:13 -0600 |
commit | 1380813fea63570f518f04e472c596877ed85c60 (patch) | |
tree | faa5710c60becdbbc743e4f8a76989951686890b | |
parent | d04bccdffd414dea6fe5247f464fd12e71470cc7 (diff) | |
download | netconfapk-1380813fea63570f518f04e472c596877ed85c60.tar.gz netconfapk-1380813fea63570f518f04e472c596877ed85c60.tar.bz2 netconfapk-1380813fea63570f518f04e472c596877ed85c60.tar.xz netconfapk-1380813fea63570f518f04e472c596877ed85c60.zip |
ncserver: Small style fixes
-rw-r--r-- | ncserver/module/nms_ifupdownng.py | 1 | ||||
-rw-r--r-- | ncserver/module/system.py | 5 | ||||
-rw-r--r-- | ncserver/server.py | 8 |
3 files changed, 9 insertions, 5 deletions
diff --git a/ncserver/module/nms_ifupdownng.py b/ncserver/module/nms_ifupdownng.py index 0c03fd9..1142f31 100644 --- a/ncserver/module/nms_ifupdownng.py +++ b/ncserver/module/nms_ifupdownng.py @@ -587,6 +587,7 @@ def rollback(): @contextmanager def transaction(): + """Context manager for NMSA transactions.""" begin_transaction() try: yield None diff --git a/ncserver/module/system.py b/ncserver/module/system.py index b40d3f0..ef14641 100644 --- a/ncserver/module/system.py +++ b/ncserver/module/system.py @@ -22,7 +22,8 @@ from socket import gethostname, sethostname from lxml.etree import QName # pylint: disable=E0611 from netconf import error, util -from ncserver.base.log import HOSTNAME_CHANGED, log_config_change, log_operation +from ncserver.base.log import HOSTNAME_CHANGED, log_config_change, \ + log_operation from ncserver.base.util import _, ensure_leaf, handle_list_operation, \ node_operation, yang_dt_for_timestamp from ncserver.util import maybe_raise_on_invalid_node, system_boot_time @@ -134,7 +135,7 @@ class NTPServer: # pylint: disable=R0902 host=self.host, nondef_args=nondef_args ) - def __str__(self): + def __str__(self): # pylint: disable=R0912 """Return the configuration line for this server.""" line = self.mode + ' ' diff --git a/ncserver/server.py b/ncserver/server.py index bbccec7..120e185 100644 --- a/ncserver/server.py +++ b/ncserver/server.py @@ -32,6 +32,7 @@ QName = etree.QName # pylint: disable=I1101 def log_read(session, rpc): """Log a configuration read.""" + # pylint: disable=I1101 LOG_CONFIG.debug( '%s on session %d is requesting configuration: %s', user_for_session(session), session.session_id, etree.tostring(rpc) @@ -40,6 +41,7 @@ def log_read(session, rpc): def log_write(session, rpc): """Log a configuration write attempt.""" + # pylint: disable=I1101 LOG_CONFIG.debug( '%s on session %d is editing configuration: %s', user_for_session(session), session.session_id, etree.tostring(rpc) @@ -117,7 +119,7 @@ class Server: """Close all connections.""" self.server.close() - def nc_append_capabilities(self, capabilities): + def nc_append_capabilities(self, capabilities): # pylint: disable=R0201 """List all capabilities of this NETCONF server.""" our_capabs = [ # Write directly to the `running` datastore. @@ -130,7 +132,7 @@ class Server: for capab in our_capabs: util.subelm(capabilities, 'capability').text = capab - def _rpc_wrapper(self, session, rpc, *params): + def _rpc_wrapper(self, session, rpc, *params): # pylint: disable=R0201 """Handle module RPCs.""" name = QName(rpc[0].tag).localname LOG_OPERATIONAL.info( @@ -176,7 +178,7 @@ class Server: return util.filter_results(rpc, root, filter_or_none, self.debug) - # pylint: disable=R0913 + # pylint: disable=R0913,R0201 def rpc_edit_config(self, session, rpc, target, config, default_or_none): """Handle the <edit-config/> RPC.""" if len(target) != 1: |