Re: [Gluster-Maintainers] Glusterfs-3.7.13 release plans

[Kaushal M]

On Fri, Jul 8, 2016 at 2:22 PM, Raghavendra Gowdappa
<rgowd...@redhat.com> wrote:
> There seems to be a major inode leak in fuse-clients:
> https://bugzilla.redhat.com/show_bug.cgi?id=1353856
>
> We have found an RCA through code reading (though have a high confidence on 
> the RCA). Do we want to include this in 3.7.13?

I'm not going to be delaying the release anymore. I'll be adding this
issue into the release-notes as a known-issue.

>
> regards,
> Raghavendra.
>
> - Original Message -
>> From: "Kaushal M" <kshlms...@gmail.com>
>> To: "Pranith Kumar Karampuri" <pkara...@redhat.com>
>> Cc: maintainers@gluster.org, "Gluster Devel" <gluster-de...@gluster.org>
>> Sent: Friday, July 8, 2016 11:51:11 AM
>> Subject: Re: [Gluster-Maintainers] Glusterfs-3.7.13 release plans
>>
>> On Fri, Jul 8, 2016 at 9:59 AM, Pranith Kumar Karampuri
>> <pkara...@redhat.com> wrote:
>> > Could you take in http://review.gluster.org/#/c/14598/ as well? It is ready
>> > for merge.
>> >
>> > On Thu, Jul 7, 2016 at 3:02 PM, Atin Mukherjee <amukh...@redhat.com> wrote:
>> >>
>> >> Can you take in http://review.gluster.org/#/c/14861 ?
>>
>> Can you get one of the maintainers to give it a +2?
>>
>> >>
>> >>
>> >> On Thursday 7 July 2016, Kaushal M <kshlms...@gmail.com> wrote:
>> >>>
>> >>> On Thu, Jun 30, 2016 at 11:08 AM, Kaushal M <kshlms...@gmail.com> wrote:
>> >>> > Hi all,
>> >>> >
>> >>> > I'm (or was) planning to do a 3.7.13 release on schedule today. 3.7.12
>> >>> > has a huge issue with libgfapi, solved by [1].
>> >>> > I'm not sure if this fixes the other issues with libgfapi noticed by
>> >>> > Lindsay on gluster-users.
>> >>> >
>> >>> > This patch has been included in the packages 3.7.12 built for CentOS,
>> >>> > Fedora, Ubuntu, Debian and SUSE. I guess Lindsay is using one of these
>> >>> > packages, so it might be that the issue seen is new. So I'd like to do
>> >>> > a quick release once we have a fix.
>> >>> >
>> >>> > Maintainers can merge changes into release-3.7 that follow the
>> >>> > criteria given in [2]. Please make sure to add the bugs for patches
>> >>> > you are merging are added as dependencies for the 3.7.13 tracker bug
>> >>> > [3].
>> >>> >
>> >>>
>> >>> I've just merged the fix for the gfapi breakage into release-3.7, and
>> >>> hope to tag 3.7.13 soon.
>> >>>
>> >>> The current head for release-3.7 is commit bddf6f8. 18 patches have
>> >>> been merged since 3.7.12 for the following components,
>> >>>  - gfapi
>> >>>  - nfs (includes ganesha related changes)
>> >>>  - glusterd/cli
>> >>>  - libglusterfs
>> >>>  - fuse
>> >>>  - build
>> >>>  - geo-rep
>> >>>  - afr
>> >>>
>> >>> I need and acknowledgement from the maintainers of the above
>> >>> components that they are ready.
>> >>> If any maintainers know of any other issues, please reply here. We'll
>> >>> decide how to address them for this release here.
>> >>>
>> >>> Also, please don't merge anymore changes into release-3.7. If you need
>> >>> to get something merged, please inform me.
>> >>>
>> >>> Thanks,
>> >>> Kaushal
>> >>>
>> >>> > Thanks,
>> >>> > Kaushal
>> >>> >
>> >>> > [1]: https://review.gluster.org/14822
>> >>> > [2]: https://public.pad.fsfe.org/p/glusterfs-release-process-201606
>> >>> > under the GlusterFS minor release heading
>> >>> > [3]: https://bugzilla.redhat.com/show_bug.cgi?id=glusterfs-3.7.13
>> >>> ___
>> >>> maintainers mailing list
>> >>> maintainers@gluster.org
>> >>> http://www.gluster.org/mailman/listinfo/maintainers
>> >>
>> >>
>> >>
>> >> --
>> >> Atin
>> >> Sent from iPhone
>> >>
>> >> ___
>> >> maintainers mailing list
>> >> maintainers@gluster.org
>> >> http://www.gluster.org/mailman/listinfo/maintainers
>> >>
>> >
>> >
>> >
>> > --
>> > Pranith
>> ___
>> maintainers mailing list
>> maintainers@gluster.org
>> http://www.gluster.org/mailman/listinfo/maintainers
>>
___
maintainers mailing list
maintainers@gluster.org
http://www.gluster.org/mailman/listinfo/maintainers

