Re: [Gluster-devel] [Gluster-Maintainers] 'Reviewd-by' tag for commits

2016-09-30 Thread Niels de Vos
On Fri, Sep 30, 2016 at 08:50:12PM +0530, Ravishankar N wrote: > On 09/30/2016 06:38 PM, Niels de Vos wrote: > > On Fri, Sep 30, 2016 at 07:11:51AM +0530, Pranith Kumar Karampuri wrote: ... > > Maybe we can add an additional tag that mentions all the people that > > did do reviews of older

Re: [Gluster-devel] [Gluster-Maintainers] GlusterFS-3.7.16 release approaching

2016-09-30 Thread Niels de Vos
On Mon, Sep 26, 2016 at 03:08:42PM +0530, Kaushal M wrote: > Hi all, > > GlusterFS-3.7.16 is on target to be released on Sep 30, 4 days from now. > > In preparation for the release, maintainers please stop merging > anymore changes into release-3.7. > If any developer has a change that needs to

Re: [Gluster-devel] [Gluster-Maintainers] 'Reviewd-by' tag for commits

2016-09-30 Thread Ravishankar N
On 09/30/2016 06:38 PM, Niels de Vos wrote: On Fri, Sep 30, 2016 at 07:11:51AM +0530, Pranith Kumar Karampuri wrote: hi, At the moment 'Reviewed-by' tag comes only if a +1 is given on the final version of the patch. But for most of the patches, different people would spend time on

Re: [Gluster-devel] Community Gluster Package Matrix, updated

2016-09-30 Thread Niels de Vos
On Thu, Sep 29, 2016 at 09:54:24PM -0400, Vijay Bellur wrote: > Thank you Kaleb for putting this together. I think it would also be useful > to list where our official container images would be present too. I think a different page would be most useful, so that we do not overwhelm users too much.

Re: [Gluster-devel] Dht readdir filtering out names

2016-09-30 Thread Prashanth Pai
readdirp needs to be explicitly disabled at 3 places if I remember correctly. At fuse mount option, at dht and at md-cache level. -Prashanth Pai - Original Message - > From: "Pranith Kumar Karampuri" > To: "Raghavendra Gowdappa" > Cc: "Gluster

Re: [Gluster-devel] [Gluster-Maintainers] 'Reviewd-by' tag for commits

2016-09-30 Thread Niels de Vos
On Fri, Sep 30, 2016 at 07:11:51AM +0530, Pranith Kumar Karampuri wrote: > hi, > At the moment 'Reviewed-by' tag comes only if a +1 is given on the > final version of the patch. But for most of the patches, different people > would spend time on different versions making the patch better,

Re: [Gluster-devel] [Gluster-Maintainers] 'Reviewd-by' tag for commits

2016-09-30 Thread Nigel Babu
If you want it to happen automatically, it's next to impossible. We'll need to fork Gerrit to make those changes. On Fri, Sep 30, 2016 at 6:08 PM, Manikandan Selvaganesh < manikandancs...@gmail.com> wrote: > +1, Pranith. I prefer adding everyone's name(who reviewed any patch set > despite the

Re: [Gluster-devel] [Gluster-Maintainers] 'Reviewd-by' tag for commits

2016-09-30 Thread Manikandan Selvaganesh
+1, Pranith. I prefer adding everyone's name(who reviewed any patch set despite the latest patch set) in the 'Reviewed-by' tag. On Sep 30, 2016 6:02 PM, "Pranith Kumar Karampuri" wrote: > hi, > At the moment 'Reviewed-by' tag comes only if a +1 is given on the > final

Re: [Gluster-devel] Dht readdir filtering out names

2016-09-30 Thread Pranith Kumar Karampuri
What if the lower xlators want to set the entry->inode to NULL and clear the entry->d_stat to force a lookup on the name? i.e. gfid-split-brain/ia_type mismatches. On Fri, Sep 30, 2016 at 10:00 AM, Raghavendra Gowdappa wrote: > > > - Original Message - > > From:

Re: [Gluster-devel] Dht readdir filtering out names

2016-09-30 Thread Raghavendra Gowdappa
- Original Message - > From: "Pranith Kumar Karampuri" > To: "Raghavendra Gowdappa" > Cc: "Shyam Ranganathan" , "Nithya Balachandran" > , "Gluster Devel" > > Sent: Friday,

