diff --git a/contrail_provisioning/common/base.py b/contrail_provisioning/common/base.py index 9958f275..389e126f 100644 --- a/contrail_provisioning/common/base.py +++ b/contrail_provisioning/common/base.py @@ -198,12 +198,12 @@ def setup_sriov_vfs(self): intf_list = sriov_string.split(",") for intf_details in intf_list: info = intf_details.split(":") - #Keep this command consistent with provision.py in fabric utils + # Keep this command consistent with provision.py in fabric utils str = 'echo %s > /sys/class/net/%s/device/sriov_numvfs; sleep 2; ifup -a' % (info[1], info[0]) # Do nothing if the entry already present in /etc/rc.local - if sudo('grep -w \'%s\' /etc/rc.local' % str, - quiet=True).succeeded: - continue + with settings(warn_only = True): + if local('grep -w \'%s\' /etc/rc.local' % str).succeeded: + continue sed = 'sudo sed -i \'/^\s*exit/i ' + str + '\' /etc/rc.local' with settings(warn_only = True): diff --git a/contrail_provisioning/common/interface_setup.py b/contrail_provisioning/common/interface_setup.py index 232896c3..e0e3724c 100755 --- a/contrail_provisioning/common/interface_setup.py +++ b/contrail_provisioning/common/interface_setup.py @@ -465,7 +465,7 @@ def _get_vf_index(self, dev): return '' def _cfg_append_spoof_vlan(self, dev, cfg): - '''Append a line to the config to turn off spoof check Also add VLAN 0 + '''Append a line to the config to turn off spoof check Also add VLAN 0 to the given VF as ixgbe seems to require it. ''' vfi = self._get_vf_index(dev)