Re: [Gluster-Maintainers] Glusterfs-3.7.13 release plans

[Raghavendra Gowdappa]

There seems to be a major inode leak in fuse-clients:
https://bugzilla.redhat.com/show_bug.cgi?id=1353856

We have found an RCA through code reading (though have a high confidence on the 
RCA). Do we want to include this in 3.7.13?

regards,
Raghavendra.

- Original Message -
> From: "Kaushal M" <kshlms...@gmail.com>
> To: "Pranith Kumar Karampuri" <pkara...@redhat.com>
> Cc: maintainers@gluster.org, "Gluster Devel" <gluster-de...@gluster.org>
> Sent: Friday, July 8, 2016 11:51:11 AM
> Subject: Re: [Gluster-Maintainers] Glusterfs-3.7.13 release plans
> 
> On Fri, Jul 8, 2016 at 9:59 AM, Pranith Kumar Karampuri
> <pkara...@redhat.com> wrote:
> > Could you take in http://review.gluster.org/#/c/14598/ as well? It is ready
> > for merge.
> >
> > On Thu, Jul 7, 2016 at 3:02 PM, Atin Mukherjee <amukh...@redhat.com> wrote:
> >>
> >> Can you take in http://review.gluster.org/#/c/14861 ?
> 
> Can you get one of the maintainers to give it a +2?
> 
> >>
> >>
> >> On Thursday 7 July 2016, Kaushal M <kshlms...@gmail.com> wrote:
> >>>
> >>> On Thu, Jun 30, 2016 at 11:08 AM, Kaushal M <kshlms...@gmail.com> wrote:
> >>> > Hi all,
> >>> >
> >>> > I'm (or was) planning to do a 3.7.13 release on schedule today. 3.7.12
> >>> > has a huge issue with libgfapi, solved by [1].
> >>> > I'm not sure if this fixes the other issues with libgfapi noticed by
> >>> > Lindsay on gluster-users.
> >>> >
> >>> > This patch has been included in the packages 3.7.12 built for CentOS,
> >>> > Fedora, Ubuntu, Debian and SUSE. I guess Lindsay is using one of these
> >>> > packages, so it might be that the issue seen is new. So I'd like to do
> >>> > a quick release once we have a fix.
> >>> >
> >>> > Maintainers can merge changes into release-3.7 that follow the
> >>> > criteria given in [2]. Please make sure to add the bugs for patches
> >>> > you are merging are added as dependencies for the 3.7.13 tracker bug
> >>> > [3].
> >>> >
> >>>
> >>> I've just merged the fix for the gfapi breakage into release-3.7, and
> >>> hope to tag 3.7.13 soon.
> >>>
> >>> The current head for release-3.7 is commit bddf6f8. 18 patches have
> >>> been merged since 3.7.12 for the following components,
> >>>  - gfapi
> >>>  - nfs (includes ganesha related changes)
> >>>  - glusterd/cli
> >>>  - libglusterfs
> >>>  - fuse
> >>>  - build
> >>>  - geo-rep
> >>>  - afr
> >>>
> >>> I need and acknowledgement from the maintainers of the above
> >>> components that they are ready.
> >>> If any maintainers know of any other issues, please reply here. We'll
> >>> decide how to address them for this release here.
> >>>
> >>> Also, please don't merge anymore changes into release-3.7. If you need
> >>> to get something merged, please inform me.
> >>>
> >>> Thanks,
> >>> Kaushal
> >>>
> >>> > Thanks,
> >>> > Kaushal
> >>> >
> >>> > [1]: https://review.gluster.org/14822
> >>> > [2]: https://public.pad.fsfe.org/p/glusterfs-release-process-201606
> >>> > under the GlusterFS minor release heading
> >>> > [3]: https://bugzilla.redhat.com/show_bug.cgi?id=glusterfs-3.7.13
> >>> ___
> >>> maintainers mailing list
> >>> maintainers@gluster.org
> >>> http://www.gluster.org/mailman/listinfo/maintainers
> >>
> >>
> >>
> >> --
> >> Atin
> >> Sent from iPhone
> >>
> >> ___
> >> maintainers mailing list
> >> maintainers@gluster.org
> >> http://www.gluster.org/mailman/listinfo/maintainers
> >>
> >
> >
> >
> > --
> > Pranith
> ___
> maintainers mailing list
> maintainers@gluster.org
> http://www.gluster.org/mailman/listinfo/maintainers
> 
___
maintainers mailing list
maintainers@gluster.org
http://www.gluster.org/mailman/listinfo/maintainers

Re: [Gluster-Maintainers] Glusterfs-3.7.13 release plans

[Kaushal M]

On Fri, Jul 8, 2016 at 9:59 AM, Pranith Kumar Karampuri
 wrote:
> Could you take in http://review.gluster.org/#/c/14598/ as well? It is ready
> for merge.
>
> On Thu, Jul 7, 2016 at 3:02 PM, Atin Mukherjee  wrote:
>>
>> Can you take in http://review.gluster.org/#/c/14861 ?

Can you get one of the maintainers to give it a +2?

>>
>>
>> On Thursday 7 July 2016, Kaushal M  wrote:
>>>
>>> On Thu, Jun 30, 2016 at 11:08 AM, Kaushal M  wrote:
>>> > Hi all,
>>> >
>>> > I'm (or was) planning to do a 3.7.13 release on schedule today. 3.7.12
>>> > has a huge issue with libgfapi, solved by [1].
>>> > I'm not sure if this fixes the other issues with libgfapi noticed by
>>> > Lindsay on gluster-users.
>>> >
>>> > This patch has been included in the packages 3.7.12 built for CentOS,
>>> > Fedora, Ubuntu, Debian and SUSE. I guess Lindsay is using one of these
>>> > packages, so it might be that the issue seen is new. So I'd like to do
>>> > a quick release once we have a fix.
>>> >
>>> > Maintainers can merge changes into release-3.7 that follow the
>>> > criteria given in [2]. Please make sure to add the bugs for patches
>>> > you are merging are added as dependencies for the 3.7.13 tracker bug
>>> > [3].
>>> >
>>>
>>> I've just merged the fix for the gfapi breakage into release-3.7, and
>>> hope to tag 3.7.13 soon.
>>>
>>> The current head for release-3.7 is commit bddf6f8. 18 patches have
>>> been merged since 3.7.12 for the following components,
>>>  - gfapi
>>>  - nfs (includes ganesha related changes)
>>>  - glusterd/cli
>>>  - libglusterfs
>>>  - fuse
>>>  - build
>>>  - geo-rep
>>>  - afr
>>>
>>> I need and acknowledgement from the maintainers of the above
>>> components that they are ready.
>>> If any maintainers know of any other issues, please reply here. We'll
>>> decide how to address them for this release here.
>>>
>>> Also, please don't merge anymore changes into release-3.7. If you need
>>> to get something merged, please inform me.
>>>
>>> Thanks,
>>> Kaushal
>>>
>>> > Thanks,
>>> > Kaushal
>>> >
>>> > [1]: https://review.gluster.org/14822
>>> > [2]: https://public.pad.fsfe.org/p/glusterfs-release-process-201606
>>> > under the GlusterFS minor release heading
>>> > [3]: https://bugzilla.redhat.com/show_bug.cgi?id=glusterfs-3.7.13
>>> ___
>>> maintainers mailing list
>>> maintainers@gluster.org
>>> http://www.gluster.org/mailman/listinfo/maintainers
>>
>>
>>
>> --
>> Atin
>> Sent from iPhone
>>
>> ___
>> maintainers mailing list
>> maintainers@gluster.org
>> http://www.gluster.org/mailman/listinfo/maintainers
>>
>
>
>
> --
> Pranith
___
maintainers mailing list
maintainers@gluster.org
http://www.gluster.org/mailman/listinfo/maintainers

