[Gluster-devel] Support to reclaim locks (posix) provided lkowner & range matches

2016-07-22 Thread Soumya Koduri
Hi, In certain scenarios (esp.,in highly available environments), the application may have to fail-over/connect to a different glusterFS client while the I/O is happening. In such cases until there is a ping timer expiry and glusterFS server cleans up the locks held by the older glusterFS cli

Re: [Gluster-devel] GF_PARENT_DOWN on SIGKILL

2016-07-22 Thread Jeff Darcy
> Does anyone know why GF_PARENT_DOWN is not triggered on SIGKILL? It will give > a chance for xlators to do any cleanup they need to do. For example ec can > complete the delayed xattrops. Nothing is triggered on SIGKILL. SIGKILL is explicitly defined to terminate a process *immediately*. Among

Re: [Gluster-devel] GF_PARENT_DOWN on SIGKILL

2016-07-22 Thread Pranith Kumar Karampuri
Gah! sorry sorry, I meant to send the mail as SIGTERM. Not SIGKILL. So xavi and I were wondering why cleanup_and_exit() is not sending GF_PARENT_DOWN event. On Fri, Jul 22, 2016 at 6:24 PM, Jeff Darcy wrote: > > Does anyone know why GF_PARENT_DOWN is not triggered on SIGKILL? It will > give > >

Re: [Gluster-devel] GF_PARENT_DOWN on SIGKILL

2016-07-22 Thread Pranith Kumar Karampuri
It is only calling fini() apart from that not much. On Fri, Jul 22, 2016 at 6:36 PM, Pranith Kumar Karampuri < pkara...@redhat.com> wrote: > Gah! sorry sorry, I meant to send the mail as SIGTERM. Not SIGKILL. So > xavi and I were wondering why cleanup_and_exit() is not sending > GF_PARENT_DOWN ev

Re: [Gluster-devel] GF_PARENT_DOWN on SIGKILL

2016-07-22 Thread Pranith Kumar Karampuri
http://review.gluster.org/14980, this is where we have all the context about why I sent out this mail. Basically the tests were failing because umount is racing with version-updation xattrop. While I fixed the test to handle that race, xavi was wondering why GF_PARENT_DOWN event didn't come. I foun

Re: [Gluster-devel] [Gluster-users] 3.7.13 & proxmox/qemu

2016-07-22 Thread Vijay Bellur
2016-07-22 1:54 GMT-04:00 Frank Rothenstein : > The point is that even if all other backend storage filesystems do correctly > untill 3.7.11 there was no error on ZFS. Something happened nobody ever > could explain in the release of 3.7.12 that makes FUSE-mount _in ovirt_ (it > partly uses dd with

Re: [Gluster-devel] Support to reclaim locks (posix) provided lkowner & range matches

2016-07-22 Thread Ira Cooper
SMB3 with persistent handles has very similar needs. Michael, You may also want to review this. Cheers, -Ira - Original Message - > Hi, > > In certain scenarios (esp.,in highly available environments), the > application may have to fail-over/connect to a different glusterFS > client w

Re: [Gluster-devel] [Gluster-users] 3.7.13 & proxmox/qemu

2016-07-22 Thread Samuli Heinonen
> On 21 Jul 2016, at 20:48, David Gossage wrote: > > Wonder if this may be related at all > > * #1347553: O_DIRECT support for sharding > https://bugzilla.redhat.com/show_bug.cgi?id=1347553 > > Is it possible to downgrade from 3.8 back to 3.7.x > > Building test box right now anyway but wond

Re: [Gluster-devel] GF_PARENT_DOWN on SIGKILL

2016-07-22 Thread Jeff Darcy
> Gah! sorry sorry, I meant to send the mail as SIGTERM. Not SIGKILL. So xavi > and I were wondering why cleanup_and_exit() is not sending GF_PARENT_DOWN > event. OK, then that grinding sound you hear is my brain shifting gears. ;) It seems that cleanup_and_exit will call xlator.fini in some few

Re: [Gluster-devel] GF_PARENT_DOWN on SIGKILL

2016-07-22 Thread Pranith Kumar Karampuri
On Fri, Jul 22, 2016 at 7:07 PM, Jeff Darcy wrote: > > Gah! sorry sorry, I meant to send the mail as SIGTERM. Not SIGKILL. So > xavi > > and I were wondering why cleanup_and_exit() is not sending GF_PARENT_DOWN > > event. > > OK, then that grinding sound you hear is my brain shifting gears. ;)

