----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16324/#review30814 -----------------------------------------------------------
Commit 80ab47c86945f8949612cbea0e169c4ca9e096bb in branch refs/heads/4.3 from Gaurav Aradhye [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=80ab47c ] CLOUDSTACK-4780: Changes related to checking snapshot path on NFS server, also made marvin import paths specific - ASF Subversion and Git Services On Dec. 23, 2013, 11:41 a.m., Gaurav Aradhye wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16324/ > ----------------------------------------------------------- > > (Updated Dec. 23, 2013, 11:41 a.m.) > > > Review request for cloudstack and Girish Shilamkar. > > > Bugs: CLOUDSTACK-4780 > https://issues.apache.org/jira/browse/CLOUDSTACK-4780 > > > Repository: cloudstack-git > > > Description > ------- > > Changes: > > 1) Appended snapshot extension to the snapshot path coming from database. As > discussed with Harikrishna, Product bug (5135) is invalid, snapshot path in > database won't contain the extension of the snapshot. So changing the code so > as to append the extension based on hypervisor type. > > 2) Code movement and cleanup - Moved function get_hypervisor_type from common > to utils to avoid cyclic dependency of libraries. Also, cleaned up the > imports. > > > Diffs > ----- > > test/integration/component/test_add_remove_network.py f1508e1 > test/integration/component/test_assign_vm.py cbdce73 > test/integration/component/test_cpu_domain_limits.py 4e8fc6d > test/integration/component/test_cpu_limits.py d721a45 > test/integration/component/test_cpu_max_limits.py 9161cee > test/integration/component/test_cpu_project_limits.py 63d1a98 > test/integration/component/test_egress_fw_rules.py 09e1dd6 > test/integration/component/test_haproxy.py c734012 > test/integration/component/test_mm_domain_limits.py c856087 > test/integration/component/test_mm_max_limits.py b1ebbb4 > test/integration/component/test_mm_project_limits.py ffeb20a > test/integration/component/test_snapshots.py d3fac42 > test/integration/component/test_vpc_network_lbrules.py e7cb823 > test/integration/component/test_vpc_network_pfrules.py 0d8e2f1 > test/integration/component/test_vpc_network_staticnatrule.py dd3d249 > test/integration/component/test_vpn_users.py 9ee907b > tools/marvin/marvin/integration/lib/common.py 096b073 > tools/marvin/marvin/integration/lib/utils.py d046235 > > Diff: https://reviews.apache.org/r/16324/diff/ > > > Testing > ------- > > Tested locally on VMware. KVM should not have impact of this change as the > snapshot created on KVM was found not to have any extension. > > Log: > test_01_snapshot_root_disk (test_snapshots.TestSnapshotRootDisk) > Test Snapshot Root Disk ... ok > > ---------------------------------------------------------------------- > Ran 1 test in 513.783s > > OK > > > Thanks, > > Gaurav Aradhye > >