Re: [Gluster-Maintainers] Glusterfs-3.7.13 release plans

[Pranith Kumar Karampuri]

Could you take in http://review.gluster.org/#/c/14598/ as well? It is ready
for merge.

On Thu, Jul 7, 2016 at 3:02 PM, Atin Mukherjee  wrote:

> Can you take in http://review.gluster.org/#/c/14861 ?
>
>
> On Thursday 7 July 2016, Kaushal M  wrote:
>
>> On Thu, Jun 30, 2016 at 11:08 AM, Kaushal M  wrote:
>> > Hi all,
>> >
>> > I'm (or was) planning to do a 3.7.13 release on schedule today. 3.7.12
>> > has a huge issue with libgfapi, solved by [1].
>> > I'm not sure if this fixes the other issues with libgfapi noticed by
>> > Lindsay on gluster-users.
>> >
>> > This patch has been included in the packages 3.7.12 built for CentOS,
>> > Fedora, Ubuntu, Debian and SUSE. I guess Lindsay is using one of these
>> > packages, so it might be that the issue seen is new. So I'd like to do
>> > a quick release once we have a fix.
>> >
>> > Maintainers can merge changes into release-3.7 that follow the
>> > criteria given in [2]. Please make sure to add the bugs for patches
>> > you are merging are added as dependencies for the 3.7.13 tracker bug
>> > [3].
>> >
>>
>> I've just merged the fix for the gfapi breakage into release-3.7, and
>> hope to tag 3.7.13 soon.
>>
>> The current head for release-3.7 is commit bddf6f8. 18 patches have
>> been merged since 3.7.12 for the following components,
>>  - gfapi
>>  - nfs (includes ganesha related changes)
>>  - glusterd/cli
>>  - libglusterfs
>>  - fuse
>>  - build
>>  - geo-rep
>>  - afr
>>
>> I need and acknowledgement from the maintainers of the above
>> components that they are ready.
>> If any maintainers know of any other issues, please reply here. We'll
>> decide how to address them for this release here.
>>
>> Also, please don't merge anymore changes into release-3.7. If you need
>> to get something merged, please inform me.
>>
>> Thanks,
>> Kaushal
>>
>> > Thanks,
>> > Kaushal
>> >
>> > [1]: https://review.gluster.org/14822
>> > [2]: https://public.pad.fsfe.org/p/glusterfs-release-process-201606
>> > under the GlusterFS minor release heading
>> > [3]: https://bugzilla.redhat.com/show_bug.cgi?id=glusterfs-3.7.13
>> ___
>> maintainers mailing list
>> maintainers@gluster.org
>> http://www.gluster.org/mailman/listinfo/maintainers
>>
>
>
> --
> Atin
> Sent from iPhone
>
> ___
> maintainers mailing list
> maintainers@gluster.org
> http://www.gluster.org/mailman/listinfo/maintainers
>
>


-- 
Pranith
___
maintainers mailing list
maintainers@gluster.org
http://www.gluster.org/mailman/listinfo/maintainers

Re: [Gluster-Maintainers] Glusterfs-3.7.13 release plans

[Niels de Vos]

