You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ts...@apache.org on 2013/06/17 14:02:20 UTC
[1/2] git commit: updated refs/heads/master to 28b598b
Updated Branches:
refs/heads/master f4dcca6e4 -> 28b598b4a
Fix VPC tests related to user roles
Many vpc tests fail because of incorrect apiClient passed in to create
the VPC, network etc. The exact method used is getUserApiClient to fetch
the apiclient for a specific user.
Signed-off-by: Prasanna Santhanam <ts...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/28b598b4
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/28b598b4
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/28b598b4
Branch: refs/heads/master
Commit: 28b598b4acde49be61caade95824a60e213108b9
Parents: a5189b1
Author: Prasanna Santhanam <ts...@apache.org>
Authored: Mon Jun 17 17:31:10 2013 +0530
Committer: Prasanna Santhanam <ts...@apache.org>
Committed: Mon Jun 17 17:32:05 2013 +0530
----------------------------------------------------------------------
setup/dev/advanced.cfg | 3 +-
test/integration/component/test_vpc.py | 158 ++++++++---------------
tools/marvin/marvin/cloudstackTestClient.py | 1 +
3 files changed, 59 insertions(+), 103 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/28b598b4/setup/dev/advanced.cfg
----------------------------------------------------------------------
diff --git a/setup/dev/advanced.cfg b/setup/dev/advanced.cfg
index 3020165..3d0854d 100644
--- a/setup/dev/advanced.cfg
+++ b/setup/dev/advanced.cfg
@@ -218,7 +218,8 @@
"mgtSvrIp": "localhost",
"passwd": "password",
"user": "root",
- "port": 8096
+ "port": 8096,
+ "hypervisor" : "simulator"
}
]
}
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/28b598b4/test/integration/component/test_vpc.py
----------------------------------------------------------------------
diff --git a/test/integration/component/test_vpc.py b/test/integration/component/test_vpc.py
index 0afeb77..510e297 100644
--- a/test/integration/component/test_vpc.py
+++ b/test/integration/component/test_vpc.py
@@ -20,6 +20,7 @@
#Import Local Modules
from nose.plugins.attrib import attr
from marvin.cloudstackTestCase import *
+from marvin.cloudstackException import cloudstackAPIException
from marvin.cloudstackAPI import *
from marvin.integration.lib.utils import *
from marvin.integration.lib.base import *
@@ -820,7 +821,7 @@ class TestVPC(cloudstackTestCase):
)
return
- @attr(tags=["advanced", "intervlan"])
+ @attr(tags=["advanced", "intervlan", "multiple"])
def test_07_restart_network_vm_running(self):
""" Test Restart VPC when there are multiple networks associated
"""
@@ -1923,8 +1924,7 @@ class TestVPC(cloudstackTestCase):
@attr(tags=["advanced", "intervlan"])
def test_14_deploy_vm_1(self):
- """ Test deployment of vm in a network from user account. But the VPC is created
- without account/domain ID
+ """ Test vm deploy in network by a user where VPC was created without account/domain ID
"""
# 1. Create VPC without providing account/domain ID.
@@ -1935,18 +1935,19 @@ class TestVPC(cloudstackTestCase):
self.apiclient,
self.services["account"]
)
- self.debug("Created account: %s" % user.account.name)
+ self.debug("Created account: %s" % user.name)
self.cleanup.append(user)
self.services["vpc"]["cidr"] = "10.1.1.1/16"
self.debug("creating a VPC network in the account: %s" %
- user.account.name)
+ user.name)
- userapiclient = self.testClient.createNewApiClient(
- UserName=user.account.name,
- DomainName=user.account.domain,
+ userapiclient = self.testClient.createUserApiClient(
+ UserName=user.name,
+ DomainName=user.domain,
acctType=0)
+
vpc = VPC.create(
userapiclient,
self.services["vpc"],
@@ -1999,8 +2000,7 @@ class TestVPC(cloudstackTestCase):
@attr(tags=["advanced", "intervlan"])
def test_15_deploy_vm_2(self):
- """ Test deployment of vm in a network from domain admin account. But the VPC is created
- without account/domain ID
+ """ Test deployment of vm in a network in a domain admin account where VPC is created without account/domain ID
"""
# 1. Create VPC without providing account/domain ID.
@@ -2016,18 +2016,18 @@ class TestVPC(cloudstackTestCase):
self.apiclient,
self.services["account"]
)
- self.debug("Created account: %s" % user.account.name)
+ self.debug("Created account: %s" % user.name)
self.cleanup.append(user)
self.services["vpc"]["cidr"] = "10.1.1.1/16"
self.debug("creating a VPC network in the account: %s" %
- user.account.name)
+ user.name)
#0 - User, 1 - Root Admin, 2 - Domain Admin
- userapiclient = self.testClient.createNewApiClient(
- UserName=user.account.name,
- DomainName=self.services["domain"]["name"],
- acctType=2)
+ userapiclient = self.testClient.getUserApiClient(
+ account=user.name,
+ domain=self.services["domain"]["name"],
+ type=2)
vpc = VPC.create(
userapiclient,
@@ -2092,23 +2092,23 @@ class TestVPC(cloudstackTestCase):
self.apiclient,
self.services["account"]
)
- self.debug("Created account: %s" % user.account.name)
+ self.debug("Created account: %s" % user.name)
self.cleanup.append(user)
self.services["vpc"]["cidr"] = "10.1.1.1/16"
self.debug("creating a VPC network in the account: %s" %
- user.account.name)
+ user.name)
- userapiclient = self.testClient.createNewApiClient(
- UserName=user.account.name,
- DomainName=user.account.domain,
- acctType=0)
+ userapiclient = self.testClient.getUserApiClient(
+ account=user.name,
+ domain=user.domain,
+ type=0)
vpc = VPC.create(
self.apiclient,
self.services["vpc"],
- account=user.account.name,
- domainid=user.account.domainid,
+ account=user.name,
+ domainid=user.domainid,
vpcofferingid=self.vpc_off.id,
zoneid=self.zone.id,
)
@@ -2176,82 +2176,39 @@ class TestVPC(cloudstackTestCase):
self.apiclient,
self.services["domain_admin"]
)
- self.debug("Created account: %s" % domain_admin.account.name)
+ self.debug("Created account: %s" % domain_admin.name)
self.cleanup.append(domain_admin)
- da_apiclient = self.testClient.createNewApiClient(
- UserName=domain_admin.account.name,
- #DomainName=self.services["domain"]["name"],
- DomainName=domain_admin.account.domain,
- acctType=2)
+ da_apiclient = self.testClient.getUserApiClient(
+ account=domain_admin.name,
+ domain=domain_admin.domain,
+ type=2)
user = Account.create(
self.apiclient,
self.services["account"]
)
- self.debug("Created account: %s" % user.account.name)
+ self.debug("Created account: %s" % user.name)
self.cleanup.append(user)
self.services["vpc"]["cidr"] = "10.1.1.1/16"
self.debug("creating a VPC network in the account: %s" %
- user.account.name)
+ user.name)
#0 - User, 1 - Root Admin, 2 - Domain Admin
- userapiclient = self.testClient.createNewApiClient(
- UserName=user.account.name,
- DomainName=user.account.domain,
- acctType=0)
-
- vpc = VPC.create(
- da_apiclient,
- self.services["vpc"],
- account=user.account.name,
- domainid=user.account.domainid,
- vpcofferingid=self.vpc_off.id,
- zoneid=self.zone.id,
- )
- self.validate_vpc_network(vpc)
-
- self.network_offering = NetworkOffering.create(
- self.apiclient,
- self.services["network_offering"],
- conservemode=False
- )
- # Enable Network offering
- self.network_offering.update(self.apiclient, state='Enabled')
- self._cleanup.append(self.network_offering)
-
- gateway = vpc.cidr.split('/')[0]
- # Split the cidr to retrieve gateway
- # for eg. cidr = 10.0.0.1/24
- # Gateway = 10.0.0.1
-
- # Creating network using the network offering created
- self.debug("Creating network with network offering: %s" %
- self.network_offering.id)
- network = Network.create(
- userapiclient,
- self.services["network"],
- networkofferingid=self.network_offering.id,
- zoneid=self.zone.id,
- gateway=gateway,
- vpcid=vpc.id
- )
- self.debug("Created network with ID: %s" % network.id)
+ userapiclient = self.testClient.getUserApiClient(
+ account=user.name,
+ domain=user.domain,
+ type=0)
- # Spawn an instance in that network
- virtual_machine = VirtualMachine.create(
- userapiclient,
- self.services["virtual_machine"],
- serviceofferingid=self.service_offering.id,
- networkids=[str(network.id)]
- )
- self.debug("Deployed VM in network: %s" % network.id)
-
- self.assertNotEqual(virtual_machine,
- None,
- "VM creation in the network failed")
-
- return
+ with self.assertRaises(cloudstackAPIException):
+ vpc = VPC.create(
+ da_apiclient,
+ self.services["vpc"],
+ account=user.name,
+ domainid=user.domainid,
+ vpcofferingid=self.vpc_off.id,
+ zoneid=self.zone.id,
+ )
@attr(tags=["advanced", "intervlan"])
def test_18_create_net_for_user_diff_domain_by_doadmin(self):
@@ -2271,29 +2228,29 @@ class TestVPC(cloudstackTestCase):
self.apiclient,
self.services["domain_admin"]
)
- self.debug("Created account: %s" % domain_admin.account.name)
+ self.debug("Created account: %s" % domain_admin.name)
self.cleanup.append(domain_admin)
- da_apiclient = self.testClient.createNewApiClient(
- UserName=domain_admin.account.name,
- DomainName=self.services["domain"]["name"],
- acctType=2)
+ da_apiclient = self.testClient.getUserApiClient(
+ account=domain_admin.name,
+ domain=self.services["domain"]["name"],
+ type=2)
user = Account.create(
self.apiclient,
self.services["account"]
)
- self.debug("Created account: %s" % user.account.name)
+ self.debug("Created account: %s" % user.name)
self.cleanup.append(user)
self.services["vpc"]["cidr"] = "10.1.1.1/16"
self.debug("creating a VPC network in the account: %s" %
- user.account.name)
+ user.name)
#0 - User, 1 - Root Admin, 2 - Domain Admin
- userapiclient = self.testClient.createNewApiClient(
- UserName=user.account.name,
- DomainName=user.account.domain,
- acctType=0)
+ userapiclient = self.testClient.getUserApiClient(
+ account=user.name,
+ domain=user.domain,
+ type=0)
vpc = VPC.create(
da_apiclient,
@@ -2467,7 +2424,4 @@ class TestVPC(cloudstackTestCase):
self.assertEqual(vpc_networks[0].displaytext,
new_display_text,
- "Updation of VPC display text failed.")
-
-
-
+ "Updation of VPC display text failed.")
\ No newline at end of file
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/28b598b4/tools/marvin/marvin/cloudstackTestClient.py
----------------------------------------------------------------------
diff --git a/tools/marvin/marvin/cloudstackTestClient.py b/tools/marvin/marvin/cloudstackTestClient.py
index d85a61c..37380d6 100644
--- a/tools/marvin/marvin/cloudstackTestClient.py
+++ b/tools/marvin/marvin/cloudstackTestClient.py
@@ -121,6 +121,7 @@ class cloudstackTestClient(object):
apiKey, securityKey, self.connection.asyncTimeout, self.connection.logging)
self.userApiClient = cloudstackAPIClient.CloudStackAPIClient(newUserConnection)
self.userApiClient.connection = newUserConnection
+ self.userApiClient.hypervisor = self.apiClient.hypervisor
return self.userApiClient
def close(self):
[2/2] git commit: updated refs/heads/master to 28b598b
Posted by ts...@apache.org.
Fix the listVPC based on user roles
listVPC based on user roles uses the old account.account reference
causing the test to fail.
Signed-off-by: Prasanna Santhanam <ts...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/a5189b15
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/a5189b15
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/a5189b15
Branch: refs/heads/master
Commit: a5189b1550139c2bcf2a0982ddbedf8effcd13d4
Parents: f4dcca6
Author: Prasanna Santhanam <ts...@apache.org>
Authored: Mon Jun 17 13:36:29 2013 +0530
Committer: Prasanna Santhanam <ts...@apache.org>
Committed: Mon Jun 17 17:32:05 2013 +0530
----------------------------------------------------------------------
test/integration/component/test_vpc.py | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/a5189b15/test/integration/component/test_vpc.py
----------------------------------------------------------------------
diff --git a/test/integration/component/test_vpc.py b/test/integration/component/test_vpc.py
index acf7a8e..0afeb77 100644
--- a/test/integration/component/test_vpc.py
+++ b/test/integration/component/test_vpc.py
@@ -795,16 +795,16 @@ class TestVPC(cloudstackTestCase):
self.services["vpc"],
vpcofferingid=self.vpc_off.id,
zoneid=self.zone.id,
- account=self.user.account.name,
- domainid=self.user.account.domainid
+ account=self.user.name,
+ domainid=self.user.domainid
)
self.validate_vpc_network(vpc_2)
self.debug("Validating list VPCs call by passing account and domain")
vpcs = VPC.list(
self.apiclient,
- account=self.user.account.name,
- domainid=self.user.account.domainid,
+ account=self.user.name,
+ domainid=self.user.domainid,
listall=True
)
self.assertEqual(