cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sebgoa <>
Subject Re: Merge Review Request 22270: Xen 2 XenServer refactoring
Date Fri, 06 Jun 2014 16:21:11 GMT

Any chance you have time to check this one review.
My git kungfu is limited and the patch seems to have conflicts.

Tim did quite a bit of work to get there, so a merge could be call quickly, if this patch
could apply, that would ease another round of rebasing..



On Jun 5, 2014, at 8:02 PM, Tim Mackey <> wrote:

> I've just submitted a review request which is essentially a merge of
> the xen2server feature branch back into master.  Since this is a
> refactoring of the Xen plugin to make it more explicitly a XenServer
> plugin per the feature:
> I wanted to ensure everyone was aware of what is changing. Diff
> details can be found here:
> The key item in this change is that what was the xen plugin has now
> moved to become the xenserver plugin to make way for a pure xen
> plugin.  If you are working on something which is XenServer specific,
> you'll probably want to take a look at what I've done, sanity check it
> against your plans and ask questions.  Additionally, if you've done
> some work in XenServer code since the start of April, you might want
> to make certain I didn't accidentally revert any of your changes
> during conflict resolution.  I tried to be careful, but accidents do
> happen.
> The merge point was commit 603eab7 (HEAD yesterday), and from there I
> did a bunch of sanity testing using XenServer 6.2.  I also tested and
> validated with the current HEAD (8b5ec64).  If you were looking at or
> testing anything on the xen2server branch, no new functionality was
> introduced from that branch; this is effectively a merge with
> conflicts resolved.
> Thanks
> -tim

View raw message