<div dir="ltr"><div>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. <br><br></div>We now need to rebase all 3.9 targeted patches (the ones which failed smoke and other tests).<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Aug 28, 2016 at 8:34 PM, Atin Mukherjee <span dir="ltr"><<a href="mailto:amukherj@redhat.com" target="_blank">amukherj@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The patch <a href="http://review.gluster.org/#/c/15331/" target="_blank">http://review.gluster.<wbr>org/#/c/15331/</a> is now ready for merge.<div><br></div><div>Niels - could you merge it?<div><div class="h5"><span></span><br><br>On Saturday 27 August 2016, Prasanna Kalever <<a href="mailto:pkalever@redhat.com" target="_blank">pkalever@redhat.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Oops!<br>
Didn't noticed these changes were part of parent/child patches, Just<br>
noticed "BUILD BROKEN" and went into action :)<br>
<br>
I'm not sure about it!<br>
<br>
If it takes time to decide on whether the other set of patches need to<br>
be taken or not, at-least my patch will fix the broken build (That<br>
much I can assure)<br>
<br>
Lets see if the regressions break after these patch goes in (mostly<br>
not, that I see from the code)<br>
<br>
Thanks,<br>
--<br>
Prasanna<br>
<br>
<br>
On Sat, Aug 27, 2016 at 8:34 PM, Atin Mukherjee<br>
<<a>atin.mukherjee83@gmail.com</a>> wrote:<br>
><br>
><br>
> On Saturday 27 August 2016, Prasanna Kalever <<a>pkalever@redhat.com</a>> wrote:<br>
>><br>
>> Here is the patch that should fix it<br>
>> <a href="http://review.gluster.org/#/c/15331/" target="_blank">http://review.gluster.org/#/c/<wbr>15331/</a><br>
><br>
><br>
> Thanks! Well thats an easy way, but the question here is dont we need the<br>
> parent patch to be merged to ensure there is no other functionality broken.<br>
> Currently I see that the parent patch has a -1, in that case is it required<br>
> to revert 15225?<br>
>><br>
>><br>
>> Happy weekend!<br>
>><br>
>> --<br>
>> Prasanna<br>
>><br>
>><br>
>> On Sat, Aug 27, 2016 at 7:49 PM, Atin Mukherjee <<a>amukherj@redhat.com</a>><br>
>> wrote:<br>
>> > [1] has broken mainline compilation and I feel this could be because its<br>
>> > parent patch is not been merged otherwise smoke should have caught it.<br>
>> > Please resolve it at earliest.<br>
>> ><br>
>> > [1] <a href="http://review.gluster.org/#/c/15225/" target="_blank">http://review.gluster.org/#/c/<wbr>15225/</a><br>
>> ><br>
>> ><br>
>> > --Atin<br>
>> ><br>
>> > ______________________________<wbr>_________________<br>
>> > Gluster-devel mailing list<br>
>> > <a>Gluster-devel@gluster.org</a><br>
>> > <a href="http://www.gluster.org/mailman/listinfo/gluster-devel" target="_blank">http://www.gluster.org/mailman<wbr>/listinfo/gluster-devel</a><br>
>> ______________________________<wbr>_________________<br>
>> Gluster-devel mailing list<br>
>> <a>Gluster-devel@gluster.org</a><br>
>> <a href="http://www.gluster.org/mailman/listinfo/gluster-devel" target="_blank">http://www.gluster.org/mailman<wbr>/listinfo/gluster-devel</a><br>
><br>
><br>
><br>
> --<br>
> --Atin<br>
</blockquote></div></div></div><span class="HOEnZb"><font color="#888888"><br><br>-- <br>--Atin<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><br></div>--Atin<br></div></div>
</div>