Addressed comments

Author: slavkap

test/integration/plugins/storpool/test_snapshot_copy_on_primary_storage.py

@@ -162,6 +162,7 @@ def test_01_take_snapshot_multi_zone(self):
         """
 
         snapshot = Snapshot.create(self.userapiclient, volume_id=self.volume.id, zoneids=[str(self.additional_zone.id)], usestoragereplication=True)
+        # the snapshot has to be deleted when the class cleanup is done. Otherwise use Time.sleep(420) ans self.cleanup to clean it after each test. This will increase the tests execution
         self._cleanup.append(snapshot)
         self.snapshot_id = snapshot.id
         self.helper.verify_snapshot_copies(self.userapiclient, self.snapshot_id, [self.zone.id, self.additional_zone.id])
@@ -174,6 +175,7 @@ def test_02_copy_snapshot_multi_pools(self):
         """
 
         snapshot = Snapshot.create(self.userapiclient, volume_id=self.volume.id)
+        # the snapshot has to be deleted when the class cleanup is done. Otherwise use Time.sleep(420) ans self.cleanup to clean it after each test. This will increase the tests execution
         self._cleanup.append(snapshot)
         self.snapshot_id = snapshot.id
         Snapshot.copy(self.userapiclient, self.snapshot_id, zone_ids=[str(self.additional_zone.id)], source_zone_id=self.zone.id, usestoragereplication=True)
@@ -220,6 +222,7 @@ def test_05_take_snapshot_multi_zone_create_volume_additional_zone(self):
         """
 
         snapshot = Snapshot.create(self.userapiclient,volume_id=self.volume.id, zoneids=[str(self.additional_zone.id)], usestoragereplication=True)
+        # the snapshot has to be deleted when the class cleanup is done. Otherwise use Time.sleep(420) ans self.cleanup to clean it after each test. This will increase the tests execution
         self._cleanup.append(snapshot)
         self.snapshot_id = snapshot.id
         self.helper.verify_snapshot_copies(self.userapiclient, self.snapshot_id, [self.zone.id, self.additional_zone.id])
@@ -246,6 +249,7 @@ def test_06_take_snapshot_multi_zone_create_template_additional_zone(self):
         """Test to take volume snapshot in multiple StorPool primary storages in diff zones and create a volume in one of the additional zones
         """
         snapshot = Snapshot.create(self.userapiclient, volume_id=self.volume.id, zoneids=[str(self.additional_zone.id)], usestoragereplication=True)
+        # the snapshot has to be deleted when the class cleanup is done. Otherwise use Time.sleep(420) ans self.cleanup to clean it after each test. This will increase the tests execution
         self._cleanup.append(snapshot)
         self.snapshot_id = snapshot.id
         self.helper.verify_snapshot_copies(self.userapiclient, self.snapshot_id, [self.zone.id, self.additional_zone.id])
@@ -261,6 +265,7 @@ def test_07_take_snapshot_multi_zone_deploy_vm_additional_zone(self):
         """Test to take volume snapshot in multiple StorPool primary storages in diff zones and deploy a VM from snapshot in one of the additional zones
         """
         snapshot = Snapshot.create(self.userapiclient, volume_id=self.volume.id, zoneids=[str(self.additional_zone.id)], usestoragereplication=True)
+        # the snapshot has to be deleted when the class cleanup is done. Otherwise use Time.sleep(420) ans self.cleanup to clean it after each test. This will increase the tests execution
         self._cleanup.append(snapshot)
         self.snapshot_id = snapshot.id
         self.helper.verify_snapshot_copies(self.userapiclient, self.snapshot_id, [self.zone.id, self.additional_zone.id])