Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-17 Thread Frank Filz
Right,



6bd32da613e26a768ac1dc4db1001395bd10c295 CMake - Have 'make dist' generate the 
correct tarball name



Went into 2.6 AFTER Bill’s Napalm. I couldn’t make enough sense of what had 
changed to confidently do a backport…



The messiness in CMakeLists.txt with multiple different changes was one of the 
reasons Bill’s Napalm got delayed a week…



Frank



From: Matt Benjamin [mailto:mbenj...@redhat.com]
Sent: Thursday, August 17, 2017 2:15 PM
To: Frank Filz <ffilz...@mindspring.com>
Cc: Malahal Naineni <mala...@gmail.com>; Soumya Koduri <skod...@redhat.com>; 
nfs-ganesha-devel <nfs-ganesha-devel@lists.sourceforge.net>
Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2



I'm confused, napalm is 2.6-dev, not 2.5.x



On Thu, Aug 17, 2017 at 5:13 PM, Frank Filz <ffilz...@mindspring.com 
<mailto:ffilz...@mindspring.com> > wrote:

Actually, the problem was Bill’s Napalm…



I was able to make sense of the manual merge for



ff98ea64b6d1228443a35b2f7ceb3c61c0a0c1d1 Build libntirpc package when not using 
system ntirpc



But not for



6bd32da613e26a768ac1dc4db1001395bd10c295 CMake - Have 'make dist' generate the 
correct tarball name



I pushed my results to



https://github.com/ffilz/nfs-ganesha/commits/V2.5-stable



Frank



From: Malahal Naineni [mailto:mala...@gmail.com <mailto:mala...@gmail.com> ]
Sent: Thursday, August 17, 2017 10:39 AM
To: Frank Filz <ffilz...@mindspring.com <mailto:ffilz...@mindspring.com> >
Cc: Matt Benjamin <mbenj...@redhat.com <mailto:mbenj...@redhat.com> >; Soumya 
Koduri <skod...@redhat.com <mailto:skod...@redhat.com> >; nfs-ganesha-devel 
<nfs-ganesha-devel@lists.sourceforge.net 
<mailto:nfs-ganesha-devel@lists.sourceforge.net> >


Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2



I did but the failover/failback code re-org looked like contributed, but I am 
not positive.



On Thu, Aug 17, 2017 at 7:40 PM, Frank Filz <ffilz...@mindspring.com 
<mailto:ffilz...@mindspring.com> > wrote:

Hmm, did you cherry pick in the original order?



I’ll take a look at this later today.



Frank



From: Malahal Naineni [mailto:mala...@gmail.com <mailto:mala...@gmail.com> ]
Sent: Wednesday, August 16, 2017 11:34 PM
To: Matt Benjamin <mbenj...@redhat.com <mailto:mbenj...@redhat.com> >
Cc: Frank Filz <ffilz...@mindspring.com <mailto:ffilz...@mindspring.com> >; 
Soumya Koduri <skod...@redhat.com <mailto:skod...@redhat.com> >; 
nfs-ganesha-devel <nfs-ganesha-devel@lists.sourceforge.net 
<mailto:nfs-ganesha-devel@lists.sourceforge.net> >


Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2



Dan, I backported everything that was needed except the following 2 as I don't 
want to mess with cmake! Can you please quickly send ported patches? Appreciate 
your help. The latest V2.5 code is at  my personal github branch V2.5-stable:



https://github.com/malahal/nfs-ganesha/commits/V2.5-stable



The following 2 commits failed to apply:



6bd32da613e26a768ac1dc4db1001395bd10c295 CMake - Have 'make dist' generate the 
correct tarball name

ff98ea64b6d1228443a35b2f7ceb3c61c0a0c1d1 Build libntirpc package when not using 
system ntirpc







On Wed, Aug 16, 2017 at 10:47 PM, Matt Benjamin <mbenj...@redhat.com 
<mailto:mbenj...@redhat.com> > wrote:

Hi Frank,

On Wed, Aug 16, 2017 at 1:11 PM, Frank Filz <ffilz...@mindspring.com 
<mailto:ffilz...@mindspring.com> > wrote:
> Oh, nice.

>
>
> Matt, what about this one?
>
>
>
> 814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified

RHCS doesn't officially support this, but I'd say it would be nice to have.

Matt


>
>
>
> Frank
>
>
>
>
>
> From: Malahal Naineni [mailto:mala...@gmail.com <mailto:mala...@gmail.com> ]
> Sent: Wednesday, August 16, 2017 9:28 AM
> To: Soumya Koduri <skod...@redhat.com <mailto:skod...@redhat.com> >
> Cc: Frank Filz <ffilz...@mindspring.com <mailto:ffilz...@mindspring.com> >; 
> d...@redhat.com <mailto:d...@redhat.com> ; Matt Benjamin
> <mbenj...@redhat.com <mailto:mbenj...@redhat.com> >; nfs-ganesha-devel
> <nfs-ganesha-devel@lists.sourceforge.net 
> <mailto:nfs-ganesha-devel@lists.sourceforge.net> >
> Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2
>
>
>
> I pushed a notes branch "refs/notes/backport" which has a note saying
> "backport to V2.5". You should be able to fetch this special branch with
> "git fetch origin refs/notes/*:refs/notes/*". After fetching this special
> branch, you should do "export GIT_NOTES_REF=refs/notes/backport" in your
> SHELL and then run the usual "git log" to see if I missed any commits you
> are interested in.
>
>
>
> 

Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-17 Thread Matt Benjamin
I'm confused, napalm is 2.6-dev, not 2.5.x

On Thu, Aug 17, 2017 at 5:13 PM, Frank Filz <ffilz...@mindspring.com> wrote:

> Actually, the problem was Bill’s Napalm…
>
>
>
> I was able to make sense of the manual merge for
>
>
>
> ff98ea64b6d1228443a35b2f7ceb3c61c0a0c1d1 Build libntirpc package when not
> using system ntirpc
>
>
>
> But not for
>
>
>
> 6bd32da613e26a768ac1dc4db1001395bd10c295 CMake - Have 'make dist'
> generate the correct tarball name
>
>
>
> I pushed my results to
>
>
>
> https://github.com/ffilz/nfs-ganesha/commits/V2.5-stable
>
>
>
> Frank
>
>
>
> *From:* Malahal Naineni [mailto:mala...@gmail.com]
> *Sent:* Thursday, August 17, 2017 10:39 AM
> *To:* Frank Filz <ffilz...@mindspring.com>
> *Cc:* Matt Benjamin <mbenj...@redhat.com>; Soumya Koduri <
> skod...@redhat.com>; nfs-ganesha-devel <nfs-ganesha-devel@lists.
> sourceforge.net>
>
> *Subject:* Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2
>
>
>
> I did but the failover/failback code re-org looked like contributed, but I
> am not positive.
>
>
>
> On Thu, Aug 17, 2017 at 7:40 PM, Frank Filz <ffilz...@mindspring.com>
> wrote:
>
> Hmm, did you cherry pick in the original order?
>
>
>
> I’ll take a look at this later today.
>
>
>
> Frank
>
>
>
> *From:* Malahal Naineni [mailto:mala...@gmail.com]
> *Sent:* Wednesday, August 16, 2017 11:34 PM
> *To:* Matt Benjamin <mbenj...@redhat.com>
> *Cc:* Frank Filz <ffilz...@mindspring.com>; Soumya Koduri <
> skod...@redhat.com>; nfs-ganesha-devel <nfs-ganesha-devel@lists.
> sourceforge.net>
>
>
> *Subject:* Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2
>
>
>
> Dan, I backported everything that was needed except the following 2 as I
> don't want to mess with cmake! Can you please quickly send ported patches?
> Appreciate your help. The latest V2.5 code is at  my personal github branch
> V2.5-stable:
>
>
>
> https://github.com/malahal/nfs-ganesha/commits/V2.5-stable
>
>
>
> The following 2 commits failed to apply:
>
>
>
> 6bd32da613e26a768ac1dc4db1001395bd10c295 CMake - Have 'make dist'
> generate the correct tarball name
>
> ff98ea64b6d1228443a35b2f7ceb3c61c0a0c1d1 Build libntirpc package when not
> using system ntirpc
>
>
>
>
>
>
>
> On Wed, Aug 16, 2017 at 10:47 PM, Matt Benjamin <mbenj...@redhat.com>
> wrote:
>
> Hi Frank,
>
> On Wed, Aug 16, 2017 at 1:11 PM, Frank Filz <ffilz...@mindspring.com>
> wrote:
> > Oh, nice.
>
> >
> >
> > Matt, what about this one?
> >
> >
> >
> > 814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified
>
> RHCS doesn't officially support this, but I'd say it would be nice to have.
>
> Matt
>
>
> >
> >
> >
> > Frank
> >
> >
> >
> >
> >
> > From: Malahal Naineni [mailto:mala...@gmail.com]
> > Sent: Wednesday, August 16, 2017 9:28 AM
> > To: Soumya Koduri <skod...@redhat.com>
> > Cc: Frank Filz <ffilz...@mindspring.com>; d...@redhat.com; Matt Benjamin
> > <mbenj...@redhat.com>; nfs-ganesha-devel
> > <nfs-ganesha-devel@lists.sourceforge.net>
> > Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2
> >
> >
> >
> > I pushed a notes branch "refs/notes/backport" which has a note saying
> > "backport to V2.5". You should be able to fetch this special branch with
> > "git fetch origin refs/notes/*:refs/notes/*". After fetching this special
> > branch, you should do "export GIT_NOTES_REF=refs/notes/backport" in your
> > SHELL and then run the usual "git log" to see if I missed any commits you
> > are interested in.
> >
> >
> >
> > Alternatively, the following are the commits that will NOT be back
> ported.
> > Let me know if you need any of these. I will cherry pick things tomorrow
> and
> > publish the branch, if there are no comments...
> >
> >
> >
> > 00b9e0798 Revert "CMake - Have 'make dist' generate the correct tarball
> > name"
> >
> > 1b60d5df2 FSAL_MEM - fix UP thread init/cleanup
> >
> > 39119aab0 FSAL_GLUSTER: Use glfs_xreaddirplus_r for readdir
> >
> > 4b4e21ed9 Manpage - Fix installing manpages in RPM
> >
> > 814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified
> >
> > b862fe360 SAL: extract fs logic from nfs4_recovery
> >
> > c29114162 Napalm dispatch plus plus
> >
> > c8bc40b69 C

Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-17 Thread Frank Filz
Actually, the problem was Bill’s Napalm…



I was able to make sense of the manual merge for



ff98ea64b6d1228443a35b2f7ceb3c61c0a0c1d1 Build libntirpc package when not using 
system ntirpc



But not for



6bd32da613e26a768ac1dc4db1001395bd10c295 CMake - Have 'make dist' generate the 
correct tarball name



I pushed my results to



https://github.com/ffilz/nfs-ganesha/commits/V2.5-stable



Frank



From: Malahal Naineni [mailto:mala...@gmail.com]
Sent: Thursday, August 17, 2017 10:39 AM
To: Frank Filz <ffilz...@mindspring.com>
Cc: Matt Benjamin <mbenj...@redhat.com>; Soumya Koduri <skod...@redhat.com>; 
nfs-ganesha-devel <nfs-ganesha-devel@lists.sourceforge.net>
Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2



I did but the failover/failback code re-org looked like contributed, but I am 
not positive.



On Thu, Aug 17, 2017 at 7:40 PM, Frank Filz <ffilz...@mindspring.com 
<mailto:ffilz...@mindspring.com> > wrote:

Hmm, did you cherry pick in the original order?



I’ll take a look at this later today.



Frank



From: Malahal Naineni [mailto:mala...@gmail.com <mailto:mala...@gmail.com> ]
Sent: Wednesday, August 16, 2017 11:34 PM
To: Matt Benjamin <mbenj...@redhat.com <mailto:mbenj...@redhat.com> >
Cc: Frank Filz <ffilz...@mindspring.com <mailto:ffilz...@mindspring.com> >; 
Soumya Koduri <skod...@redhat.com <mailto:skod...@redhat.com> >; 
nfs-ganesha-devel <nfs-ganesha-devel@lists.sourceforge.net 
<mailto:nfs-ganesha-devel@lists.sourceforge.net> >


Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2



Dan, I backported everything that was needed except the following 2 as I don't 
want to mess with cmake! Can you please quickly send ported patches? Appreciate 
your help. The latest V2.5 code is at  my personal github branch V2.5-stable:



https://github.com/malahal/nfs-ganesha/commits/V2.5-stable



The following 2 commits failed to apply:



6bd32da613e26a768ac1dc4db1001395bd10c295 CMake - Have 'make dist' generate the 
correct tarball name

ff98ea64b6d1228443a35b2f7ceb3c61c0a0c1d1 Build libntirpc package when not using 
system ntirpc







On Wed, Aug 16, 2017 at 10:47 PM, Matt Benjamin <mbenj...@redhat.com 
<mailto:mbenj...@redhat.com> > wrote:

Hi Frank,

On Wed, Aug 16, 2017 at 1:11 PM, Frank Filz <ffilz...@mindspring.com 
<mailto:ffilz...@mindspring.com> > wrote:
> Oh, nice.

>
>
> Matt, what about this one?
>
>
>
> 814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified

RHCS doesn't officially support this, but I'd say it would be nice to have.

Matt


>
>
>
> Frank
>
>
>
>
>
> From: Malahal Naineni [mailto:mala...@gmail.com <mailto:mala...@gmail.com> ]
> Sent: Wednesday, August 16, 2017 9:28 AM
> To: Soumya Koduri <skod...@redhat.com <mailto:skod...@redhat.com> >
> Cc: Frank Filz <ffilz...@mindspring.com <mailto:ffilz...@mindspring.com> >; 
> d...@redhat.com <mailto:d...@redhat.com> ; Matt Benjamin
> <mbenj...@redhat.com <mailto:mbenj...@redhat.com> >; nfs-ganesha-devel
> <nfs-ganesha-devel@lists.sourceforge.net 
> <mailto:nfs-ganesha-devel@lists.sourceforge.net> >
> Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2
>
>
>
> I pushed a notes branch "refs/notes/backport" which has a note saying
> "backport to V2.5". You should be able to fetch this special branch with
> "git fetch origin refs/notes/*:refs/notes/*". After fetching this special
> branch, you should do "export GIT_NOTES_REF=refs/notes/backport" in your
> SHELL and then run the usual "git log" to see if I missed any commits you
> are interested in.
>
>
>
> Alternatively, the following are the commits that will NOT be back ported.
> Let me know if you need any of these. I will cherry pick things tomorrow and
> publish the branch, if there are no comments...
>
>
>
> 00b9e0798 Revert "CMake - Have 'make dist' generate the correct tarball
> name"
>
> 1b60d5df2 FSAL_MEM - fix UP thread init/cleanup
>
> 39119aab0 FSAL_GLUSTER: Use glfs_xreaddirplus_r for readdir
>
> 4b4e21ed9 Manpage - Fix installing manpages in RPM
>
> 814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified
>
> b862fe360 SAL: extract fs logic from nfs4_recovery
>
> c29114162 Napalm dispatch plus plus
>
> c8bc40b69 CMake - Have 'make dist' generate the correct tarball name
>
> cb787a1cf SAL: introduce new recovery backend based on rados kv store
>
> eadfc762e New (empty) sample config
>
> eb4eea134 config: add new config options for rados_kv recovery backend
>
> fbc905015 cmake: make modulized recovery backends compile as modules
>
>
>
>
>
> On Fri, Aug 11, 2017 at 

Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-17 Thread Frank Filz
Ok, I’ll have a look at backporting this afternoon.



Frank



From: Malahal Naineni [mailto:mala...@gmail.com]
Sent: Thursday, August 17, 2017 10:39 AM
To: Frank Filz <ffilz...@mindspring.com>
Cc: Matt Benjamin <mbenj...@redhat.com>; Soumya Koduri <skod...@redhat.com>; 
nfs-ganesha-devel <nfs-ganesha-devel@lists.sourceforge.net>
Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2



I did but the failover/failback code re-org looked like contributed, but I am 
not positive.



On Thu, Aug 17, 2017 at 7:40 PM, Frank Filz <ffilz...@mindspring.com 
<mailto:ffilz...@mindspring.com> > wrote:

Hmm, did you cherry pick in the original order?



I’ll take a look at this later today.



Frank



From: Malahal Naineni [mailto:mala...@gmail.com <mailto:mala...@gmail.com> ]
Sent: Wednesday, August 16, 2017 11:34 PM
To: Matt Benjamin <mbenj...@redhat.com <mailto:mbenj...@redhat.com> >
Cc: Frank Filz <ffilz...@mindspring.com <mailto:ffilz...@mindspring.com> >; 
Soumya Koduri <skod...@redhat.com <mailto:skod...@redhat.com> >; 
nfs-ganesha-devel <nfs-ganesha-devel@lists.sourceforge.net 
<mailto:nfs-ganesha-devel@lists.sourceforge.net> >


Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2



Dan, I backported everything that was needed except the following 2 as I don't 
want to mess with cmake! Can you please quickly send ported patches? Appreciate 
your help. The latest V2.5 code is at  my personal github branch V2.5-stable:



https://github.com/malahal/nfs-ganesha/commits/V2.5-stable



The following 2 commits failed to apply:



6bd32da613e26a768ac1dc4db1001395bd10c295 CMake - Have 'make dist' generate the 
correct tarball name

ff98ea64b6d1228443a35b2f7ceb3c61c0a0c1d1 Build libntirpc package when not using 
system ntirpc







On Wed, Aug 16, 2017 at 10:47 PM, Matt Benjamin <mbenj...@redhat.com 
<mailto:mbenj...@redhat.com> > wrote:

Hi Frank,

On Wed, Aug 16, 2017 at 1:11 PM, Frank Filz <ffilz...@mindspring.com 
<mailto:ffilz...@mindspring.com> > wrote:
> Oh, nice.

>
>
> Matt, what about this one?
>
>
>
> 814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified

RHCS doesn't officially support this, but I'd say it would be nice to have.

Matt


>
>
>
> Frank
>
>
>
>
>
> From: Malahal Naineni [mailto:mala...@gmail.com <mailto:mala...@gmail.com> ]
> Sent: Wednesday, August 16, 2017 9:28 AM
> To: Soumya Koduri <skod...@redhat.com <mailto:skod...@redhat.com> >
> Cc: Frank Filz <ffilz...@mindspring.com <mailto:ffilz...@mindspring.com> >; 
> d...@redhat.com <mailto:d...@redhat.com> ; Matt Benjamin
> <mbenj...@redhat.com <mailto:mbenj...@redhat.com> >; nfs-ganesha-devel
> <nfs-ganesha-devel@lists.sourceforge.net 
> <mailto:nfs-ganesha-devel@lists.sourceforge.net> >
> Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2
>
>
>
> I pushed a notes branch "refs/notes/backport" which has a note saying
> "backport to V2.5". You should be able to fetch this special branch with
> "git fetch origin refs/notes/*:refs/notes/*". After fetching this special
> branch, you should do "export GIT_NOTES_REF=refs/notes/backport" in your
> SHELL and then run the usual "git log" to see if I missed any commits you
> are interested in.
>
>
>
> Alternatively, the following are the commits that will NOT be back ported.
> Let me know if you need any of these. I will cherry pick things tomorrow and
> publish the branch, if there are no comments...
>
>
>
> 00b9e0798 Revert "CMake - Have 'make dist' generate the correct tarball
> name"
>
> 1b60d5df2 FSAL_MEM - fix UP thread init/cleanup
>
> 39119aab0 FSAL_GLUSTER: Use glfs_xreaddirplus_r for readdir
>
> 4b4e21ed9 Manpage - Fix installing manpages in RPM
>
> 814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified
>
> b862fe360 SAL: extract fs logic from nfs4_recovery
>
> c29114162 Napalm dispatch plus plus
>
> c8bc40b69 CMake - Have 'make dist' generate the correct tarball name
>
> cb787a1cf SAL: introduce new recovery backend based on rados kv store
>
> eadfc762e New (empty) sample config
>
> eb4eea134 config: add new config options for rados_kv recovery backend
>
> fbc905015 cmake: make modulized recovery backends compile as modules
>
>
>
>
>
> On Fri, Aug 11, 2017 at 8:08 AM, Soumya Koduri <skod...@redhat.com 
> <mailto:skod...@redhat.com> > wrote:
>
>
>> commit 7f2d461277521301a417ca368d3c7656edbfc903
>>  FSAL_GLUSTER: Reset caller_garray to NULL upon free
>>
>
> Yes
>
> On 08/09/2017 08:57 PM, Frank Filz wrote:
>
> 39119aa Sou

Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-17 Thread Malahal Naineni
I did but the failover/failback code re-org looked like contributed, but I
am not positive.

On Thu, Aug 17, 2017 at 7:40 PM, Frank Filz <ffilz...@mindspring.com> wrote:

> Hmm, did you cherry pick in the original order?
>
>
>
> I’ll take a look at this later today.
>
>
>
> Frank
>
>
>
> *From:* Malahal Naineni [mailto:mala...@gmail.com]
> *Sent:* Wednesday, August 16, 2017 11:34 PM
> *To:* Matt Benjamin <mbenj...@redhat.com>
> *Cc:* Frank Filz <ffilz...@mindspring.com>; Soumya Koduri <
> skod...@redhat.com>; nfs-ganesha-devel <nfs-ganesha-devel@lists.
> sourceforge.net>
>
> *Subject:* Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2
>
>
>
> Dan, I backported everything that was needed except the following 2 as I
> don't want to mess with cmake! Can you please quickly send ported patches?
> Appreciate your help. The latest V2.5 code is at  my personal github branch
> V2.5-stable:
>
>
>
> https://github.com/malahal/nfs-ganesha/commits/V2.5-stable
>
>
>
> The following 2 commits failed to apply:
>
>
>
> 6bd32da613e26a768ac1dc4db1001395bd10c295 CMake - Have 'make dist'
> generate the correct tarball name
>
> ff98ea64b6d1228443a35b2f7ceb3c61c0a0c1d1 Build libntirpc package when not
> using system ntirpc
>
>
>
>
>
>
>
> On Wed, Aug 16, 2017 at 10:47 PM, Matt Benjamin <mbenj...@redhat.com>
> wrote:
>
> Hi Frank,
>
> On Wed, Aug 16, 2017 at 1:11 PM, Frank Filz <ffilz...@mindspring.com>
> wrote:
> > Oh, nice.
>
> >
> >
> > Matt, what about this one?
> >
> >
> >
> > 814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified
>
> RHCS doesn't officially support this, but I'd say it would be nice to have.
>
> Matt
>
>
> >
> >
> >
> > Frank
> >
> >
> >
> >
> >
> > From: Malahal Naineni [mailto:mala...@gmail.com]
> > Sent: Wednesday, August 16, 2017 9:28 AM
> > To: Soumya Koduri <skod...@redhat.com>
> > Cc: Frank Filz <ffilz...@mindspring.com>; d...@redhat.com; Matt Benjamin
> > <mbenj...@redhat.com>; nfs-ganesha-devel
> > <nfs-ganesha-devel@lists.sourceforge.net>
> > Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2
> >
> >
> >
> > I pushed a notes branch "refs/notes/backport" which has a note saying
> > "backport to V2.5". You should be able to fetch this special branch with
> > "git fetch origin refs/notes/*:refs/notes/*". After fetching this special
> > branch, you should do "export GIT_NOTES_REF=refs/notes/backport" in your
> > SHELL and then run the usual "git log" to see if I missed any commits you
> > are interested in.
> >
> >
> >
> > Alternatively, the following are the commits that will NOT be back
> ported.
> > Let me know if you need any of these. I will cherry pick things tomorrow
> and
> > publish the branch, if there are no comments...
> >
> >
> >
> > 00b9e0798 Revert "CMake - Have 'make dist' generate the correct tarball
> > name"
> >
> > 1b60d5df2 FSAL_MEM - fix UP thread init/cleanup
> >
> > 39119aab0 FSAL_GLUSTER: Use glfs_xreaddirplus_r for readdir
> >
> > 4b4e21ed9 Manpage - Fix installing manpages in RPM
> >
> > 814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified
> >
> > b862fe360 SAL: extract fs logic from nfs4_recovery
> >
> > c29114162 Napalm dispatch plus plus
> >
> > c8bc40b69 CMake - Have 'make dist' generate the correct tarball name
> >
> > cb787a1cf SAL: introduce new recovery backend based on rados kv store
> >
> > eadfc762e New (empty) sample config
> >
> > eb4eea134 config: add new config options for rados_kv recovery backend
> >
> > fbc905015 cmake: make modulized recovery backends compile as modules
> >
> >
> >
> >
> >
> > On Fri, Aug 11, 2017 at 8:08 AM, Soumya Koduri <skod...@redhat.com>
> wrote:
> >
> >
> >> commit 7f2d461277521301a417ca368d3c7656edbfc903
> >>  FSAL_GLUSTER: Reset caller_garray to NULL upon free
> >>
> >
> > Yes
> >
> > On 08/09/2017 08:57 PM, Frank Filz wrote:
> >
> > 39119aa Soumya Koduri FSAL_GLUSTER: Use glfs_xreaddirplus_r for
> > readdir
> >
> > Yes? No? It's sort of a new feature, but may be critical for some use
> cases.
> > I'd rather it go into stable than end up separately backported for
> > downstream.
> >
> >
> > Right..as it is more of a n

Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-17 Thread Frank Filz
Hmm, did you cherry pick in the original order?



I’ll take a look at this later today.



Frank



From: Malahal Naineni [mailto:mala...@gmail.com]
Sent: Wednesday, August 16, 2017 11:34 PM
To: Matt Benjamin <mbenj...@redhat.com>
Cc: Frank Filz <ffilz...@mindspring.com>; Soumya Koduri <skod...@redhat.com>; 
nfs-ganesha-devel <nfs-ganesha-devel@lists.sourceforge.net>
Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2



Dan, I backported everything that was needed except the following 2 as I don't 
want to mess with cmake! Can you please quickly send ported patches? Appreciate 
your help. The latest V2.5 code is at  my personal github branch V2.5-stable:



https://github.com/malahal/nfs-ganesha/commits/V2.5-stable



The following 2 commits failed to apply:



6bd32da613e26a768ac1dc4db1001395bd10c295 CMake - Have 'make dist' generate the 
correct tarball name

ff98ea64b6d1228443a35b2f7ceb3c61c0a0c1d1 Build libntirpc package when not using 
system ntirpc







On Wed, Aug 16, 2017 at 10:47 PM, Matt Benjamin <mbenj...@redhat.com 
<mailto:mbenj...@redhat.com> > wrote:

Hi Frank,

On Wed, Aug 16, 2017 at 1:11 PM, Frank Filz <ffilz...@mindspring.com 
<mailto:ffilz...@mindspring.com> > wrote:
> Oh, nice.

>
>
> Matt, what about this one?
>
>
>
> 814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified

RHCS doesn't officially support this, but I'd say it would be nice to have.

Matt


>
>
>
> Frank
>
>
>
>
>
> From: Malahal Naineni [mailto:mala...@gmail.com <mailto:mala...@gmail.com> ]
> Sent: Wednesday, August 16, 2017 9:28 AM
> To: Soumya Koduri <skod...@redhat.com <mailto:skod...@redhat.com> >
> Cc: Frank Filz <ffilz...@mindspring.com <mailto:ffilz...@mindspring.com> >; 
> d...@redhat.com <mailto:d...@redhat.com> ; Matt Benjamin
> <mbenj...@redhat.com <mailto:mbenj...@redhat.com> >; nfs-ganesha-devel
> <nfs-ganesha-devel@lists.sourceforge.net 
> <mailto:nfs-ganesha-devel@lists.sourceforge.net> >
> Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2
>
>
>
> I pushed a notes branch "refs/notes/backport" which has a note saying
> "backport to V2.5". You should be able to fetch this special branch with
> "git fetch origin refs/notes/*:refs/notes/*". After fetching this special
> branch, you should do "export GIT_NOTES_REF=refs/notes/backport" in your
> SHELL and then run the usual "git log" to see if I missed any commits you
> are interested in.
>
>
>
> Alternatively, the following are the commits that will NOT be back ported.
> Let me know if you need any of these. I will cherry pick things tomorrow and
> publish the branch, if there are no comments...
>
>
>
> 00b9e0798 Revert "CMake - Have 'make dist' generate the correct tarball
> name"
>
> 1b60d5df2 FSAL_MEM - fix UP thread init/cleanup
>
> 39119aab0 FSAL_GLUSTER: Use glfs_xreaddirplus_r for readdir
>
> 4b4e21ed9 Manpage - Fix installing manpages in RPM
>
> 814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified
>
> b862fe360 SAL: extract fs logic from nfs4_recovery
>
> c29114162 Napalm dispatch plus plus
>
> c8bc40b69 CMake - Have 'make dist' generate the correct tarball name
>
> cb787a1cf SAL: introduce new recovery backend based on rados kv store
>
> eadfc762e New (empty) sample config
>
> eb4eea134 config: add new config options for rados_kv recovery backend
>
> fbc905015 cmake: make modulized recovery backends compile as modules
>
>
>
>
>
> On Fri, Aug 11, 2017 at 8:08 AM, Soumya Koduri <skod...@redhat.com 
> <mailto:skod...@redhat.com> > wrote:
>
>
>> commit 7f2d461277521301a417ca368d3c7656edbfc903
>>  FSAL_GLUSTER: Reset caller_garray to NULL upon free
>>
>
> Yes
>
> On 08/09/2017 08:57 PM, Frank Filz wrote:
>
> 39119aa Soumya Koduri FSAL_GLUSTER: Use glfs_xreaddirplus_r for
> readdir
>
> Yes? No? It's sort of a new feature, but may be critical for some use cases.
> I'd rather it go into stable than end up separately backported for
> downstream.
>
>
> Right..as it is more of a new feature, wrt upstream we wanted it to be part
> of only 2.6 on wards so as not to break stable branch (in case if there are
> nit issues).
>
> But yes we may end up back-porting to downstream if we do not rebase to 2.6
> by then.
>
> Thanks,
> Soumya
>
>
>
> --
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> ___
> Nfs-ganesha-

Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-17 Thread Malahal Naineni
Dan, I backported everything that was needed except the following 2 as I
don't want to mess with cmake! Can you please quickly send ported patches?
Appreciate your help. The latest V2.5 code is at  my personal github branch
V2.5-stable:

https://github.com/malahal/nfs-ganesha/commits/V2.5-stable

The following 2 commits failed to apply:

6bd32da613e26a768ac1dc4db1001395bd10c295 CMake - Have 'make dist' generate
the correct tarball name
ff98ea64b6d1228443a35b2f7ceb3c61c0a0c1d1 Build libntirpc package when not
using system ntirpc



On Wed, Aug 16, 2017 at 10:47 PM, Matt Benjamin <mbenj...@redhat.com> wrote:

> Hi Frank,
>
> On Wed, Aug 16, 2017 at 1:11 PM, Frank Filz <ffilz...@mindspring.com>
> wrote:
> > Oh, nice.
>
> >
> >
> > Matt, what about this one?
> >
> >
> >
> > 814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified
>
> RHCS doesn't officially support this, but I'd say it would be nice to have.
>
> Matt
>
> >
> >
> >
> > Frank
> >
> >
> >
> >
> >
> > From: Malahal Naineni [mailto:mala...@gmail.com]
> > Sent: Wednesday, August 16, 2017 9:28 AM
> > To: Soumya Koduri <skod...@redhat.com>
> > Cc: Frank Filz <ffilz...@mindspring.com>; d...@redhat.com; Matt Benjamin
> > <mbenj...@redhat.com>; nfs-ganesha-devel
> > <nfs-ganesha-devel@lists.sourceforge.net>
> > Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2
> >
> >
> >
> > I pushed a notes branch "refs/notes/backport" which has a note saying
> > "backport to V2.5". You should be able to fetch this special branch with
> > "git fetch origin refs/notes/*:refs/notes/*". After fetching this special
> > branch, you should do "export GIT_NOTES_REF=refs/notes/backport" in your
> > SHELL and then run the usual "git log" to see if I missed any commits you
> > are interested in.
> >
> >
> >
> > Alternatively, the following are the commits that will NOT be back
> ported.
> > Let me know if you need any of these. I will cherry pick things tomorrow
> and
> > publish the branch, if there are no comments...
> >
> >
> >
> > 00b9e0798 Revert "CMake - Have 'make dist' generate the correct tarball
> > name"
> >
> > 1b60d5df2 FSAL_MEM - fix UP thread init/cleanup
> >
> > 39119aab0 FSAL_GLUSTER: Use glfs_xreaddirplus_r for readdir
> >
> > 4b4e21ed9 Manpage - Fix installing manpages in RPM
> >
> > 814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified
> >
> > b862fe360 SAL: extract fs logic from nfs4_recovery
> >
> > c29114162 Napalm dispatch plus plus
> >
> > c8bc40b69 CMake - Have 'make dist' generate the correct tarball name
> >
> > cb787a1cf SAL: introduce new recovery backend based on rados kv store
> >
> > eadfc762e New (empty) sample config
> >
> > eb4eea134 config: add new config options for rados_kv recovery backend
> >
> > fbc905015 cmake: make modulized recovery backends compile as modules
> >
> >
> >
> >
> >
> > On Fri, Aug 11, 2017 at 8:08 AM, Soumya Koduri <skod...@redhat.com>
> wrote:
> >
> >
> >> commit 7f2d461277521301a417ca368d3c7656edbfc903
> >>  FSAL_GLUSTER: Reset caller_garray to NULL upon free
> >>
> >
> > Yes
> >
> > On 08/09/2017 08:57 PM, Frank Filz wrote:
> >
> > 39119aa Soumya Koduri FSAL_GLUSTER: Use glfs_xreaddirplus_r for
> > readdir
> >
> > Yes? No? It's sort of a new feature, but may be critical for some use
> cases.
> > I'd rather it go into stable than end up separately backported for
> > downstream.
> >
> >
> > Right..as it is more of a new feature, wrt upstream we wanted it to be
> part
> > of only 2.6 on wards so as not to break stable branch (in case if there
> are
> > nit issues).
> >
> > But yes we may end up back-porting to downstream if we do not rebase to
> 2.6
> > by then.
> >
> > Thanks,
> > Soumya
> >
> >
> >
> > 
> --
> > Check out the vibrant tech community on one of the world's most
> > engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> > ___
> > Nfs-ganesha-devel mailing list
> > Nfs-ganesha-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
> >
> >
> >
> >
> > Virus-free. www.avast.com
> >
> > 
> --
> > Check out the vibrant tech community on one of the world's most
> > engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> > ___
> > Nfs-ganesha-devel mailing list
> > Nfs-ganesha-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
> >
>
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-16 Thread Matt Benjamin
Hi Frank,

On Wed, Aug 16, 2017 at 1:11 PM, Frank Filz <ffilz...@mindspring.com> wrote:
> Oh, nice.

>
>
> Matt, what about this one?
>
>
>
> 814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified

RHCS doesn't officially support this, but I'd say it would be nice to have.

Matt

>
>
>
> Frank
>
>
>
>
>
> From: Malahal Naineni [mailto:mala...@gmail.com]
> Sent: Wednesday, August 16, 2017 9:28 AM
> To: Soumya Koduri <skod...@redhat.com>
> Cc: Frank Filz <ffilz...@mindspring.com>; d...@redhat.com; Matt Benjamin
> <mbenj...@redhat.com>; nfs-ganesha-devel
> <nfs-ganesha-devel@lists.sourceforge.net>
> Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2
>
>
>
> I pushed a notes branch "refs/notes/backport" which has a note saying
> "backport to V2.5". You should be able to fetch this special branch with
> "git fetch origin refs/notes/*:refs/notes/*". After fetching this special
> branch, you should do "export GIT_NOTES_REF=refs/notes/backport" in your
> SHELL and then run the usual "git log" to see if I missed any commits you
> are interested in.
>
>
>
> Alternatively, the following are the commits that will NOT be back ported.
> Let me know if you need any of these. I will cherry pick things tomorrow and
> publish the branch, if there are no comments...
>
>
>
> 00b9e0798 Revert "CMake - Have 'make dist' generate the correct tarball
> name"
>
> 1b60d5df2 FSAL_MEM - fix UP thread init/cleanup
>
> 39119aab0 FSAL_GLUSTER: Use glfs_xreaddirplus_r for readdir
>
> 4b4e21ed9 Manpage - Fix installing manpages in RPM
>
> 814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified
>
> b862fe360 SAL: extract fs logic from nfs4_recovery
>
> c29114162 Napalm dispatch plus plus
>
> c8bc40b69 CMake - Have 'make dist' generate the correct tarball name
>
> cb787a1cf SAL: introduce new recovery backend based on rados kv store
>
> eadfc762e New (empty) sample config
>
> eb4eea134 config: add new config options for rados_kv recovery backend
>
> fbc905015 cmake: make modulized recovery backends compile as modules
>
>
>
>
>
> On Fri, Aug 11, 2017 at 8:08 AM, Soumya Koduri <skod...@redhat.com> wrote:
>
>
>> commit 7f2d461277521301a417ca368d3c7656edbfc903
>>  FSAL_GLUSTER: Reset caller_garray to NULL upon free
>>
>
> Yes
>
> On 08/09/2017 08:57 PM, Frank Filz wrote:
>
> 39119aa Soumya Koduri FSAL_GLUSTER: Use glfs_xreaddirplus_r for
> readdir
>
> Yes? No? It's sort of a new feature, but may be critical for some use cases.
> I'd rather it go into stable than end up separately backported for
> downstream.
>
>
> Right..as it is more of a new feature, wrt upstream we wanted it to be part
> of only 2.6 on wards so as not to break stable branch (in case if there are
> nit issues).
>
> But yes we may end up back-porting to downstream if we do not rebase to 2.6
> by then.
>
> Thanks,
> Soumya
>
>
>
> --
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> ___
> Nfs-ganesha-devel mailing list
> Nfs-ganesha-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
>
>
>
>
> Virus-free. www.avast.com
>
> --
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> ___
> Nfs-ganesha-devel mailing list
> Nfs-ganesha-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
>