Re: [Gluster-devel] [Gluster-users] 3.7.13 & proxmox/qemu

2016-07-22 Thread David Gossage
On Fri, Jul 22, 2016 at 8:23 AM, Samuli Heinonen wrote: > > > On 21 Jul 2016, at 20:48, David Gossage > wrote: > > > > Wonder if this may be related at all > > > > * #1347553: O_DIRECT support for sharding > > https://bugzilla.redhat.com/show_bug.cgi?id=1347553 > > > > Is it possible to downgrad

Re: [Gluster-devel] [Gluster-users] 3.7.13 & proxmox/qemu

2016-07-22 Thread David Gossage
On Fri, Jul 22, 2016 at 8:12 AM, Vijay Bellur wrote: > 2016-07-22 1:54 GMT-04:00 Frank Rothenstein < > f.rothenst...@bodden-kliniken.de>: > > The point is that even if all other backend storage filesystems do > correctly > > untill 3.7.11 there was no error on ZFS. Something happened nobody ever

Re: [Gluster-devel] [Gluster-users] 3.7.13 & proxmox/qemu

2016-07-22 Thread Samuli Heinonen
Here is a quick way how to test this: GlusterFS 3.7.13 volume with default settings with brick on ZFS dataset. gluster-test1 is server and gluster-test2 is client mounting with FUSE. Writing file with oflag=direct is not ok: [root@gluster-test2 gluster]# dd if=/dev/zero of=file oflag=direct count