Re: [Gluster-devel] Dht readdir filtering out names

2016-09-30 Thread Poornima Gurusiddaiah
In gfapi, we pass down readdirp, irrespective of whether the application called readdir/readdirp. Hence the behaviour will be same for samba and Ganesha i suppose. Regards, Poornima - Original Message - > From: "Pranith Kumar Karampuri" > To: "Raghavendra

Re: [Gluster-devel] Community Gluster Package Matrix, updated

2016-09-30 Thread Vijay Bellur
Thank you Kaleb for putting this together. I think it would also be useful to list where our official container images would be present too. Should we make this content persistent somewhere on our website and have a link from the release notes? The complaints that we encountered after releasing

Re: [Gluster-devel] Dht readdir filtering out names

2016-09-30 Thread Raghavendra Gowdappa
- Original Message - > From: "Pranith Kumar Karampuri" > To: "Raghavendra Gowdappa" > Cc: "Shyam Ranganathan" , "Nithya Balachandran" > , "Gluster Devel" > > Sent: Friday,

Re: [Gluster-devel] RPC, DHT and AFR logging errors that are expected, reduce log level?

2016-09-30 Thread Pranith Kumar Karampuri
On Thu, Sep 29, 2016 at 7:51 PM, Niels de Vos wrote: > Hello, > > When NFS-Ganesha does an UNLINK of a filename on an inode, it does a > follow-up check to see if the inode has been deleted or if there are > still other filenames linked (like hardlinks) to it. > > Users are

[Gluster-devel] 'Reviewd-by' tag for commits

2016-09-30 Thread Pranith Kumar Karampuri
hi, At the moment 'Reviewed-by' tag comes only if a +1 is given on the final version of the patch. But for most of the patches, different people would spend time on different versions making the patch better, they may not get time to do the review for every version of the patch. Is it

[Gluster-devel] Dht readdir filtering out names

2016-09-30 Thread Pranith Kumar Karampuri
hi, In dht_readdirp_cbk() there is a check about skipping files without ia_type. Could you help me understand why this check is added? There are times when users have to delete gfid of the entries and trigger something like 'find . | xargs stat' to heal the gfids. This case would fail if we

Re: [Gluster-devel] Dht readdir filtering out names

2016-09-30 Thread Pranith Kumar Karampuri
On Fri, Sep 30, 2016 at 9:58 AM, Raghavendra Gowdappa wrote: > > > - Original Message - > > From: "Pranith Kumar Karampuri" > > To: "Raghavendra Gowdappa" > > Cc: "Shyam Ranganathan" , "Nithya

Re: [Gluster-devel] Dht readdir filtering out names

2016-09-30 Thread Raghavendra Gowdappa
dht_readdirp_cbk has different behaviour for directories and files. 1. If file, pick the dentry (passed from subvols as part of readdirp response) if the it corresponds to data file. 2. If directory pick the dentry if readdirp response is from hashed-subvol. In all other cases, the dentry is

Re: [Gluster-devel] logs/cores for smoke failures

2016-09-30 Thread Nigel Babu
On Tue, Sep 27, 2016 at 01:13:15PM +0530, Nigel Babu wrote: > On Tue, Sep 27, 2016 at 12:52:45PM +0530, Pranith Kumar Karampuri wrote: > > On Tue, Sep 27, 2016 at 12:39 PM, Nigel Babu wrote: > > > > > On Tue, Sep 27, 2016 at 12:00:40PM +0530, Pranith Kumar Karampuri wrote: > >

Re: [Gluster-devel] Dht readdir filtering out names

2016-09-30 Thread Soumya Koduri
On 09/30/2016 10:08 AM, Pranith Kumar Karampuri wrote: Does samba/gfapi/nfs-ganesha have options to disable readdirp? AFAIK, currently there is no option to disable/enable readdirp in gfapi & nfs-ganesha (not sure about samba). But looks like nfs-ganesha seem to be always using readdir,

Re: [Gluster-devel] RPC, DHT and AFR logging errors that are expected, reduce log level?

2016-09-30 Thread Raghavendra Gowdappa
- Original Message - > From: "Niels de Vos" > To: gluster-devel@gluster.org > Cc: "Raghavendra Gowdappa" , "Nithya Balachandran" > , "Pranith Kumar > Karampuri" > Sent: Thursday, September 29, 2016

Re: [Gluster-devel] GlusterFs upstream bugzilla components Fine graining

2016-09-30 Thread Prasanna Kalever
On Fri, Sep 30, 2016 at 3:16 PM, Niels de Vos wrote: > On Wed, Sep 28, 2016 at 10:09:34PM +0530, Prasanna Kalever wrote: >> On Wed, Sep 28, 2016 at 11:24 AM, Muthu Vigneshwaran >> wrote: >> > >> > Hi, >> > >> > This an update to the previous mail about

Re: [Gluster-devel] Dht readdir filtering out names

2016-09-30 Thread Raghavendra Gowdappa
- Original Message - > From: "Pranith Kumar Karampuri" > To: "Raghavendra Gowdappa" > Cc: "Shyam Ranganathan" , "Nithya Balachandran" > , "Gluster Devel" > > Sent: Friday,

Re: [Gluster-devel] Dht readdir filtering out names

2016-09-30 Thread Pranith Kumar Karampuri
On Fri, Sep 30, 2016 at 9:13 AM, Raghavendra Gowdappa wrote: > dht_readdirp_cbk has different behaviour for directories and files. > > 1. If file, pick the dentry (passed from subvols as part of readdirp > response) if the it corresponds to data file. > 2. If directory pick

Re: [Gluster-devel] logs/cores for smoke failures

2016-09-30 Thread Pranith Kumar Karampuri
On Fri, Sep 30, 2016 at 4:00 PM, Nigel Babu wrote: > On Tue, Sep 27, 2016 at 01:13:15PM +0530, Nigel Babu wrote: > > On Tue, Sep 27, 2016 at 12:52:45PM +0530, Pranith Kumar Karampuri wrote: > > > On Tue, Sep 27, 2016 at 12:39 PM, Nigel Babu > wrote: > > > >

Re: [Gluster-devel] Dht readdir filtering out names

2016-09-30 Thread Raghavendra Gowdappa
- Original Message - > From: "Raghavendra Gowdappa" > To: "Pranith Kumar Karampuri" > Cc: "Shyam Ranganathan" , "Nithya Balachandran" > , "Gluster Devel" > > Sent: Friday,

Re: [Gluster-devel] [Gluster-Maintainers] Disable experimental features in Release-3.9

2016-09-30 Thread Niels de Vos
On Thu, Sep 29, 2016 at 11:29:30AM +0530, Aravinda wrote: > Hi, > > Following features are not ready or not planned for 3.9 release. Do we have > any option to disable these features using ./configure options in > release-3.9 branch or we need to revert all the patches related to the > feature. >

Re: [Gluster-devel] Dht readdir filtering out names

2016-09-30 Thread Pranith Kumar Karampuri
On Fri, Sep 30, 2016 at 9:50 AM, Raghavendra Gowdappa wrote: > > > - Original Message - > > From: "Pranith Kumar Karampuri" > > To: "Raghavendra Gowdappa" > > Cc: "Shyam Ranganathan" , "Nithya

Re: [Gluster-devel] Dht readdir filtering out names

2016-09-30 Thread Soumya Koduri
On 09/30/2016 03:02 PM, Poornima Gurusiddaiah wrote: In gfapi, we pass down readdirp, irrespective of whether the application called readdir/readdirp. Hence the behaviour will be same for samba and Ganesha i suppose. But in gfapi, I see clear distinction for readdir and readdirp calls

Re: [Gluster-devel] Dht readdir filtering out names

2016-09-30 Thread Pranith Kumar Karampuri
Does samba/gfapi/nfs-ganesha have options to disable readdirp? On Fri, Sep 30, 2016 at 10:04 AM, Pranith Kumar Karampuri < pkara...@redhat.com> wrote: > What if the lower xlators want to set the entry->inode to NULL and clear > the entry->d_stat to force a lookup on the name? i.e. >

Re: [Gluster-devel] libgfapi zero copy write - application in samba, nfs-ganesha

2016-09-30 Thread Raghavendra G
On Thu, Sep 29, 2016 at 11:11 AM, Raghavendra G wrote: > > > On Wed, Sep 28, 2016 at 7:37 PM, Shyam wrote: > >> On 09/27/2016 04:02 AM, Poornima Gurusiddaiah wrote: >> >>> W.r.t Samba consuming this, it requires a great deal of code change in >>>

Re: [Gluster-devel] GlusterFs upstream bugzilla components Fine graining

2016-09-30 Thread Niels de Vos
On Wed, Sep 28, 2016 at 10:09:34PM +0530, Prasanna Kalever wrote: > On Wed, Sep 28, 2016 at 11:24 AM, Muthu Vigneshwaran > wrote: > > > > Hi, > > > > This an update to the previous mail about Fine graining of the > > GlusterFS upstream bugzilla components. > > > > Finally we

[Gluster-devel] Disable experimental features in Release-3.9

2016-09-30 Thread Aravinda
Hi, Following features are not ready or not planned for 3.9 release. Do we have any option to disable these features using ./configure options in release-3.9 branch or we need to revert all the patches related to the feature. - dht2 - jbr Please add if we need to disable/remove any other

Re: [Gluster-devel] libgfapi zero copy write - application in samba, nfs-ganesha

2016-09-30 Thread Raghavendra G
On Wed, Sep 28, 2016 at 7:37 PM, Shyam wrote: > On 09/27/2016 04:02 AM, Poornima Gurusiddaiah wrote: > >> W.r.t Samba consuming this, it requires a great deal of code change in >> Samba. >> Currently samba has no concept of getting buf from the underlying file >> system, >>

Re: [Gluster-devel] [Gluster-infra] Renaming the log files for regression runs

2016-09-30 Thread Nigel Babu
On Tue, Aug 02, 2016 at 10:02:41AM -0400, Vijay Bellur wrote: > On Tue, Aug 2, 2016 at 1:59 AM, Nigel Babu wrote: > > Hello, > > > > Nithya suggested we rename the log files to have the Jenkins Job ID, making > > it > > easier to match the log file to the job. I think this is

Re: [Gluster-devel] [regression tests] Aborted runs and diagnostic information

2016-09-30 Thread Raghavendra Gowdappa
- Original Message - > From: "Nigel Babu" > To: "Raghavendra Gowdappa" > Cc: "gluster-infra" , "Gluster Devel" > > Sent: Wednesday, September 28, 2016 6:28:11 PM > Subject: Re: [regression

Re: [Gluster-devel] [Gluster-users] An Update on GlusterD-2.0

2016-09-30 Thread Vijay Bellur
On 09/29/2016 12:48 AM, Kaushal M wrote: On Thu, Sep 29, 2016 at 10:10 AM, Vijay Bellur wrote: On 09/22/2016 07:28 AM, Kaushal M wrote: The first preview/dev release of GlusterD-2.0 is available now. A prebuilt binary is available for download from the release-page[1].

Re: [Gluster-devel] [Gluster-users] An Update on GlusterD-2.0

2016-09-30 Thread Kaushal M
On Thu, Sep 29, 2016 at 10:10 AM, Vijay Bellur wrote: > On 09/22/2016 07:28 AM, Kaushal M wrote: >> >> The first preview/dev release of GlusterD-2.0 is available now. A >> prebuilt binary is available for download from the release-page[1]. >> >> This is just a preview of what

Re: [Gluster-devel] [Gluster-users] An Update on GlusterD-2.0

2016-09-30 Thread Vijay Bellur
On 09/22/2016 07:28 AM, Kaushal M wrote: The first preview/dev release of GlusterD-2.0 is available now. A prebuilt binary is available for download from the release-page[1]. This is just a preview of what has been happening in GD2, to give users a taste of how GD2 is evolving. GD2 can now

Re: [Gluster-devel] GlusterFs upstream bugzilla components Fine graining

2016-09-30 Thread Prasanna Kalever
On Wed, Sep 28, 2016 at 11:24 AM, Muthu Vigneshwaran wrote: > > Hi, > > This an update to the previous mail about Fine graining of the > GlusterFS upstream bugzilla components. > > Finally we have come out a new structure that would help in easy > access of the bug for