--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-16 Thread Frank Filz
Oh, nice.



I think the following would be good to backport also since they help packaging:



4b4e21ed9 Manpage - Fix installing manpages in RPM

eadfc762e New (empty) sample config



Matt, what about this one?



814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified



Frank





From: Malahal Naineni [mailto:mala...@gmail.com]
Sent: Wednesday, August 16, 2017 9:28 AM
To: Soumya Koduri <skod...@redhat.com>
Cc: Frank Filz <ffilz...@mindspring.com>; d...@redhat.com; Matt Benjamin 
<mbenj...@redhat.com>; nfs-ganesha-devel 
<nfs-ganesha-devel@lists.sourceforge.net>
Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2



I pushed a notes branch "refs/notes/backport" which has a note saying "backport 
to V2.5". You should be able to fetch this special branch with "git fetch 
origin refs/notes/*:refs/notes/*". After fetching this special branch, you 
should do "export GIT_NOTES_REF=refs/notes/backport" in your SHELL and then run 
the usual "git log" to see if I missed any commits you are interested in.



Alternatively, the following are the commits that will NOT be back ported. Let 
me know if you need any of these. I will cherry pick things tomorrow and 
publish the branch, if there are no comments...



00b9e0798 Revert "CMake - Have 'make dist' generate the correct tarball name"

1b60d5df2 FSAL_MEM - fix UP thread init/cleanup

39119aab0 FSAL_GLUSTER: Use glfs_xreaddirplus_r for readdir

4b4e21ed9 Manpage - Fix installing manpages in RPM

814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified

b862fe360 SAL: extract fs logic from nfs4_recovery

c29114162 Napalm dispatch plus plus

c8bc40b69 CMake - Have 'make dist' generate the correct tarball name

cb787a1cf SAL: introduce new recovery backend based on rados kv store

eadfc762e New (empty) sample config

eb4eea134 config: add new config options for rados_kv recovery backend

fbc905015 cmake: make modulized recovery backends compile as modules





On Fri, Aug 11, 2017 at 8:08 AM, Soumya Koduri <skod...@redhat.com 
<mailto:skod...@redhat.com> > wrote:


> commit 7f2d461277521301a417ca368d3c7656edbfc903
>  FSAL_GLUSTER: Reset caller_garray to NULL upon free
>

Yes

On 08/09/2017 08:57 PM, Frank Filz wrote:

39119aa Soumya Koduri FSAL_GLUSTER: Use glfs_xreaddirplus_r for
readdir

Yes? No? It's sort of a new feature, but may be critical for some use cases.
I'd rather it go into stable than end up separately backported for
downstream.


Right..as it is more of a new feature, wrt upstream we wanted it to be part of 
only 2.6 on wards so as not to break stable branch (in case if there are nit 
issues).

But yes we may end up back-porting to downstream if we do not rebase to 2.6 by 
then.

Thanks,
Soumya



--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net 
<mailto:Nfs-ganesha-devel@lists.sourceforge.net>
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel





---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-16 Thread Malahal Naineni
I pushed a notes branch "refs/notes/backport" which has a note saying
"backport to V2.5". You should be able to fetch this special branch with
"git fetch origin refs/notes/*:refs/notes/*". After fetching this special
branch, you should do "export GIT_NOTES_REF=refs/notes/backport" in your
SHELL and then run the usual "git log" to see if I missed any commits you
are interested in.

Alternatively, the following are the commits that will NOT be back ported.
Let me know if you need any of these. I will cherry pick things tomorrow
and publish the branch, if there are no comments...

00b9e0798 Revert "CMake - Have 'make dist' generate the correct tarball
name"
1b60d5df2 FSAL_MEM - fix UP thread init/cleanup
39119aab0 FSAL_GLUSTER: Use glfs_xreaddirplus_r for readdir
4b4e21ed9 Manpage - Fix installing manpages in RPM
814e9cd65 FSAL_RGW: adopt new rgw_mount2 with bucket specified
b862fe360 SAL: extract fs logic from nfs4_recovery
c29114162 Napalm dispatch plus plus
c8bc40b69 CMake - Have 'make dist' generate the correct tarball name
cb787a1cf SAL: introduce new recovery backend based on rados kv store
eadfc762e New (empty) sample config
eb4eea134 config: add new config options for rados_kv recovery backend
fbc905015 cmake: make modulized recovery backends compile as modules


On Fri, Aug 11, 2017 at 8:08 AM, Soumya Koduri  wrote:

>
> > commit 7f2d461277521301a417ca368d3c7656edbfc903
> >  FSAL_GLUSTER: Reset caller_garray to NULL upon free
> >
>
> Yes
>
> On 08/09/2017 08:57 PM, Frank Filz wrote:
>
>> 39119aa Soumya Koduri FSAL_GLUSTER: Use glfs_xreaddirplus_r for
>>> readdir
>>>
>> Yes? No? It's sort of a new feature, but may be critical for some use
>> cases.
>> I'd rather it go into stable than end up separately backported for
>> downstream.
>>
>>
> Right..as it is more of a new feature, wrt upstream we wanted it to be
> part of only 2.6 on wards so as not to break stable branch (in case if
> there are nit issues).
>
> But yes we may end up back-porting to downstream if we do not rebase to
> 2.6 by then.
>
> Thanks,
> Soumya
>
>
> 
> --
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> ___
> Nfs-ganesha-devel mailing list
> Nfs-ganesha-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
>
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-10 Thread Soumya Koduri


> commit 7f2d461277521301a417ca368d3c7656edbfc903
>  FSAL_GLUSTER: Reset caller_garray to NULL upon free
>

Yes

On 08/09/2017 08:57 PM, Frank Filz wrote:

39119aa Soumya Koduri FSAL_GLUSTER: Use glfs_xreaddirplus_r for
readdir

Yes? No? It's sort of a new feature, but may be critical for some use cases.
I'd rather it go into stable than end up separately backported for
downstream.



Right..as it is more of a new feature, wrt upstream we wanted it to be 
part of only 2.6 on wards so as not to break stable branch (in case if 
there are nit issues).


But yes we may end up back-porting to downstream if we do not rebase to 
2.6 by then.


Thanks,
Soumya

--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-09 Thread Frank Filz
> Can you also add, it is a minor change in the value of
> FATTR4_XATTR_SUPPORT Thanks, Marc.
> 
> commit ac16ad42a514de218debc9b21e92cf9ca353ccc4
> Author: Marc Eshel <es...@us.ibm.com>
> Date:   Sun Jul 30 12:50:14 2017 -0700
> 
> update FATTR4_XATTR_SUPPORT
> 
> update FATTR4_XATTR_SUPPORT to 82 based on latest draft-ietf-nfsv4-
> xattrs-06.txt
> Push to version 2.5
> 
> Change-Id: Id9afe2cedc9de080e5008a4a8bb502f47ebd0116
> Signed-off-by: Marc Eshel <es...@us.ibm.com>

Yes, already tagged that one in my e-mail listing the patches from
V2.6-dev-2 to backport.

Frank

> From:   "Frank Filz" <ffilz...@mindspring.com>
> To: <d...@redhat.com>, "'Matt Benjamin'" <mbenj...@redhat.com>
> Cc: 'nfs-ganesha-devel' <nfs-ganesha-devel@lists.sourceforge.net>
> Date:   08/09/2017 09:05 AM
> Subject:Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2
> 
> 
> 
> > On 08/09/2017 11:29 AM, Frank Filz wrote:
> > > Candidates not merged into 2.6 yet:
> > >
> > > Fix rgw_mount2 check when RGW not installed Daniel
> > Gryniewicz
> > > CMake - Have 'make dist' generate the correct tarball name Daniel
> > > Gryniewicz
> > > Build libntirpc package when not using system ntirpc Daniel
> > > Gryniewicz
> >
> > These could be useful to upstream Ceph for building.  However, it's
> > not
> vital.
> > For now, they have to have a separate branch with patches to do this.
> 
> If it helps upstream Ceph for building, that seems like a good reason to
take
> them.
> 
> Frank
> 
> 
> ---
> This email has been checked for viruses by Avast antivirus software.
> https://www.avast.com/antivirus
> 
> 
>

--
> Check out the vibrant tech community on one of the world's most engaging
> tech sites, Slashdot.org! http://sdm.link/slashdot
> ___
> Nfs-ganesha-devel mailing list
> Nfs-ganesha-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
> 
> 
> 



--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-09 Thread Marc Eshel
Can you also add, it is a minor change in the value of 
FATTR4_XATTR_SUPPORT
Thanks, Marc.

commit ac16ad42a514de218debc9b21e92cf9ca353ccc4
Author: Marc Eshel <es...@us.ibm.com>
Date:   Sun Jul 30 12:50:14 2017 -0700

update FATTR4_XATTR_SUPPORT
 
update FATTR4_XATTR_SUPPORT to 82 based on latest 
draft-ietf-nfsv4-xattrs-06.txt
Push to version 2.5
 
Change-Id: Id9afe2cedc9de080e5008a4a8bb502f47ebd0116
Signed-off-by: Marc Eshel <es...@us.ibm.com>



From:   "Frank Filz" <ffilz...@mindspring.com>
To: <d...@redhat.com>, "'Matt Benjamin'" <mbenj...@redhat.com>
Cc: 'nfs-ganesha-devel' <nfs-ganesha-devel@lists.sourceforge.net>
Date:   08/09/2017 09:05 AM
Subject:Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2



> On 08/09/2017 11:29 AM, Frank Filz wrote:
> > Candidates not merged into 2.6 yet:
> >
> > Fix rgw_mount2 check when RGW not installed Daniel
> Gryniewicz
> > CMake - Have 'make dist' generate the correct tarball name Daniel
> > Gryniewicz
> > Build libntirpc package when not using system ntirpc Daniel
> > Gryniewicz
> 
> These could be useful to upstream Ceph for building.  However, it's not 
vital.
> For now, they have to have a separate branch with patches to do this.

If it helps upstream Ceph for building, that seems like a good reason to 
take them.

Frank


---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus


--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel






--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-09 Thread Frank Filz
> On 08/09/2017 11:29 AM, Frank Filz wrote:
> > Candidates not merged into 2.6 yet:
> >
> > Fix rgw_mount2 check when RGW not installed Daniel
> Gryniewicz
> > CMake - Have 'make dist' generate the correct tarball name Daniel
> > Gryniewicz
> > Build libntirpc package when not using system ntirpcDaniel
> > Gryniewicz
> 
> These could be useful to upstream Ceph for building.  However, it's not vital.
> For now, they have to have a separate branch with patches to do this.

If it helps upstream Ceph for building, that seems like a good reason to take 
them.

Frank


---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus


--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-09 Thread Daniel Gryniewicz

On 08/09/2017 11:29 AM, Frank Filz wrote:

Candidates not merged into 2.6 yet:

Fix rgw_mount2 check when RGW not installed Daniel Gryniewicz
CMake - Have 'make dist' generate the correct tarball name
Daniel Gryniewicz
Build libntirpc package when not using system ntirpcDaniel
Gryniewicz


These could be useful to upstream Ceph for building.  However, it's not 
vital.  For now, they have to have a separate branch with patches to do 
this.


Daniel

--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-09 Thread Frank Filz
Candidates not merged into 2.6 yet:

Fix rgw_mount2 check when RGW not installed Daniel Gryniewicz
CMake - Have 'make dist' generate the correct tarball name
Daniel Gryniewicz
Build libntirpc package when not using system ntirpcDaniel
Gryniewicz

Are any of the above three relevant to 2.5? If so, yes.

Ensure partition lock and qlane lock ordering   Daniel Gryniewicz

Yes

Set op_ctx when calling mdcache_lru_clean() in the context of background
thread. Pradeepthomas

Yes

Fixed mappings of Detached_Mult and Dir_Chunk to the right
mdcache_parameter.  Pradeepthomas

Yes

MDCACHE LRU: Increase log level from LogDebug to LogFullDebug
Supriti

Yes

FSAL: Provision for performance stats.  Sachin Punadikar

Probably not unless IBM needs this one.

Allow '.' and '..' as target path in symlink creation for NFSv4
Sachin Punadikar

Yes

 [rpm] Fix rpm release number.  Malahal

Yes

> -Original Message-
> From: Matt Benjamin [mailto:mbenj...@redhat.com]
> Sent: Wednesday, August 9, 2017 7:23 AM
> To: Gryniewicz, Daniel <d...@redhat.com>
> Cc: nfs-ganesha-devel <nfs-ganesha-devel@lists.sourceforge.net>
> Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2
> 
> does this include the fix for  readdir chunking config parsing issue?
> I thought that affected 2.5.x
> 
> Matt
> 
> On Wed, Aug 9, 2017 at 9:55 AM, Daniel Gryniewicz <d...@redhat.com>
> wrote:
> > Here's my proposed backports for 2.5.2:
> >
> > commit 7f2d461277521301a417ca368d3c7656edbfc903
> > FSAL_GLUSTER: Reset caller_garray to NULL upon free
> >
> > commit 114c38ce9fcf20878ffce3b454e106089a34ab5d
> > Decrement FD count in fsal_close even if obj_ops.close() fails
> >
> > commit 7f7c92363ba5c96e6fe5add81c81e65e90063cb2
> > MDCACHE - Fix rename/getattrs deadlock
> >
> > commit ef48ab5bc4e3f1a0fd6879433d774975622e8383
> > Dbus config: Allow only root users
> >
> > commit 2b88bcd16aaa653874079dc7401a49bb61421401
> > nfs_init_complete() is called too soon!
> >
> > commit 4aec4811f1c83578df8ecaacf4c8adda58cc3393
> > Fix deadlock in lru_reap_impl and simplify reapers
> >
> > commit d2b701662db9468c967ed6ecfd274625b1902a6c
> > Add new Reaper_Work_Per_Lane option
> >
> >
> > Daniel
> >
> > --
> >  Check out the vibrant tech community on one of the world's
> > most engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> > ___
> > Nfs-ganesha-devel mailing list
> > Nfs-ganesha-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
> 
>

--
> Check out the vibrant tech community on one of the world's most engaging
> tech sites, Slashdot.org! http://sdm.link/slashdot
> ___
> Nfs-ganesha-devel mailing list
> Nfs-ganesha-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus


--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-09 Thread Frank Filz
I had posted this earlier as a response to the dev-2 announcement...

Since we are talking about tagging V2.5.2 this week in addition to
V2.6-dev-4, I thought I'd indicate which patches from dev-2 probably should
be taken:

There will be more from dev-4 (but nothing from dev-3)

> 860368f Malahal Naineni Add dbus interface to recalculate fd limits

Yes if IBM needs this

> c8bc40b Daniel Gryniewicz CMake - Have 'make dist' generate the 
> correct
tarball name

No

> 814e9cd Gui Hecheng FSAL_RGW: adopt new rgw_mount2 with bucket 
> specified

No

> 39119aa Soumya Koduri FSAL_GLUSTER: Use glfs_xreaddirplus_r for 
> readdir

Yes? No? It's sort of a new feature, but may be critical for some use cases.
I'd rather it go into stable than end up separately backported for
downstream.

> 3441818 Dominique Martinet GPFS: add sys/sysmacros.h include to 
> export.c

Yes

> d2b7016 Daniel Gryniewicz Add new Reaper_Work_Per_Lane option

Yes (it's a new option, but I think this is critical for some use cases of
Ganesha, let's get it into stable sooner than later)

> 18baba1 Kinglong Mee export: add a parameter to indicating lock type 
> for
foreach_gsh_export

Yes

> ac16ad4 Marc Eshel update FATTR4_XATTR_SUPPORT

Yes (spec compliance)

> f0a6a4e Frank S. Filz Fix CREATE_SESSION replay cache

Yes (spec compliance)

> 18ca372 Frank S. Filz Fix EID6h and EID7 pynfs 4.1 test cases

Yes (spec compliance and also fixes a refcount leak)

> 4aec481 Frank S. Filz Fix deadlock in lru_reap_impl and simplify 
> reapers

Yes

Frank


---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus

> -Original Message-
> From: Daniel Gryniewicz [mailto:d...@redhat.com]
> Sent: Wednesday, August 9, 2017 7:49 AM
> To: Matt Benjamin <mbenj...@redhat.com>
> Cc: nfs-ganesha-devel <nfs-ganesha-devel@lists.sourceforge.net>
> Subject: Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2
> 
> That's not in 2.6 yet.  As soon as it is, yes.
> 
> Daniel
> 
> On 08/09/2017 10:22 AM, Matt Benjamin wrote:
> > does this include the fix for  readdir chunking config parsing issue?
> > I thought that affected 2.5.x
> >
> > Matt
> >
> > On Wed, Aug 9, 2017 at 9:55 AM, Daniel Gryniewicz <d...@redhat.com>
> wrote:
> >> Here's my proposed backports for 2.5.2:
> >>
> >> commit 7f2d461277521301a417ca368d3c7656edbfc903
> >>  FSAL_GLUSTER: Reset caller_garray to NULL upon free
> >>
> >> commit 114c38ce9fcf20878ffce3b454e106089a34ab5d
> >>  Decrement FD count in fsal_close even if obj_ops.close() fails
> >>
> >> commit 7f7c92363ba5c96e6fe5add81c81e65e90063cb2
> >>  MDCACHE - Fix rename/getattrs deadlock
> >>
> >> commit ef48ab5bc4e3f1a0fd6879433d774975622e8383
> >>  Dbus config: Allow only root users
> >>
> >> commit 2b88bcd16aaa653874079dc7401a49bb61421401
> >>  nfs_init_complete() is called too soon!
> >>
> >> commit 4aec4811f1c83578df8ecaacf4c8adda58cc3393
> >>  Fix deadlock in lru_reap_impl and simplify reapers
> >>
> >> commit d2b701662db9468c967ed6ecfd274625b1902a6c
> >>  Add new Reaper_Work_Per_Lane option
> >>
> >>
> >> Daniel
> >>
> >> -
> >> - Check out the vibrant tech community on one of the world's
> >> most engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> >> ___
> >> Nfs-ganesha-devel mailing list
> >> Nfs-ganesha-devel@lists.sourceforge.net
> >> https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
> 
> 
>

--
> Check out the vibrant tech community on one of the world's most engaging
> tech sites, Slashdot.org! http://sdm.link/slashdot
> ___
> Nfs-ganesha-devel mailing list
> Nfs-ganesha-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-09 Thread Daniel Gryniewicz

That's not in 2.6 yet.  As soon as it is, yes.

Daniel

On 08/09/2017 10:22 AM, Matt Benjamin wrote:

does this include the fix for  readdir chunking config parsing issue?
I thought that affected 2.5.x

Matt

On Wed, Aug 9, 2017 at 9:55 AM, Daniel Gryniewicz  wrote:

Here's my proposed backports for 2.5.2:

commit 7f2d461277521301a417ca368d3c7656edbfc903
 FSAL_GLUSTER: Reset caller_garray to NULL upon free

commit 114c38ce9fcf20878ffce3b454e106089a34ab5d
 Decrement FD count in fsal_close even if obj_ops.close() fails

commit 7f7c92363ba5c96e6fe5add81c81e65e90063cb2
 MDCACHE - Fix rename/getattrs deadlock

commit ef48ab5bc4e3f1a0fd6879433d774975622e8383
 Dbus config: Allow only root users

commit 2b88bcd16aaa653874079dc7401a49bb61421401
 nfs_init_complete() is called too soon!

commit 4aec4811f1c83578df8ecaacf4c8adda58cc3393
 Fix deadlock in lru_reap_impl and simplify reapers

commit d2b701662db9468c967ed6ecfd274625b1902a6c
 Add new Reaper_Work_Per_Lane option


Daniel

--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel



--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


Re: [Nfs-ganesha-devel] Proposed backports for 2.5.2

2017-08-09 Thread Matt Benjamin
does this include the fix for  readdir chunking config parsing issue?
I thought that affected 2.5.x

Matt

On Wed, Aug 9, 2017 at 9:55 AM, Daniel Gryniewicz  wrote:
> Here's my proposed backports for 2.5.2:
>
> commit 7f2d461277521301a417ca368d3c7656edbfc903
> FSAL_GLUSTER: Reset caller_garray to NULL upon free
>
> commit 114c38ce9fcf20878ffce3b454e106089a34ab5d
> Decrement FD count in fsal_close even if obj_ops.close() fails
>
> commit 7f7c92363ba5c96e6fe5add81c81e65e90063cb2
> MDCACHE - Fix rename/getattrs deadlock
>
> commit ef48ab5bc4e3f1a0fd6879433d774975622e8383
> Dbus config: Allow only root users
>
> commit 2b88bcd16aaa653874079dc7401a49bb61421401
> nfs_init_complete() is called too soon!
>
> commit 4aec4811f1c83578df8ecaacf4c8adda58cc3393
> Fix deadlock in lru_reap_impl and simplify reapers
>
> commit d2b701662db9468c967ed6ecfd274625b1902a6c
> Add new Reaper_Work_Per_Lane option
>
>
> Daniel
>
> --
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> ___
> Nfs-ganesha-devel mailing list
> Nfs-ganesha-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel

--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel