I can fix it, let me see what I broke. Darren
> On Oct 23, 2013, at 1:03 PM, "SuichII, Christopher" <chris.su...@netapp.com> > wrote: > > It looks like the changes from us didn’t make it through your merge at all. I > should have been more clear in my review - there are logic changes that > needed to be carried from my changes into yours, not simply a rebase. I will > work on those changes and try to get a patch up shortly. > > -- > Chris Suich > chris.su...@netapp.com > NetApp Software Engineer > Data Center Platforms – Cloud Solutions > Citrix, Cisco & Red Hat > >> On Oct 23, 2013, at 3:52 PM, Darren Shepherd <darren.s.sheph...@gmail.com> >> wrote: >> >> Chris, Edison, >> >> You guys just committed 'Support Revert VM Disk from Snapshot.' At >> the same time I was merging both my txn-refactor and >> spring-modularization branches. They are really tricky merges and >> each time I have to rebase it takes awhile to figure out. Anyhow, >> your change + my changes breaks master. So I quickly rebased rb14823 >> and committed to master. rb14823 is the patch that makes the Storage >> Strategies work with my spring work plus clean up some things. >> Additionally I found out you can't inject List<SnapshotStrategy> to >> the Snapshot object, so we really have to go with my change to >> centralize the ownership of the strategies to a single class. >> >> Can you please pull master and revalidate that I didn't break >> anything, if its not too much of a pain. >> >> Thanks, >> Darren >