[GitHub] cloudstack issue #1949: Automated Cloudstack bugs 9277 9276 9275 9274 9273 9...

2017-03-02 Thread shwetaag
Github user shwetaag commented on the issue:

https://github.com/apache/cloudstack/pull/1949
  
@rhtyd : I executed all the travis failures manually on  my setup and both 
test case passed
copy pasting the result:
Test Create/delete a volume and verify correct usage is recorded ... === 
TestName: test_01_volume_usage | Status : SUCCESS ===
ok

--
Ran 1 test in 820.009s

OK

Test Create/Destroy VM and verify usage calculation ... === TestName: 
test_01_vm_usage | Status : SUCCESS ===
ok

--
Ran 1 test in 197.750s

OK





---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1949: Automated Cloudstack bugs 9277 9276 9275 9274 9273 9...

2017-03-01 Thread shwetaag
Github user shwetaag commented on the issue:

https://github.com/apache/cloudstack/pull/1949
  
Closing and reopening this PR as two travis runs have given two different 
failures


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1949: Automated Cloudstack bugs 9277 9276 9275 9274...

2017-03-01 Thread shwetaag
Github user shwetaag closed the pull request at:

https://github.com/apache/cloudstack/pull/1949


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1949: Automated Cloudstack bugs 9277 9276 9275 9274...

2017-03-01 Thread shwetaag
GitHub user shwetaag reopened a pull request:

https://github.com/apache/cloudstack/pull/1949

Automated Cloudstack bugs 9277 9276 9275 9274 9273 9179 9178 9177 

Addressed all review comments. Changed all URL to pubic available url where 
ever possible . For few  tests needing rhel7 template or windows server 
templates added a comment in test case to provide a valid urls for those 
templates before executing those test cases

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shwetaag/cloudstack cruds2

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1949.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1949


commit 3e3a403208d07a2cbd61da916224a4ffda17fbc7
Author: shweta agarwal <shweta.agar...@citrix.com>
Date:   2016-02-04T06:49:39Z

automated 9277 9276 9275 9274 9273 9179 9178 9177

commit 15badc456b039ab40378c30df7285cf638cc1fd7
Author: shweta <shweta@shweta.local>
Date:   2017-02-17T10:03:45Z

Adressed review comment for automating bugs 9277 9276 9275 9274 9273 9179 
9178 9177




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1949: Automated Cloudstack bugs 9277 9276 9275 9274...

2017-02-27 Thread shwetaag
GitHub user shwetaag reopened a pull request:

https://github.com/apache/cloudstack/pull/1949

Automated Cloudstack bugs 9277 9276 9275 9274 9273 9179 9178 9177 

Addressed all review comments. Changed all URL to pubic available url where 
ever possible . For few  tests needing rhel7 template or windows server 
templates added a comment in test case to provide a valid urls for those 
templates before executing those test cases

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shwetaag/cloudstack cruds2

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1949.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1949


commit 3e3a403208d07a2cbd61da916224a4ffda17fbc7
Author: shweta agarwal <shweta.agar...@citrix.com>
Date:   2016-02-04T06:49:39Z

automated 9277 9276 9275 9274 9273 9179 9178 9177

commit 15badc456b039ab40378c30df7285cf638cc1fd7
Author: shweta <shweta@shweta.local>
Date:   2017-02-17T10:03:45Z

Adressed review comment for automating bugs 9277 9276 9275 9274 9273 9179 
9178 9177




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1949: Automated Cloudstack bugs 9277 9276 9275 9274...

2017-02-27 Thread shwetaag
Github user shwetaag closed the pull request at:

https://github.com/apache/cloudstack/pull/1949


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1949: Automated Cloudstack bugs 9277 9276 9275 9274 9273 9...

2017-02-26 Thread shwetaag
Github user shwetaag commented on the issue:

https://github.com/apache/cloudstack/pull/1949
  
@rhtyd  any updates on this PR .. it shows Travis build failed ..Does this 
need any action on my part ?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1949: Cruds2

2017-02-17 Thread shwetaag
Github user shwetaag commented on the issue:

https://github.com/apache/cloudstack/pull/1949
  
@borisstoyanov  This PR is about automating these 8 cloudstack bugs.  
automated these 8 test bugs 9277 9276 9275 9274 9273 9179 9178 9177

- Test to verify Scale CentOS7 VM does not fails with error "Cannot 
scale up the vm because of memory constraint violation"
- Test to verify Private template is visible in project
- Test to verify attaching 7th Disk to windows server 2012R2 instance
- API listVMSnapshot returns value for tags ‘project’ and 
‘projectid’ when called with project id param
- 'listProjects' doesn't return tags 'vmstopped' or 'vmrunning' when 
their value is zero
- Verify register ssh key pair can not be used to register same ssh 
keys with different name
- Verify correct Guest OS (RHEL 7) Mapping for vmware
- Atomation of Usage events for "uploadedVolume"

the details of these bugs can be seen in related bugs


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1949: Cruds2

2017-02-17 Thread shwetaag
GitHub user shwetaag opened a pull request:

https://github.com/apache/cloudstack/pull/1949

Cruds2

Addressed all review comments. Changed all URL to pubic available url where 
ever possible . For few  tests needing rhel7 template or windows server 
templates added a comment in test case to provide a valid urls for those 
templates before executing those test cases

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shwetaag/cloudstack cruds2

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1949.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1949


commit 3e3a403208d07a2cbd61da916224a4ffda17fbc7
Author: shweta agarwal <shweta.agar...@citrix.com>
Date:   2016-02-04T06:49:39Z

automated 9277 9276 9275 9274 9273 9179 9178 9177

commit 15badc456b039ab40378c30df7285cf638cc1fd7
Author: shweta <shweta@shweta.local>
Date:   2017-02-17T10:03:45Z

Adressed review comment for automating bugs 9277 9276 9275 9274 9273 9179 
9178 9177




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1401: automated 9277 9276 9275 9274 9273 9179 9178 ...

2017-02-17 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1401#discussion_r101708312
  
--- Diff: test/integration/component/test_project_usage.py ---
@@ -221,10 +219,35 @@ def test_01_vm_usage(self):
 #VM.Destroy and volume .delete Event for the created account
 # 4. Delete the account
 
+projectlist=Project.list(self.apiclient,account=self.account.name,
+ 
domainid=self.account.domainid,id=self.project.id)
+
+self.assertEqual(projectlist[0].vmrunning,0,"vmrunning value is 
not returned")
--- End diff --

added hasattr function to verify attributes presence


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1401: automated 9277 9276 9275 9274 9273 9179 9178 ...

2017-02-17 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1401#discussion_r101708342
  
--- Diff: test/integration/component/test_project_usage.py ---
@@ -1819,3 +1844,183 @@ def test_01_vpn_usage(self):
 "Check VPN.USER.ADD in events table"
 )
 return
