You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by gi...@apache.org on 2013/11/15 06:54:05 UTC

git commit: updated refs/heads/4.3 to 0a02c7e

Updated Branches:
  refs/heads/4.3 3c4d4cc03 -> 0a02c7e14


CLOUDSTACK-5168: Fixed test script issue related to SSH
 command


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

Branch: refs/heads/4.3
Commit: 0a02c7e1474d12fb494e6938d5e67018790c2bd2
Parents: 3c4d4cc
Author: Ashutosh K <as...@clogeny.com>
Authored: Fri Nov 15 11:20:44 2013 +0530
Committer: Girish Shilamkar <gi...@clogeny.com>
Committed: Fri Nov 15 11:23:26 2013 +0530

----------------------------------------------------------------------
 test/integration/component/test_egress_rules.py | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/0a02c7e1/test/integration/component/test_egress_rules.py
----------------------------------------------------------------------
diff --git a/test/integration/component/test_egress_rules.py b/test/integration/component/test_egress_rules.py
index 983b7ce..eb38c1a 100644
--- a/test/integration/component/test_egress_rules.py
+++ b/test/integration/component/test_egress_rules.py
@@ -907,7 +907,7 @@ class TestRevokeEgressRule(cloudstackTestCase):
         #    CIDR: 0.0.0.0/0
         # 5. deployVirtualMachine into this security group (ssh)
         # 6. deployed VM should be Running, ssh should be allowed into the VM,
-        #    ping out to google.com from the VM should fail,
+        #    ping out to google.com from the VM should be successful,
         #    ssh from within VM to mgt server should pass
         # 7. Revoke egress rule. Verify ping and SSH access to management server
         #    is restored
@@ -1005,7 +1005,7 @@ class TestRevokeEgressRule(cloudstackTestCase):
 
         result = str(res)
         self.assertEqual(
-                         result.count("0 received"),
+                         result.count("1 received"),
                          1,
                          "Ping to outside world from VM should be successful"
                          )
@@ -1014,7 +1014,7 @@ class TestRevokeEgressRule(cloudstackTestCase):
             self.debug("SSHing into management server from VM")
             res = ssh.execute("ssh %s@%s" % (
                                     self.services["mgmt_server"]["username"],
-                                    self.services["mgmt_server"]["ipaddress"]
+                                    self.apiclient.connection.mgtSvr
                                  ))
             self.debug("SSH result: %s" % str(res))
 
@@ -1062,16 +1062,16 @@ class TestRevokeEgressRule(cloudstackTestCase):
 
         result = str(res)
         self.assertEqual(
-                         result.count("1 received"),
+                         result.count("0 received"),
                          1,
-                         "Ping to outside world from VM should be successful"
+                         "Ping to outside world from VM should fail"
                          )
 
         try:
             self.debug("SSHing into management server from VM")
             res = ssh.execute("ssh %s@%s" % (
                                     self.services["mgmt_server"]["username"],
-                                    self.services["mgmt_server"]["ipaddress"]
+                                    self.apiclient.connection.mgtSvr
                                  ))
             self.debug("SSH result: %s" % str(res))