Re: [Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Jeff Darcy
> I attempted to get us more space on NetBSD by creating a new partition called > /data and putting /build as a symlink to /data/build. This has caused > problems > with tests/basic/quota.t. It's marked as bad for master, but not for > release-3.7. This is possibly because we have a hard-coded grep

Re: [Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Nithya Balachandran
On Fri, Jul 22, 2016 at 7:31 PM, Jeff Darcy wrote: > > I attempted to get us more space on NetBSD by creating a new partition > called > > /data and putting /build as a symlink to /data/build. This has caused > > problems > > with tests/basic/quota.t. It's marked as bad for master, but not for >

Re: [Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Pranith Kumar Karampuri
On Fri, Jul 22, 2016 at 7:42 PM, Pranith Kumar Karampuri < pkara...@redhat.com> wrote: > > > On Fri, Jul 22, 2016 at 7:39 PM, Nithya Balachandran > wrote: > >> >> >> On Fri, Jul 22, 2016 at 7:31 PM, Jeff Darcy wrote: >> >>> > I attempted to get us more space on NetBSD by creating a new partition

[Gluster-devel] Notifications (was Re: GF_PARENT_DOWN on SIGKILL)

2016-07-22 Thread Jeff Darcy
> I don't think we need any list traversal because notify sends it down > the graph. Good point. I think we need to change that, BTW. Relying on translators to propagate notifications has proven very fragile, as many of those events are overloaded to mean very different things to different trans

Re: [Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Nithya Balachandran
On Fri, Jul 22, 2016 at 7:42 PM, Pranith Kumar Karampuri < pkara...@redhat.com> wrote: > > > On Fri, Jul 22, 2016 at 7:39 PM, Nithya Balachandran > wrote: > >> >> >> On Fri, Jul 22, 2016 at 7:31 PM, Jeff Darcy wrote: >> >>> > I attempted to get us more space on NetBSD by creating a new partition

Re: [Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Pranith Kumar Karampuri
On Fri, Jul 22, 2016 at 7:39 PM, Nithya Balachandran wrote: > > > On Fri, Jul 22, 2016 at 7:31 PM, Jeff Darcy wrote: > >> > I attempted to get us more space on NetBSD by creating a new partition >> called >> > /data and putting /build as a symlink to /data/build. This has caused >> > problems >>

Re: [Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Jeff Darcy
> > Excluding /build/install for no obvious reason at all > > This looks like it was done to remove the /build/install components from the > df -h outputs. Changing the path to /data/build/install broke this as it did > not strip the "/data" from the paths. > It did work when I changed the sed to

Re: [Gluster-devel] [Gluster-users] 3.7.13 & proxmox/qemu

2016-07-22 Thread Vijay Bellur
On Fri, Jul 22, 2016 at 10:03 AM, Samuli Heinonen wrote: > Here is a quick way how to test this: > GlusterFS 3.7.13 volume with default settings with brick on ZFS dataset. > gluster-test1 is server and gluster-test2 is client mounting with FUSE. > > Writing file with oflag=direct is not ok: > [ro

Re: [Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Jeff Darcy
> This path might be available as an env variable - but is there a better way > to figure out the aux mount without bothering with df -h? I believe so. The function that creates the mount is: glusterd_create_quota_auxiliary_mount That function *also* creates log and pid files. The pid file

Re: [Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Pranith Kumar Karampuri
I am playing with the following diff, let me see. diff --git a/tests/volume.rc b/tests/volume.rc index 331a802..b288508 100644 --- a/tests/volume.rc +++ b/tests/volume.rc @@ -579,7 +579,9 @@ function num_graphs function get_aux() { ##Check if a auxiliary mount is there -df -h 2>&1 | sed 's#/bui

Re: [Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Pranith Kumar Karampuri
On Fri, Jul 22, 2016 at 4:46 PM, Nigel Babu wrote: > Hello, > > I attempted to get us more space on NetBSD by creating a new partition > called > /data and putting /build as a symlink to /data/build. This has caused > problems > with tests/basic/quota.t. It's marked as bad for master, but not for

Re: [Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Pranith Kumar Karampuri
On Fri, Jul 22, 2016 at 8:12 PM, Pranith Kumar Karampuri < pkara...@redhat.com> wrote: > I am playing with the following diff, let me see. > > diff --git a/tests/volume.rc b/tests/volume.rc > index 331a802..b288508 100644 > --- a/tests/volume.rc > +++ b/tests/volume.rc > @@ -579,7 +579,9 @@ functi

Re: [Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Jeff Darcy
> Based on what I saw in code, this seems to get the job done. Comments > welcome: > http://review.gluster.org/14988 Good thinking. Thanks, Pranith! ___ Gluster-devel mailing list Gluster-devel@gluster.org http://www.gluster.org/mailman/listinfo/gluster

Re: [Gluster-devel] [Gluster-users] 3.7.13 & proxmox/qemu

2016-07-22 Thread David Gossage
On Fri, Jul 22, 2016 at 9:37 AM, Vijay Bellur wrote: > On Fri, Jul 22, 2016 at 10:03 AM, Samuli Heinonen > wrote: > > Here is a quick way how to test this: > > GlusterFS 3.7.13 volume with default settings with brick on ZFS dataset. > gluster-test1 is server and gluster-test2 is client mounting

Re: [Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Pranith Kumar Karampuri
On Fri, Jul 22, 2016 at 10:25 PM, Jeff Darcy wrote: > > Based on what I saw in code, this seems to get the job done. Comments > > welcome: > > http://review.gluster.org/14988 > > Good thinking. Thanks, Pranith! > Nitya clarified my doubts as well on IRC :-). -- Pranith __

[Gluster-devel] Open Source Storage Summit

2016-07-22 Thread Amye Scavarda
This was a little late being announced officially, but EMC{code} is hosting Open Source Storage Summits for the next two LinuxCons: North American and Europe. https://www.linux.com/blog/announcing-open-source-storage-summit At LinuxCon Japan, I was asked to speak for the Gluster project, and it e

Re: [Gluster-devel] [Gluster-users] 3.7.13 & proxmox/qemu

2016-07-22 Thread Frank Rothenstein
I can't tell myself, I'm using the ovirt-4.0-centos-gluster37 repo (from ovirt-release40). I have a second gluster-cluster as storage, I didn't dare to upgrade, as it simply works...not as an ovirt/vm storage. Am Freitag, den 22.07.2016, 08:28 +0200 schrieb Gandalf Corvotempesta: > > Il 22 lug 2016

Re: [Gluster-devel] [Gluster-users] 3.7.13 & proxmox/qemu

2016-07-22 Thread Frank Rothenstein
The point is that even if all other backend storage filesystems do correctly untill 3.7.11 there was no error on ZFS. Something happened nobody ever could explain in the release of 3.7.12 that makes FUSE- mount _in ovirt_ (it partly uses dd with iflag=direct  , using iflag=direct yourself gives als

[Gluster-devel] spurious failure in tests/basic/gfapi/libgfapi-fini-hang.t

2016-07-22 Thread Pranith Kumar Karampuri
I see both of your names in git blame output. https://build.gluster.org/job/rackspace-regression-2GB-triggered/22439/console has more information about the failure. This failure happened on http://review.gluster.org/#/c/14985/ which changes only .t files so I believe the reason for the failure to b

Re: [Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Nithya Balachandran
On Fri, Jul 22, 2016 at 9:07 PM, Pranith Kumar Karampuri < pkara...@redhat.com> wrote: > > > On Fri, Jul 22, 2016 at 8:12 PM, Pranith Kumar Karampuri < > pkara...@redhat.com> wrote: > >> I am playing with the following diff, let me see. >> >> diff --git a/tests/volume.rc b/tests/volume.rc >> index

Re: [Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Nithya Balachandran
On Sat, Jul 23, 2016 at 9:45 AM, Nithya Balachandran wrote: > > > On Fri, Jul 22, 2016 at 9:07 PM, Pranith Kumar Karampuri < > pkara...@redhat.com> wrote: > >> >> >> On Fri, Jul 22, 2016 at 8:12 PM, Pranith Kumar Karampuri < >> pkara...@redhat.com> wrote: >> >>> I am playing with the following di

Re: [Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Pranith Kumar Karampuri
On Sat, Jul 23, 2016 at 10:17 AM, Nithya Balachandran wrote: > > > On Sat, Jul 23, 2016 at 9:45 AM, Nithya Balachandran > wrote: > >> >> >> On Fri, Jul 22, 2016 at 9:07 PM, Pranith Kumar Karampuri < >> pkara...@redhat.com> wrote: >> >>> >>> >>> On Fri, Jul 22, 2016 at 8:12 PM, Pranith Kumar Kara

Re: [Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Atin Mukherjee
AFAIK, an explicit rebase is required. On Saturday 23 July 2016, Pranith Kumar Karampuri wrote: > > > On Sat, Jul 23, 2016 at 10:17 AM, Nithya Balachandran > wrote: > >> >> >> On Sat, Jul 23, 2016 at 9:45 AM, Nithya Balachandran > > wrote: >> >>> >>> >>> On Fri, Jul 22, 2016 at 9:07 PM, Pranith

Re: [Gluster-devel] [Gluster-users] 3.7.13 & proxmox/qemu

2016-07-22 Thread David Gossage
2016-07-22 2:32 GMT-05:00 Frank Rothenstein < f.rothenst...@bodden-kliniken.de>: > I can't tell myself, I'm using the ovirt-4.0-centos-gluster37 repo > (from ovirt-release40). I have a second gluster-cluster as storage, I > didn't dare to upgrade, as it simply works...not as an ovirt/vm storage. >

[Gluster-devel] GF_PARENT_DOWN on SIGKILL

2016-07-22 Thread Pranith Kumar Karampuri
Does anyone know why GF_PARENT_DOWN is not triggered on SIGKILL? It will give a chance for xlators to do any cleanup they need to do. For example ec can complete the delayed xattrops. -- Pranith ___ Gluster-devel mailing list Gluster-devel@gluster.org h

Re: [Gluster-devel] Question on merging zfs snapshot support into the mainline glusterfs

2016-07-22 Thread Rajesh Joseph
On Thu, Jul 21, 2016 at 3:07 AM, Vijay Bellur wrote: > On 07/19/2016 11:01 AM, Atin Mukherjee wrote: > >> >> >> On Tue, Jul 19, 2016 at 7:29 PM, Rajesh Joseph > > wrote: >> >> >> >> On Tue, Jul 19, 2016 at 11:23 AM, > > wrote: >> >>

[Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Nigel Babu
Hello, I attempted to get us more space on NetBSD by creating a new partition called /data and putting /build as a symlink to /data/build. This has caused problems with tests/basic/quota.t. It's marked as bad for master, but not for release-3.7. This is possibly because we have a hard-coded grep f

Re: [Gluster-devel] 3.7 regressions on NetBSD

2016-07-22 Thread Atin Mukherjee
On Fri, Jul 22, 2016 at 4:46 PM, Nigel Babu wrote: > Hello, > > I attempted to get us more space on NetBSD by creating a new partition > called > /data and putting /build as a symlink to /data/build. This has caused > problems > with tests/basic/quota.t. It's marked as bad for master, but not for