From dev-return-29555-archive-asf-public=cust-asf.ponee.io@geode.apache.org Wed Aug 29 23:21:14 2018 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx-eu-01.ponee.io (Postfix) with SMTP id 7D36D180657 for ; Wed, 29 Aug 2018 23:21:13 +0200 (CEST) Received: (qmail 29993 invoked by uid 500); 29 Aug 2018 21:21:12 -0000 Mailing-List: contact dev-help@geode.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@geode.apache.org Delivered-To: mailing list dev@geode.apache.org Received: (qmail 29982 invoked by uid 99); 29 Aug 2018 21:21:12 -0000 Received: from mail-relay.apache.org (HELO mailrelay2-lw-us.apache.org) (207.244.88.137) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 29 Aug 2018 21:21:12 +0000 Received: from mail-lf1-f70.google.com (mail-lf1-f70.google.com [209.85.167.70]) by mailrelay2-lw-us.apache.org (ASF Mail Server at mailrelay2-lw-us.apache.org) with ESMTPSA id 532A42048 for ; Wed, 29 Aug 2018 21:21:11 +0000 (UTC) Received: by mail-lf1-f70.google.com with SMTP id c82-v6so1276625lfh.12 for ; Wed, 29 Aug 2018 14:21:10 -0700 (PDT) X-Gm-Message-State: APzg51CD5l3GFd76DJOne5+iAIWPxevMs5oH1CyUWoZ3YonJ27fMcWzs gAznPwwoMhxLT3NFszs/YmfHSzJGel9GJpyjDDWHDo07bAAOiUTA6j3FzHruIzvD6rKi4v6wkbh qk14EP/ykBjHFKm4dndfw2kFZnFoRa+K3D9Ooe616NZMUUtlqwXy1ahM= X-Received: by 2002:a2e:80d3:: with SMTP id r19-v6mr5251266ljg.85.1535577668976; Wed, 29 Aug 2018 14:21:08 -0700 (PDT) X-Google-Smtp-Source: ANB0VdZRrLdbkVPiExggqkvOiPvF+Wrpsup6AGLLhQmbJo23d8pL6IrQqVq8ai9UXMSl3sdJeIK7JlKlHi37AG8QqTc= X-Received: by 2002:a2e:80d3:: with SMTP id r19-v6mr5251261ljg.85.1535577668769; Wed, 29 Aug 2018 14:21:08 -0700 (PDT) MIME-Version: 1.0 References: <7C0913BA-DFD7-48BC-B012-5CEEE34605AC@pivotal.io> <78a9c054-eeb7-cf42-39d7-ba5e9dc08868@pivotal.io> <518c398a-ac97-be81-424b-16a25881b627@pivotal.io> In-Reply-To: From: Nabarun Nag Date: Wed, 29 Aug 2018 14:20:56 -0700 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [DISCUSS] Apache Geode 1.7.0 release branch created To: dev@geode.apache.org Content-Type: multipart/alternative; boundary="00000000000078f8210574998922" --00000000000078f8210574998922 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Thank you Sai, We are also waiting on documentation to be updated to reflect the changes made by these resolved JIRAs. Once that is completed, I believe that we will be ready for the new branch. Regards Nabarun Nag On Wed, Aug 29, 2018 at 11:45 AM Sai Boorlagadda wrote: > I have merged GEODE-5594 to develop. > > GEODE-5338 is now waiting for PR review and precheckin. > > Sai > > On Tue, Aug 28, 2018 at 10:30 AM Sai Boorlagadda < > sai.boorlagadda@gmail.com> > wrote: > > > GEODE-5338 is downvoted for the security concerns related to trusting > > the default trust store and thus resulted in an improvement to add a > > hostname > > validation as a feature before we can support trusting default trust > store. > > > > So GEODE-5338 is blocked by GEODE-5594. > > > > Once I merge GEODE-5594, I will reinitiate review on GEODE-5338 PR. > > > > Sai > > > > On Tue, Aug 28, 2018 at 10:15 AM Alexander Murmann > > wrote: > > > >> Looks like we are now waiting for these tickets: > >> > >> GEODE-5601 which is a dup of GEODE-5590 which has this open PR: > >> https://github.com/apache/geode/pull/2368. > >> GEODE-5594 has open PR: https://github.com/apache/geode/pull/2346 > >> GEODE-5338 has > >> open PR: https://github.com/apache/geode/pull/2244. > >> > >> Does this look right? > >> > >> The GEODE-5338 ticket is the most concerning to me right now. The PR w= as > >> down voted, had some down voted discussion and nothing since. Sai > >> mentioned > >> yesterday that this might be able to merge. That's surprising given th= e > >> downvotes and lack of discussion. Sai, do you want to give us a update= , > >> maybe on the PR? > >> > >> On Tue, Aug 28, 2018 at 1:31 AM, Juan Jos=C3=A9 Ramos > >> wrote: > >> > >> > Thanks!! > >> > > >> > On Tue, Aug 28, 2018 at 9:13 AM Nabarun Nag wrote: > >> > > >> > > Hi Juan, > >> > > > >> > > GEODE-5618 as PR#2360 has been merged in to develop. The new branc= h > >> has > >> > not > >> > > yet been created hence this fix will be in 1.7.0 > >> > > > >> > > Regards > >> > > Nabarun Nag > >> > > > >> > > On Tue, Aug 28, 2018 at 12:33 AM Juan Jos=C3=A9 Ramos > > >> > > wrote: > >> > > > >> > > > Hello team, > >> > > > > >> > > > Can we also include GEODE-5618 in the next release?. The pull > >> request > >> > has > >> > > > been approved already, it just needs to be merged. > >> > > > Best regards. > >> > > > > >> > > > > >> > > > On Mon, Aug 27, 2018 at 11:45 PM Bruce Schuchardt < > >> > > bschuchardt@pivotal.io> > >> > > > wrote: > >> > > > > >> > > > > great! thanks > >> > > > > > >> > > > > > >> > > > > On 8/27/18 1:42 PM, Nabarun Nag wrote: > >> > > > > > I completely agree. Once the branch is created, it will > undergo > >> all > >> > > > > > compatibility and upgrade tests. > >> > > > > > > >> > > > > > The commit that you have mentioned will be reverted in 1.7.0= , > as > >> > well > >> > > > as > >> > > > > > any related commits > >> > > > > > > >> > > > > > Regards > >> > > > > > Nabarun Nag > >> > > > > > > >> > > > > > On Mon, Aug 27, 2018 at 1:34 PM Bruce Schuchardt < > >> > > > bschuchardt@pivotal.io > >> > > > > > > >> > > > > > wrote: > >> > > > > > > >> > > > > >> I don't think it's as easy as doing a rebase. Someone adde= d > >> the > >> > 1.8 > >> > > > > >> version to Version.java and we need to revert that. We als= o > >> need > >> > to > >> > > > see > >> > > > > >> if it's being used anywhere for backward-compatibility. If > >> it's > >> > in > >> > > > use > >> > > > > >> those changes need to be examined and probably undone on th= e > >> > branch > >> > > if > >> > > > > >> they're targeting 1.7 peers/clients. > >> > > > > >> > >> > > > > >> On 8/27/18 12:11 PM, Nabarun Nag wrote: > >> > > > > >>> @Bruce those changes were done when 1.7.0 release process > was > >> > > > > >> in-progress, > >> > > > > >>> and a release branch was already created. But we stopped > that > >> > > process > >> > > > > mid > >> > > > > >>> way. This happened in May 2018. > >> > > > > >>> We are planning to rebase the 1.7.0 brach with the current > >> > develop > >> > > > > pretty > >> > > > > >>> soon. > >> > > > > >>> > >> > > > > >>> Regards > >> > > > > >>> Nabarun > >> > > > > >>> On Mon, Aug 27, 2018 at 12:02 PM Bruce Schuchardt < > >> > > > > >> bschuchardt@pivotal.io> > >> > > > > >>> wrote: > >> > > > > >>> > >> > > > > >>>> It looks like we've cut a 1.7.0 release branch that says > its > >> > > 1.8.0. > >> > > > > Is > >> > > > > >>>> that intentional? > >> > > > > >>>> > >> > > > > >>>> > >> > > > > >>>> private static final byte GEODE_180_ORDINAL =3D95; > >> > > > > >>>> > >> > > > > >>>> public static final VersionGEODE_180 =3D > >> > > > > >>>> new Version("GEODE","1.8.0", (byte)1, (byte)8, > >> (byte)0, > >> > > > > >>>> (byte)0,GEODE_180_ORDINAL); > >> > > > > >>>> > >> > > > > >>>> > >> > > > > >>>> On 8/27/18 9:50 AM, Sai Boorlagadda wrote: > >> > > > > >>>>> After reading through the weekend, validating against CN > as > >> a > >> > > > > >>>>> fallback should be acceptable and dont have any further > >> > concerns > >> > > > > >>>>> with default JDK's implementation as expressed[1]. > >> > > > > >>>>> > >> > > > > >>>>> Planning to merge GEODE-5594 today and following with > >> > GEODE-5338. > >> > > > > >>>>> > >> > > > > >>>>> Sai > >> > > > > >>>>> [1] > >> > > > > >>>>> > >> > > > > >> > >> > > > > > >> > > > > >> > > https://lists.apache.org/thread.html/906540e18fa6f85fc77c88c28fc74= a > >> > 61402471d2eed4ee9dab4813c9@%3Cdev.geode.apache.org%3E > >> > > > > >>>>> On Fri, Aug 24, 2018 at 5:07 PM Sai Boorlagadda < > >> > > > > >>>> sai.boorlagadda@gmail.com> > >> > > > > >>>>> wrote: > >> > > > > >>>>> > >> > > > > >>>>>> Regarding GEODE-5594, though the current implementation > is > >> > good > >> > > > and > >> > > > > >>>> needed > >> > > > > >>>>>> more coverage. > >> > > > > >>>>>> > >> > > > > >>>>>> While adding tests to cover negative cases, I found > >> something > >> > > > about > >> > > > > >>>> JDK's > >> > > > > >>>>>> default implementation of > >> > > > > >>>>>> hostname validation which I am not happy about and so i= t > >> > needs a > >> > > > > >>>>>> rethought. It could result in > >> > > > > >>>>>> implementing our own custom algorithm to do hostname > >> > validation. > >> > > > > >>>>>> > >> > > > > >>>>>> I will send out details and seek to advise on what we > >> should > >> > do > >> > > > in a > >> > > > > >>>>>> different thread. > >> > > > > >>>>>> > >> > > > > >>>>>> Sai > >> > > > > >>>>>> > >> > > > > >>>>>> On Fri, Aug 24, 2018 at 10:52 AM Alexander Murmann < > >> > > > > >> amurmann@pivotal.io > >> > > > > >>>>>> wrote: > >> > > > > >>>>>> > >> > > > > >>>>>>> To summarize where we are right now in this discussion= , > I > >> see > >> > > the > >> > > > > >>>>>>> following > >> > > > > >>>>>>> tickets listed in this thread as want-to-haves for 1.7= : > >> > > > > >>>>>>> > >> > > > > >>>>>>> - GEODE-5615 - =E2=9C=85 resolved > >> > > > > >>>>>>> - GEODE-5601 - =F0=9F=8F=83=E2=80=8D=E2=99=80=EF= =B8=8F in progress > >> > > > > >>>>>>> - GEODE-5594 - =F0=9F=8F=83=E2=80=8D=E2=99=80=EF= =B8=8F waiting for PR review > >> > > > > >>>>>>> - GEODE-5338 - =F0=9F=8F=83=E2=80=8D=E2=99=80=EF= =B8=8F waiting for PR review > >> > > > > >>>>>>> - GEODE-5619 - =F0=9F=99=84 in progress in JIRA = but has > merged > >> > PR. > >> > > > What > >> > > > > >> does > >> > > > > >>>> it > >> > > > > >>>>>>> mean? > >> > > > > >>>>>>> > >> > > > > >>>>>>> Is there anything else that needs to go into 1.7? > >> > > > > >>>>>>> > >> > > > > >>>>>>> It seems like the best we all can do is to review Sai'= s > >> PRs. > >> > Is > >> > > > > that > >> > > > > >>>>>>> correct? > >> > > > > >>>>>>> > >> > > > > >>>>>>> On Wed, Aug 22, 2018 at 10:59 AM, Jens Deppe < > >> > > jdeppe@pivotal.io> > >> > > > > >>>> wrote: > >> > > > > >>>>>>>> I'd also like to include GEODE-5619 > >> > > > > >>>>>>>> > >> > > > > >>>>>>>> On Tue, Aug 21, 2018 at 3:59 PM Xiaojian Zhou < > >> > > gzhou@pivotal.io > >> > > > > > >> > > > > >>>> wrote: > >> > > > > >>>>>>>>> +1 > >> > > > > >>>>>>>>> > >> > > > > >>>>>>>>> The release will be a great one with so many > historical > >> > bugs > >> > > > > fixed. > >> > > > > >>>>>>>>> > >> > > > > >>>>>>>>> Today I tried to use IJ to build and run with latest > >> > > > build.gradle > >> > > > > >> and > >> > > > > >>>>>>>>> recent moved test packages, it worked. So this > >> refactoring > >> > is > >> > > > > also > >> > > > > >>>>>>>> success. > >> > > > > >>>>>>>>> On Tue, Aug 21, 2018 at 3:52 PM, Anthony Baker < > >> > > > > abaker@pivotal.io> > >> > > > > >>>>>>>> wrote: > >> > > > > >>>>>>>>>> I most definitely agree! > >> > > > > >>>>>>>>>> > >> > > > > >>>>>>>>>> Anthony > >> > > > > >>>>>>>>>> > >> > > > > >>>>>>>>>> > >> > > > > >>>>>>>>>>> On Aug 21, 2018, at 2:26 PM, Dan Smith < > >> > dsmith@pivotal.io> > >> > > > > >> wrote: > >> > > > > >>>>>>>>>>> I think we do want to wait for GEODE-5615 > >> > (DistributedTest > >> > > > > OOMEs) > >> > > > > >>>>>>> and > >> > > > > >>>>>>>>>>> GEODE-5601 (AcceptanceTest port conflicts) to be > fixed > >> > > before > >> > > > > >>>>>>> cutting > >> > > > > >>>>>>>>> the > >> > > > > >>>>>>>>>>> new 1.7 branch. It would be better if we don't > create > >> a > >> > > > release > >> > > > > >>>>>>>> branch > >> > > > > >>>>>>>>>> from > >> > > > > >>>>>>>>>>> a point where we have these systematic issues with > our > >> > > > > pipeline. > >> > > > > >>>>>>>>>>> > >> > > > > >>>>>>>>>>> -Dan > >> > > > > >> > >> > > > > > >> > > > > > >> > > > > >> > > > -- > >> > > > Juan Jos=C3=A9 Ramos Cassella > >> > > > Senior Technical Support Engineer > >> > > > Email: jramos@pivotal.io > >> > > > Office#: +353 21 4238611 <+353%2021%20423%208611> > <+353%2021%20423%208611> > >> > > > Mobile#: +353 87 2074066 <+353%2087%20207%204066> > <+353%2087%20207%204066> > >> > > > After Hours Contact#: +1 877 477 2269 <(877)%20477-2269> > <(877)%20477-2269> > >> > > > Office Hours: Mon - Thu 08:30 - 17:00 GMT. Fri 08:30 - 16:00 GMT > >> > > > How to upload artifacts: > >> > > > https://support.pivotal.io/hc/en-us/articles/204369073 > >> > > > How to escalate a ticket: > >> > > > https://support.pivotal.io/hc/en-us/articles/203809556 > >> > > > > >> > > > [image: support] [image: twitter] > >> > > > [image: linkedin] > >> > > > [image: facebook] > >> > > > [image: google plus] > >> > > > [image: youtube] > >> > > > < > >> > > > >> > https://www.youtube.com/playlist?list=3DPLAdzTan_eSPScpj2J50ErtzR9ANSzv3k= l > >> > > > >> > > > > >> > > > >> > > >> > > >> > -- > >> > Juan Jos=C3=A9 Ramos Cassella > >> > Senior Technical Support Engineer > >> > Email: jramos@pivotal.io > >> > Office#: +353 21 4238611 <+353%2021%20423%208611> > >> > Mobile#: +353 87 2074066 <+353%2087%20207%204066> > >> > After Hours Contact#: +1 877 477 2269 <(877)%20477-2269> > >> > Office Hours: Mon - Thu 08:30 - 17:00 GMT. Fri 08:30 - 16:00 GMT > >> > How to upload artifacts: > >> > https://support.pivotal.io/hc/en-us/articles/204369073 > >> > How to escalate a ticket: > >> > https://support.pivotal.io/hc/en-us/articles/203809556 > >> > > >> > [image: support] [image: twitter] > >> > [image: linkedin] > >> > [image: facebook] > >> > [image: google plus] > >> > [image: youtube] > >> > < > >> > https://www.youtube.com/playlist?list=3DPLAdzTan_eSPScpj2J50ErtzR9ANSzv3k= l> > >> > > >> > > > --00000000000078f8210574998922--