+
+class TestVMSnapshotUsage(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+cls.testClient = super(TestVMSnapshotUsage, cls).getClsTestClient()
+cls.api_client = cls.testClient.getApiClient()
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+cls.snapshotSupported = True
+cls._cleanup = []
+if cls.hypervisor.lower() in ['hyperv', 'lxc', 'kvm','baremetal']:
+cls.snapshotSupported = False
+return
+cls.services = Services().services
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.api_client)
+cls.zone = get_zone(cls.api_client, 
cls.testClient.getZoneForTests())
+cls.services['mode'] = cls.zone.networktype
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+template = get_template(
+cls.api_client,
+cls.zone.id,
+cls.services["ostype"]
+)
+cls.services["server"]["zoneid"] = cls.zone.id
+
+cls.services["template"] = template.id
+
+# Create Account, VMs etc
+cls.account = Account.create(
+cls.api_client,
+cls.services["account"],
+domainid=cls.domain.id
+)
+
+cls.services["account"] = cls.account.name
+
+cls.project = Project.create(
+ cls.api_client,
+ cls.services["project"],
+ account=cls.account.name,
+ domainid=cls.account.domainid
+ )
+
+cls.service_offering = ServiceOffering.create(
+cls.api_client,
+
cls.services["service_offering"]
+)
+cls.virtual_machine = VirtualMachine.create(
+cls.api_client,
+cls.services["server"],
+templateid=template.id,
+serviceofferingid=cls.service_offering.id,
+projectid=cls.project.id
+)
+cls._cleanup = [
+cls.project,
+cls.service_offering,
+cls.account,
+]
+return
+
+@classmethod
+def tearDownClass(cls):
+try:
+#Cleanup resources used
+cleanup_resources(cls.api_client, cls._cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+return
+
+def setUp(self):
+self.apiclient = self.testClient.getApiClient()
+self.dbclient = self.testClient.getDbConnection()
+self.cleanup = []
+if not self.snapshotSupported:
+self.skipTest("Snapshots are not supported on %s" % 
self.hypervisor)
+return
+
+def tearDown(self):
+try:
+#Clean up, terminate the created instance and snapshots
+cleanup_resources(self.apiclient, self.cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+return
+
+@attr(speed = "slow")
+@attr(tags=["advanced", "basic", "sg", "eip", "advancedns", 
"simulator"], required_hardware="false")
--- End diff --

Done



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1401: automated 9277 9276 9275 9274 9273 9179 9178 ...

2017-02-17 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1401#discussion_r101708208
  
--- Diff: test/integration/component/test_escalations_instances.py ---
@@ -2068,6 +2068,94 @@ def test_13_vm_nics(self):
 "VM NIC is not same as expected"
 )
 return
+@attr(tags=["advanced", "basic"], required_hardware="true")
+def test_14_Create_vm_with_same_sshkey(self):
+"""
+@Desc: Test to verify API call Register ssh key pair fails when 
uses same public key for differnet key name
+"""
+
+
+# Listing all the SSH Key pairs
+list_keypairs_before = SSHKeyPair.list(
+self.userapiclient
+)
+list_keypairs_before_size = 0
+if list_keypairs_before is not None:
+list_keypairs_before_size = len(list_keypairs_before)
+
+# Registering first Key pair
+new_keypair1 = SSHKeyPair.register(
+self.userapiclient,
+name="keypair1",
+publickey="ssh-rsa: 
e6:9a:1e:b5:98:75:88:5d:56:bc:92:7b:43:48:05:b2")
+self.assertIsNotNone(
+new_keypair1,
+"New Key pair generation failed"
+)
+self.assertEquals(
+"keypair1",
+new_keypair1.name,
+"Key Pair not created with given name"
+)
+# Listing all the SSH Key pairs again
+list_keypairs_after = SSHKeyPair.list(
+self.userapiclient
+)
+status = validateList(list_keypairs_after)
+self.assertEquals(
+PASS,
+status[0],
+"Listing of Key pairs failed"
+)
+# Verifying that list size is increased by 1
+self.assertEquals(
+list_keypairs_before_size + 1,
+len(list_keypairs_after),
+"List count is not matching"
+)
+try:
+
+# Registering second key pair using same public key
+new_keypair2 = SSHKeyPair.register(
+self.userapiclient,
+name="keypair2",
+publickey="ssh-rsa: 
e6:9a:1e:b5:98:75:88:5d:56:bc:92:7b:43:48:05:b2")
+self.fail("SSH Key creation passed using same public key ")
+except CloudstackAPIException  as e:
+self.assertRaises("Exception Raised : %s" % e)
--- End diff --

Moved assertRaise in the end of the test case to execute the case completely


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1401: automated 9277 9276 9275 9274 9273 9179 9178 ...

2017-02-17 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1401#discussion_r101708141
  
--- Diff: test/integration/component/maint/test_escalations_instances.py ---
@@ -0,0 +1,270 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+#Import Local Modules
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.utils import (cleanup_resources,
+  validateList,
+  get_hypervisor_type)
+from marvin.lib.base import (Account,
+ VirtualMachine,
+ ServiceOffering,
+ Volume,
+ DiskOffering,
+ Template,
+ listConfigurations,Configurations)
+from marvin.lib.common import (get_domain,list_isos,
+   get_zone,
+   get_template)
+from nose.plugins.attrib import attr
+from ast import literal_eval
+from marvin.codes import PASS
+from marvin.cloudstackException import CloudstackAPIException
+from marvin.sshClient import SshClient
+from marvin.cloudstackException import CloudstackAPIException
+import time
+
+class TestInstance(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+try:
+cls._cleanup = []
+cls.testClient = super(TestInstance, cls).getClsTestClient()
+cls.api_client = cls.testClient.getApiClient()
+cls.services = cls.testClient.getParsedTestDataConfig()
+# Get Domain, Zone, Template
+cls.domain = get_domain(cls.api_client)
+cls.zone = get_zone(cls.api_client, 
cls.testClient.getZoneForTests())
+cls.template = get_template(
+cls.api_client,
+cls.zone.id,
+cls.services["ostype"]
+)
+if cls.zone.localstorageenabled:
+cls.storagetype = 'local'
+cls.services["service_offerings"]["tiny"]["storagetype"] = 
'local'
+cls.services["disk_offering"]["storagetype"] = 'local'
+else:
+cls.storagetype = 'shared'
+cls.services["service_offerings"]["tiny"]["storagetype"] = 
'shared'
+cls.services["disk_offering"]["storagetype"] = 'shared'
+
+cls.services['mode'] = cls.zone.networktype
+cls.services["virtual_machine"]["hypervisor"] = 
cls.testClient.getHypervisorInfo()
+cls.services["virtual_machine"]["zoneid"] = cls.zone.id
+cls.services["virtual_machine"]["template"] = cls.template.id
+cls.services["custom_volume"]["zoneid"] = cls.zone.id
+# Creating Disk offering, Service Offering and Account
+cls.disk_offering = DiskOffering.create(
+cls.api_client,
+cls.services["disk_offering"]
+)
+cls.service_offering = ServiceOffering.create(
+cls.api_client,
+cls.services["service_offerings"]["small"]
+)
+cls.account = Account.create(
+cls.api_client,
+cls.services["account"],
+domainid=cls.domain.id
+)
+# Getting authentication for user in newly created Account
+cls.user = cls.account.user[0]
+cls.userapiclient = 
cls.testClient.getUserApiClient(cls.user.username, cls.domain.name)
+cls._cleanup.append(cls.disk_offering)
+cls._cleanup.app

[GitHub] cloudstack pull request #1401: automated 9277 9276 9275 9274 9273 9179 9178 ...

2017-02-17 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1401#discussion_r101708093
  
--- Diff: tools/marvin/marvin/config/test_data.py ---
@@ -790,6 +799,26 @@
 "format": "OVA",
 "ispublic": "true"
 },
+"CentOS7template": {
+"displaytext": "Centos",
+"name": "Centos",
+"passwordenabled": False,
+"isdynamicallyscalable":True,
+"ostype": "CentOS 7",
+"url": "http://10.147.28.7/templates/cenots7/Centos7.vhd;,
--- End diff --

Changed the file location to public available link


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1401: automated 9277 9276 9275 9274 9273 9179 9178 ...

2017-02-17 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1401#discussion_r101708018
  
--- Diff: tools/marvin/marvin/config/test_data.py ---
@@ -781,6 +781,15 @@
 "ostype": "CentOS 5.6 (64-bit)"
 
 },
+"coreos_volume": {
+"diskname": "Volume_core",
+
"urlvmware":"http://10.147.28.7/templates/coreos/coreos_production_vmware.ova;,
--- End diff --

@remibergsma  Removed the link and added a public link  like 

http://dl.openvm.eu/cloudstack/coreos/x86_64/coreos_production_cloudstack_image-vmware.ova


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1250: automated CLOUDSTACK 9177 9178 9179

2017-02-17 Thread shwetaag
Github user shwetaag commented on the issue:

https://github.com/apache/cloudstack/pull/1250
  
as these cases are also included in PR 1401 so closing this PR


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1250: automated CLOUDSTACK 9177 9178 9179

2017-02-17 Thread shwetaag
Github user shwetaag closed the pull request at:

https://github.com/apache/cloudstack/pull/1250


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

2016-04-19 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1500#discussion_r60193647
  
--- Diff: test/integration/component/test_volumes.py ---
@@ -603,7 +603,134 @@ def test_01_volume_attach_detach(self):
 "Check the state of VM"
 )
 except Exception as e:
-self.fail("Exception occuered: %s" % e)
+self.fail("Exception occurred: %s" % e)
+return
+
+@attr(tags=["advanced", "advancedns"])
+def test_02_root_volume_attach_detach(self):
+"""Test Root Volume attach/detach to VM
+"""
+
+# Validate the following
+# 1. Deploy a VM
+# 2. Check for root volume
+# 3. Stop VM
+# 4. Detach root volume
+# 5. Verify root volume detached
+# 6. Attach root volume
+# 7. Start VM
+
+try:
+# Check for root volume
+root_volume_response = Volume.list(
+self.apiclient,
+virtualmachineid=self.virtual_machine.id,
+type='ROOT',
+listall=True
+)
+self.assertNotEqual(
+root_volume_response,
+None,
+"Check if root volume exists in ListVolumes"
+)
+self.assertEqual(
+isinstance(root_volume_response, list),
+True,
+"Check list volumes response for valid list"
+)
+# Grab the root volume for later use
+root_volume = root_volume_response[0]
+
+# Stop VM
+self.debug("Stopping the VM: %s" % self.virtual_machine.id)
+self.virtual_machine.stop(self.apiclient)
+
+# Ensure VM is stopped before detaching the root volume
+time.sleep(self.services["sleep"])
+
+vm_response = VirtualMachine.list(
+self.apiclient,
+id=self.virtual_machine.id,
+)
+vm = vm_response[0]
+self.assertEqual(
+vm.state,
+'Stopped',
+"Check the state of VM"
+)
+
+# Detach root volume from VM
+self.virtual_machine.detach_volume(
+self.apiclient,
+root_volume
+)
+
+# Verify that root disk is gone
+no_root_volume_response = Volume.list(
+self.apiclient,
+virtualmachineid=self.virtual_machine.id,
+type='ROOT',
+listall=True
+)
+self.assertEqual(
+no_root_volume_response,
+None,
+"Check if root volume exists in ListVolumes"
+)
+
+# Attach root volume to VM
+self.virtual_machine.attach_volume(
+self.apiclient,
+root_volume,
+0
+)
+
+# Check for root volume
+new_root_volume_response = Volume.list(
+self.apiclient,
+virtualmachineid=self.virtual_machine.id,
+type='ROOT',
+listall=True
+)
+self.assertNotEqual(
+new_root_volume_response,
+None,
+"Check if root volume exists in ListVolumes"
+)
+self.assertEqual(
+isinstance(new_root_volume_response, list),
+True,
+"Check list volumes response for valid list"
+)
+
+# Start VM
+self.virtual_machine.start(self.apiclient)
+# Sleep to ensure that VM is in ready state
+time.sleep(self.services["sleep"])
+
+vm_response = VirtualMachine.list(
+self.apiclient,
+id=self.virtual_machine.id,
+)
+# Verify VM response to check whether VM deployment was 
successful
+self.assertEqual(
+isinstance(vm_response, list),
+True,
+"Check list VM response for valid list"
+)
+sel

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

2016-04-19 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1500#discussion_r60193089
  
--- Diff: test/integration/component/test_volumes.py ---
@@ -603,7 +603,134 @@ def test_01_volume_attach_detach(self):
 "Check the state of VM"
 )
 except Exception as e:
-self.fail("Exception occuered: %s" % e)
+self.fail("Exception occurred: %s" % e)
+return
+
+@attr(tags=["advanced", "advancedns"])
+def test_02_root_volume_attach_detach(self):
+"""Test Root Volume attach/detach to VM
+"""
+
+# Validate the following
+# 1. Deploy a VM
+# 2. Check for root volume
+# 3. Stop VM
+# 4. Detach root volume
+# 5. Verify root volume detached
+# 6. Attach root volume
+# 7. Start VM
+
+try:
+# Check for root volume
--- End diff --

First Verify the hypervisor type is KVM  in order for this test to execute 
else skip this test . As root volume detach is supported for KVM .


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-9331:added code in marv...

2016-04-06 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/1458#issuecomment-206273934
  
LGTM  based on code walk through 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: [CLOUDSTACK-9337]Enhance vcenter.py to cr...

2016-04-06 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/1464#issuecomment-206273155
  
Did a code walk through . LGTM . 
Get the CI result for this PR


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-9331:added code in marv...

2016-04-01 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1458#discussion_r58182115
  
--- Diff: tools/marvin/marvin/config/Baremtal_AdvZone_Vmware.cfg ---
@@ -0,0 +1,191 @@
+{
+"zones": [
+{
+"name": "ZoneBaremetal",
+"guestcidraddress": "10.1.1.0/24",
+"providers": [
+{
+"broadcastdomainrange": "ZONE",
+"name": "VirtualRouter"
+}
+],
+"dns1": "10.x.x.x",
+"vlan": "1020-1025",
+"physical_networks": [
+{
+"broadcastdomainrange": "Zone",
+"name": "Sandbox-pnet",
+"traffictypes": [
+{
+"typ": "Guest"
+},
+{
+"typ": "Management"
+},
+{
+"typ": "Public"
+}
+],
+"providers": [
+{
+"broadcastdomainrange": "ZONE",
+"name": "VirtualRouter"
+}
+]
+}
+],
+   "logger": {
+"LogFolderPath": "C:/Marvin"
+   },
+"ipranges": [
+{
+"startip": "10.x.x.x",
+"endip": "10.x.x.x",
+"netmask": "255.255.255.0",
+"vlan": "1220",
+"gateway": "10.x.x.1"
+}
+],
+"networktype": "Advanced",
+"pods": [
+{
+"endip": "10.x.x.x",
+"name": "POD01",
+"startip": "10.x.x.x",
+"netmask": "255.255.255.192",
+   
"vmwaredc":{"name":"dc-bare","vcenter":"10.x.x.x","username":"Administrator@vsphere.local","password":"Password"},
+"clusters": [
+{
+"clustername": "10.x.x.x/dc-bar/cluster1",
+"hypervisor": "VmWare",
+"username": "Administrator@vsphere.local",
+"clustertype": "ExternalManaged",
+"password": "Password",
+"url": "http://10.x.x.x/dc-bare/cluster1;,
+"hosts": [
+{
+"username": "root",
+"url": "http://10.x.x.x;,
+"password": "password"
+},
+{
+"username": "root",
+"url": "http://10.x.x.x;,
+"password": "password"
+}
+],
+"primaryStorages": [
+{
+"url": 
"nfs://10.x.x.x:/export/home/sadhu/primrc2",
+"name": "PS0"
+}
+]
+},
+   {
+"clustername": "C1",
+"hypervisor": "BareMetal",
+"

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-9331:added code in marv...

2016-04-01 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1458#discussion_r58181661
  
--- Diff: tools/marvin/marvin/config/Baremtal_AdvZone_Vmware.cfg ---
@@ -0,0 +1,191 @@
+{
+"zones": [
+{
+"name": "ZoneBaremetal",
+"guestcidraddress": "10.1.1.0/24",
+"providers": [
+{
+"broadcastdomainrange": "ZONE",
+"name": "VirtualRouter"
+}
+],
+"dns1": "10.x.x.x",
+"vlan": "1020-1025",
+"physical_networks": [
+{
+"broadcastdomainrange": "Zone",
+"name": "Sandbox-pnet",
+"traffictypes": [
+{
+"typ": "Guest"
+},
+{
+"typ": "Management"
+},
+{
+"typ": "Public"
+}
+],
+"providers": [
+{
+"broadcastdomainrange": "ZONE",
+"name": "VirtualRouter"
+}
+]
+}
+],
+   "logger": {
+"LogFolderPath": "C:/Marvin"
+   },
+"ipranges": [
+{
+"startip": "10.x.x.x",
+"endip": "10.x.x.x",
+"netmask": "255.255.255.0",
+"vlan": "1220",
+"gateway": "10.x.x.1"
+}
+],
+"networktype": "Advanced",
+"pods": [
+{
+"endip": "10.x.x.x",
+"name": "POD01",
+"startip": "10.x.x.x",
+"netmask": "255.255.255.192",
+   
"vmwaredc":{"name":"dc-bare","vcenter":"10.x.x.x","username":"Administrator@vsphere.local","password":"Password"},
+"clusters": [
+{
+"clustername": "10.x.x.x/dc-bar/cluster1",
+"hypervisor": "VmWare",
+"username": "Administrator@vsphere.local",
+"clustertype": "ExternalManaged",
+"password": "Password",
+"url": "http://10.x.x.x/dc-bare/cluster1;,
+"hosts": [
+{
+"username": "root",
+"url": "http://10.x.x.x;,
+"password": "password"
+},
+{
+"username": "root",
+"url": "http://10.x.x.x;,
+"password": "password"
+}
+],
+"primaryStorages": [
+{
+"url": 
"nfs://10.x.x.x:/export/home/sadhu/primrc2",
+"name": "PS0"
+}
+]
+},
+   {
+"clustername": "C1",
+"hypervisor": "BareMetal",
+"

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-9331:added code in marv...

2016-04-01 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1458#discussion_r58180544
  
--- Diff: tools/marvin/marvin/config/Baremtal_AdvZone_Vmware.cfg ---
@@ -0,0 +1,191 @@
+{
+"zones": [
+{
+"name": "ZoneBaremetal",
+"guestcidraddress": "10.1.1.0/24",
+"providers": [
+{
+"broadcastdomainrange": "ZONE",
+"name": "VirtualRouter"
+}
+],
+"dns1": "10.x.x.x",
+"vlan": "1020-1025",
+"physical_networks": [
+{
+"broadcastdomainrange": "Zone",
+"name": "Sandbox-pnet",
+"traffictypes": [
+{
+"typ": "Guest"
+},
+{
+"typ": "Management"
+},
+{
+"typ": "Public"
+}
+],
+"providers": [
+{
+"broadcastdomainrange": "ZONE",
+"name": "VirtualRouter"
+}
+]
+}
+],
+   "logger": {
+"LogFolderPath": "C:/Marvin"
+   },
+"ipranges": [
+{
+"startip": "10.x.x.x",
+"endip": "10.x.x.x",
+"netmask": "255.255.255.0",
+"vlan": "1220",
+"gateway": "10.x.x.1"
+}
+],
+"networktype": "Advanced",
+"pods": [
+{
+"endip": "10.x.x.x",
+"name": "POD01",
+"startip": "10.x.x.x",
+"netmask": "255.255.255.192",
+   
"vmwaredc":{"name":"dc-bare","vcenter":"10.x.x.x","username":"Administrator@vsphere.local","password":"Password"},
+"clusters": [
+{
+"clustername": "10.x.x.x/dc-bar/cluster1",
+"hypervisor": "VmWare",
+"username": "Administrator@vsphere.local",
+"clustertype": "ExternalManaged",
+"password": "Password",
+"url": "http://10.x.x.x/dc-bare/cluster1;,
+"hosts": [
+{
+"username": "root",
+"url": "http://10.x.x.x;,
+"password": "password"
+},
+{
+"username": "root",
+"url": "http://10.x.x.x;,
+"password": "password"
+}
+],
+"primaryStorages": [
+{
+"url": 
"nfs://10.x.x.x:/export/home/sadhu/primrc2",
+"name": "PS0"
+}
+]
+},
+   {
+"clustername": "C1",
+"hypervisor": "BareMetal",
+"hosts": [
+{
+"username": "root",
+"url": "http://10.x.x.x;,
+"password": "password",
+"hostmac": "b2:ac:21:s2:1c:20",
+"cpunumber": "1",
+"cpuspeed": "1000",
+"memory": "1024",
+   
"hosttags": "host15"
--- End diff --

indentation issues


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: [CLOUDSTACK-9328]: Fix vlan issues from t...

2016-03-31 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/1455#issuecomment-203818359
  
Did Code walk through . LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: [CLOUDSTACK-9328]: Fix vlan issues from t...

2016-03-28 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1455#discussion_r57562054
  
--- Diff: test/integration/smoke/test_privategw_acl.py ---
@@ -287,6 +298,16 @@ def performVPCTests(self, vpc_off, 
restart_with_cleanup = False):
 if not physical_networks:
 self.fail("No Physical Networks found!")
 
+qresultset = self.dbclient.execute(
+"select vnet from op_dc_vnet_alloc where physical_network_id=\
+(select id from physical_network where uuid='%s' ) and taken 
is NULL;" % physical_networks[0].id
+)
+self.assertEqual(validateList(qresultset)[0],
+ PASS,
+ "Invalid sql query response"
+)
+#vlans = physical_networks[0].vlan.split('-')
+vlans = qresultset
 vlans = physical_networks[0].vlan.split('-')
--- End diff --

this split is not needed for getting correct vlan entry


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9066: Update testpath to delet...

2016-02-25 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/1078#issuecomment-188677423
  
did code walk through. LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9140: Testcase to verify if De...

2016-02-24 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/1218#issuecomment-188178629
  
did code walk through. LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9026: Modifying testpath for a...

2016-02-19 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/1031#issuecomment-186124001
  
did cod walk thought. Looks good to me .


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9012 :automation of cores feat...

2016-02-04 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1011#discussion_r51857877
  
--- Diff: test/integration/component/test_coreos.py ---
@@ -0,0 +1,291 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+""" P1 tests for coreos template and vm
+"""
+
+from marvin.lib.base import (
+VirtualMachine,
+ServiceOffering,
+Account,
+SSHKeyPair,
+Host, Template)
+from marvin.lib.common import (
+get_zone,
+get_template,
+list_routers)
+from marvin.lib.utils import (
+cleanup_resources,
+get_hypervisor_type,
+get_process_status)
+from marvin.cloudstackTestCase import cloudstackTestCase
+# Import System modules
+import tempfile
+import time
+import os
+import base64
+from nose.plugins.attrib import attr
+from marvin.sshClient import SshClient
+
+
+class TestDeployVmWithCoreosTemplate(cloudstackTestCase):
+"""Tests for deploying VM  using coreos template
+"""
+
+@classmethod
+def setUpClass(cls):
+cls._cleanup = []
+cls.testClient = super(
+TestDeployVmWithCoreosTemplate,
+cls).getClsTestClient()
+cls.api_client = cls.testClient.getApiClient()
+cls.services = cls.testClient.getParsedTestDataConfig()
+cls.zone = get_zone(cls.api_client, 
cls.testClient.getZoneForTests())
+cls.service_offering = ServiceOffering.create(
+cls.api_client, 
services=cls.services["service_offerings"]["medium"])
+cls.account = Account.create(
+cls.api_client, services=cls.services["account"])
+cls.cleanup = [cls.account]
+cls.template = get_template(
+cls.api_client,
+cls.zone.id,
+cls.services["ostype"]
+)
+cls.services["coreos"][
+"hypervisor"] = cls.testClient.getHypervisorInfo()
+cls.userdata = '#cloud-config\n\ncoreos:\n  units:\n- name: 
docker.service\n  command: start\n' \
+   '- name: web.service\n  command: start\n  
content: |\n[Unit]\n' \
+   'After=docker.service\n
Requires=docker.service\nDescription=Starts web server ' \
+   'container\n[Service]\n
TimeoutStartSec=0\n' \
+   'ExecStartPre=/usr/bin/docker pull httpd:2.4\n  
  ' \
+   'ExecStart=/usr/bin/docker run -d -p 8000:80 
httpd:2.4'
+cls.services["virtual_machine"]["userdata"] = cls.userdata
+
+cls.keypair = SSHKeyPair.create(
+cls.api_client,
+name="coreos",
+account=cls.account.name,
+domainid=cls.account.domainid
+)
+cls.debug("Created a new keypair with name: %s" % cls.keypair.name)
+
+cls.debug("Writing the private key to local file")
+cls.keyPairFilePath = tempfile.gettempdir() + os.sep + 
cls.keypair.name
+# Clenaup at end of execution
+cls.cleanup.append(cls.keyPairFilePath)
+
+cls.debug("File path: %s" % cls.keyPairFilePath)
+
+f = open(cls.keyPairFilePath, "w+")
+f.write(cls.keypair.privatekey)
+f.close()
+os.system("chmod 400 " + cls.keyPairFilePath)
+cls.mgtSvrDetails = cls.config.__dict__["mgtSvr"][0].__dict__
+return
+
+def setUp(self):
+self.api_client = self.testClient.getApiClient()
+return
+
+def tearDown(self):
+# Clean up, terminate the created volumes
+cleanup_resources

[GitHub] cloudstack pull request: automated 9277 9276 9275 9274 9273 9179 9...

2016-02-04 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1401#discussion_r51846084
  
--- Diff: tools/marvin/marvin/config/test_data.py ---
@@ -781,6 +781,15 @@
 "ostype": "CentOS 5.6 (64-bit)"
 
 },
+"coreos_volume": {
+"diskname": "Volume_core",
+
"urlvmware":"http://10.147.28.7/templates/coreos/coreos_production_vmware.ova;,
--- End diff --

@remibergsma Sure. I will remove my uploaded results file 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: automated 9277 9276 9275 9274 9273 9179 9...

2016-02-04 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1401#discussion_r51847088
  
--- Diff: tools/marvin/marvin/config/test_data.py ---
@@ -790,6 +799,26 @@
 "format": "OVA",
 "ispublic": "true"
 },
+"CentOS7template": {
+"displaytext": "Centos",
+"name": "Centos",
+"passwordenabled": False,
+"isdynamicallyscalable":True,
+"ostype": "CentOS 7",
+"url": "http://10.147.28.7/templates/cenots7/Centos7.vhd;,
--- End diff --

@remibergsma  This will certainly work . Any centos 7 vhd that is 
downloadable by CS setup should work 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: automated 9277 9276 9275 9274 9273 9179 9...

2016-02-04 Thread shwetaag
GitHub user shwetaag opened a pull request:

https://github.com/apache/cloudstack/pull/1401

automated 9277 9276 9275 9274 9273 9179 9178 9177

automated these 8  test bugs 9277 9276 9275 9274 9273 9179 9178 9177

1. - Test to verify Scale CentOS7 VM does not fails with error "Cannot 
scale up the vm because of memory constraint violation"
2. - Test to verify Private template is visible in project
3. - Test to verify attaching 7th Disk to windows server 2012R2 instance
4. - API listVMSnapshot returns value for tags ‘project’ and 
‘projectid’ when called with project id param
5. - 'listProjects' doesn't return tags 'vmstopped' or 'vmrunning' when 
their value is zero
6. - Verify register ssh key pair can not be used to register same ssh keys 
with different name
7. - Verify correct Guest OS (RHEL 7) Mapping for vmware
8. - Atomation of Usage events for "uploadedVolume" 

the details of these bugs can be seen in related bugs
[results.txt](https://github.com/apache/cloudstack/files/117080/results.txt)
[results.txt](https://github.com/apache/cloudstack/files/117081/results.txt)
[results.txt](https://github.com/apache/cloudstack/files/117082/results.txt)
[results.txt](https://github.com/apache/cloudstack/files/117083/results.txt)
[results.txt](https://github.com/apache/cloudstack/files/117084/results.txt)
[results.txt](https://github.com/apache/cloudstack/files/117085/results.txt)
[runinfo.txt](https://github.com/apache/cloudstack/files/117088/runinfo.tx
[results.txt](https://github.com/apache/cloudstack/files/117089/results.txt)
t)








You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shwetaag/cloudstack cruds

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1401.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1401


commit 3e3a403208d07a2cbd61da916224a4ffda17fbc7
Author: shweta agarwal <shweta.agar...@citrix.com>
Date:   2016-02-04T06:49:39Z

automated 9277 9276 9275 9274 9273 9179 9178 9177




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: automated 9277 9276 9275 9274 9273 9179 9...

2016-02-04 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/1401#issuecomment-179712919
  
copy pasting results of each test cases
@desc:Incorrect guest os mapping in vmware for Rhel7 ... === TestName: 
test3_attach_ISO_in_RHEL7OSVM | Status : SUCCESS ===
ok

--
Ran 1 test in 280.817s

OK
@desc: Unable to attach 7th Disk to windows server 2012R2 instance ... === 
TestName: test1_attach_volume | Status : SUCCESS ===
ok

--
Ran 1 test in 1545.088s

OK
Test Templates creation in projects ... === TestName: 
test_04_public_private_template_use_in_project | Status : SUCCESS ===
ok

--
Ran 1 test in 177.475s

OK
Test Create/Destroy VM and verify usage calculation ... === TestName: 
test_01_vm_usage | Status : SUCCESS ===
ok

--
Ran 1 test in 99.517s

OK
Test Create/delete a volume and verify correct usage is recorded ... === 
TestName: test_01_volume_usage | Status : SUCCESS ===
ok

--
Ran 1 test in 207.581s

OK
@desc: ... === TestName: test_Scale_VM | Status : SUCCESS ===
ok

--
Ran 1 test in 607.857s

OK
@Desc: Test to verify API call Register ssh key pair fails when uses same 
public key for differnet key name ... === TestName: 
test_14_Create_vm_with_same_sshkey | Status : SUCCESS ===
ok

--
Ran 1 test in 80.050s

OK



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9128: Testcase to verify physi...

2016-02-02 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/1199#issuecomment-179035607
  
Did code review. Code looks  good to me


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: Automation for CLOUDSTACK-9214 , detect w...

2016-01-08 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/1315#issuecomment-169947496
  
@DaanHoogland  I did the code review by code walk through and the code 
looks good to me


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: automated CLOUDSTACK 9177 9178 9179

2015-12-16 Thread shwetaag
GitHub user shwetaag opened a pull request:

https://github.com/apache/cloudstack/pull/1250

automated CLOUDSTACK 9177 9178 9179

Automated three test cases for Cloud stack 
First is  Atomation of Usage events for "uploadedVolume" 
Second  is Verify correct Guest OS (RHEL 7) Mapping for vmware  
Third is  Verify register ssh key pair can not be used to register same ssh 
keys with different name

More details on the given  bugs 

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shwetaag/cloudstack bug_fixes

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1250.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1250


commit 3efd5d60871510984c47a78113f704019e37bb73
Author: shweta agarwal <shweta.agar...@citrix.com>
Date:   2015-12-16T10:35:33Z

automated CLOUDSTACK 9177 9178 9179




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9012 :automation of cores feat...

2015-10-30 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/1011#issuecomment-152510674
  
pasting result.txt

test1_coreos_VM_creation 
(integration.component.test_coreos.TestDeployVmWithCoreosTemplate) ... === 
TestName: test1_coreos_VM_creation | Status : SUCCESS ===
ok

--
Ran 1 test in 838.303s

OK



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9012 :automation of cores feat...

2015-10-30 Thread shwetaag
GitHub user shwetaag opened a pull request:

https://github.com/apache/cloudstack/pull/1011

CLOUDSTACK-9012 :automation of cores feature test path

https://issues.apache.org/jira/browse/CLOUDSTACK-9012
Automated a full scenario of coreos guest OS support:
it includes registering coreos templates present at 
http://dl.openvm.eu/cloudstack/coreos/x86_64/
1. based on hypervisor types of zone
2. creating ssh key pair
3. creating a sample user data
4. creating a coreos virtual machine using this ssh keypair and userdata
5. verifying ssh access to coreo os machine using keypair and core username
6. verifying userdata is applied on virtual machine and the service asked 
in sample data is actually running
7. Verifying userdata in router vm as well

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shwetaag/cloudstack coreos

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1011.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1011


commit 95355960746fbe1bdc649a483bff250b115ee7e7
Author: shweta agarwal <shweta.agar...@citrix.com>
Date:   2015-10-30T11:24:16Z

automation of cores feature test path

corrected some entires in test data




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8756:Incorrect guest os mappin...

2015-09-01 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/728#discussion_r38500164
  
--- Diff: tools/marvin/marvin/config/test_data.py ---
@@ -775,6 +775,15 @@
 "ostype": "CentOS 5.6 (64-bit)"
 
 },
+"CentOS6.3template": {
+"displaytext": "Centos",
+"name": "Centos",
+"passwordenabled": False,
+"ostype": "CentOS 6.3 (64-bit)",
+"url": "http://10.147.28.7/templates/centos63.ova;,
--- End diff --

@bhaisaab this is a simple centos template for vmware so nothing special 
about it. for this test to pass in one's environment he should have centos 
template uploaded in their http server and change the url accordingly . 
If you have a place where we can share this file publicly i will be more 
than happy to upload it there. Not only this file, i will upload  lot other  
files  whose urls are a private http server mentioned in test_data.py 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8757:FTP modules are not loade...

2015-08-25 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/729#issuecomment-134502839
  
Result information: 
test_01_FTPModulesInVR 
(integration.component.maint.test_escalations_routers.TestVR) ... === TestName: 
test_01_FTPModulesInVR | Status : SUCCESS ===
ok

--
Ran 1 test in 233.348s

OK



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8756:Incorrect guest os mappin...

2015-08-25 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/728#issuecomment-134504120
  
result information :

@desc:Incorrect guest os mapping in vmware for CentOS 5.9 and above ... === 
TestName: test2_attach_ISO_in_CentOSVM | Status : SUCCESS ===
ok

--
Ran 1 test in 392.003s

OK



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8755: xs-tools.iso missing fro...

2015-08-25 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/727#issuecomment-134504724
  
Result information : 

Test delete ISO ... === TestName: test_07_list_default_iso | Status : 
SUCCESS ===
ok

--
Ran 1 test in 384.111s

OK



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8757:FTP modules are not loade...

2015-08-23 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/729#issuecomment-134030869
  
@remibergsma : i have added details to the bug . Copy pasting for your 
reference 
To have Active FTP working in isolated networks VRs need the have the 
following modules loaded (In order to map the FTP Data and FTP CMD to the same 
frontend IP):

modprobe nf_conntrack_ftp
modprobe nf_nat_ftp

in some versions of Cloudstack we noted that these modules are not loaded 
on CS Routers by default  . So this test case is for verifying that these 
modules are loaded by default in VR and if not report an error


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8756:Incorrect guest os mappin...

2015-08-23 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/728#issuecomment-134034345
  
@remibergsma   the issue was of incorrect mapping of centos guest os 
(grater than 5.9 ) in cloud stack .CS was sending wrong info to Vcenter to 
launch VMs and centos VMs guest os is set to others .
This will cause CentOS( 5.9 or any other greater version)  unable to be 
provided on VMware certain operations like attach and detach iso.
so this test case is about verifying that attach and detach iso is working 
fine with vmware VMs  especially when using centos guest os


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8757:FTP modules are not loade...

2015-08-21 Thread shwetaag
GitHub user shwetaag opened a pull request:

https://github.com/apache/cloudstack/pull/729

CLOUDSTACK-8757:FTP modules are not loaded in VR

check bug 8757 for more details .

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shwetaag/cloudstack router

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/729.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #729


commit cc308050b8e50516c50b02d99934d5ea14304ac4
Author: shweta agarwal shweta.agar...@citrix.com
Date:   2015-08-21T12:34:19Z

CLOUDSTACK-8757:FTP modules are not loaded in VR




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8756:Incorrect guest os mappin...

2015-08-21 Thread shwetaag
GitHub user shwetaag opened a pull request:

https://github.com/apache/cloudstack/pull/728

CLOUDSTACK-8756:Incorrect guest os mapping in CCP 4.2.1-6 for CentOS 5.9

Check the bug 8756 for more details 

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shwetaag/cloudstack attach_iso

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/728.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #728


commit bbdb47854df106f6c95b9c8935ffc3717eac3617
Author: shweta agarwal shweta.agar...@citrix.com
Date:   2015-08-21T12:17:01Z

CLOUDSTACK-8756:Incorrect guest os mapping in CCP 4.2.1-6 for CentOS 5.9




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: Customer escalations

2015-08-21 Thread shwetaag
Github user shwetaag closed the pull request at:

https://github.com/apache/cloudstack/pull/724


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8755: xs-tools.iso missing fro...

2015-08-21 Thread shwetaag
GitHub user shwetaag opened a pull request:

https://github.com/apache/cloudstack/pull/727

CLOUDSTACK-8755: xs-tools.iso missing from ISOs in CCP GUI

updated the automation for correct verification in case of listing default 
isos 

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shwetaag/cloudstack list_iso

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/727.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #727


commit b6fafbfcbe527b65b80cc11d394421dc363e007f
Author: shweta agarwal shweta.agar...@citrix.com
Date:   2015-08-21T12:03:23Z

CLOUDSTACK-8755: xs-tools.iso missing from ISOs in CCP GUI




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: Customer escalations

2015-08-21 Thread shwetaag
GitHub user shwetaag opened a pull request:

https://github.com/apache/cloudstack/pull/724

Customer escalations

Automated two test cases and modified one testcase
First one is  to verify correct guest os mapping in vmware for CentOS 5.9 
and above
Step1 :Register an CentOS 6.3 template
Step2 :Launch a VM
Step3: Try to attach VMware Tools ISO
Step4: Verify VMware tools ISO attached correctly
Second one is 
 Verify FTP modules are loaded in VR of advance zone
step1 : create a VR in advance zone
step2: Verify FTP modules are there in created VR
and modified test case for listing default ISO which was not codded 
correctly and hence not giving error even after failing


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shwetaag/cloudstack customer_escalations

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/724.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #724


commit d1311b90ac0764bd137f7140572a71a7b33da10c
Author: shweta agarwal shweta.agar...@citrix.com
Date:   2015-07-30T11:57:22Z

test case automated for list template pagination

commit 4ff606bce8d587c91a1bef2447ac109ec23b3ebc
Author: shweta agarwal shweta.agar...@citrix.com
Date:   2015-08-10T11:18:03Z

additional verification added

commit f05fec9044c28db6802c9c1c593b11424cf79294
Author: shweta agarwal shweta.agar...@citrix.com
Date:   2015-08-19T05:28:02Z

Merge branch 'master' of https://git-wip-us.apache.org/repos/asf/cloudstack 
into customer_escalations

commit 9dacbd615276fbc33d6d37a5fe6c2a181a49ce2a
Author: shweta agarwal shweta.agar...@citrix.com
Date:   2015-08-21T07:19:13Z

added 3 test case 39918 38555 22659




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: test case automated for list template pag...

2015-08-10 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/646#discussion_r36620849
  
--- Diff: test/integration/component/maint/test_escalation_templates.py ---
@@ -0,0 +1,394 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# License); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# AS IS BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+from marvin.cloudstackTestCase import cloudstackTestCase, unittest
+from marvin.lib.base import (Account,
+ Domain, Template, 
Configurations,VirtualMachine,Snapshot,ServiceOffering
+ )
+from marvin.lib.utils import (cleanup_resources, validateList)
+from marvin.lib.common import (get_zone, get_template, 
get_builtin_template_info,update_resource_limit,list_volumes )
+from nose.plugins.attrib import attr
+from marvin.codes import PASS
+from marvin.sshClient import SshClient
+from marvin.cloudstackException import CloudstackAPIException
+import time
+
+
+class TestlistTemplates(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+
+testClient = super(
+TestlistTemplates, cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.testdata = testClient.getParsedTestDataConfig()
+cls.zone = get_zone(cls.apiclient, 
cls.testClient.getZoneForTests())
+cls.template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.testdata[ostype]
+)
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+builtin_info = get_builtin_template_info(cls.apiclient, 
cls.zone.id)
+cls.testdata[templates][url] = builtin_info[0]
+cls.testdata[templates][hypervisor] = builtin_info[1]
+cls.testdata[templates][format] = builtin_info[2]
+if cls.zone.localstorageenabled:
+cls.storagetype = 'local'
+cls.testdata[service_offerings][tiny][storagetype] = 
'local'
+cls.testdata[disk_offering][storagetype] = 'local'
+else:
+cls.storagetype = 'shared'
+cls.testdata[service_offerings][tiny][storagetype] = 
'shared'
+cls.testdata[disk_offering][storagetype] = 'shared'
+cls.testdata[virtual_machine][hypervisor] = cls.hypervisor
+cls.testdata[virtual_machine][zoneid] = cls.zone.id
+cls.testdata[virtual_machine][template] = cls.template.id
+cls.testdata[custom_volume][zoneid] = cls.zone.id
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata[service_offerings][tiny]
+)
+cls.mgtSvrDetails = cls.config.__dict__[mgtSvr][0].__dict__
+cls.cleanup = []
+
+# Create 1 domain admin account
+
+cls.domain = Domain.create(
+cls.apiclient,
+cls.testdata[domain])
+
+cls.account = Account.create(
+cls.apiclient,
+cls.testdata[account],
+admin=True,
+domainid=cls.domain.id)
+
+cls.debug(Created account %s in domain %s %
+  (cls.account.name, cls.domain.id))
+
+cls.cleanup.append(cls.account)
+cls.cleanup.append(cls.domain)
+
+@classmethod
+def tearDownClass(cls):
+try:
+# Cleanup resources used
+cleanup_resources(cls.apiclient, cls.cleanup)
+
+except Exception as e:
+raise Exception(Warning: Exception during cleanup : %s % e)
+return
+def RestartServers(self):
+ Restart management server and usage server 
+
+sshClient = SshClient(
+self.mgtSvrDetails[mgtSvrIp],
+22,
+self.mgtSvrDetails[user],
+self.mgtSvrDetails[passwd]
+)
+command = service cloudstack-management restart
+sshClient.execute(command

[GitHub] cloudstack pull request: test case automated for list template pag...

2015-08-10 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/646#discussion_r36621050
  
--- Diff: test/integration/component/maint/test_escalation_templates.py ---
@@ -0,0 +1,394 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# License); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# AS IS BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+from marvin.cloudstackTestCase import cloudstackTestCase, unittest
+from marvin.lib.base import (Account,
+ Domain, Template, 
Configurations,VirtualMachine,Snapshot,ServiceOffering
+ )
+from marvin.lib.utils import (cleanup_resources, validateList)
+from marvin.lib.common import (get_zone, get_template, 
get_builtin_template_info,update_resource_limit,list_volumes )
+from nose.plugins.attrib import attr
+from marvin.codes import PASS
+from marvin.sshClient import SshClient
+from marvin.cloudstackException import CloudstackAPIException
+import time
+
+
+class TestlistTemplates(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+
+testClient = super(
+TestlistTemplates, cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.testdata = testClient.getParsedTestDataConfig()
+cls.zone = get_zone(cls.apiclient, 
cls.testClient.getZoneForTests())
+cls.template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.testdata[ostype]
+)
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+builtin_info = get_builtin_template_info(cls.apiclient, 
cls.zone.id)
+cls.testdata[templates][url] = builtin_info[0]
+cls.testdata[templates][hypervisor] = builtin_info[1]
+cls.testdata[templates][format] = builtin_info[2]
+if cls.zone.localstorageenabled:
+cls.storagetype = 'local'
+cls.testdata[service_offerings][tiny][storagetype] = 
'local'
+cls.testdata[disk_offering][storagetype] = 'local'
+else:
+cls.storagetype = 'shared'
+cls.testdata[service_offerings][tiny][storagetype] = 
'shared'
+cls.testdata[disk_offering][storagetype] = 'shared'
+cls.testdata[virtual_machine][hypervisor] = cls.hypervisor
+cls.testdata[virtual_machine][zoneid] = cls.zone.id
+cls.testdata[virtual_machine][template] = cls.template.id
+cls.testdata[custom_volume][zoneid] = cls.zone.id
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata[service_offerings][tiny]
+)
+cls.mgtSvrDetails = cls.config.__dict__[mgtSvr][0].__dict__
+cls.cleanup = []
+
+# Create 1 domain admin account
+
+cls.domain = Domain.create(
+cls.apiclient,
+cls.testdata[domain])
+
+cls.account = Account.create(
+cls.apiclient,
+cls.testdata[account],
+admin=True,
+domainid=cls.domain.id)
+
+cls.debug(Created account %s in domain %s %
+  (cls.account.name, cls.domain.id))
+
+cls.cleanup.append(cls.account)
+cls.cleanup.append(cls.domain)
+
+@classmethod
+def tearDownClass(cls):
+try:
+# Cleanup resources used
+cleanup_resources(cls.apiclient, cls.cleanup)
+
+except Exception as e:
+raise Exception(Warning: Exception during cleanup : %s % e)
+return
+def RestartServers(self):
+ Restart management server and usage server 
+
+sshClient = SshClient(
+self.mgtSvrDetails[mgtSvrIp],
+22,
+self.mgtSvrDetails[user],
+self.mgtSvrDetails[passwd]
+)
+command = service cloudstack-management restart
+sshClient.execute(command

[GitHub] cloudstack pull request: test case automated for list template pag...

2015-07-31 Thread shwetaag
GitHub user shwetaag opened a pull request:

https://github.com/apache/cloudstack/pull/646

test case automated for list template pagination

verify list template gives same result with page size=500page=(1,2) and 
page size=1000 when there are around 1000 templates .
Moved one test case from component folder to maint folder as it requires MS 
restart

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shwetaag/cloudstack customer_escalations

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/646.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #646


commit d1311b90ac0764bd137f7140572a71a7b33da10c
Author: shweta agarwal shweta.agar...@citrix.com
Date:   2015-07-30T11:57:22Z

test case automated for list template pagination




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: verifying template creation permission ba...

2015-06-22 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/153#issuecomment-114021543
  
@gauravaradhye  and @DaanHoogland  another request for this was created 

https://github.com/apache/cloudstack/pull/184
and @sanjeevneelarapu reviewed it and checked it . Most of your review 
comments were incorporated in the new request except moving it to maintenance 
folder. 





---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: verifying template creation permission ba...

2015-06-22 Thread shwetaag
Github user shwetaag closed the pull request at:

https://github.com/apache/cloudstack/pull/153


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: Automation for issue, Private template no...

2015-05-12 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/241#discussion_r30117632
  
--- Diff: test/integration/component/test_project_template.py ---
@@ -0,0 +1,162 @@
+from marvin.codes import FAILED, PASS
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.base import Account, VirtualMachine, 
ServiceOffering,Project,Network,NetworkOffering,Domain,Volume,Template
+from marvin.lib.common import get_zone, get_domain, get_template
+from marvin.lib.utils import cleanup_resources
+from nose.plugins.attrib import attr
+
+
+
+class TestProjectPrivateTemplate(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestDeployVM, cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.services = testClient.getParsedTestDataConfig()
+cls.cleanup = []
+
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+cls.services['mode'] = cls.zone.networktype
+
+template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.services[ostype]
+)
+if template == FAILED:
+assert False, get_template() failed to return template with 
description %s % cls.services[ostype]
+
+# Set Zones and disk offerings
+cls.services[small][zoneid] = cls.zone.id
+cls.services[small][template] = template.id
+
+cls.services[medium][zoneid] = cls.zone.id
+cls.services[medium][template] = template.id
+cls.services[iso1][zoneid] = cls.zone.id
+
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.services[service_offerings][tiny]
+)
+
+cls.new_domain = Domain.create(
+   cls.apiclient,
+   cls.services[domain],
+   )
+cls.account = Account.create(
+cls.apiclient,
+cls.services[account],
+admin=True,
+domainid=cls.new_domain.id
+)
+
+cls.debug(cls.account.id)
+#Fetch an api client with the user account created
+
+
+def test_project_private_template(self):
+self.api_client = 
self.testClient.getUserApiClient(UserName=self.account.name, 
DomainName=self.account.domain)
+
+#Create a new project using the account created
+self.project = 
Project.create(self.api_client,self.services[project],account=self.account.name,domainid=self.new_domain.id)
+self.debug(The project has been created)
+
+self.network_offering = NetworkOffering.create(
+self.apiclient,
+self.services[network_offering],
+)
+# Enable Network offering
+self.network_offering.update(self.apiclient, state='Enabled')
+
+self.services[network][networkoffering] = 
self.network_offering.id
+
+
+
+#cls.api_client = 
testClient.getUserApiClient(UserName=cls.account.name, 
DomainName=cls.account.domain)
+
+self.project_network = Network.create(
+self.api_client,
+self.services[network],
+networkofferingid=self.network_offering.id,
+zoneid=self.zone.id,
+projectid=self.project.id
+)
+
+
+self.virtual_machine = VirtualMachine.create(
+self.api_client,
+self.services[small],
+serviceofferingid=self.service_offering.id,
+projectid=self.project.id,
+networkids=self.project_network.id,
+zoneid=self.zone.id
+)
+
+#Stop virtual machine
+self.virtual_machine.stop(self.api_client)
+
+list_volumes = Volume.list(
+   self.api_client,
+   projectid=self.project.id,
+   
virtualmachineid=self.virtual_machine.id,
+   type='ROOT',
+   listall=True
+   )
+
+self.volume = list_volumes[0]
+
+#Create template from Virtual machine and Volume ID
+self.services[projtemplate]={}
+self.services[projtemplate][name] = Project_Private_template

[GitHub] cloudstack pull request: Automation for issue, Private template no...

2015-05-12 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/241#discussion_r30117593
  
--- Diff: test/integration/component/test_project_template.py ---
@@ -0,0 +1,162 @@
+from marvin.codes import FAILED, PASS
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.base import Account, VirtualMachine, 
ServiceOffering,Project,Network,NetworkOffering,Domain,Volume,Template
+from marvin.lib.common import get_zone, get_domain, get_template
+from marvin.lib.utils import cleanup_resources
+from nose.plugins.attrib import attr
+
+
+
+class TestProjectPrivateTemplate(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestDeployVM, cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.services = testClient.getParsedTestDataConfig()
+cls.cleanup = []
+
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+cls.services['mode'] = cls.zone.networktype
+
+template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.services[ostype]
+)
+if template == FAILED:
+assert False, get_template() failed to return template with 
description %s % cls.services[ostype]
+
+# Set Zones and disk offerings
+cls.services[small][zoneid] = cls.zone.id
+cls.services[small][template] = template.id
+
+cls.services[medium][zoneid] = cls.zone.id
+cls.services[medium][template] = template.id
+cls.services[iso1][zoneid] = cls.zone.id
+
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.services[service_offerings][tiny]
+)
+
+cls.new_domain = Domain.create(
+   cls.apiclient,
+   cls.services[domain],
+   )
+cls.account = Account.create(
+cls.apiclient,
+cls.services[account],
+admin=True,
+domainid=cls.new_domain.id
+)
+
+cls.debug(cls.account.id)
+#Fetch an api client with the user account created
+
+
+def test_project_private_template(self):
+self.api_client = 
self.testClient.getUserApiClient(UserName=self.account.name, 
DomainName=self.account.domain)
+
+#Create a new project using the account created
+self.project = 
Project.create(self.api_client,self.services[project],account=self.account.name,domainid=self.new_domain.id)
+self.debug(The project has been created)
+
+self.network_offering = NetworkOffering.create(
+self.apiclient,
+self.services[network_offering],
+)
+# Enable Network offering
+self.network_offering.update(self.apiclient, state='Enabled')
+
+self.services[network][networkoffering] = 
self.network_offering.id
+
+
+
+#cls.api_client = 
testClient.getUserApiClient(UserName=cls.account.name, 
DomainName=cls.account.domain)
+
+self.project_network = Network.create(
+self.api_client,
+self.services[network],
+networkofferingid=self.network_offering.id,
+zoneid=self.zone.id,
+projectid=self.project.id
+)
+
+
+self.virtual_machine = VirtualMachine.create(
+self.api_client,
+self.services[small],
+serviceofferingid=self.service_offering.id,
+projectid=self.project.id,
+networkids=self.project_network.id,
+zoneid=self.zone.id
+)
+
+#Stop virtual machine
+self.virtual_machine.stop(self.api_client)
+
+list_volumes = Volume.list(
+   self.api_client,
+   projectid=self.project.id,
+   
virtualmachineid=self.virtual_machine.id,
+   type='ROOT',
+   listall=True
+   )
+
+self.volume = list_volumes[0]
+
+#Create template from Virtual machine and Volume ID
+self.services[projtemplate]={}
+self.services[projtemplate][name] = Project_Private_template

[GitHub] cloudstack pull request: Automation for issue, Private template no...

2015-05-12 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/241#discussion_r30117163
  
--- Diff: test/integration/component/test_project_template.py ---
@@ -0,0 +1,162 @@
+from marvin.codes import FAILED, PASS
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.base import Account, VirtualMachine, 
ServiceOffering,Project,Network,NetworkOffering,Domain,Volume,Template
+from marvin.lib.common import get_zone, get_domain, get_template
+from marvin.lib.utils import cleanup_resources
+from nose.plugins.attrib import attr
+
+
+
+class TestProjectPrivateTemplate(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestDeployVM, cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.services = testClient.getParsedTestDataConfig()
+cls.cleanup = []
+
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+cls.services['mode'] = cls.zone.networktype
+
+template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.services[ostype]
+)
+if template == FAILED:
+assert False, get_template() failed to return template with 
description %s % cls.services[ostype]
+
+# Set Zones and disk offerings
+cls.services[small][zoneid] = cls.zone.id
+cls.services[small][template] = template.id
+
+cls.services[medium][zoneid] = cls.zone.id
+cls.services[medium][template] = template.id
+cls.services[iso1][zoneid] = cls.zone.id
+
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.services[service_offerings][tiny]
+)
+
+cls.new_domain = Domain.create(
+   cls.apiclient,
+   cls.services[domain],
+   )
+cls.account = Account.create(
+cls.apiclient,
+cls.services[account],
+admin=True,
+domainid=cls.new_domain.id
+)
+
+cls.debug(cls.account.id)
+#Fetch an api client with the user account created
+
+
+def test_project_private_template(self):
+self.api_client = 
self.testClient.getUserApiClient(UserName=self.account.name, 
DomainName=self.account.domain)
+
+#Create a new project using the account created
+self.project = 
Project.create(self.api_client,self.services[project],account=self.account.name,domainid=self.new_domain.id)
+self.debug(The project has been created)
+
+self.network_offering = NetworkOffering.create(
+self.apiclient,
+self.services[network_offering],
+)
+# Enable Network offering
+self.network_offering.update(self.apiclient, state='Enabled')
+
+self.services[network][networkoffering] = 
self.network_offering.id
+
+
+
+#cls.api_client = 
testClient.getUserApiClient(UserName=cls.account.name, 
DomainName=cls.account.domain)
+
+self.project_network = Network.create(
+self.api_client,
+self.services[network],
+networkofferingid=self.network_offering.id,
+zoneid=self.zone.id,
+projectid=self.project.id
+)
+
+
+self.virtual_machine = VirtualMachine.create(
+self.api_client,
+self.services[small],
+serviceofferingid=self.service_offering.id,
+projectid=self.project.id,
+networkids=self.project_network.id,
+zoneid=self.zone.id
+)
+
+#Stop virtual machine
+self.virtual_machine.stop(self.api_client)
+
+list_volumes = Volume.list(
+   self.api_client,
+   projectid=self.project.id,
+   
virtualmachineid=self.virtual_machine.id,
+   type='ROOT',
+   listall=True
+   )
+
+self.volume = list_volumes[0]
+
+#Create template from Virtual machine and Volume ID
+self.services[projtemplate]={}
+self.services[projtemplate][name] = Project_Private_template

[GitHub] cloudstack pull request: Automation for issue, Private template no...

2015-05-12 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/241#discussion_r30118073
  
--- Diff: test/integration/component/test_user_taglisting.py ---
@@ -0,0 +1,129 @@
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.base import (Account,
+ ServiceOffering,
+ VirtualMachine,
+ NetworkOffering,
+ Network,
+ Tag
+)
+from marvin.lib.common import (get_domain,
+get_zone,
+get_template)
+from marvin.cloudstackAPI import *
+from nose.plugins.attrib import attr
+from marvin.codes import FAILED, PASS
+
+class TestDeployVMWithTags(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+cls.apiclient = cls.testClient.getApiClient()
+cls.dbclient = cls.testClient.getDbConnection()
+cls.cleanup = []
+cls.services = cls.testClient.getParsedTestDataConfig()
+
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, 
cls.testClient.getZoneForTests())
+cls.services['mode'] = cls.zone.networktype
+
+cls.account = Account.create(
+cls.apiclient,
+cls.services[account],
+admin=True,
+domainid=cls.domain.id
+)
+
+
+template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.services[ostype]
+)
+if template == FAILED:
+assert False, get_template() failed to return template with 
description %s % cls.services[ostype]
+
+# Set Zones and disk offerings
+cls.services[small][zoneid] = cls.zone.id
+cls.services[small][template] = template.id
+
+cls.services[medium][zoneid] = cls.zone.id
+cls.services[medium][template] = template.id
+cls.services[iso1][zoneid] = cls.zone.id
+
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.services[service_offerings][tiny]
+)
+
+cls.network_offering = NetworkOffering.create(
+cls.apiclient,
+cls.services[network_offering],
+)
+# Enable Network offering
+cls.network_offering.update(cls.apiclient, state='Enabled')
+
+cls.services[network][networkoffering] = 
cls.network_offering.id
+cls.api_client = 
cls.testClient.getUserApiClient(UserName=cls.account.name, 
DomainName=cls.account.domain)
+cls.network = Network.create(
+cls.api_client,
+cls.services[network],
+networkofferingid=cls.network_offering.id,
+accountid=cls.account.name,
+domainid=cls.domain.id,
+zoneid=cls.zone.id
+)
+
+
+def test_deploy_vm_with_tags(self):
+Test Deploy Virtual Machine
+
+# Validate the following:
+# 1. User tags are created for VM
+# 2. listing the tags for the VM will list the tag only once and 
doesn't list the same tag multiple times
+self.virtual_machine = VirtualMachine.create(
+self.api_client,
+self.services[small],
+serviceofferingid=self.service_offering.id,
+accountid=self.account.name,
+domainid=self.account.domainid,
+networkids=self.network.id,
+zoneid=self.zone.id
+)
+
+tag1 = Tag.create(
+self.apiclient,
+resourceIds=self.virtual_machine.id,
+resourceType='userVM',
+tags={'vmtag': 'autotag'}
+)
+tag2 = Tag.create(
+self.apiclient,
+resourceIds=self.virtual_machine.id,
+resourceType='userVM',
+tags={'vmtag1': 'autotag'}
+)
+
+tags = Tag.list(
+self.apiclient,
+listall=True,
+resourceType='userVM',
+key='vmtag',
+value='autotag'
+)
+
+self.assertEqual(
+   len(tags),
+1,
+There is just one tag listed with list tags
+)
+
+self.assertGreater(
+len(tags

[GitHub] cloudstack pull request: Automation for issue, Private template no...

2015-05-11 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/241#discussion_r30032217
  
--- Diff: test/integration/component/test_project_template.py ---
@@ -0,0 +1,162 @@
+from marvin.codes import FAILED, PASS
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.base import Account, VirtualMachine, 
ServiceOffering,Project,Network,NetworkOffering,Domain,Volume,Template
+from marvin.lib.common import get_zone, get_domain, get_template
+from marvin.lib.utils import cleanup_resources
+from nose.plugins.attrib import attr
+
+
+
+class TestProjectPrivateTemplate(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestDeployVM, cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.services = testClient.getParsedTestDataConfig()
+cls.cleanup = []
+
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+cls.services['mode'] = cls.zone.networktype
+
+template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.services[ostype]
+)
+if template == FAILED:
+assert False, get_template() failed to return template with 
description %s % cls.services[ostype]
+
+# Set Zones and disk offerings
+cls.services[small][zoneid] = cls.zone.id
+cls.services[small][template] = template.id
+
+cls.services[medium][zoneid] = cls.zone.id
+cls.services[medium][template] = template.id
+cls.services[iso1][zoneid] = cls.zone.id
+
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.services[service_offerings][tiny]
+)
+
+cls.new_domain = Domain.create(
+   cls.apiclient,
+   cls.services[domain],
+   )
+cls.account = Account.create(
+cls.apiclient,
+cls.services[account],
+admin=True,
+domainid=cls.new_domain.id
+)
+
+cls.debug(cls.account.id)
+#Fetch an api client with the user account created
+
+
+def test_project_private_template(self):
+self.api_client = 
self.testClient.getUserApiClient(UserName=self.account.name, 
DomainName=self.account.domain)
+
+#Create a new project using the account created
+self.project = 
Project.create(self.api_client,self.services[project],account=self.account.name,domainid=self.new_domain.id)
+self.debug(The project has been created)
+
+self.network_offering = NetworkOffering.create(
--- End diff --

no need to create a network offering for this test case .  


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: Automation for issue, Private template no...

2015-05-11 Thread shwetaag
Github user shwetaag commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/241#discussion_r30032273
  
--- Diff: test/integration/component/test_project_template.py ---
@@ -0,0 +1,162 @@
+from marvin.codes import FAILED, PASS
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.base import Account, VirtualMachine, 
ServiceOffering,Project,Network,NetworkOffering,Domain,Volume,Template
+from marvin.lib.common import get_zone, get_domain, get_template
+from marvin.lib.utils import cleanup_resources
+from nose.plugins.attrib import attr
+
+
+
+class TestProjectPrivateTemplate(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestDeployVM, cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.services = testClient.getParsedTestDataConfig()
+cls.cleanup = []
+
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+cls.services['mode'] = cls.zone.networktype
+
+template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.services[ostype]
+)
+if template == FAILED:
+assert False, get_template() failed to return template with 
description %s % cls.services[ostype]
+
+# Set Zones and disk offerings
+cls.services[small][zoneid] = cls.zone.id
+cls.services[small][template] = template.id
+
+cls.services[medium][zoneid] = cls.zone.id
+cls.services[medium][template] = template.id
+cls.services[iso1][zoneid] = cls.zone.id
+
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.services[service_offerings][tiny]
+)
+
+cls.new_domain = Domain.create(
+   cls.apiclient,
+   cls.services[domain],
+   )
+cls.account = Account.create(
+cls.apiclient,
+cls.services[account],
+admin=True,
+domainid=cls.new_domain.id
+)
+
+cls.debug(cls.account.id)
+#Fetch an api client with the user account created
+
+
+def test_project_private_template(self):
+self.api_client = 
self.testClient.getUserApiClient(UserName=self.account.name, 
DomainName=self.account.domain)
+
+#Create a new project using the account created
+self.project = 
Project.create(self.api_client,self.services[project],account=self.account.name,domainid=self.new_domain.id)
+self.debug(The project has been created)
+
+self.network_offering = NetworkOffering.create(
+self.apiclient,
+self.services[network_offering],
+)
+# Enable Network offering
+self.network_offering.update(self.apiclient, state='Enabled')
+
+self.services[network][networkoffering] = 
self.network_offering.id
+
+
+
+#cls.api_client = 
testClient.getUserApiClient(UserName=cls.account.name, 
DomainName=cls.account.domain)
+
+self.project_network = Network.create(
--- End diff --

no need to create this network explicitly .This can be created as a part of 
deploy virtual machine
 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: tool to aid db comaprision for upgrade te...

2015-04-24 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/188#issuecomment-95824131
  
@agneya2001  this is version 1. 
I know enhancement are possible .
Like db comparison based on versions but that will come in next version


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: tool to aid db comaprision for upgrade te...

2015-04-23 Thread shwetaag
GitHub user shwetaag opened a pull request:

https://github.com/apache/cloudstack/pull/188

tool to aid db comaprision for upgrade testing

What this tool is capable to do is

1.  Compare all the tables schema between upgraded setup and fresh install 
setup  and find if there is any schema difference between any tables
2.  Compare global configuration between upgraded and fresh install setup 
and find out if there is any difference between the two on following fields
a.  Value
b.  Scope
c.  Description
d.  Component
e.  Category


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shwetaag/cloudstack tools

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/188.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #188


commit 7f917f26a8b9700310144e031ecda5d82b00435e
Author: shweta agarwal shweta.agar...@citrix.com
Date:   2015-04-23T11:42:30Z

tool to aid db comaprision for upgrade testing




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: Escalations

2015-04-23 Thread shwetaag
Github user shwetaag closed the pull request at:

https://github.com/apache/cloudstack/pull/184


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: tool to aid db comaprision for upgrade te...

2015-04-23 Thread shwetaag
Github user shwetaag closed the pull request at:

https://github.com/apache/cloudstack/pull/188


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: Escalations

2015-04-21 Thread shwetaag
GitHub user shwetaag opened a pull request:

https://github.com/apache/cloudstack/pull/184

Escalations

incorporated review comments

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shwetaag/cloudstack escalations

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/184.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #184


commit 382a171230d4f36a38c6d4518713e87934861748
Author: shweta agarwal shweta.agar...@citrix.com
Date:   2015-04-09T11:21:50Z

verifying template creation permission

commit 3714a4e545bb05984666ab3643995469269cc6ff
Author: shweta agarwal shweta.agar...@citrix.com
Date:   2015-04-21T12:25:56Z

incorporated review comments




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: verifying template creation permission ba...

2015-04-09 Thread shwetaag
GitHub user shwetaag opened a pull request:

https://github.com/apache/cloudstack/pull/153

verifying template creation permission based on global parameter

Test to create Public Template by registering or by snapshot and volume when
Global parameter 'allow.public.user.template' is set to  False
@steps:
1.Set Global parameter 'allow.public.user.template' as False. 
Restart Management server
2. Create a domain
3. Create a domain admin and a domain user
4. Create a vm as domain user
5. take snapshot of root disk as user vm
6. try to create public template from snapshot . It should fail
7. stop the VM
8. take the public template from volume. it should fail
9. register a public template as a domain user . it should fail
10. create a VM  as domain admin
11. create a snapshot of root disk as domain admin
12 create a public template of the snapshot .it should fail
13. Register a public template as domain admin. it should fail
14 Stop the vm as domain admin
15. Create a template from volume as domain admin . it should fail

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shwetaag/cloudstack escalations

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/153.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #153


commit 01f86972a9317ebf98a1c4677d15f7c1cc0c0698
Author: shweta agarwal shweta.agar...@citrix.com
Date:   2015-04-09T11:21:50Z

verifying template creation permission




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: Resetvm

2015-04-07 Thread shwetaag
GitHub user shwetaag opened a pull request:

https://github.com/apache/cloudstack/pull/148

Resetvm

Added two tests to verify reset vm functionality. 

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shwetaag/cloudstack resetvm

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/148.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #148


commit 433e6a2bd4634f3a1ce8525ccd5601b7e27190b7
Author: shweta agarwal shweta.agar...@citrix.com
Date:   2015-03-16T07:25:41Z

added two test case related to restore VM functionality

commit 62a4c74007fab83124f36acb20c74013f9a89c96
Author: shweta agarwal shweta.agar...@citrix.com
Date:   2015-04-07T05:07:53Z

Restore VM verification tests




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---