[Gluster-Maintainers] mainline compilation fails

Atin Mukherjee amukherj at redhat.com
Mon Aug 29 03:57:15 UTC 2016


I've merged this patch to unblock the other patches to go through the smoke
and other regression. Given the volume of patches we have in the pipeline
for 3.9 and we are just couple of days away to 3.9 branching, this needed
an immediate attention.

We now need to rebase all 3.9 targeted patches (the ones which failed smoke
and other tests).

On Sun, Aug 28, 2016 at 8:34 PM, Atin Mukherjee <amukherj at redhat.com> wrote:

> The patch http://review.gluster.org/#/c/15331/ is now ready for merge.
>
> Niels - could you merge it?
>
>
> On Saturday 27 August 2016, Prasanna Kalever <pkalever at redhat.com> wrote:
>
>> Oops!
>> Didn't noticed these changes were part of parent/child patches, Just
>> noticed "BUILD BROKEN" and went into action :)
>>
>> I'm not sure about it!
>>
>> If it takes time to decide on whether the other set of patches need to
>> be taken or not, at-least my patch will fix the broken build (That
>> much I can assure)
>>
>> Lets see if the regressions break after these patch goes in (mostly
>> not, that I see from the code)
>>
>> Thanks,
>> --
>> Prasanna
>>
>>
>> On Sat, Aug 27, 2016 at 8:34 PM, Atin Mukherjee
>> <atin.mukherjee83 at gmail.com> wrote:
>> >
>> >
>> > On Saturday 27 August 2016, Prasanna Kalever <pkalever at redhat.com>
>> wrote:
>> >>
>> >> Here is the patch that should fix it
>> >> http://review.gluster.org/#/c/15331/
>> >
>> >
>> > Thanks! Well thats an easy way, but the question here is dont we need
>> the
>> > parent patch to be merged to ensure there is no other functionality
>> broken.
>> > Currently I see that the parent patch has a -1, in that case is it
>> required
>> > to revert 15225?
>> >>
>> >>
>> >> Happy weekend!
>> >>
>> >> --
>> >> Prasanna
>> >>
>> >>
>> >> On Sat, Aug 27, 2016 at 7:49 PM, Atin Mukherjee <amukherj at redhat.com>
>> >> wrote:
>> >> > [1] has broken mainline compilation and I feel this could be because
>> its
>> >> > parent patch is not been merged otherwise smoke should have caught
>> it.
>> >> > Please resolve it at earliest.
>> >> >
>> >> > [1] http://review.gluster.org/#/c/15225/
>> >> >
>> >> >
>> >> > --Atin
>> >> >
>> >> > _______________________________________________
>> >> > Gluster-devel mailing list
>> >> > Gluster-devel at gluster.org
>> >> > http://www.gluster.org/mailman/listinfo/gluster-devel
>> >> _______________________________________________
>> >> Gluster-devel mailing list
>> >> Gluster-devel at gluster.org
>> >> http://www.gluster.org/mailman/listinfo/gluster-devel
>> >
>> >
>> >
>> > --
>> > --Atin
>>
>
>
> --
> --Atin
>



-- 

--Atin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.gluster.org/pipermail/maintainers/attachments/20160829/61b7418c/attachment.html>


More information about the maintainers mailing list