On Thu, Jun 30, 2016 at 03:18:18PM +0530, Kaushal M wrote:
> On Thu, Jun 30, 2016 at 2:53 PM, Niels de Vos  wrote:
> > On Thu, Jun 30, 2016 at 02:13:36PM +0530, Kaushal M wrote:
> >> On Thu, Jun 30, 2016 at 1:57 PM, Niels de Vos  wrote:
> >> > On Thu, Jun 30, 2016 at 12:46:57PM +0530, Atin Mukherjee wrote:
> >> >> On Thu, Jun 30, 2016 at 11:56 AM, Atin Mukherjee 
> >> >> wrote:
> >> >>
> >> >> >
> >> >> >
> >> >> > On Thu, Jun 30, 2016 at 11:08 AM, Kaushal M  
> >> >> > wrote:
> >> >> >
> >> >> >> Hi all,
> >> >> >>
> >> >> >> I'm (or was) planning to do a 3.7.13 release on schedule today. 
> >> >> >> 3.7.12
> >> >> >> has a huge issue with libgfapi, solved by [1].
> >> >> >> I'm not sure if this fixes the other issues with libgfapi noticed by
> >> >> >> Lindsay on gluster-users.
> >> >> >>
> >> >> >> This patch has been included in the packages 3.7.12 built for CentOS,
> >> >> >> Fedora, Ubuntu, Debian and SUSE. I guess Lindsay is using one of 
> >> >> >> these
> >> >> >> packages, so it might be that the issue seen is new. So I'd like to 
> >> >> >> do
> >> >> >> a quick release once we have a fix.
> >> >> >>
> >> >> >
> >> >> >  http://review.gluster.org/14835 probably is the one you are looking 
> >> >> > for.
> >> >> >
> >> >> >
> >> >>
> >> >> Ignore it. I had a chance to talk to Poornima and she mentioned that 
> >> >> this
> >> >> is a different problem.
> >> >
> >> > The patch that fixes the problem is http://review.gluster.org/14822 and
> >> > I've merged it yesterday. The problem was introduced by
> >> > http://review.gluster.org/14822 (similar subject as 12835 above).
> >>
> >> This probably should be another review, the same change cannot
> >> possibly introduce and fix a problem.
> >
> > Nigel is correct, http://review.gluster.org/14148 caused it.
> >
> >> But are you sure that the VM pauses observed were due to buffer
> >> overflows, which the patch fixes?
> >> I think this is a different problem, as I'm pretty sure Lindsay was
> >> using packages that included this patch.
> >
> > Without this patch, all VMs that run with an updated libgfapi will fail.
> > I do not know if Lindsay has this patch applied.
> 
> We have confirmation that Lindsay and Kevin are using packages from
> download.gluster.org, built by Kaleb.

Is there a bug filed for this already? If 3.7.13 is waiting for its fix
it should have been added to the tracker...

  https://bugzilla.redhat.com/showdependencytree.cgi?id=glusterfs-3.7.13

Without a bug report and details on what has been investigated already,
it does not make a lot of sense for me to look into it. I hate to do the
same work someone else did before me.

Thanks,
Niels


> > The nightly builds are available for everyone to test, maybe someone
> > could try it out?
> >   http://artifacts.ci.centos.org/gluster/nightly/
> >
> > HTH,
> > Niels


signature.asc
Description: PGP signature
___
maintainers mailing list
maintainers@gluster.org
http://www.gluster.org/mailman/listinfo/maintainers

Re: [Gluster-Maintainers] Glusterfs-3.7.13 release plans

[Niels de Vos]

On Thu, Jun 30, 2016 at 02:13:36PM +0530, Kaushal M wrote:
> On Thu, Jun 30, 2016 at 1:57 PM, Niels de Vos  wrote:
> > On Thu, Jun 30, 2016 at 12:46:57PM +0530, Atin Mukherjee wrote:
> >> On Thu, Jun 30, 2016 at 11:56 AM, Atin Mukherjee 
> >> wrote:
> >>
> >> >
> >> >
> >> > On Thu, Jun 30, 2016 at 11:08 AM, Kaushal M  wrote:
> >> >
> >> >> Hi all,
> >> >>
> >> >> I'm (or was) planning to do a 3.7.13 release on schedule today. 3.7.12
> >> >> has a huge issue with libgfapi, solved by [1].
> >> >> I'm not sure if this fixes the other issues with libgfapi noticed by
> >> >> Lindsay on gluster-users.
> >> >>
> >> >> This patch has been included in the packages 3.7.12 built for CentOS,
> >> >> Fedora, Ubuntu, Debian and SUSE. I guess Lindsay is using one of these
> >> >> packages, so it might be that the issue seen is new. So I'd like to do
> >> >> a quick release once we have a fix.
> >> >>
> >> >
> >> >  http://review.gluster.org/14835 probably is the one you are looking for.
> >> >
> >> >
> >>
> >> Ignore it. I had a chance to talk to Poornima and she mentioned that this
> >> is a different problem.
> >
> > The patch that fixes the problem is http://review.gluster.org/14822 and
> > I've merged it yesterday. The problem was introduced by
> > http://review.gluster.org/14822 (similar subject as 12835 above).
> 
> This probably should be another review, the same change cannot
> possibly introduce and fix a problem.

Nigel is correct, http://review.gluster.org/14148 caused it.

> But are you sure that the VM pauses observed were due to buffer
> overflows, which the patch fixes?
> I think this is a different problem, as I'm pretty sure Lindsay was
> using packages that included this patch.

Without this patch, all VMs that run with an updated libgfapi will fail.
I do not know if Lindsay has this patch applied.

The nightly builds are available for everyone to test, maybe someone
could try it out?
  http://artifacts.ci.centos.org/gluster/nightly/

HTH,
Niels


signature.asc
Description: PGP signature
___
maintainers mailing list
maintainers@gluster.org
http://www.gluster.org/mailman/listinfo/maintainers

Re: [Gluster-Maintainers] Glusterfs-3.7.13 release plans

[Kaushal M]

On Thu, Jun 30, 2016 at 1:57 PM, Niels de Vos  wrote:
> On Thu, Jun 30, 2016 at 12:46:57PM +0530, Atin Mukherjee wrote:
>> On Thu, Jun 30, 2016 at 11:56 AM, Atin Mukherjee 
>> wrote:
>>
>> >
>> >
>> > On Thu, Jun 30, 2016 at 11:08 AM, Kaushal M  wrote:
>> >
>> >> Hi all,
>> >>
>> >> I'm (or was) planning to do a 3.7.13 release on schedule today. 3.7.12
>> >> has a huge issue with libgfapi, solved by [1].
>> >> I'm not sure if this fixes the other issues with libgfapi noticed by
>> >> Lindsay on gluster-users.
>> >>
>> >> This patch has been included in the packages 3.7.12 built for CentOS,
>> >> Fedora, Ubuntu, Debian and SUSE. I guess Lindsay is using one of these
>> >> packages, so it might be that the issue seen is new. So I'd like to do
>> >> a quick release once we have a fix.
>> >>
>> >
>> >  http://review.gluster.org/14835 probably is the one you are looking for.
>> >
>> >
>>
>> Ignore it. I had a chance to talk to Poornima and she mentioned that this
>> is a different problem.
>
> The patch that fixes the problem is http://review.gluster.org/14822 and
> I've merged it yesterday. The problem was introduced by
> http://review.gluster.org/14822 (similar subject as 12835 above).

This probably should be another review, the same change cannot
possibly introduce and fix a problem.

But are you sure that the VM pauses observed were due to buffer
overflows, which the patch fixes?
I think this is a different problem, as I'm pretty sure Lindsay was
using packages that included this patch.

> Unfortunately none of the libgfapi maintainer did completely review the
> change before it got merged. It also seems that minimal testing was done
> after the change got included (last minute change in 3.8, quickly
> backported as well).
>
> In order to make Gluster more stable, and prevent problems like this
> again, we really need to work on automating test cases. I hope all
> maintainers are thinking about how they want to test the components they
> are responsible for. For example, I'm planning to run the upstream QEMU
> tests against our nightly builds (libgfapi), and similar for the
> connectathon tests (Gluster/NFS). At one point it should be possible to
> wrap these in DiSTAF, but the DiSTAF job in the CentOS CI is not ready
> yet.
>
> Thanks,
> Niels
>
>
>>
>>
>> >
>> >> Maintainers can merge changes into release-3.7 that follow the
>> >> criteria given in [2]. Please make sure to add the bugs for patches
>> >> you are merging are added as dependencies for the 3.7.13 tracker bug
>> >> [3].
>> >>
>> >> Thanks,
>> >> Kaushal
>> >>
>> >> [1]: https://review.gluster.org/14822
>> >> [2]: https://public.pad.fsfe.org/p/glusterfs-release-process-201606
>> >> under the GlusterFS minor release heading
>> >> [3]: https://bugzilla.redhat.com/show_bug.cgi?id=glusterfs-3.7.13
>> >> ___
>> >> maintainers mailing list
>> >> maintainers@gluster.org
>> >> http://www.gluster.org/mailman/listinfo/maintainers
>> >>
>> >
>> >
>
>> ___
>> maintainers mailing list
>> maintainers@gluster.org
>> http://www.gluster.org/mailman/listinfo/maintainers
>
___
maintainers mailing list
maintainers@gluster.org
http://www.gluster.org/mailman/listinfo/maintainers

Re: [Gluster-Maintainers] Glusterfs-3.7.13 release plans

[Niels de Vos]

On Thu, Jun 30, 2016 at 12:46:57PM +0530, Atin Mukherjee wrote:
> On Thu, Jun 30, 2016 at 11:56 AM, Atin Mukherjee 
> wrote:
> 
> >
> >
> > On Thu, Jun 30, 2016 at 11:08 AM, Kaushal M  wrote:
> >
> >> Hi all,
> >>
> >> I'm (or was) planning to do a 3.7.13 release on schedule today. 3.7.12
> >> has a huge issue with libgfapi, solved by [1].
> >> I'm not sure if this fixes the other issues with libgfapi noticed by
> >> Lindsay on gluster-users.
> >>
> >> This patch has been included in the packages 3.7.12 built for CentOS,
> >> Fedora, Ubuntu, Debian and SUSE. I guess Lindsay is using one of these
> >> packages, so it might be that the issue seen is new. So I'd like to do
> >> a quick release once we have a fix.
> >>
> >
> >  http://review.gluster.org/14835 probably is the one you are looking for.
> >
> >
> 
> Ignore it. I had a chance to talk to Poornima and she mentioned that this
> is a different problem.

The patch that fixes the problem is http://review.gluster.org/14822 and
I've merged it yesterday. The problem was introduced by
http://review.gluster.org/14822 (similar subject as 12835 above).
Unfortunately none of the libgfapi maintainer did completely review the
change before it got merged. It also seems that minimal testing was done
after the change got included (last minute change in 3.8, quickly
backported as well).

In order to make Gluster more stable, and prevent problems like this
again, we really need to work on automating test cases. I hope all
maintainers are thinking about how they want to test the components they
are responsible for. For example, I'm planning to run the upstream QEMU
tests against our nightly builds (libgfapi), and similar for the
connectathon tests (Gluster/NFS). At one point it should be possible to
wrap these in DiSTAF, but the DiSTAF job in the CentOS CI is not ready
yet.

Thanks,
Niels


> 
> 
> >
> >> Maintainers can merge changes into release-3.7 that follow the
> >> criteria given in [2]. Please make sure to add the bugs for patches
> >> you are merging are added as dependencies for the 3.7.13 tracker bug
> >> [3].
> >>
> >> Thanks,
> >> Kaushal
> >>
> >> [1]: https://review.gluster.org/14822
> >> [2]: https://public.pad.fsfe.org/p/glusterfs-release-process-201606
> >> under the GlusterFS minor release heading
> >> [3]: https://bugzilla.redhat.com/show_bug.cgi?id=glusterfs-3.7.13
> >> ___
> >> maintainers mailing list
> >> maintainers@gluster.org
> >> http://www.gluster.org/mailman/listinfo/maintainers
> >>
> >
> >

> ___
> maintainers mailing list
> maintainers@gluster.org
> http://www.gluster.org/mailman/listinfo/maintainers



signature.asc
Description: PGP signature
___
maintainers mailing list
maintainers@gluster.org
http://www.gluster.org/mailman/listinfo/maintainers

Re: [Gluster-Maintainers] Glusterfs-3.7.13 release plans

[Atin Mukherjee]

On Thu, Jun 30, 2016 at 11:56 AM, Atin Mukherjee 
wrote:

