[GitHub] cloudstack pull request: CLOUDSTACK-9168: Testpath to check if wro...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1245#issuecomment-190057225
  
Changed the tags to advanced zone:

Testcase result:
Check Netmask value in database ... === TestName: 
test_01_netmask_value_check | Status : SUCCESS ===
ok

--
Ran 1 test in 122.485s

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-9026: Modifying testpath for a...

[pritisarap12]

Github user pritisarap12 commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1031#discussion_r53309123
  
--- Diff: test/integration/testpaths/testpath_storage_migration.py ---
@@ -248,6 +248,11 @@ def setUpClass(cls):
 DomainName=cls.account.domain
 )
 # Create Service offering
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata["service_offering"]
+)
+
--- End diff --

Added service_offering in cleanup.


---
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-9091: Update testpath for para...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1130#issuecomment-185610104
  
Testcase result after modifications:

Test Volume (root) Snapshot ... === TestName: test_01_volume_snapshot | 
Status : SUCCESS ===
ok

--
Ran 1 test in 1012.564s

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-9168: Testpath to check if wro...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1245#issuecomment-185550931
  
Updated the testpath with review comments:
--Validation of netmask 
--check it is from correct CIDR



---
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-9168: Testpath to check if wro...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1245#issuecomment-185550227
  
Test result after modifications:

Check Netmask value in database ... === TestName: 
test_01_netmask_value_check | Status : SUCCESS ===
ok

--
Ran 1 test in 121.854s

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-9026: Modifying testpath for a...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1031#issuecomment-185115938
  
Testcase result:

Test migrate Volume (root and data disk) ... === TestName: 
test_01_migrate_root_and_data_disk_nonlive | Status : SUCCESS ===
ok

--
Ran 1 test in 1688.400s

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-9066: Update testpath to delet...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1078#issuecomment-185115973
  
Testcase result:

snapshot hardning ... === TestName: test_01_snapshot_hardning_kvm | Status 
: SUCCESS ===
ok
snapshot hardning ... === SKIP: Skip test for server other than XenServer

--
Ran 2 tests in 1259.500s

OK (SKIP=1)




---
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-8717: Failed to start instance...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1416#issuecomment-185092942
  
@remibergsma and @runseb  Please give one LGTM to close the PR as it is 
just the change in commit message and PR title .


---
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-8717: Failed to start instance...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1416#issuecomment-185092215
  
@DaanHoogland  Please give one LGTM to close the PR as it is just the 
change in commit message and PR title .




---
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...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1218#issuecomment-185067571
  
Testcase result after modifications:

Dedicated cluster and router allocation ... === TestName: 
test_01_dedicated_cluster_allocation | Status : SUCCESS ===
ok

--
Ran 1 test in 278.037s

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-9140: Testcase to verify if De...

[pritisarap12]

Github user pritisarap12 commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1218#discussion_r53127878
  
--- Diff: test/integration/component/maint/testpath_disable_enable_zone.py 
---
@@ -1691,3 +1696,159 @@ def test_01_disable_enable_host(self):
  )
 
 return
