Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-31 Thread Vijay Bellur
On 05/28/2015 08:16 AM, Vijay Bellur wrote: On 05/22/2015 11:45 PM, Vijay Bellur wrote: On 05/21/2015 12:07 AM, Vijay Bellur wrote: On 05/19/2015 11:56 PM, Vijay Bellur wrote: On 05/18/2015 08:03 PM, Vijay Bellur wrote: On 05/16/2015 03:34 PM, Vijay Bellur wrote: I will send daily status

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-27 Thread Vijay Bellur
On 05/22/2015 11:45 PM, Vijay Bellur wrote: On 05/21/2015 12:07 AM, Vijay Bellur wrote: On 05/19/2015 11:56 PM, Vijay Bellur wrote: On 05/18/2015 08:03 PM, Vijay Bellur wrote: On 05/16/2015 03:34 PM, Vijay Bellur wrote: I will send daily status updates from Monday (05/18) about this so

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-26 Thread Vijaikumar M
Here is the status on quota test-case spurious failure: There were 3 issues 1) Quota exceeding the limit because of parallel writes - Merged Upstream, patch submitted to release-3.7 #10910 ./tests/bugs/quota/bug-1038598.t ./tests/bugs/distribute/bug-1161156.t 2) Quoting

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-26 Thread Raghavendra Gowdappa
: Re: [Gluster-devel] Moratorium on new patch acceptance Here is the status on quota test-case spurious failure: There were 3 issues 1) Quota exceeding the limit because of parallel writes - Merged Upstream, patch submitted to release-3.7 #10910 ./tests/bugs/quota/bug-1038598.t

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-22 Thread Vijay Bellur
On 05/21/2015 12:07 AM, Vijay Bellur wrote: On 05/19/2015 11:56 PM, Vijay Bellur wrote: On 05/18/2015 08:03 PM, Vijay Bellur wrote: On 05/16/2015 03:34 PM, Vijay Bellur wrote: I will send daily status updates from Monday (05/18) about this so that we are clear about where we are and what

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-21 Thread Vijaikumar M
On Tuesday 19 May 2015 09:50 PM, Shyam wrote: On 05/19/2015 11:23 AM, Vijaikumar M wrote: On Tuesday 19 May 2015 08:36 PM, Shyam wrote: On 05/19/2015 08:10 AM, Raghavendra G wrote: After discussion with Vijaykumar mallikarjuna and other inputs in this thread, we are proposing all quota

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-21 Thread Shyam
On 05/21/2015 04:04 AM, Vijaikumar M wrote: On Tuesday 19 May 2015 09:50 PM, Shyam wrote: On 05/19/2015 11:23 AM, Vijaikumar M wrote: Did that (in the attached script that I sent) and it still failed. Please note: - This dd command passes (or fails with EDQUOT) - dd if=/dev/zero

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-21 Thread Shyam
On 05/21/2015 09:33 AM, Vijaikumar M wrote: On Thursday 21 May 2015 06:48 PM, Shyam wrote: On 05/21/2015 04:04 AM, Vijaikumar M wrote: On Tuesday 19 May 2015 09:50 PM, Shyam wrote: On 05/19/2015 11:23 AM, Vijaikumar M wrote: Did that (in the attached script that I sent) and it still

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-21 Thread Vijaikumar M
On Thursday 21 May 2015 06:48 PM, Shyam wrote: On 05/21/2015 04:04 AM, Vijaikumar M wrote: On Tuesday 19 May 2015 09:50 PM, Shyam wrote: On 05/19/2015 11:23 AM, Vijaikumar M wrote: Did that (in the attached script that I sent) and it still failed. Please note: - This dd command passes (or

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-20 Thread Raghavendra G
On Tue, May 19, 2015 at 8:36 PM, Shyam srang...@redhat.com wrote: On 05/19/2015 08:10 AM, Raghavendra G wrote: After discussion with Vijaykumar mallikarjuna and other inputs in this thread, we are proposing all quota tests to comply to following criteria: * use dd always with oflag=append

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-20 Thread Vijaikumar M
On Tuesday 19 May 2015 09:50 PM, Shyam wrote: On 05/19/2015 11:23 AM, Vijaikumar M wrote: On Tuesday 19 May 2015 08:36 PM, Shyam wrote: On 05/19/2015 08:10 AM, Raghavendra G wrote: After discussion with Vijaykumar mallikarjuna and other inputs in this thread, we are proposing all quota

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-20 Thread Vijay Bellur
On 05/19/2015 11:56 PM, Vijay Bellur wrote: On 05/18/2015 08:03 PM, Vijay Bellur wrote: On 05/16/2015 03:34 PM, Vijay Bellur wrote: I will send daily status updates from Monday (05/18) about this so that we are clear about where we are and what needs to be done to remove this moratorium.

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-20 Thread Pranith Kumar Karampuri
On 05/21/2015 12:07 AM, Vijay Bellur wrote: On 05/19/2015 11:56 PM, Vijay Bellur wrote: On 05/18/2015 08:03 PM, Vijay Bellur wrote: On 05/16/2015 03:34 PM, Vijay Bellur wrote: I will send daily status updates from Monday (05/18) about this so that we are clear about where we are and what

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-19 Thread Vijay Bellur
On 05/19/2015 12:21 PM, Raghavendra Gowdappa wrote: Yes, this is a possible scenario. There is a finite time window between, 1. Querying the size of a directory. In other words checking whether current write can be allowed 2. The effect of this write getting reflected in size of all the

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-19 Thread Vijaikumar M
On Tuesday 19 May 2015 06:13 AM, Shyam wrote: On 05/18/2015 07:05 PM, Shyam wrote: On 05/18/2015 03:49 PM, Shyam wrote: On 05/18/2015 10:33 AM, Vijay Bellur wrote: The etherpad did not call out, ./tests/bugs/distribute/bug-1161156.t which did not have an owner, and so I took a stab at it

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-19 Thread Raghavendra Gowdappa
- Original Message - From: Shyam srang...@redhat.com To: gluster-devel@gluster.org Sent: Tuesday, May 19, 2015 6:13:06 AM Subject: Re: [Gluster-devel] Moratorium on new patch acceptance On 05/18/2015 07:05 PM, Shyam wrote: On 05/18/2015 03:49 PM, Shyam wrote: On 05/18/2015 10

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-19 Thread Raghavendra Gowdappa
- Original Message - From: Raghavendra Gowdappa rgowd...@redhat.com To: Shyam srang...@redhat.com Cc: gluster-devel@gluster.org Sent: Tuesday, May 19, 2015 11:46:19 AM Subject: Re: [Gluster-devel] Moratorium on new patch acceptance - Original Message - From: Shyam

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-19 Thread Raghavendra Gowdappa
- Original Message - From: Vijay Bellur vbel...@redhat.com To: Raghavendra Gowdappa rgowd...@redhat.com, Shyam srang...@redhat.com Cc: gluster-devel@gluster.org Sent: Tuesday, May 19, 2015 1:29:57 PM Subject: Re: [Gluster-devel] Moratorium on new patch acceptance On 05/19/2015

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-19 Thread Jeff Darcy
No, my suggestion was aimed at not having parallel writes. In this case quota won't even fail the writes with EDQUOT because of reasons explained above. Yes, we need to disable flush-behind along with this so that errors are delivered to application. Would conv=sync help here? That should

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-19 Thread Raghavendra G
On Tue, May 19, 2015 at 4:26 PM, Jeff Darcy jda...@redhat.com wrote: No, my suggestion was aimed at not having parallel writes. In this case quota won't even fail the writes with EDQUOT because of reasons explained above. Yes, we need to disable flush-behind along with this so that errors

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-19 Thread Raghavendra G
After discussion with Vijaykumar mallikarjuna and other inputs in this thread, we are proposing all quota tests to comply to following criteria: * use dd always with oflag=append (to make sure there are no parallel writes) and conv=fdatasync (to make sure errors, if any are delivered to

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-19 Thread Raghavendra G
On Tue, May 19, 2015 at 5:40 PM, Raghavendra G raghaven...@gluster.com wrote: After discussion with Vijaykumar mallikarjuna and other inputs in this thread, we are proposing all quota tests to comply to following criteria: * use dd always with oflag=append (to make sure there are no parallel

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-19 Thread Jeff Darcy
* fdatasync is issued only once at the end of all writes when conv=fdatasync * for some strange reason no fsync or fdatasync is issued at all when conv=sync That's because of my typo. I meant oflag=sync, not conv=sync. Sorry. So, using conv=fdatasync in the test cannot prevent

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-19 Thread Vijaikumar M
On Tuesday 19 May 2015 08:36 PM, Shyam wrote: On 05/19/2015 08:10 AM, Raghavendra G wrote: After discussion with Vijaykumar mallikarjuna and other inputs in this thread, we are proposing all quota tests to comply to following criteria: * use dd always with oflag=append (to make sure there

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-19 Thread Shyam
On 05/19/2015 08:10 AM, Raghavendra G wrote: After discussion with Vijaykumar mallikarjuna and other inputs in this thread, we are proposing all quota tests to comply to following criteria: * use dd always with oflag=append (to make sure there are no parallel writes) and conv=fdatasync (to make

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-19 Thread Vijay Bellur
On 05/18/2015 08:03 PM, Vijay Bellur wrote: On 05/16/2015 03:34 PM, Vijay Bellur wrote: I will send daily status updates from Monday (05/18) about this so that we are clear about where we are and what needs to be done to remove this moratorium. Appreciate your help in having a clean set of

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-19 Thread Shyam
On 05/19/2015 11:23 AM, Vijaikumar M wrote: On Tuesday 19 May 2015 08:36 PM, Shyam wrote: On 05/19/2015 08:10 AM, Raghavendra G wrote: After discussion with Vijaykumar mallikarjuna and other inputs in this thread, we are proposing all quota tests to comply to following criteria: * use dd

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-18 Thread Vijay Bellur
On 05/16/2015 03:34 PM, Vijay Bellur wrote: I will send daily status updates from Monday (05/18) about this so that we are clear about where we are and what needs to be done to remove this moratorium. Appreciate your help in having a clean set of regression tests going forward! We have made

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-18 Thread Jeff Darcy
./tests/bugs/glusterd/bug-974007.t I looked at the core generated by this test and it turned to be a mem pool corruption. I will continue to investigate on this and keep you posted. Thank you. It looks like we have another generic memory-management problem that has surfaced more than once,

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-18 Thread Shyam
On 05/18/2015 03:49 PM, Shyam wrote: On 05/18/2015 10:33 AM, Vijay Bellur wrote: The etherpad did not call out, ./tests/bugs/distribute/bug-1161156.t which did not have an owner, and so I took a stab at it and below are the results. I also think failure in ./tests/bugs/quota/bug-1038598.t is

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-18 Thread Shyam
On 05/18/2015 07:05 PM, Shyam wrote: On 05/18/2015 03:49 PM, Shyam wrote: On 05/18/2015 10:33 AM, Vijay Bellur wrote: The etherpad did not call out, ./tests/bugs/distribute/bug-1161156.t which did not have an owner, and so I took a stab at it and below are the results. I also think failure in

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-17 Thread Emmanuel Dreyfus
Vijay Bellur vbel...@redhat.com wrote: We made a commitment to fix all regression tests before doing anything else for 3.7.1 and beyond. What about adding a fix Jenkins requirement? It keeps trashing NetBSD VM randomly, which makes regression tests hard to obtain. -- Emmanuel Dreyfus

Re: [Gluster-devel] Moratorium on new patch acceptance

2015-05-17 Thread Vijay Bellur
On 05/17/2015 02:17 PM, Emmanuel Dreyfus wrote: Vijay Bellur vbel...@redhat.com wrote: We made a commitment to fix all regression tests before doing anything else for 3.7.1 and beyond. What about adding a fix Jenkins requirement? It keeps trashing NetBSD VM randomly, which makes regression

[Gluster-devel] Moratorium on new patch acceptance

2015-05-16 Thread Vijay Bellur
Hey All, We made a commitment to fix all regression tests before doing anything else for 3.7.1 and beyond. Now is the time to honor the commitment. I request all maintainers to stop merging patches till the following tests are fixed and removed from is_bad_test() in run-tests.sh: