Return-Path: X-Original-To: apmail-cloudstack-dev-archive@www.apache.org Delivered-To: apmail-cloudstack-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 27CB618FE2 for ; Mon, 14 Dec 2015 06:53:49 +0000 (UTC) Received: (qmail 51977 invoked by uid 500); 14 Dec 2015 06:53:48 -0000 Delivered-To: apmail-cloudstack-dev-archive@cloudstack.apache.org Received: (qmail 51920 invoked by uid 500); 14 Dec 2015 06:53:48 -0000 Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cloudstack.apache.org Delivered-To: mailing list dev@cloudstack.apache.org Received: (qmail 51908 invoked by uid 99); 14 Dec 2015 06:53:48 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 14 Dec 2015 06:53:48 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 587CBDFBDB; Mon, 14 Dec 2015 06:53:48 +0000 (UTC) From: pritisarap12 To: dev@cloudstack.apache.org Reply-To: dev@cloudstack.apache.org References: In-Reply-To: Subject: [GitHub] cloudstack pull request: CLOUDSTACK-9128: Testcase to verify physi... Content-Type: text/plain Message-Id: <20151214065348.587CBDFBDB@git1-us-west.apache.org> Date: Mon, 14 Dec 2015 06:53:48 +0000 (UTC) Github user pritisarap12 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1199#discussion_r47465693 --- Diff: test/integration/testpaths/testpath_snapshot_limits.py --- @@ -358,3 +362,129 @@ def test_01_storage_snapshots_limits(self): ) return + + @attr(tags=["advanced", "basic"], required_hardware="true") + def test_02_snapshot_size_check(self): + """ Check Snapshots size in database + 1. Create file on ROOT disk of deployed VM. + 2. Create Snapshot of ROOT disk. + 3. Check if physiacal_size parameter of snapshot_store_ref table + has physical size of snapshot + """ + if self.hypervisor.lower() not in ["xenserver", "vmware"]: + self.skipTest("Test not to be run on %s" % self.hypervisor) --- End diff -- Here we are running the test for Xenserver and VMWare i.e. skip the test if the hypervisor is not XS or VMWare . So the test run 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 infrastructure@apache.org or file a JIRA ticket with INFRA. ---