Darren, Would you be able to look into copy the logic back into your refactoring today or tomorrow? If not, I may be able to in the morning.
-Chris -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Oct 23, 2013, at 5:56 PM, SuichII, Christopher <chris.su...@netapp.com> wrote: > My understanding is that it is still a work in progress to get those test > back running. Is this correct, Edison? > > -- > Chris Suich > chris.su...@netapp.com > NetApp Software Engineer > Data Center Platforms – Cloud Solutions > Citrix, Cisco & Red Hat > > On Oct 23, 2013, at 5:48 PM, Darren Shepherd <darren.s.sheph...@gmail.com> > wrote: > >> I fixed all the compilation errors in engine/storage/integration-test. >> I don't know how to run those test though, so I can't validate the >> changes. >> >> Darren >> >> On Wed, Oct 23, 2013 at 1:53 PM, SuichII, Christopher >> <chris.su...@netapp.com> wrote: >>> Yep. I’m running on a clean master. >>> >>> -- >>> Chris Suich >>> chris.su...@netapp.com >>> NetApp Software Engineer >>> Data Center Platforms – Cloud Solutions >>> Citrix, Cisco & Red Hat >>> >>> On Oct 23, 2013, at 4:30 PM, Darren Shepherd <darren.s.sheph...@gmail.com> >>> wrote: >>> >>>> Okay let me look at that. Are you 100% sure your looking at a clean >>>> version of master? >>>> >>>> Darren >>>> >>>>> On Oct 23, 2013, at 1:17 PM, "SuichII, Christopher" >>>>> <chris.su...@netapp.com> wrote: >>>>> >>>>> Er, sorry. That was poorly worded on my part. Some classes, like >>>>> SnapshotTest.java and all the storage providers, did not get updated >>>>> references to your refactoring. They still reference >>>>> StrategyPriority.pickStrategy(), etc. Additionally, I changed the >>>>> pickStrategy() logic from using a comparator to looping over the list >>>>> once keeping a reference to the best result. This logic was lost in the >>>>> merge. >>>>> >>>>> -- >>>>> Chris Suich >>>>> chris.su...@netapp.com >>>>> NetApp Software Engineer >>>>> Data Center Platforms – Cloud Solutions >>>>> Citrix, Cisco & Red Hat >>>>> >>>>>> On Oct 23, 2013, at 4:13 PM, Darren Shepherd >>>>>> <darren.s.sheph...@gmail.com> wrote: >>>>>> >>>>>> The transaction API was changed in the merge. I could have maybe >>>>>> missed updating a class. Let me check. When you said "It looks like >>>>>> the changes from us didn’t make it through your merge at all," can you >>>>>> point to something specific that got lost? >>>>>> >>>>>> Darren >>>>>> >>>>>> On Wed, Oct 23, 2013 at 1:05 PM, SuichII, Christopher >>>>>> <chris.su...@netapp.com> wrote: >>>>>>> And it looks like some of your changes may have not merged correctly. >>>>>>> I’m getting compile errors like: >>>>>>> >>>>>>> The method close() is undefined for the type Transaction >>>>>>> >>>>>>> This shouldn’t have come from our merge. >>>>>>> >>>>>>> -- >>>>>>> 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 >>>>> >>> >