>
>
> On Thu, Jun 30, 2016 at 11:08 AM, Kaushal M  wrote:
>
>> Hi all,
>>
>> I'm (or was) planning to do a 3.7.13 release on schedule today. 3.7.12
>> has a huge issue with libgfapi, solved by [1].
>> I'm not sure if this fixes the other issues with libgfapi noticed by
>> Lindsay on gluster-users.
>>
>> This patch has been included in the packages 3.7.12 built for CentOS,
>> Fedora, Ubuntu, Debian and SUSE. I guess Lindsay is using one of these
>> packages, so it might be that the issue seen is new. So I'd like to do
>> a quick release once we have a fix.
>>
>
>  http://review.gluster.org/14835 probably is the one you are looking for.
>
>

Ignore it. I had a chance to talk to Poornima and she mentioned that this
is a different problem.


>
>> Maintainers can merge changes into release-3.7 that follow the
>> criteria given in [2]. Please make sure to add the bugs for patches
>> you are merging are added as dependencies for the 3.7.13 tracker bug
>> [3].
>>
>> Thanks,
>> Kaushal
>>
>> [1]: https://review.gluster.org/14822
>> [2]: https://public.pad.fsfe.org/p/glusterfs-release-process-201606
>> under the GlusterFS minor release heading
>> [3]: https://bugzilla.redhat.com/show_bug.cgi?id=glusterfs-3.7.13
>> ___
>> maintainers mailing list
>> maintainers@gluster.org
>> http://www.gluster.org/mailman/listinfo/maintainers
>>
>
>
___
maintainers mailing list
maintainers@gluster.org
http://www.gluster.org/mailman/listinfo/maintainers

Re: [Gluster-Maintainers] Glusterfs-3.7.13 release plans

[Atin Mukherjee]

On Thu, Jun 30, 2016 at 11:08 AM, Kaushal M  wrote:

> Hi all,
>
> I'm (or was) planning to do a 3.7.13 release on schedule today. 3.7.12
> has a huge issue with libgfapi, solved by [1].
> I'm not sure if this fixes the other issues with libgfapi noticed by
> Lindsay on gluster-users.
>
> This patch has been included in the packages 3.7.12 built for CentOS,
> Fedora, Ubuntu, Debian and SUSE. I guess Lindsay is using one of these
> packages, so it might be that the issue seen is new. So I'd like to do
> a quick release once we have a fix.
>

 http://review.gluster.org/14835 probably is the one you are looking for.


>
> Maintainers can merge changes into release-3.7 that follow the
> criteria given in [2]. Please make sure to add the bugs for patches
> you are merging are added as dependencies for the 3.7.13 tracker bug
> [3].
>
> Thanks,
> Kaushal
>
> [1]: https://review.gluster.org/14822
> [2]: https://public.pad.fsfe.org/p/glusterfs-release-process-201606
> under the GlusterFS minor release heading
> [3]: https://bugzilla.redhat.com/show_bug.cgi?id=glusterfs-3.7.13
> ___
> maintainers mailing list
> maintainers@gluster.org
> http://www.gluster.org/mailman/listinfo/maintainers
>
___
maintainers mailing list
maintainers@gluster.org
http://www.gluster.org/mailman/listinfo/maintainers

[Gluster-Maintainers] Glusterfs-3.7.13 release plans

[Kaushal M]

Hi all,

I'm (or was) planning to do a 3.7.13 release on schedule today. 3.7.12
has a huge issue with libgfapi, solved by [1].
I'm not sure if this fixes the other issues with libgfapi noticed by
Lindsay on gluster-users.

This patch has been included in the packages 3.7.12 built for CentOS,
Fedora, Ubuntu, Debian and SUSE. I guess Lindsay is using one of these
packages, so it might be that the issue seen is new. So I'd like to do
a quick release once we have a fix.

Maintainers can merge changes into release-3.7 that follow the
criteria given in [2]. Please make sure to add the bugs for patches
you are merging are added as dependencies for the 3.7.13 tracker bug
[3].

Thanks,
Kaushal

[1]: https://review.gluster.org/14822
[2]: https://public.pad.fsfe.org/p/glusterfs-release-process-201606
under the GlusterFS minor release heading
[3]: https://bugzilla.redhat.com/show_bug.cgi?id=glusterfs-3.7.13
___
maintainers mailing list
maintainers@gluster.org
http://www.gluster.org/mailman/listinfo/maintainers