Merge pull request apache#869 from pritisarap12/CLOUDSTACK-8895-Verify-if-storage-can-be-selected-when-attaching-uploaded-data-volume-to-VM

sanjeev · sanjeev · commit f61f23c7fa2c · 2016-02-10T10:57:19.000+05:30
CLOUDSTACK-8895: Verify if storage on storage pool can be attached to VMTest 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.

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

Signed-off-by: sanjeev &lt;sanjeev@apache.org&gt;
diff --git a/test/integration/testpaths/testpath_attach_disk_zwps.py b/test/integration/testpaths/testpath_attach_disk_zwps.py
@@ -34,7 +34,8 @@
                                )
 
 from marvin.codes import (PASS,
-                          ZONETAG1)
+                          ZONETAG1,
+                          CLUSTERTAG1)
 
 
 class TestAttachDataDisk(cloudstackTestCase):
@@ -207,3 +208,188 @@ def test_01_attach_datadisk_to_vm_on_zwps(self):
             "Check: Data if Disk is attached to VM")
 
         return
+
+
+class TestAttachDataDiskOnCWPS(cloudstackTestCase):
+
+    @classmethod
+    def setUpClass(cls):
+        testClient = super(TestAttachDataDiskOnCWPS, 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,
+                scope="CLUSTER")
+        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 = ServiceOffering.create(
+                cls.apiclient,
+                cls.testdata["service_offering"],
+            )
+            cls._cleanup.append(cls.service_offering)
+
+            # Create Disk offering
+            cls.disk_offering = DiskOffering.create(
+                cls.apiclient,
+                cls.testdata["disk_offering"],
+                custom=True,
+                tags=CLUSTERTAG1,
+            )
+
+            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,
+                    id=self.data_volume_created.id,
+                    virtualmachineid=self.vm.id
+                )
+                if data_volumes_list:
+                    self.vm.detach_volume(
+                        self.userapiclient,
+                        data_volumes_list[0]
+                    )
+
+                status = validateList(data_volumes_list)
+                self.assertEqual(
+                    status[0],
+                    PASS,
+                    "DATA Volume List Validation Failed")
+
+            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="true")
+    def test_01_attach_datadisk_to_vm_on_zwps(self):
+        """ Attach Data Disk on CWPS To VM 
+            1.  Check if zwps storage pool exists.
+            2.  Adding tag to zone wide primary storage
+            3.  Launch a VM
+            4.  Attach data disk to vm.
+            5.  Verify disk is attached and in correct storage pool.
+        """
+
+        # Step 1
+        if len(list(self.pools)) < 1:
+            self.skipTest("There must be at least one zone wide \
+                storage pools available in the setup")
+
+        # Step 2
+        # Adding tags to Storage Pools
+        StoragePool.update(
+            self.apiclient,
+            id=self.pools[0].id,
+            tags=[CLUSTERTAG1])
+
+        # Launch VM
+        self.vm = VirtualMachine.create(
+            self.apiclient,
+            self.testdata["small"],
+            templateid=self.template.id,
+            accountid=self.account.name,
+            domainid=self.account.domainid,
+            serviceofferingid=self.service_offering_zone1.id,
+            zoneid=self.zone.id
+        )
+
+        self.testdata["volume"]["zoneid"] = self.zone.id
+        self.testdata["volume"]["customdisksize"] = 1
+        self.data_volume_created = Volume.create_custom_disk(
+            self.userapiclient,
+            self.testdata["volume"],
+            account=self.account.name,
+            domainid=self.account.domainid,
+            diskofferingid=self.disk_offering.id,
+        )
+
+        self.cleanup.append(self.data_volume_created)
+
+        # Step 4
+        self.vm.attach_volume(
+            self.userapiclient,
+            self.data_volume_created
+        )
+
+        data_volumes_list = Volume.list(
+            self.userapiclient,
+            virtualmachineid=self.vm.id,
+            type="DATA",
+            listall=True
+        )
+
+        self.debug("list volumes using vm id %s" % dir(data_volumes_list[0]))
+
+        data_volumes_list = Volume.list(self.apiclient,
+                                        id=self.data_volume_created.id,
+                                        listall=True)
+        data_volume = data_volumes_list[0]
+        status = validateList(data_volume)
+        # Step 5
+        self.assertEqual(
+            status[0],
+            PASS,
+            "Check: volume list is valid")
+
+        self.assertEqual(
+            data_volume.state,
+            "Ready",
+            "Check: Data volume is attached to VM")
+
+        if data_volume.storage != self.pools[0].name:
+            self.fail("check if volume is created in correct storage pool")
+        return
