diff options
| author |   Marcel Enguehard <mengueha+fdio@cisco.com> | 2017-04-25 13:37:27 +0200 |
|---|---|---|
| committer | Marcel Enguehard <mengueha+fdio@cisco.com> | 2017-04-25 13:37:27 +0200 |
| commit | a42b1e0f2ef7ca75bfece00a0ea597b95fae88c1 (patch) | |
| tree | a170b045f9320d12f9365765a7a13ba2fca56f4a /vicn/resource | |
| parent | ba05bc2b35dd94577e4279fb875d576874b88086 (diff) | |
Typo for VPP interfaces + More explicit exceptions for BashTask + Forgotten if-statement in LxcContainer setup
Change-Id: I7b450b13e632899bef5ff37168a9454fe4e92640
Signed-off-by: Marcel Enguehard <mengueha+fdio@cisco.com>
Diffstat (limited to 'vicn/resource')
| -rw-r--r-- | vicn/resource/linux/link.py | 12 | ||||
| -rw-r--r-- | vicn/resource/lxd/lxc_container.py | 6 |
2 files changed, 10 insertions, 8 deletions
diff --git a/vicn/resource/linux/link.py b/vicn/resource/linux/link.py index b3ba4e07..a930b8a3 100644 --- a/vicn/resource/linux/link.py +++ b/vicn/resource/linux/link.py @@ -118,17 +118,17 @@ class Link(Channel): #XXX VPP if hasattr(self.src_node, 'vpp') and not self.src_node.vpp is None: - vpp_src = VPPInterface(parent = self.src, + vpp_src = VPPInterface(parent = self._src, vpp = self.src_node.vpp, - ip_address = Reference(self.src, 'ip_address'), - device_name = 'vpp' + self.src.device_name) + ip_address = Reference(self._src, 'ip_address'), + device_name = 'vpp' + self._src.device_name) manager.commit_resource(vpp_src) if hasattr(self.dst_node, 'vpp') and not self.dst_node.vpp is None: - vpp_dst = VPPInterface(parent = self.dst, + vpp_dst = VPPInterface(parent = self._dst, vpp = self.dst_node.vpp, - ip_address = Reference(self.dst, 'ip_address'), - device_name = 'vpp' + self.dst.device_name) + ip_address = Reference(self._dst, 'ip_address'), + device_name = 'vpp' + self._dst.device_name) manager.commit_resource(vpp_dst) #-------------------------------------------------------------------------- diff --git a/vicn/resource/lxd/lxc_container.py b/vicn/resource/lxd/lxc_container.py index afa64aba..7c678e57 100644 --- a/vicn/resource/lxd/lxc_container.py +++ b/vicn/resource/lxd/lxc_container.py @@ -33,7 +33,7 @@ from vicn.core.commands import ReturnValue from vicn.core.exception import ResourceNotFound from vicn.core.requirement import Requirement from vicn.core.resource_mgr import wait_resource_task -from vicn.core.task import task, inline_task, BashTask +from vicn.core.task import task, inline_task, BashTask, EmptyTask from vicn.resource.linux.net_device import NetDevice from vicn.resource.node import Node from vicn.resource.vpp.scripts import APPARMOR_VPP_PROFILE @@ -140,7 +140,9 @@ class LxcContainer(Node): """ Make sure vpp_host is instanciated before starting the container. """ - wait_vpp_host = wait_resource_task(self.node.vpp_host) + wait_vpp_host = EmptyTask() + if 'vpp' in self.profiles: + wait_vpp_host = wait_resource_task(self.node.vpp_host) create = self._create_container() start = self.__method_start__() return wait_vpp_host > (create > start) |