Re: [Gluster-devel] Need inputs on patch #17985

2017-12-06 Thread Milind Changire
> From: "FNU Raghavendra Manjunath" >>> > To: "Raghavendra Gowdappa" >>> > Cc: "Raghavendra G" , "Nithya Balachandran" >>> , anoo...@redhat.com, >>> > "Gluster Devel" , "Raghavendra Bhat&q

Re: [Gluster-devel] Need inputs on patch #17985

2017-11-29 Thread Raghavendra G
y and hence no nodeid corresponding to >> .snaps is passed to kernel and kernel is forced to do an explicit lookup". >> >> > >> > Regards, >> > Raghavendra >> > >> > >> > On Tue, Aug 29, 2017 at 12:53 AM, Raghavendra Gowdappa &l

Re: [Gluster-devel] Need inputs on patch #17985

2017-09-11 Thread Raghavendra G
"Raghavendra Gowdappa" > > Cc: "Raghavendra G" , "Nithya Balachandran" < > nbala...@redhat.com>, anoo...@redhat.com, > > "Gluster Devel" , "Raghavendra Bhat" < > raghaven...@redhat.com> > > Sent: Thursday, September 7, 2017

Re: [Gluster-devel] Need inputs on patch #17985

2017-09-07 Thread Raghavendra Gowdappa
cit lookup". > > Regards, > Raghavendra > > > On Tue, Aug 29, 2017 at 12:53 AM, Raghavendra Gowdappa > wrote: > > > > > > > - Original Message - > > > From: "Raghavendra G" > > > To: "Nithya Balacha

Re: [Gluster-devel] Need inputs on patch #17985

2017-09-07 Thread FNU Raghavendra Manjunath
t;Gluster Devel" , > > raghaven...@redhat.com > > Sent: Tuesday, August 29, 2017 8:52:28 AM > > Subject: Re: [Gluster-devel] Need inputs on patch #17985 > > > > On Thu, Aug 24, 2017 at 2:53 PM, Nithya Balachandran < > nbala...@redhat.com> > > wrote: &

Re: [Gluster-devel] Need inputs on patch #17985

2017-09-06 Thread Raghavendra G
Balachandran" , anoo...@redhat.com, > "Gluster Devel" , > > raghaven...@redhat.com > > Sent: Tuesday, August 29, 2017 10:23:49 AM > > Subject: Re: [Gluster-devel] Need inputs on patch #17985 > > > > > > > > - Original Message - >

Re: [Gluster-devel] Need inputs on patch #17985

2017-08-29 Thread Raghavendra Gowdappa
- Original Message - > From: "Raghavendra Gowdappa" > To: "Raghavendra G" > Cc: "Nithya Balachandran" , anoo...@redhat.com, "Gluster > Devel" , > raghaven...@redhat.com > Sent: Tuesday, August 29, 2017 10:23:49 AM

Re: [Gluster-devel] Need inputs on patch #17985

2017-08-28 Thread Raghavendra Gowdappa
- Original Message - > From: "Raghavendra G" > To: "Nithya Balachandran" > Cc: "Raghavendra Gowdappa" , anoo...@redhat.com, > "Gluster Devel" , > raghaven...@redhat.com > Sent: Tuesday, August 29, 2017 8:52:28 AM > Subject

Re: [Gluster-devel] Need inputs on patch #17985

2017-08-28 Thread Raghavendra G
On Thu, Aug 24, 2017 at 2:53 PM, Nithya Balachandran wrote: > It has been a while but iirc snapview client (loaded abt dht/tier etc) had > some issues when we ran tiering tests. Rafi might have more info on this - > basically it was expecting to find the inode_ctx populated but it was not. > Tha

Re: [Gluster-devel] Need inputs on patch #17985

2017-08-24 Thread Nithya Balachandran
It has been a while but iirc snapview client (loaded abt dht/tier etc) had some issues when we ran tiering tests. Rafi might have more info on this - basically it was expecting to find the inode_ctx populated but it was not. On 24 August 2017 at 10:13, Raghavendra G wrote: > Note that we need to

Re: [Gluster-devel] Need inputs on patch #17985

2017-08-23 Thread Raghavendra G
Note that we need to consider xlators on brick stack too. I've added maintainers/peers of xlators on brick stack. Please explicitly ack/nack whether this patch affects your component. For reference, following are the xlators loaded in brick stack storage/posix features/trash features/changetimere

Re: [Gluster-devel] Need inputs on patch #17985

2017-08-22 Thread Raghavendra Gowdappa
Thanks Pranith and Ashish for your inputs. - Original Message - > From: "Pranith Kumar Karampuri" > To: "Ashish Pandey" > Cc: "Raghavendra Gowdappa" , "Xavier Hernandez" > , "Gluster Devel" > > Sent: Wednesday, August 23, 2017 11:55:19 AM > Subject: Re: Need inputs on patch #17985 > >

Re: [Gluster-devel] Need inputs on patch #17985

2017-08-22 Thread Pranith Kumar Karampuri
Raghavendra, As Ashish mentioned, there aren't any known problems if upper xlators don't send lookups in EC at the moment. On Wed, Aug 23, 2017 at 9:07 AM, Ashish Pandey wrote: > Raghvendra, > > I have provided my comment on this patch. > I think EC will not have any issue with this approach

Re: [Gluster-devel] Need inputs on patch #17985

2017-08-22 Thread Ashish Pandey
Raghvendra, I have provided my comment on this patch. I think EC will not have any issue with this approach. However, I would welcome comments from Xavi and Pranith too for any side effects which I may not be able to foresee. Ashish - Original Message - From: "Raghavendra Gowdappa

[Gluster-devel] Need inputs on patch #17985

2017-08-22 Thread Raghavendra Gowdappa
Hi Ashish, Following are the blockers for making a decision on whether patch [1] can be merged or not: * Evaluation of dentry operations (like rename etc) in dht * Whether EC works fine if a non-lookup fop (like open(dir), stat, chmod etc) hits EC without a single lookup performed on file/inode