[Gluster-devel] Announcing release 3.11 : Scope, schedule and feature tracking
Shyam
srangana at redhat.com
Tue Apr 25 23:30:02 UTC 2017
On 04/25/2017 10:16 AM, Shyam wrote:
> On 04/20/2017 02:46 AM, Kaushal M wrote:
>>> 2) IPv6 support (@kaushal)
>> This is under review at https://review.gluster.org/16228 . The patch
>> mostly looks fine.
>>
>> The only issue is that it currently depends and links with an internal
>> FB fork of tirpc (mainly for some helper functions and utilities).
>> This makes it hard for the community to make actual use of and test,
>> the IPv6 features/fixes introduced by the change.
>>
>> If the change were refactored the use publicly available versions of
>> tirpc or ntirpc, I'm OK for it to be merged. I did try it out myself.
>> While I was able to build it against available versions of tirpc, I
>> wasn't able to get it working correctly.
>>
>
> I checked the patch and here are my comments on merging this,
>
> 1) We are encouraging FB to actually not use FB specific configure time
> options, and instead use a site.h like approach (wherein we can build
> with different site.h files and not proliferate options). This
> discussion I realize is not public, nor is there a github issue for the
> same.
>
> Considering this, we would need this patch to change appropriately.
>
> 2) I also agree on the tirpc dependency, if we could make it work with
> the publicly available tirpc, it is better as otherwise it is difficult
> to use by the community.
>
> Considering this, I would suggest we (as in all concerned) work on these
> aspects and get it right in master before we take it in for a release.
Forgot to mention, please add a github issue to track this feature
against the release scope (3.11 or otherwise).
>
> Thanks,
> Shyam
> _______________________________________________
> Gluster-devel mailing list
> Gluster-devel at gluster.org
> http://lists.gluster.org/mailman/listinfo/gluster-devel
More information about the Gluster-devel
mailing list