Syed, Any test that is calling is_snapshot_on_nfs from utils.py will fail for Xen as it ends up appending an additional “.vhd” to the install_path. There are several of them both in smoke and component folder. The reason this is under scrutiny is die to marvin test failures.
On 06/12/16, 9:33 AM, "Syed Ahmed" <sah...@cloudops.com> wrote: >Yes this will break managed snapshot archives. I want to understand what >this revert is trying to fix and what tests were failing. >On Mon, Dec 5, 2016 at 22:38 abhinandanprateek <g...@git.apache.org> wrote: > >> Github user abhinandanprateek commented on the issue: >> >> https://github.com/apache/cloudstack/pull/1807 >> >> @syed can you look at this 'revert' i guess this is going to be an >> issue with the managed storage change that has been put in. cc @rhtyd >> >> >> --- >> 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. >> --- >> abhinandan.prat...@shapeblue.com www.shapeblue.com 53 Chandos Place, Covent Garden, London WC2N 4HSUK @shapeblue
