diff options
| author |   Tibor Frank <tifrank@cisco.com> | 2016-12-15 10:52:33 +0100 |
|---|---|---|
| committer | Tibor Frank <tifrank@cisco.com> | 2016-12-16 08:35:32 +0100 |
| commit | cf561a6e3d4c4fbd78ab6c9d0a9aa817bb3300fc (patch) | |
| tree | 0b4d5d2dbda78223d77d76e237285396d42c868e /resources/libraries/python/SetupFramework.py | |
| parent | 677ae40a2be6db9f49b19ada3c4299a8904f7784 (diff) | |
Pylint fixes
- Fix PyLint errors
- Fix comments in touched python modules
Change-Id: I26db2d292a41969cf38b9b0bdd49c4fb15349102
Signed-off-by: Tibor Frank <tifrank@cisco.com>
Diffstat (limited to 'resources/libraries/python/SetupFramework.py')
| -rw-r--r-- | resources/libraries/python/SetupFramework.py | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/resources/libraries/python/SetupFramework.py b/resources/libraries/python/SetupFramework.py index 5d3d59ab56..82c4ae2a65 100644 --- a/resources/libraries/python/SetupFramework.py +++ b/resources/libraries/python/SetupFramework.py @@ -71,7 +71,7 @@ def copy_tarball_to_node(tarball, node): :param node: Dictionary created from topology. :type tarball: str :type node: dict - :return: nothing + :returns: nothing """ logger.console('Copying tarball to {0}'.format(node['host'])) ssh = SSH() @@ -89,7 +89,7 @@ def extract_tarball_at_node(tarball, node): :param node: Dictionary created from topology. :type tarball: str :type node: dict - :return: nothing + :returns: nothing """ logger.console('Extracting tarball to {0} on {1}'.format( con.REMOTE_FW_DIR, node['host'])) @@ -133,8 +133,7 @@ def setup_node(args): :param args: All parameters needed to setup one node. :type args: tuple - :return: nothing - :return: True - success, False - error + :returns: True - success, False - error :rtype: bool """ tarball, remote_tarball, node = args @@ -156,12 +155,12 @@ def delete_local_tarball(tarball): :param tarball: Path to tarball to upload. :type tarball: str - :return: nothing + :returns: nothing """ call(split('sh -c "rm {0} > /dev/null 2>&1"'.format(tarball))) -class SetupFramework(object): # pylint: disable=too-few-public-methods +class SetupFramework(object): """Setup suite run on topology nodes. Many VAT/CLI based tests need the scripts at remote hosts before executing @@ -199,4 +198,3 @@ class SetupFramework(object): # pylint: disable=too-few-public-methods logger.trace('Test framework copied to all topology nodes') delete_local_tarball(tarball) logger.console('All nodes are ready') - |