+
+
+class TestClusterDedication(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestClusterDedication, cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.testdata = testClient.getParsedTestDataConfig()
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+
+cls.template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.testdata["ostype"])
+
+cls.Skiptest = False
+cls._cleanup = []
+cls.clusters = Cluster.list(cls.apiclient, zoneid=cls.zone.id)
+if len(cls.clusters) < 2:
+cls.Skiptest = True
+
+try:
+# Create an account
+cls.account_1 = Account.create(
+cls.apiclient,
+cls.testdata["account"],
+domainid=cls.domain.id
+)
+
+cls._cleanup.append(cls.account_1)
+
+cls.account_2 = Account.create(
+cls.apiclient,
+cls.testdata["account"],
+domainid=cls.domain.id
+)
+
+cls._cleanup.append(cls.account_2)
+# Create user api client of the account
+cls.userapiclient_1 = testClient.getUserApiClient(
+UserName=cls.account_1.name,
+DomainName=cls.account_1.domain
+)
+cls.userapiclient_2 = testClient.getUserApiClient(
+UserName=cls.account_2.name,
+DomainName=cls.account_2.domain
+)
+
+# Create Service offering
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata["service_offering"],
+)
+cls._cleanup.append(cls.service_offering)
+
+cls.disk_offering = DiskOffering.create(
+cls.apiclient,
+cls.testdata["disk_offering"],
+)
+
+cls._cleanup.append(cls.disk_offering)
+
+except Exception as e:
+cls.tearDownClass()
+raise e
+return
+
+@classmethod
+def tearDownClass(cls):
+try:
+cleanup_resources(cls.apiclient, cls._cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+
+def setUp(self):
+
+if self.Skiptest:
+self.skipTest("Insufficient clusters to run the test")
+
+self.apiclient = self.testClient.getApiClient()
+self.dbclient = self.testClient.getDbConnection()
+self.cleanup = []
+
+def tearDown(self):
+try:
+listClusterscmd = 
listDedicatedClusters.listDedicatedClustersCmd()
+listClusterscmd.clusterid = self.clusters[0].id
+ret_list = 
self.apiclient.listDedicatedClusters(listClusterscmd)
+if ret_list:
+dedicateCmd = 
releaseDedicatedCluster.releaseDedicatedClusterCmd()
+dedicateCmd.clusterid = self.clusters[0].id
+self.apiclient.releaseDedicatedCluster(dedicateCmd)
+
+cleanup_resources(self.apiclient, self.cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+return
+
+@attr(tags=["basic", "advanced"], required_hardware="false")
+def test_01_dedicated_cluster_allocation(self):
+""" Dedicated cluster and router allocation
+1.   Dedicate a cluster to one account
+2.   Deploy a VM on dedicated account
+3.   Deploy another VM on another account.
+4.   Verify the dedicated cluster is not used for
+virtual routers that belong to non-dedicated account
+"""
+
+# Step 1
+dedicateCmd = d

[GitHub] cloudstack pull request: CLOUDSTACK-8717: Failed to start instance...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8717: Failed to start instance after restoring the running 
instance 

Changing PR title and commit message 
In continuation with PR #1411  and  #667 

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8717-Failed-to-start-instance-after-restoring-the-running-instance

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

https://github.com/apache/cloudstack/pull/1416.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 #1416


commit 06289da022e264c3135413cdb064a414a35e1c8c
Author: Priti Sarap <priti.sa...@clogeny.com>
Date:   2016-02-17T05:02:01Z

CLOUDSTACK-8717: Failed to start instance after restoring the running 
instance




---
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...

[pritisarap12]

GitHub user pritisarap12 reopened a pull request:

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

CLOUDSTACK-9026: Modifying testpath for adding missing parameter

Adding service_offering creation in testpath_storage_migration.py testpath 
which is missing right now 

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-9026-Modifying-testpath-for-adding-missing-parameter

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

https://github.com/apache/cloudstack/pull/1031.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 #1031


commit c2376334592835c30b865c2b08f8c179eed8d328
Author: Priti Sarap <priti.sa...@clogeny.com>
Date:   2015-11-04T09:18:38Z

CLOUDSTACK-9026: Modifying testpath for adding missing parameter




---
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-8717: Failed to start instance...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/667#issuecomment-182778439
  
Testcase results:

Test Restore VM on VMWare ... === TestName: test_01_recover_VM | Status : 
SUCCESS ===
ok

--
Ran 1 test in 414.293s

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: Squashing two commits in to single commit

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

Squashing two commits in to single commit

CLOUDSTACK-8717: Failed to start instance after restoring the running 
instance
-Modified code to add tag to aonly one cluster wide SP
-Added validateList function
-Added code to clear tags in tearDown class

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8717-Failed-to-start-instance-after-restoring-the-running-instance

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

https://github.com/apache/cloudstack/pull/1411.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 #1411


commit 8c7f5ab93d56c2aa0cde194509b728511173f7a9
Author: Priti Sarap <priti.sa...@clogeny.com>
Date:   2015-08-07T12:27:54Z

Squashing two commits in to single commit

CLOUDSTACK-8717: Failed to start instance after restoring the running 
instance
-Modified code to add tag to aonly one cluster wide SP
-Added validateList function
-Added code to clear tags in tearDown class




---
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...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1199#issuecomment-182284142
  
Rebased with current master.


---
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-9168: Testpath to check if wro...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-9168: Testpath to check if wrong value is inserted into nics 
table netmask field when creating a VM

Testcase Result:

Check Netmask value in database ... === TestName: 
test_01_netmask_value_check | Status : SUCCESS ===
ok

--
Ran 1 test in 143.944s

OK


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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-9168-Check-if-wrong-value-is-inserted-into-nics-table-netmask-field-when-creating-a-VM

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

https://github.com/apache/cloudstack/pull/1245.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 #1245


commit b7e0941158521b1e1330b51f1a151f6b9b9f543b
Author: Priti Sarap <priti.sa...@clogeny.com>
Date:   2015-12-15T11:08:17Z

CLOUDSTACK-9168: Testpath to check if wrong value is inserted into nics 
table netmask field when creating a VM




---
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...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-9140: Testcase to verify if Dedicated cluster is used for 
virtual routers that belong to non dedicated account



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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-9140-Testcase-to-verify-if-Dedicated-cluster-is-used-for-virtual-routers-that-belong-to-non-dedicated-account

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

https://github.com/apache/cloudstack/pull/1218.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 #1218


commit dff3bf2c0ee5dc8bbc8de82e6eb9a3a974a3d94a
Author: Priti Sarap <priti.sa...@clogeny.com>
Date:   2015-12-11T10:17:21Z

CLOUDSTACK-9140: Testcase to verify if Dedicated cluster is used for 
virtual routers that belong to non dedicated account




---
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...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1218#issuecomment-163903907
  
Testcase Result:

Dedicated cluster and router allocation ... === TestName: 
test_01_dedicated_cluster_allocation | Status : SUCCESS ===
ok

--
Ran 1 test in 302.277s

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...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-9128: Testcase to verify physical_size attribute of 
snapshot_store_ref table 

Verify if physical_size attribute of snapshot_store_ref table stores actual 
physical size of the snapshot 

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-9128-Testcase-to-verify-if-snapshot_store_ref-table-stores-actual-size-of-back-snapshot-in-secondary-storage

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

https://github.com/apache/cloudstack/pull/1199.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 #1199


commit be88dacf1e5890a176abaed09bcc325fb3684a51
Author: Priti Sarap <priti.sa...@clogeny.com>
Date:   2015-12-09T09:34:50Z

CLOUDSTACK-9128: Testcase to verify if snapshot_store_ref table stores 
actual size of back snapshot in secondary storage




---
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...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1199#issuecomment-163163707
  
Testcase result:

Check Snapshots size in database ... === TestName: 
test_02_snapshot_size_check | Status : SUCCESS ===
ok

--
Ran 1 test in 337.826s

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-9121: Adding VmSnapshot valida...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-9121: Adding VmSnapshot validation in testpath_revert_snap.py

In testpath_revert_snap.py, there was no code to check if VM snapshot is 
created or not hence adding code for snapshot validation.

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-9121-Adding-VmSnapshot-validation-in-testpath_revert_snap.py

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

https://github.com/apache/cloudstack/pull/1190.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 #1190


commit 1cf44dee3a94dcac765aff9b2827e20cd05f971f
Author: Priti Sarap <priti.sa...@clogeny.com>
Date:   2015-12-08T10:59:13Z

CLOUDSTACK-9121: Adding VmSnapshot validation in testpath_revert_snap.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-9121: Adding VmSnapshot valida...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1190#issuecomment-162851170
  
Testpath Result:

Test revert snapshot on XenServer ... === TestName: 
test_01_check_revert_snapshot | Status : SUCCESS ===
ok

--
Ran 1 test in 119.808s

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-9091: Update testpath for para...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-9091: Update testpath for parameter issues

In testpath_volume_snapshot testpath creating volume from snapshot function 
is passing zonid parameter to function in base class but there it doesn't take 
as separate parameter it takes it from "services" so updating it.

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-9091-Update-testpath-for-parameter-issues

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

https://github.com/apache/cloudstack/pull/1130.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 #1130


commit 358115de151ababd45d31a743f71edadc8cfafae
Author: Priti Sarap <priti.sa...@clogeny.com>
Date:   2015-11-27T07:21:50Z

CLOUDSTACK-9091: Update testpath for parameter 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-9066: Update testpath to delet...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-9066: Update testpath to delete account after deleting VM's of 
that account 

In testpath_snapshot_hardning.py testpath account was getting cleared prior 
to VM's of that account hence while cleaning up the account the VM's in that 
account also gets deleted hence while clearing VM's it gives exception as "No 
VM found".

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-9066-update-testpath-to-delete-account-after-deleting-VMs-of-that-account

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

https://github.com/apache/cloudstack/pull/1078.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 #1078


commit a8e60499fe7a895f03d875e27105e017fcc2e615
Author: Priti Sarap <priti.sa...@clogeny.com>
Date:   2015-11-17T11:55:23Z

CLOUDSTACK-9066: Update testpath to delete account after deleting VMs of 
that account




---
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-9041: Modifying template creat...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-9041: Modifying template creation from snapshot function 

In create_from_snapshot function of Template class there is no parameter to 
accept if the template is public hence default it is creating private templates 
Hence adding this parameter.

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-9041-Modifying-template-creation-from-snapshot-function-in-base.py

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

https://github.com/apache/cloudstack/pull/1041.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 #1041


commit bbe0fc4be9527d51820b067a602886003991db4d
Author: Priti Sarap <priti.sa...@clogeny.com>
Date:   2015-11-06T09:06:32Z

CLOUDSTACK-9041: Modifying template creation from snapshot function in 
base.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-9026: Modifying testpath for a...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-9026: Modifying testpath for adding missing parameter

Addning service_offering creation in testpath_storage_migration.py testpath 
which is missing right now 

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-9026-Modifying-testpath-for-adding-missing-parameter

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

https://github.com/apache/cloudstack/pull/1031.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 #1031


commit c2376334592835c30b865c2b08f8c179eed8d328
Author: Priti Sarap <priti.sa...@clogeny.com>
Date:   2015-11-04T09:18:38Z

CLOUDSTACK-9026: Modifying testpath for adding missing parameter




---
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-9005: Modifying tearDown funct...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/1000#issuecomment-153302753
  
Test Result:

Storage and Snapshot Limit ...
--
Ran 1 test in 970.317s

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-9001: Modifying snapshot resul...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/994#issuecomment-153302400
  
Test Result:

Test events table ... === TestName: test_01_verify_events_table | Status : 
SUCCESS ===
ok

--
Ran 1 test in 86.360s

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-8996: Reducing Virual Machine ...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8996: Reducing Virual Machine Deployments

Combining testcases to reduce the VM deployments wherever possible.

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8996-Reducing-Virual-Machine-Deployments

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

https://github.com/apache/cloudstack/pull/1010.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 #1010


commit a49dec37b28ba2b8af22d6444f40f72b33b63547
Author: Priti Sarap <priti.sa...@clogeny.com>
Date:   2015-10-30T04:34:30Z

CLOUDSTACK-8996: Reducing Virual Machine Deployments




---
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-9001: Modifying snapshot resul...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-9001: Modifying snapshot results validation 

Currently snapshots results validation is based on length of snapshot 
result but if snapshot creation fails then None type object will not have "len" 
attribute hence modifying the validation.

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

$ git pull https://github.com/pritisarap12/cloudstack 
snapshot-has-no-attribute-len

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

https://github.com/apache/cloudstack/pull/994.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 #994


commit 0e01c7f357cd6596570fd966e0f2ac65c4bbedb1
Author: Priti Sarap <priti.sa...@clogeny.com>
Date:   2015-10-28T05:49:36Z

CLOUDSTACK-9001: Modifying snapshot results validation in 
testpath_uuid_event testpath




---
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-8728: Testcase to Verify if VR...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/684#issuecomment-148614442
  
@remibergsma:  I force pushed the commit and it is showing green signal now 
you can run the test.


---
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-8728: Testcase to Verify if VR...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/684#issuecomment-144003970
  
Merged the testcase in the existing testcase 
test_03_restart_network_cleanup by adding support for basic setup.


---
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-8895: Verify if storage on sto...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8895: Verify if storage on storage pool can be attached to VM

Test case to verify if data volume uploaded in a storage pool(Cluster wide 
storage pool) is available for attachment to a Virtual Machine.and also check 
that after attachment the volume is in correct storage pool.

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8895-Verify-if-storage-can-be-selected-when-attaching-uploaded-data-volume-to-VM

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

https://github.com/apache/cloudstack/pull/869.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 #869


commit 48544cc7d007f05927bb291ee50871276e75
Author: Priti Sarap <priti.sa...@clogeny.com>
Date:   2015-09-22T07:41:44Z

CLOUDSTACK-8895: Verify if storage can be selected when attaching uploaded 
data volume to VM




---
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-8895: Verify if storage on sto...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/869#issuecomment-142205006
  
Test case result:

Attach Data Disk on CWPS To VM ... === TestName: 
test_01_attach_datadisk_to_vm_on_zwps | Status : SUCCESS ===
ok

--
Ran 1 test in 188.574s

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-8728: Testcase to Verify if VR...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/684#issuecomment-134488914
  
Merged the testcase in test_routers.py 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: CLOUDSTACK-8728: Testcase to Verify if VR...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/684#issuecomment-134489114
  
Testcase result:

Check if Virtual Router has same IP after network restart or not. ... === 
TestName: test_10_vr_has_same_ip | Status : SUCCESS ===
ok

--
Ran 1 test in 294.769s

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-8693-Adding-missing-code-in-te...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/668#issuecomment-130566681
  
Updated testpath to get single commit.


---
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-8728: Testcase to Verify if VR...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8728: Testcase to Verify if VRs IP 

Testcase to check if VR's IP changes if it is destroyed and re-created in 
basic zone.

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8728-Testcase-to-Verify-if-VRs-IP-changes-if-it-is-destroyed-and-re-created-in-Basic-Zone

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

https://github.com/apache/cloudstack/pull/684.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 #684


commit ea0d5ca141c3b73f212be37953b96b823b179600
Author: Priti Sarap priti.sa...@clogeny.com
Date:   2015-08-12T11:58:04Z

CLOUDSTACK:8728-Testcase to Verify if VRs IP changes if it is destroyed and 
re created in Basic Zone




---
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-8728: Testcase to Verify if VR...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/684#issuecomment-130273467
  
Testcase result:

Check if Virtual Router has same IP after network restart or not. ... === 
TestName: test_01_vr_has_same_ip | Status : SUCCESS ===
ok

--
Ran 1 test in 245.730s

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-8723: Verify API call listUsa...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8723: Verify API call listUsageRecords returns usage of new 
volume created after restore VM

After restoring a running VM current ROOT disk gets destroyed and new ROOT 
disk gets created.
This testcase is to check if volume usage of this newly created volume is 
listed in listUsageRecords API.


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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8723-Verify-API-call-listUsageRecords-returns-usage-of-new-volume-created-after-restore-VM

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

https://github.com/apache/cloudstack/pull/675.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 #675


commit c7e30029c17cc9c0d1f72803da481cc896ecc5e7
Author: Priti Sarap priti.sa...@clogeny.com
Date:   2015-08-11T08:45:13Z

CLOUDSTACK-8723: Verify API call listUsageRecords returns usage of new 
volume created after restore VM




---
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-8723: Verify API call listUsa...

[pritisarap12]

Github user pritisarap12 commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/675#discussion_r36734678
  
--- Diff: test/integration/testpaths/testpath_usage.py ---
@@ -2879,6 +2882,60 @@ def test_07_positive_tests_usage(self):
 # aggregation period and current period will give the network usage
 return
 
+@attr(tags=[advanced, basic], required_hardware=false)
+def test_08_checkNewVolumein_listUsageRecords(self):
+ Test case to check if new volume crated after
+restore VM is listed in listUsageRecords
+# 1. Launch a VM
+# 2. Restore the VM
+# 3. Check if the new volume created is listed in listUsageRecords 
API
+
+
+# Step 1
+vm_cluster = VirtualMachine.create(
+self.userapiclient,
+self.testdata[small],
+templateid=self.template.id,
+accountid=self.account.name,
+domainid=self.account.domainid,
+serviceofferingid=self.service_offering.id,
+zoneid=self.zone.id,
+)
+
+volumes_root_list = list_volumes(
+self.apiclient,
+virtualmachineid=vm_cluster.id,
+type='ROOT',
+listall=True
+)
+
+root_volume = volumes_root_list[0]
+
+# Step 2
+vm_cluster.restore(self.apiclient)
+
+qresultset = self.dbclient.execute(
+select id from volumes where name='%s' and state='Ready'; %
+root_volume.name)
+
+self.assertNotEqual(
+len(qresultset),
+0,
+Check DB Query result set
+)
+
+volumeCheck = Volume Id:  + str(qresultset[0][0]) +  usage time
+
+response = self.listUsageRecords(usagetype=6)
+self.assertEqual(response[0], PASS, response[1])
+UsageRecords = [record for record in response[1]
+if volumeCheck in record.description]
+# Step 3
+self.assertNotEqual(
--- End diff --

No, In step 3 we are verifying that UsageRecords should not be None.
or I have updated test case to fail it if UsageRecords  is empty.


---
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-8693: Adding missing code in t...

[pritisarap12]

Github user pritisarap12 closed the pull request at:

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


---
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-8693: Adding missing code in t...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/645#issuecomment-129316634
  
C


---
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-8693-Adding-missing-code-in-te...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8693-Adding-missing-code-in-testpath_same_vm_name.py_testpath

-Adding cls.hypervisor = cls.testClient.getHypervisorInfo()
-Fixed pep8 issues

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8693-Adding-missing-code-in-testpath_same_vm_name.py_testpatha

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

https://github.com/apache/cloudstack/pull/668.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 #668


commit 23ac5588fb354b9ba36c9e0096287c700a85775f
Author: Priti Sarap priti.sa...@clogeny.com
Date:   2015-08-10T06:00:55Z

CLOUDSTACK-8693-Adding-missing-code-in-testpath_same_vm_name.py_testpatha
-Adding cls.hypervisor = cls.testClient.getHypervisorInfo()




---
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-8717: Failed to start instance...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/667#issuecomment-129310396
  
Updated testcase with 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: CLOUDSTACK-8716: Verify creation of snaps...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/665#issuecomment-129314182
  
Updated testcase:
-Removed redundant code 
-Added validate_list function for list snapshot operation 


---
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-8716: Verify creation of snaps...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8716: Verify creation of snapshot from volume when the task is 
performed repeatedly in zone wide primary Storage

On VMWare with a Zone wide primary storage and more than two clusters 
verify successful creation of snapshot multiple times.


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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8716-Verify-creation-of-snapshot-from-volume-when-the-task-is-performed-repeatedly-in-zone-wide-primary-Storage

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

https://github.com/apache/cloudstack/pull/665.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 #665


commit 44ba3a652e3c13c79f65aafd3e68c4b72e1dc134
Author: Priti Sarap priti.sa...@clogeny.com
Date:   2015-08-07T11:48:42Z

CLOUDSTACK-8716: Verify creation of snapshot from volume when the task is 
performed repeatedly in zone wide primary Storage




---
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-8717: Failed to start instance...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8717: Failed to start instance after restoring the running 
instance 

On setup with two cluster wide primary storage verify restoring a running 
instance.(As while restoring instance root disk may get created on another 
primary storage.)


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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8717-Failed-to-start-instance-after-restoring-the-running-instance

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

https://github.com/apache/cloudstack/pull/667.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 #667


commit f163cbcb44d1c712deb5722f9623cb4675f01bf9
Author: Priti Sarap priti.sa...@clogeny.com
Date:   2015-08-07T12:27:54Z

CLOUDSTACK-8717: Failed to start instance after restoring the running 
instance




---
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-8686:Verify data disk attachme...

[pritisarap12]

Github user pritisarap12 commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/631#discussion_r36058447
  
--- Diff: test/integration/testpaths/testpath_attach_disk_zwps.py ---
@@ -0,0 +1,209 @@
+# 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.
+ Test case for Data Disk Attach to VM on ZWPS Test Path
+
+
+from nose.plugins.attrib import attr
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.utils import (cleanup_resources,
+  validateList)
+from marvin.lib.base import (Account,
+ ServiceOffering,
+ DiskOffering,
+ Volume,
+ VirtualMachine,
+ StoragePool
+ )
+from marvin.lib.common import (get_domain,
+   get_zone,
+   get_template
+   )
+
+from marvin.codes import (PASS,
+  ZONETAG1)
+
+
+class TestAttachDataDisk(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestAttachDataDisk, cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.testdata = testClient.getParsedTestDataConfig()
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+cls._cleanup = []
+cls.template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.testdata[ostype])
+cls.skiptest = False
+
+try:
+cls.pools = StoragePool.list(cls.apiclient, zoneid=cls.zone.id)
+except Exception as e:
+cls.skiptest = True
+return
+try:
+
+# Create an account
+cls.account = Account.create(
+cls.apiclient,
+cls.testdata[account],
+domainid=cls.domain.id
+)
+cls._cleanup.append(cls.account)
+
+# Create user api client of the account
+cls.userapiclient = testClient.getUserApiClient(
+UserName=cls.account.name,
+DomainName=cls.account.domain
+)
+# Create Service offering
+cls.service_offering_zone1 = ServiceOffering.create(
+cls.apiclient,
+cls.testdata[service_offering],
+tags=ZONETAG1
+)
+cls._cleanup.append(cls.service_offering_zone1)
+
+# Create Disk offering
+cls.disk_offering = DiskOffering.create(
+cls.apiclient,
+cls.testdata[disk_offering]
+)
+
+cls._cleanup.append(cls.disk_offering)
+
+except Exception as e:
+cls.tearDownClass()
+raise e
+return
+
+@classmethod
+def tearDownClass(cls):
+try:
+cleanup_resources(cls.apiclient, cls._cleanup)
+except Exception as e:
+raise Exception(Warning: Exception during cleanup : %s % e)
+
+def setUp(self):
+self.apiclient = self.testClient.getApiClient()
+self.dbclient = self.testClient.getDbConnection()
+self.cleanup = []
+
+def tearDown(self):
+try:
+for storagePool in self.pools:
+StoragePool.update(self.apiclient, id=storagePool.id, 
tags=)
+
+if hasattr(self, data_volume_created):
+data_volumes_list = Volume.list(
+self.userapiclient

[GitHub] cloudstack pull request: Cloudstack-8693: Adding missing code in t...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

Cloudstack-8693: Adding missing code in testpath same vm name.py testpath

Adding missing code to get hypervisor information

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8693-Adding-missing-code-in-testpath_same_vm_name.py_testpath

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

https://github.com/apache/cloudstack/pull/645.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 #645


commit a2df28d2f7f2596a121a53ee064250b62bb3b6c2
Author: Priti Sarap priti.sa...@clogeny.com
Date:   2015-07-15T09:46:24Z

CLOUDSTACK-8637: Verify that VM with same Display name creation fails if 
vm.instancename.flag=true

commit cb0e47f3eecd3328f6b87e1ec4d520d2d5704c20
Author: Priti Sarap priti.sa...@clogeny.com
Date:   2015-07-31T06:51:00Z

CLOUDSTACK-8693: Adding missing code in testpath_same_vm_name.py_testpath
 -Adding missing code to get hypervisor information




---
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-8689: Verify effect of changin...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/638#issuecomment-126606147
  
Modified test to check for empty vmList and updating maxguestlimit 
accordingly.
And the test result is-

Test hypervisor max limits effect ... === TestName: 
test_check_hypervisor_max_limit_effect | Status : SUCCESS ===
ok

--
Ran 1 test in 142.770s

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-8689: Verify effect of changin...

[pritisarap12]

Github user pritisarap12 commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/638#discussion_r35955311
  
--- Diff: test/integration/testpaths/testpath_hypervisor_limit.py ---
@@ -0,0 +1,198 @@
+# 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.
+ Test cases for Testing Max Hypervisor Limit
+
+from nose.plugins.attrib import attr
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.utils import (cleanup_resources,
+  validateList)
+from marvin.lib.base import (Account,
+ ServiceOffering,
+ VirtualMachine,
+ Host
+ )
+from marvin.lib.common import (get_domain,
+   get_zone,
+   get_template,
+   list_virtual_machines
+   )
+
+
+from marvin.cloudstackAPI import (updateHypervisorCapabilities,
+  listHypervisorCapabilities)
+
+from marvin.codes import PASS
+
+
+class TestMaxHyperviosrLimit(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestMaxHyperviosrLimit, cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.testdata = testClient.getParsedTestDataConfig()
+
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+# Get Zone, Domain and templates
+
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+
+cls.template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.testdata[ostype])
+
+cls._cleanup = []
+hostList = Host.list(cls.apiclient, zoneid=cls.zone.id, 
type=Routing)
+cls.host = Host(hostList[0].__dict__)
+Host.update(cls.apiclient, id=cls.host.id, hosttags=host1)
+try:
+cls.skiptest = False
+if cls.hypervisor.lower() not in ['xenserver']:
+cls.skiptest = True
+return
+
+# Create an account
+cls.account = Account.create(
+cls.apiclient,
+cls.testdata[account],
+domainid=cls.domain.id
+)
+
+# Create user api client of the account
+cls.userapiclient = testClient.getUserApiClient(
+UserName=cls.account.name,
+DomainName=cls.account.domain
+)
+# Create Service offering
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata[service_offering],
+hosttags=host1
+)
+
+cls._cleanup = [
+cls.account,
+cls.service_offering,
+]
+except Exception as e:
+cls.tearDownClass()
+raise e
+return
+
+@classmethod
+def tearDownClass(cls):
+try:
+cleanup_resources(cls.apiclient, cls._cleanup)
+except Exception as e:
+raise Exception(Warning: Exception during cleanup : %s % e)
+
+def setUp(self):
+self.cleanup = []
+if self.skiptest:
+self.skipTest(This test is to be checked on xenserver \
+only  Hence, skip for %s % self.hypervisor)
+
+self.apiclient = self.testClient.getApiClient()
+self.dbclient = self.testClient.getDbConnection()
+
+def tearDown(self):
+try:
+
+cmd = 
updateHypervisorCapabilities.updateHypervisorCapabilitiesCmd()
+cmd.id = self.hostCapId

[GitHub] cloudstack pull request: CLOUDSTACK-8689: Verify effect of changin...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/638#issuecomment-126636676
  
Updated the testcase for:
 -Count SSVM's on selected host
 -Count VR on the host
 -Count user VM's on the host
 -Then update the maxguestlimit for that host as(ssvm+vr+user_vm+1)
 -Added validateList for host list


---
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-8689: Verify effect of changin...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/638#issuecomment-126637367
  
Testcase Result-

Test hypervisor max limits effect ... === TestName: 
test_check_hypervisor_max_limit_effect | Status : SUCCESS ===
ok

Ran 1 test in 159.787s

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-8689: Verify effect of changin...

[pritisarap12]

Github user pritisarap12 commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/638#discussion_r35862152
  
--- Diff: test/integration/testpaths/testpath_hypervisor_limit.py ---
@@ -0,0 +1,191 @@
+# 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.
+
+ Test cases to Verify effect of changing value of XenServer
+Max guest limit on previously added hosts
+
+
+from nose.plugins.attrib import attr
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.utils import (cleanup_resources,
+validateList)
+from marvin.lib.base import (Account,
+ ServiceOffering,
+ VirtualMachine,
+ Host
+ )
+from marvin.lib.common import (get_domain,
+   get_zone,
+   get_template,
+  )
+
+
+from marvin.cloudstackAPI import (updateHypervisorCapabilities,
+  listHypervisorCapabilities)
+
+from marvin.codes import PASS
+
+
+
+class TestMaxHyperviosrLimit(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestMaxHyperviosrLimit, cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.testdata = testClient.getParsedTestDataConfig()
+
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+
+cls.template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.testdata[ostype])
+
+cls._cleanup = []
+hostList = Host.list(cls.apiclient, zoneid=cls.zone.id, 
type=Routing)
+cls.host = Host(hostList[0].__dict__)
+try:
+
+cls.skiptest = False
+if cls.hypervisor.lower() not in ['xenserver']:
+cls.skiptest = True
+return
+# Create an account
+cls.account = Account.create(
+cls.apiclient,
+cls.testdata[account],
+domainid=cls.domain.id
+)
+
+# Create user api client of the account
+cls.userapiclient = testClient.getUserApiClient(
+UserName=cls.account.name,
+DomainName=cls.account.domain
+)
+# Create Service offering
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata[service_offering],
+)
+
+cls._cleanup = [
+cls.account,
+cls.service_offering,
+]
+except Exception as e:
+cls.tearDownClass()
+raise e
+return
+
+@classmethod
+def tearDownClass(cls):
+try:
+cleanup_resources(cls.apiclient, cls._cleanup)
+except Exception as e:
+raise Exception(Warning: Exception during cleanup : %s % e)
+
+def setUp(self):
+self.cleanup = []
+if self.skiptest:
+self.skipTest(This test is to be checked on xenserver \
+only  Hence, skip for %s  % self.hypervisor)
+
+self.apiclient = self.testClient.getApiClient()
+self.dbclient = self.testClient.getDbConnection()
+
+def tearDown(self):
+try:
+
+cmd = 
updateHypervisorCapabilities.updateHypervisorCapabilitiesCmd()
+cmd.id = self.hostCapId
+cmd.maxguestslimit = self.originalLimit
+self.apiclient.updateHypervisorCapabilities(cmd

[GitHub] cloudstack pull request: CLOUDSTACK-8686:Verify data disk attachme...

[pritisarap12]

Github user pritisarap12 commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/631#discussion_r35947714
  
--- Diff: test/integration/testpaths/testpath_attach_disk_zwps.py ---
@@ -0,0 +1,185 @@
+# 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.
+ Test case for Data Disk Attach to VM on ZWPS Test Path
+
+
+from nose.plugins.attrib import attr
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.utils import (cleanup_resources,
+  validateList)
+from marvin.lib.base import (Account,
+ ServiceOffering,
+ DiskOffering,
+ Volume,
+ VirtualMachine,
+ StoragePool
+ )
+from marvin.lib.common import (get_domain,
+   get_zone,
+   get_template
+   )
+
+from marvin.codes import (PASS,
+  ZONETAG1)
+
+
+class TestAttachDataDisk(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestAttachDataDisk, cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.testdata = testClient.getParsedTestDataConfig()
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+cls._cleanup = []
+cls.template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.testdata[ostype])
+cls.skiptest = False
+
+try:
+cls.pools = StoragePool.list(cls.apiclient, zoneid=cls.zone.id)
+except Exception as e:
+cls.skiptest = True
+return
+try:
+
+# Create an account
+cls.account = Account.create(
+cls.apiclient,
+cls.testdata[account],
+domainid=cls.domain.id
+)
+cls._cleanup.append(cls.account)
+
+# Create user api client of the account
+cls.userapiclient = testClient.getUserApiClient(
+UserName=cls.account.name,
+DomainName=cls.account.domain
+)
+# Create Service offering
+cls.service_offering_zone1 = ServiceOffering.create(
+cls.apiclient,
+cls.testdata[service_offering],
+tags=ZONETAG1
+)
+cls._cleanup.append(cls.service_offering_zone1)
+
+# Create Disk offering
+cls.disk_offering = DiskOffering.create(
+cls.apiclient,
+cls.testdata[disk_offering]
+)
+
+cls._cleanup.append(cls.disk_offering)
+
+except Exception as e:
+cls.tearDownClass()
+raise e
+return
+
+@classmethod
+def tearDownClass(cls):
+try:
+cleanup_resources(cls.apiclient, cls._cleanup)
+except Exception as e:
+raise Exception(Warning: Exception during cleanup : %s % e)
+
+def setUp(self):
+self.apiclient = self.testClient.getApiClient()
+self.dbclient = self.testClient.getDbConnection()
+self.cleanup = []
+
+def tearDown(self):
+try:
+if hasattr(self, data_volume_created):
+data_volumes_list = Volume.list(
+self.userapiclient,
+id=self.data_volume_created.id,
+virtualmachineid=self.vm.id

[GitHub] cloudstack pull request: CLOUDSTACK-8689: Verify effect of changin...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8689: Verify effect of changing value of XenServer Max guest 
limit

Verify effect of changing value of Max guest limit of XenServer on 
previously added hosts
-Check if it reads changed value and act accordingly

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8689-Verify-effect-of-changing-value-of-XenServer-Max-guest-limitis-on-previously-added-hosts

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

https://github.com/apache/cloudstack/pull/638.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 #638


commit e2c578f7fa29404213a26c80458e1e51fc494759
Author: Priti Sarap priti.sa...@clogeny.com
Date:   2015-07-30T07:30:39Z

CLOUDSTACK-8689: Verify effect of changing value of XenServer Max guest 
limitis on previously added hosts




---
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-8686:Verify data disk attachme...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8686:Verify data disk attachment for clusters with only zone 
wide primary

-Attaching the uploaded/allocated volume to a VM on zwps

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8686-Data-disk-attach-failed-for-clusters-with-only-zone-wide-primary

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

https://github.com/apache/cloudstack/pull/631.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 #631


commit 608fff5412a6cbb4a7f19849c61b69c049cc3e4e
Author: Priti Sarap priti.sa...@clogeny.com
Date:   2015-07-29T06:14:59Z

CLOUDSTACK-8686: Data disk attach failed for clusters with only zone wide 
primary
-Attaching the uploaded/allocated volume to a VM on zwps




---
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-8664: Verify if queryAsyncJobR...

[pritisarap12]

Github user pritisarap12 commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/619#discussion_r35508814
  
--- Diff: test/integration/testpaths/testpath_queryAsyncJobResult.py ---
@@ -0,0 +1,133 @@
+# 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.
+ Test case to check if queryAsyncJobResult returns jobinstanceid 
+
+from nose.plugins.attrib import attr
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.utils import (cleanup_resources)
+from marvin.lib.base import (Account,
+ ServiceOffering,
+ VirtualMachine,
+ )
+from marvin.lib.common import (get_domain,
+   get_zone,
+   get_template,
+  )
+
+from marvin.cloudstackAPI import queryAsyncJobResult
+
+class TestJobinstanceid(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestJobinstanceid, cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.testdata = testClient.getParsedTestDataConfig()
+
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+
+cls.template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.testdata[ostype])
+
+cls._cleanup = []
+
+try:
+
+# Create an account
+cls.account = Account.create(
+cls.apiclient,
+cls.testdata[account],
+domainid=cls.domain.id
+)
+# Create user api client of the account
+cls.userapiclient = testClient.getUserApiClient(
+UserName=cls.account.name,
+DomainName=cls.account.domain
+)
+
+   # Create Service offering
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata[service_offering],
+)
+
+cls._cleanup = [
+cls.account,
+cls.service_offering,
+]
+except Exception as e:
+cls.tearDownClass()
+raise e
+return
+
+
+@classmethod
+def tearDownClass(cls):
+try:
+cleanup_resources(cls.apiclient, cls._cleanup)
+except Exception as e:
+raise Exception(Warning: Exception during cleanup : %s % e)
+
+def setUp(self):
+self.apiclient = self.testClient.getApiClient()
+self.dbclient = self.testClient.getDbConnection()
+self.cleanup = []
+
+def tearDown(self):
+try:
+cleanup_resources(self.apiclient, self.cleanup)
+except Exception as e:
+raise Exception(Warning: Exception during cleanup : %s % e)
+return
+
+@attr(tags=[advanced, basic], required_hardware=false)
+def test_queryAsyncJobResult_jobinstanceid(self):
+ Test queryAsyncJobResult api return jobinstanceid
+
+# 1. Deploy a VM 
+# 2. Call queryAsyncJobResult API with jobid of previous step  
+# 3. Verify that queryAsyncJobResult returns jobinstanceid
+
+
+# Step 1
+
--- End diff --

Fixed pep8 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-8659: Verify presentation of v...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/613#issuecomment-123986839
  
Updated the testcase as per 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: CLOUDSTACK-8664: Verify if queryAsyncJobR...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8664: Verify if queryAsyncJobResult api return jobinstanceid



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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8664-Verify-if-queryAsyncJobResult-api-return-jobinstanceid

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

https://github.com/apache/cloudstack/pull/619.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 #619


commit 292734324c81f8fd1fa92606aa42e73dc583c15c
Author: Priti Sarap priti.sa...@clogeny.com
Date:   2015-07-23T09:38:35Z

CLOUDSTACK-8664: Verify if queryAsyncJobResult api return jobinstanceid




---
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-8659: Verify presentation of v...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8659: Verify presentation of volume ID in events table

Verify if events table records UUID of the volume instead of volume ID in 
description from which snapshot is created for 'SNAPSHOT.CREATE' type .

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8659-Verify-presentation-of-volume-id-in-description-of-events-table-for-SNAPSHOT.CREATE-type

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

https://github.com/apache/cloudstack/pull/613.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 #613


commit a634ad2e3dcbc1ec668fdc135db9891717bf8a7d
Author: Priti Sarap priti.sa...@clogeny.com
Date:   2015-07-22T07:24:46Z

CLOUDSTACK-8659: Verify presentation of volume id in description of events 
table for 'SNAPSHOT.CREATE' type




---
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-8637: Verify that VM with same...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8637: Verify that VM with same Display name creation fails if 
vm.instancename.flag=true

When vm.instancename.flag=true it will only set the instancename without 
any additional prefix so name of the instance on host will also be same hence 
verify that when tried to create another instance with same name on same host 
should fail on VMWare as it should have every instance with unique name.

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8637-Verify-that-cloudstack-gives-error-on-creation-of-VMs-with-the-same-Display-name-when-vm.instancename.flag=true

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

https://github.com/apache/cloudstack/pull/594.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 #594


commit a2df28d2f7f2596a121a53ee064250b62bb3b6c2
Author: Priti Sarap priti.sa...@clogeny.com
Date:   2015-07-15T09:46:24Z

CLOUDSTACK-8637: Verify that VM with same Display name creation fails if 
vm.instancename.flag=true




---
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-8636: Verify failure of creati...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8636: Verify failure of creation of Custom disk offering with 
disksize parameter

Custom disk creation should not take disksize parameter while creating the 
disk offering

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8636-Verify-failure-of-creation-of-Custom-disk-offering-with-disksize-parameter

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

https://github.com/apache/cloudstack/pull/592.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 #592


commit b3c01f8bc3f46b65cb996373cd962392d4588a92
Author: Priti Sarap priti.sa...@clogeny.com
Date:   2015-07-15T06:56:45Z

CLOUDSTACK-8636: Verify failure of creation of Custom disk offering with 
disksize parameter




---
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-8632: Unable to revert snapsho...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8632: Unable to revert snapshot on XenServer

On XenServer check that volume snapshot fails for the ROOT volume of VM 
which has VM snapshot created. 

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8632-Unable-to-revert-snapshot-on-XenServer

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

https://github.com/apache/cloudstack/pull/590.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 #590


commit e5aee72fd4940e4bd2a90f690e5ed033b54f1821
Author: Gaurav Aradhye gaurav.arad...@clogeny.com
Date:   2015-07-15T05:45:47Z

CLOUDSTACK-8632: Unable to revert snapshot on XenServer




---
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-8588 - Remove redundant skip t...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8588 - Remove redundant skip test for LXC



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

$ git pull https://github.com/pritisarap12/cloudstack 
removeRedundantSkipTest

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

https://github.com/apache/cloudstack/pull/526.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 #526


commit 90b1b34ea19c1cb7dd88e2c7c8fb57b90f89b784
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-06-25T08:47:02Z

CLOUDSTACK-8588 - Remove redundant skip test for LXC




---
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-8556: Unable to delete attache...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/481#issuecomment-115109007
  
Delete volume was failing in cleanup before because volume was not in 
detached state as snapshot creation failed and detach volume command did not 
get executed so volume remains in attached state.
Now the detach volume is executed in tearDown class itself so clean up does 
not fail for volume.


---
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-8576: Skip tests for LXC

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

 CLOUDSTACK-8576: Skip tests for LXC

Skipping testcases for LXC as it doesn't support snapshots and templates 
creation also excluded unittest as it was not used.

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

$ git pull https://github.com/pritisarap12/cloudstack resourceLimits

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

https://github.com/apache/cloudstack/pull/506.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 #506


commit 5c8922e640e7ebe5dedc378779ef465987606ef7
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-06-22T11:14:07Z

 CLOUDSTACK-8576: Skip tests as snapshots and template are not supported on 
LXC




---
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-8572:Unable to deploy VM as no...

[pritisarap12]

Github user pritisarap12 closed the pull request at:

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


---
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-8574: Skip testcases for LXC i...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8574: Skip testcases for LXC if storagePool type is not RBD

As LXC DATA disk is supported only on RBD StoragePool so skip testcases for 
LXC hypervisor if StoragePool type is not RBD 

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

$ git pull https://github.com/pritisarap12/cloudstack lxcRBD

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

https://github.com/apache/cloudstack/pull/488.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 #488


commit 35d7b745193f6f0f7ad9181b01860119222e9a45
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-06-19T10:41:46Z

CLOUDSTACK-8574: Skip testcases including data disk creation for LXC if 
storagePool type is not RBD




---
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-8556: Unable to delete attache...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/481#issuecomment-113485973
  
Removed reboot as it is not needed.
Below is the test result after removing reboot function:

Storage and Snapshot Limit ... === TestName: 
test_01_storage_snapshots_limits | Status : SUCCESS ===
ok

--
Ran 1 test in 300.585s

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-8308:Adding automation testcas...

[pritisarap12]

Github user pritisarap12 commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/384#discussion_r32823312
  
--- Diff: test/integration/testpaths/testpath_delta_snapshots.py ---
@@ -0,0 +1,516 @@
+# 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.
+ Test cases for Delta Snapshots Test Path
+
+
+from nose.plugins.attrib import attr
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.utils import (cleanup_resources,
+  validateList,
+  is_snapshot_on_nfs)
+from marvin.lib.base import (Account,
+ ServiceOffering,
+ Template,
+ VirtualMachine,
+ Volume,
+ Configurations,
+ Snapshot
+ )
+from marvin.lib.common import (get_domain,
+   get_zone,
+   get_template,
+   list_volumes,
+   createChecksum,
+   compareChecksum
+   )
+from marvin.sshClient import SshClient
+from marvin.codes import (FAIL)
+import time
+
+
+def checkIntegrityOfSnapshot(
+self, snapshotsToRestore, checksumToCompare, disk_type=root):
+
+if disk_type == root:
+# Create template from snapshot
+template_from_snapshot = Template.create_from_snapshot(
+self.apiclient,
+snapshotsToRestore,
+self.testdata[template_2])
+
+self.assertNotEqual(
+template_from_snapshot,
+None,
+Check if result exists in list item call
+)
+
+time.sleep(60)
+
+# Deploy VM
+vm_from_temp = VirtualMachine.create(
+self.apiclient,
+self.testdata[small],
+templateid=template_from_snapshot.id,
+accountid=self.account.name,
+domainid=self.account.domainid,
+serviceofferingid=self.service_offering.id,
+zoneid=self.zone.id,
+mode=self.zone.networktype
+)
+
+self.assertNotEqual(
+vm_from_temp,
+None,
+Check if result exists in list item call
+)
+time.sleep(60)
+# Verify contents of ROOT disk match with snapshot
+
+compareChecksum(
+self.apiclient,
+service=self.testdata,
+original_checksum=checksumToCompare,
+disk_type=rootdiskdevice,
+virt_machine=vm_from_temp
+)
+
+vm_from_temp.delete(self.apiclient)
+template_from_snapshot.delete(self.apiclient)
+else:
+volumeFormSnap = Volume.create_from_snapshot(
+self.apiclient,
+snapshotsToRestore.id,
+self.testdata[volume],
+account=self.account.name,
+domainid=self.account.domainid,
+zoneid=self.zone.id
+)
+
+temp_vm = VirtualMachine.create(
+self.apiclient,
+self.testdata[small],
+templateid=self.template.id,
+accountid=self.account.name,
+domainid=self.account.domainid,
+serviceofferingid=self.service_offering.id,
+zoneid=self.zone.id,
+mode=self.zone.networktype
+)
+temp_vm.attach_volume(
+self.apiclient,
+volumeFormSnap
+)
+
+temp_vm.reboot(self.apiclient)
+
+compareChecksum(
+self.apiclient,
+self.testdata,
+checksumToCompare

[GitHub] cloudstack pull request: CLOUDSTACK-8572:Unable to deploy VM as no...

[pritisarap12]

Github user pritisarap12 commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/480#discussion_r32804688
  
--- Diff: test/integration/testpaths/testpath_vmlc.py ---
@@ -176,6 +176,19 @@ def setUpClass(cls):
 cls.zone = get_zone(cls.apiclient)
 cls._cleanup = []
 
+cls.skiptest = False
--- End diff --

This test does not require any special storage pool or with any tags. This 
check was put since it did not find any storage pool in UP state.i.e storage 
pool was present but was not in UP state.
So should I remove the check for skipTest?


---
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-8572:Unable to deploy VM as no...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8572:Unable to deploy VM as no storage pool found in UP state



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

$ git pull https://github.com/pritisarap12/cloudstack shivani

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

https://github.com/apache/cloudstack/pull/480.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 #480


commit b3dab6031adbcb5f08b75e2ccb44874cf0ee688b
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-06-18T08:40:40Z

CLOUDSTACK-8572:Unable to deploy VM as no storage pool found in UP state




---
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-8556: Unable to delete attache...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8556: Unable to delete attached volume in cleanup



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

$ git pull https://github.com/pritisarap12/cloudstack deleteVolume

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

https://github.com/apache/cloudstack/pull/481.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 #481


commit 9f13d8ebc9e4db6ac2ea39122b7406f809ccc591
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-06-18T09:03:03Z

CLOUDSTACK-8556: Unable to delete attached volume in cleanup




---
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-8555:Skip resize volume operat...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8555:Skip resize volume operation for HyperV

Skipping resize volume operation for HyperV as it doesn't support the 
operation.
It is in extension with https://github.com/apache/cloudstack/pull/391/files 

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

$ git pull https://github.com/pritisarap12/cloudstack SkipTestforresize

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

https://github.com/apache/cloudstack/pull/465.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 #465


commit 7441863ac32f5244519b2a8a26aef2231c8a490e
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-06-16T10:46:44Z

CLOUDSTACK-8555:Skip resize volume operation for HyperV




---
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-8555:Skip testcase for HyperV ...

[pritisarap12]

Github user pritisarap12 commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/391#discussion_r32393907
  
--- Diff: test/integration/component/test_browse_volumes.py ---
@@ -2028,6 +2028,10 @@ def test_01_Browser_volume_Life_cycle_tpath(self):
 
 Test Browser_volume_Life_cycle - This includes upload 
volume,attach to a VM, write data ,Stop ,Start, Reboot,Reset  of a VM, 
detach,attach back to the VM, delete volumes  
 
+if self.hypervisor.lower() == 'hyperv':
--- End diff --

Yes, we can skip only resize operation and done with modification of the 
code doing the same.


---
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-8555:Skip testcase for HyperV ...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/391#issuecomment-111934294
  
Done with the review changes.


---
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-8555:Skip testcase for HyperV ...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8555:Skip testcase for HyperV as it doesn't support volume 
resize operation



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

$ git pull https://github.com/pritisarap12/cloudstack test_browse_volume

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

https://github.com/apache/cloudstack/pull/391.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 #391


commit 190171998a02facac513e4a1335b26bfd5338739
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-06-12T08:55:49Z

CLOUDSTACK-8555:Skip testcase for HyperV as it doesn't support volume 
resize operation




---
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-8553:Unable to launch VM from ...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8553:Unable to launch VM from template because of permission 
issue

Non-root user failed to create VM from a template created by Root user

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

$ git pull https://github.com/pritisarap12/cloudstack launch-vm-permission

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

https://github.com/apache/cloudstack/pull/387.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 #387


commit 913661303aa0b11258bdde7d13c8e5daceef2031
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-06-11T08:45:06Z

CLOUDSTACK-8553:Unable to launch VM from template because of permission 
issue




---
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-8308:Adding automation testcas...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8308:Adding automation testcases for Delta Snapshot testpath



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

$ git pull https://github.com/pritisarap12/cloudstack delta-snapshot

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

https://github.com/apache/cloudstack/pull/384.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 #384


commit e05f97bf900bfa9ddf66bbd7f816687597879462
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-06-11T06:38:46Z

CLOUDSTACK-8308:Adding automation testcases for Delta Snapshot




---
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-8550 - Attempt to delete alrea...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8550 - Attempt to delete already deleted VM in tearDownClass()



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

$ git pull https://github.com/pritisarap12/cloudstack 
unable-to-delete-vm-hardning

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

https://github.com/apache/cloudstack/pull/385.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 #385


commit 6373d9d28a3975bb34b78a3bd6cb4d8c7d77c2ea
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-06-11T06:57:51Z

CLOUDSTACK-8550 - Attempt to delete already deleted VM




---
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: UDSTACK-8549:Update test_concurrent_snaps...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

UDSTACK-8549:Update test_concurrent_snapshots_limits.py asesrt statement

Update test_concurrent_snapshots_limits.py to check if the output(config 
value) is list or not only if it is not Null

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

$ git pull https://github.com/pritisarap12/cloudstack snapshot_cunc

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

https://github.com/apache/cloudstack/pull/386.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 #386


commit c7afddd210bbc070baea4f83613942b0ed4ac9a4
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-06-11T08:20:26Z

UDSTACK-8549-Update test_concurrent_snapshots_limits.py asesrt statement




---
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-8547: Modify hypervisor check ...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8547: Modify hypervisor check in testpath_snapshot_hardning.py

--Modify hypervisor check to skip testcase for any hypervisor
   other than KVM

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

$ git pull https://github.com/pritisarap12/cloudstack 
my-testcases-regression

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

https://github.com/apache/cloudstack/pull/378.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 #378


commit ff01687a5d3b9fdc432b85a817e4e2f693380e25
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-06-10T07:01:16Z

CLOUDSTACK-8547 - Modify hypervisor check in testpath_snapshot_hardning.py 
testpath
--Modify hypervisor check to skip testcase for any hypervisor
  other than 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: Cloudstack 8308 Adding automation test ca...

[pritisarap12]

Github user pritisarap12 closed the pull request at:

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


---
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-8549:Update assert statements ...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8549:Update assert statements in 
testpath_disable_enable_zone.py…

--Update the assertNotEquals for verifying the state of VM's to
assertEqual(running)
-- Also added missing VM deployment by admin in disabled zone state

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

$ git pull https://github.com/pritisarap12/cloudstack disable_not_equal

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

https://github.com/apache/cloudstack/pull/382.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 #382


commit 08c8d0a253401a9a701ac8e3a1ef52bb7209b52d
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-06-10T09:48:53Z

CLOUDSTACK-8549:Update assert statements in testpath_disable_enable_zone.py 
testpath
--Update the assertNotEquals for verifying the state of VM's to
assertEqual(running)




---
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: Disable enable Zone Pod Cluster Host

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/356#issuecomment-109176867
  
Done with rebased with upstream master. 


---
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 8308 Adding automation test ca...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

Cloudstack 8308 Adding automation test cases for vm/volume snapshot hardning

Updating testcase as per the discussion that while volume snapshot is in 
progress that volume 
can not be attached to a VM in different cluster 

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-hardning

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

https://github.com/apache/cloudstack/pull/359.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 #359


commit 93a59f52a7197316551c3ddcfa93fff7825d3037
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-05-26T07:28:18Z


CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-hardning:
--Automating testcases for volume snapshot hardning testpath

commit 4cfa771b6e1db392bf2f2d4b2985d8769aa7c937
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-06-05T08:41:51Z

CLOUDSTACK-8308-Add test cases for volume/VM snapshot test path:
--Modifying the testpath as per the discussion that while Volume
snapshot is in progress that volume can not be attached to VM in
different Cluster




---
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: Disable enable Zone Pod Cluster Host

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

Disable enable Zone Pod Cluster Host

Updating the testcase as Admin user is also not able to deploy new VM on 
Host in disabled state

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

$ git pull https://github.com/pritisarap12/cloudstack 
Disable-Enable-Zone-Pod-Cluster-1

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

https://github.com/apache/cloudstack/pull/356.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 #356


commit 69193d588fb5713b2d419ffe906f5f2239942f3e
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-05-22T11:58:40Z

CLOUDSTACK-8476-Disable enable zone pod cluster and host:
--Test cases for testing the behavior of resources running on
zone, cluster, pod, host and admin/non-admin user after disabling
the zone, cluster, pod, host respectively

commit 983945be0b65fd0bd54c42795576e859765822f3
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-05-25T05:40:30Z

Disable enable zone pod cluster and host:
--Adding missing strings in codes.py

commit 2524d0b0edc3ae5359feacf75868a228ca02a1fa
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-05-26T07:35:27Z

Disable-Enable-Zone-Pod-Cluster-1:
--Modified the string comparison by using lowercase string

commit c5c30f6c0c24224c67c7b249f81e46ded0935d4f
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-06-04T09:31:08Z

Disable-Enable-Zone-Pod-Cluster:
--Updating Disable/enable host testpath as admin user should not be
able to deploy vm on disabled host




---
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: Local variable accessed as a class variab...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

 Local variable accessed as a class variable

Modified test_lb_secondary_ip.py in which local variable was accessed as 
class variable

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

$ git pull https://github.com/pritisarap12/cloudstack 
Regrassion-testing-test-lb-secondary-ip

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

https://github.com/apache/cloudstack/pull/344.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 #344


commit 1216156abca712c983151ba0d58ac2d34f9ca1d9
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-06-02T07:49:19Z

Modified test_lb_secondary_ip.py file:
Local variable accessed as class variable




---
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: Modification in setupClass to skip testca...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/343#issuecomment-107824912
  
Done with the review changes.


---
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: Modification in setupClass to skip testca...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

Modification in setupClass to skip testcases rather than throwing exception 

Skip the testcase which is throwing an exception on Exceeding connection 
limit to NetScaler device

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

$ git pull https://github.com/pritisarap12/cloudstack 
Regrassion-testing-test_netscaler_lb_algo

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

https://github.com/apache/cloudstack/pull/343.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 #343


commit 3dcfa82de109264a724b5aeca0809271295727a6
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-06-02T05:34:34Z

Modification in setupClass to skip testcases rather than throwing an
exception on Exceeding connection limit to Netscaler device




---
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: Disable enable zone pod cluster and host

[pritisarap12]

Github user pritisarap12 closed the pull request at:

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


---
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: UDSTACK-8513-Fixed-VM-Deployment-Issue:

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

UDSTACK-8513-Fixed-VM-Deployment-Issue:

--Fixed the vm deployment issue

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8513-Fixed-VM-Deployment-Issue

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

https://github.com/apache/cloudstack/pull/312.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 #312


commit 10ff4804fbf20abcde5c17a2b5f72da5a9c1ce6c
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-05-27T06:22:27Z

UDSTACK-8513-Fixed-VM-Deployment-Issue:
--Fixed the vm deployment issue




---
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-8476-Disable enable host:

[pritisarap12]

Github user pritisarap12 closed the pull request at:

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


---
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-8476-Disable enable pod:

[pritisarap12]

Github user pritisarap12 closed the pull request at:

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


---
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: Fixed VM Deployment Issue in(integration....

[pritisarap12]

Github user pritisarap12 closed the pull request at:

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


---
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: Disable enable zone pod cluster and host

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/285#issuecomment-105429711
  
Done with review changes.


---
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-8308-Adding-automation-test-ca...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-h…

…ardning:

--Automating testcases for volume snapshot hardning testpath

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-hardning

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

https://github.com/apache/cloudstack/pull/301.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 #301


commit 93a59f52a7197316551c3ddcfa93fff7825d3037
Author: pritisarap12 priti.sa...@clogeny.com
Date:   2015-05-26T07:28:18Z


CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-hardning:
--Automating testcases for volume snapshot hardning testpath




---
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.
---