You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ja...@apache.org on 2015/06/17 11:48:33 UTC

git commit: updated refs/heads/dhcpoffload to b641231

Repository: cloudstack
Updated Branches:
  refs/heads/dhcpoffload [created] b641231ab


Fixed issue in adding vm SG rules on vm reboot for xenserver 6.5


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/b641231a
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/b641231a
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/b641231a

Branch: refs/heads/dhcpoffload
Commit: b641231ab01f13685b8a237abceebf446a532169
Parents: 4727855
Author: Jayapal <ja...@apache.org>
Authored: Wed Jun 17 15:16:54 2015 +0530
Committer: Jayapal <ja...@apache.org>
Committed: Wed Jun 17 15:17:17 2015 +0530

----------------------------------------------------------------------
 scripts/vm/hypervisor/xenserver/vmops | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/b641231a/scripts/vm/hypervisor/xenserver/vmops
----------------------------------------------------------------------
diff --git a/scripts/vm/hypervisor/xenserver/vmops b/scripts/vm/hypervisor/xenserver/vmops
index d195f53..0dfa6da 100755
--- a/scripts/vm/hypervisor/xenserver/vmops
+++ b/scripts/vm/hypervisor/xenserver/vmops
@@ -927,10 +927,9 @@ def network_rules_for_rebooted_vm(session, vmName):
         for cmd in [delcmd, delcmd2, inscmd, inscmd2, inscmd3, inscmd4]:
             cmds = util.pread2(['/bin/bash', '-c', cmd]).split('\n')
             cmds.pop()
-            for c in cmds:
+            for c in filter(None,cmds):
                     ipt = c.split(' ')
                     ipt.insert(0, 'iptables')
-                    ipt.pop()
                     ipts.append(ipt)
 
         for ipt in ipts: