You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ya...@apache.org on 2013/07/23 22:59:29 UTC

[1/2] git commit: updated refs/heads/master to 1898fa5

Updated Branches:
  refs/heads/master 7faa71dc6 -> 1898fa504


CLOUDSTACK-3697 LB rule clean up already done in cleanup project


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

Branch: refs/heads/master
Commit: c6ea21866e008c467afdcc8b5d505c1c7978e164
Parents: 7faa71d
Author: Sowmya Krishnan <so...@citrix.com>
Authored: Tue Jul 23 21:16:11 2013 +0530
Committer: Sheng Yang <sh...@citrix.com>
Committed: Tue Jul 23 13:59:15 2013 -0700

----------------------------------------------------------------------
 test/integration/component/test_project_resources.py | 1 -
 1 file changed, 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/c6ea2186/test/integration/component/test_project_resources.py
----------------------------------------------------------------------
diff --git a/test/integration/component/test_project_resources.py b/test/integration/component/test_project_resources.py
index 378238e..02de854 100644
--- a/test/integration/component/test_project_resources.py
+++ b/test/integration/component/test_project_resources.py
@@ -1044,7 +1044,6 @@ class TestPublicIpAddress(cloudstackTestCase):
                                           public_ip.ipaddress.id,
                                           projectid=self.project.id
                                           )
-        self.cleanup.append(lb_rule)
         self.debug("Assigning VM: %s to LB rule: %s" % (
                                                     self.virtual_machine.name,
                                                     lb_rule.id


[2/2] git commit: updated refs/heads/master to 1898fa5

Posted by ya...@apache.org.
CLOUDSTACK-3693 Fix more account objects


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

Branch: refs/heads/master
Commit: 1898fa5046b6b73c85d9d0ed9490156d2278ece8
Parents: c6ea218
Author: Sowmya Krishnan <so...@citrix.com>
Authored: Tue Jul 23 22:16:02 2013 +0530
Committer: Sheng Yang <sh...@citrix.com>
Committed: Tue Jul 23 13:59:20 2013 -0700

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


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/1898fa50/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 c8b3eee..10e0d03 100644
--- a/test/integration/component/test_egress_rules.py
+++ b/test/integration/component/test_egress_rules.py
@@ -1264,7 +1264,7 @@ class TestMultipleAccountsEgressRuleNeg(cloudstackTestCase):
                             cls.services["account"],
                             domainid=cls.domain.id
                             )
-        cls.services["account"] = cls.accountA.account.name
+        cls.services["account"] = cls.accountA.name
         cls._cleanup = [
                         cls.accountA,
                         cls.accountB,
@@ -1308,16 +1308,16 @@ class TestMultipleAccountsEgressRuleNeg(cloudstackTestCase):
         security_group = SecurityGroup.create(
                                         self.apiclient,
                                         self.services["security_group"],
-                                        account=self.accountA.account.name,
-                                        domainid=self.accountA.account.domainid
+                                        account=self.accountA.name,
+                                        domainid=self.accountA.domainid
                                         )
         self.debug("Created security group with ID: %s" % security_group.id)
 
         # Default Security group should not have any ingress rule
         sercurity_groups = SecurityGroup.list(
                                         self.apiclient,
-                                        account=self.accountA.account.name,
-                                        domainid=self.accountA.account.domainid
+                                        account=self.accountA.name,
+                                        domainid=self.accountA.domainid
                                         )
         self.assertEqual(
                          isinstance(sercurity_groups, list),
@@ -1335,13 +1335,13 @@ class TestMultipleAccountsEgressRuleNeg(cloudstackTestCase):
                 "Authorizing egress rule for sec group ID: %s for ssh access"
                                                         % security_group.id)
         # Authorize to only account not CIDR
-        user_secgrp_list = {self.accountB.account.name: 'default'}
+        user_secgrp_list = {self.accountB.name: 'default'}
 
         egress_rule = security_group.authorizeEgress(
                                         self.apiclient,
                                         self.services["sg_account"],
-                                        account=self.accountA.account.name,
-                                        domainid=self.accountA.account.domainid,
+                                        account=self.accountA.name,
+                                        domainid=self.accountA.domainid,
                                         user_secgrp_list=user_secgrp_list
                                         )
 
@@ -1359,8 +1359,8 @@ class TestMultipleAccountsEgressRuleNeg(cloudstackTestCase):
         ingress_rule = security_group.authorize(
                                         self.apiclient,
                                         self.services["security_group"],
-                                        account=self.accountA.account.name,
-                                        domainid=self.accountA.account.domainid
+                                        account=self.accountA.name,
+                                        domainid=self.accountA.domainid
                                         )
 
         self.assertEqual(
@@ -1374,13 +1374,13 @@ class TestMultipleAccountsEgressRuleNeg(cloudstackTestCase):
         self.virtual_machineA = VirtualMachine.create(
                                     self.apiclient,
                                     self.services["virtual_machine"],
-                                    accountid=self.accountA.account.name,
-                                    domainid=self.accountA.account.domainid,
+                                    accountid=self.accountA.name,
+                                    domainid=self.accountA.domainid,
                                     serviceofferingid=self.service_offering.id,
                                     securitygroupids=[security_group.id]
                                 )
         self.cleanup.append(self.virtual_machineA)
-        self.debug("Deploying VM in account: %s" % self.accountA.account.name)
+        self.debug("Deploying VM in account: %s" % self.accountA.name)
         vms = VirtualMachine.list(
                                   self.apiclient,
                                   id=self.virtual_machineA.id,
@@ -1402,12 +1402,12 @@ class TestMultipleAccountsEgressRuleNeg(cloudstackTestCase):
         self.virtual_machineB = VirtualMachine.create(
                                     self.apiclient,
                                     self.services["virtual_machine"],
-                                    accountid=self.accountB.account.name,
-                                    domainid=self.accountB.account.domainid,
+                                    accountid=self.accountB.name,
+                                    domainid=self.accountB.domainid,
                                     serviceofferingid=self.service_offering.id
                                 )
         self.cleanup.append(self.virtual_machineB)
-        self.debug("Deploying VM in account: %s" % self.accountB.account.name)
+        self.debug("Deploying VM in account: %s" % self.accountB.name)
 
         vms = VirtualMachine.list(
                                   self.apiclient,
@@ -1512,7 +1512,7 @@ class TestMultipleAccountsEgressRule(cloudstackTestCase):
                             cls.services["account"],
                             domainid=cls.domain.id
                             )
-        cls.services["account"] = cls.accountA.account.name
+        cls.services["account"] = cls.accountA.name
         cls._cleanup = [
                         cls.accountA,
                         cls.accountB,
@@ -1556,16 +1556,16 @@ class TestMultipleAccountsEgressRule(cloudstackTestCase):
         security_groupA = SecurityGroup.create(
                                         self.apiclient,
                                         self.services["security_group"],
-                                        account=self.accountA.account.name,
-                                        domainid=self.accountA.account.domainid
+                                        account=self.accountA.name,
+                                        domainid=self.accountA.domainid
                                         )
         self.debug("Created security group with ID: %s" % security_groupA.id)
 
         # Default Security group should not have any ingress rule
         sercurity_groups = SecurityGroup.list(
                                         self.apiclient,
-                                        account=self.accountA.account.name,
-                                        domainid=self.accountA.account.domainid
+                                        account=self.accountA.name,
+                                        domainid=self.accountA.domainid
                                         )
         self.assertEqual(
                          isinstance(sercurity_groups, list),
@@ -1582,16 +1582,16 @@ class TestMultipleAccountsEgressRule(cloudstackTestCase):
         security_groupB = SecurityGroup.create(
                                         self.apiclient,
                                         self.services["security_group"],
-                                        account=self.accountB.account.name,
-                                        domainid=self.accountB.account.domainid
+                                        account=self.accountB.name,
+                                        domainid=self.accountB.domainid
                                         )
         self.debug("Created security group with ID: %s" % security_groupB.id)
 
         # Default Security group should not have any ingress rule
         sercurity_groups = SecurityGroup.list(
                                     self.apiclient,
-                                    account=self.accountB.account.name,
-                                    domainid=self.accountB.account.domainid
+                                    account=self.accountB.name,
+                                    domainid=self.accountB.domainid
                                     )
         self.assertEqual(
                          isinstance(sercurity_groups, list),
@@ -1610,13 +1610,13 @@ class TestMultipleAccountsEgressRule(cloudstackTestCase):
                 "Authorizing egress rule for sec group ID: %s for ssh access"
                                                         % security_groupA.id)
         # Authorize to only account not CIDR
-        user_secgrp_list = {self.accountB.account.name: security_groupB.name}
+        user_secgrp_list = {self.accountB.name: security_groupB.name}
 
         egress_rule = security_groupA.authorizeEgress(
                                         self.apiclient,
                                         self.services["sg_account"],
-                                        account=self.accountA.account.name,
-                                        domainid=self.accountA.account.domainid,
+                                        account=self.accountA.name,
+                                        domainid=self.accountA.domainid,
                                         user_secgrp_list=user_secgrp_list
                                         )
 
@@ -1634,8 +1634,8 @@ class TestMultipleAccountsEgressRule(cloudstackTestCase):
         ingress_ruleA = security_groupA.authorize(
                                         self.apiclient,
                                         self.services["security_group"],
-                                        account=self.accountA.account.name,
-                                        domainid=self.accountA.account.domainid
+                                        account=self.accountA.name,
+                                        domainid=self.accountA.domainid
                                         )
 
         self.assertEqual(
@@ -1649,13 +1649,13 @@ class TestMultipleAccountsEgressRule(cloudstackTestCase):
         self.virtual_machineA = VirtualMachine.create(
                                     self.apiclient,
                                     self.services["virtual_machine"],
-                                    accountid=self.accountA.account.name,
-                                    domainid=self.accountA.account.domainid,
+                                    accountid=self.accountA.name,
+                                    domainid=self.accountA.domainid,
                                     serviceofferingid=self.service_offering.id,
                                     securitygroupids=[security_groupA.id]
                                 )
         self.cleanup.append(self.virtual_machineA)
-        self.debug("Deploying VM in account: %s" % self.accountA.account.name)
+        self.debug("Deploying VM in account: %s" % self.accountA.name)
 
         vms = VirtualMachine.list(
                                   self.apiclient,
@@ -1682,8 +1682,8 @@ class TestMultipleAccountsEgressRule(cloudstackTestCase):
         ingress_ruleB = security_groupB.authorize(
                                         self.apiclient,
                                         self.services["security_group"],
-                                        account=self.accountB.account.name,
-                                        domainid=self.accountB.account.domainid
+                                        account=self.accountB.name,
+                                        domainid=self.accountB.domainid
                                         )
 
         self.assertEqual(
@@ -1697,13 +1697,13 @@ class TestMultipleAccountsEgressRule(cloudstackTestCase):
         self.virtual_machineB = VirtualMachine.create(
                                     self.apiclient,
                                     self.services["virtual_machine"],
-                                    accountid=self.accountB.account.name,
-                                    domainid=self.accountB.account.domainid,
+                                    accountid=self.accountB.name,
+                                    domainid=self.accountB.domainid,
                                     serviceofferingid=self.service_offering.id,
                                     securitygroupids=[security_groupB.id]
                                 )
         self.cleanup.append(self.virtual_machineB)
-        self.debug("Deploying VM in account: %s" % self.accountB.account.name)
+        self.debug("Deploying VM in account: %s" % self.accountB.name)
 
         vms = VirtualMachine.list(
                                   self.apiclient,