I see. I was more interested in the functionality that is broken on Cloudstack. I have verified that older operations are still working correctly. Did I miss anything? On Mon, Dec 5, 2016 at 23:44 Abhinandan Prateek < abhinandan.prat...@shapeblue.com> wrote:
> 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 > > > >
