FIX: Pylint + Container mount 39/22039/2
authorPeter Mikus <pmikus@cisco.com>
Fri, 13 Sep 2019 08:13:09 +0000 (08:13 +0000)
committerPeter Mikus <pmikus@cisco.com>
Fri, 13 Sep 2019 12:15:34 +0000 (12:15 +0000)
- Mount is affecting only LXC container

Signed-off-by: Peter Mikus <pmikus@cisco.com>
Change-Id: I84140ea5e48120792fc84182a97eb7d06c954a84

resources/libraries/python/ContainerUtils.py
resources/libraries/python/VPPUtil.py
resources/libraries/python/VppCounters.py

index cd48fc6..6fea804 100644 (file)
@@ -635,13 +635,16 @@ class LXC(ContainerEngine):
         if self.container.mnt:
             for mount in self.container.mnt:
                 host_dir, guest_dir = mount.split(':')
+                if host_dir.endswith('/'):
+                    self.container.ssh.exec_command_sudo(
+                        "sh -c 'mkdir -p {host_dir}'".format(host_dir=host_dir))
                 options = 'bind,create=dir' \
                     if guest_dir.endswith('/') else 'bind,create=file'
                 entry = 'lxc.mount.entry = {host_dir} '\
-                    '/var/lib/lxc/{c.name}/rootfs{guest_dir} none ' \
+                    '{guest_dir} none ' \
                     '{options} 0 0'.format(c=self.container,
                                            host_dir=host_dir,
-                                           guest_dir=guest_dir,
+                                           guest_dir=guest_dir[1:],
                                            options=options)
                 ret, _, _ = self.container.ssh.exec_command_sudo(
                     "sh -c 'echo \"{e}\" >> /var/lib/lxc/{c.name}/config'".
index 8f9b593..4e2b237 100644 (file)
@@ -17,7 +17,6 @@ from robot.api import logger
 
 from resources.libraries.python.Constants import Constants
 from resources.libraries.python.DUTSetup import DUTSetup
-from resources.libraries.python.L2Util import L2Util
 from resources.libraries.python.PapiExecutor import PapiSocketExecutor
 from resources.libraries.python.ssh import exec_cmd_no_error
 from resources.libraries.python.topology import NodeType
index 65d6477..c115efc 100644 (file)
@@ -71,7 +71,7 @@ class VppCounters(object):
                     return
 
                 runtime = []
-                runtime_non_zero = []
+                runtime_nz = []
 
                 for name in names:
                     runtime.append({'name': name})
@@ -100,18 +100,20 @@ class VppCounters(object):
 
                     if (sum(calls_th) or sum(vectors_th) or
                             sum(suspends_th) or sum(clocks_th)):
-                        runtime_non_zero.append(runtime_item)
+                        runtime_nz.append(runtime_item)
 
                 if log_zeros:
                     logger.info(
-                        "stats runtime ({host} - {socket}):\n{runtime}".format(
-                        host=node['host'], runtime=pformat(runtime),
-                        socket=socket))
+                        "stats runtime ({host} - {socket}):\n{runtime}".
+                        format(
+                            host=node['host'], runtime=pformat(runtime),
+                            socket=socket))
                 else:
                     logger.info(
-                        "stats runtime ({host} - {socket}):\n{runtime}".format(
-                        host=node['host'], runtime=pformat(runtime_non_zero),
-                        socket=socket))
+                        "stats runtime ({host} - {socket}):\n{runtime}".
+                        format(
+                            host=node['host'], runtime=pformat(runtime_nz),
+                            socket=socket))
 
     @staticmethod
     def vpp_show_runtime_counters_on_all_duts(nodes):