Return-Path: X-Original-To: apmail-curator-dev-archive@minotaur.apache.org Delivered-To: apmail-curator-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 430971045D for ; Fri, 11 Jul 2014 03:54:49 +0000 (UTC) Received: (qmail 42978 invoked by uid 500); 11 Jul 2014 03:54:49 -0000 Delivered-To: apmail-curator-dev-archive@curator.apache.org Received: (qmail 42920 invoked by uid 500); 11 Jul 2014 03:54:49 -0000 Mailing-List: contact dev-help@curator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@curator.apache.org Delivered-To: mailing list dev@curator.apache.org Received: (qmail 42908 invoked by uid 99); 11 Jul 2014 03:54:48 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 11 Jul 2014 03:54:48 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of mckenzie.cam@gmail.com designates 209.85.215.50 as permitted sender) Received: from [209.85.215.50] (HELO mail-la0-f50.google.com) (209.85.215.50) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 11 Jul 2014 03:54:44 +0000 Received: by mail-la0-f50.google.com with SMTP id pv20so406023lab.9 for ; Thu, 10 Jul 2014 20:54:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=QJPK1cvflImNnG+oTjcVlnKFKd96YxSrJYT5JUqcgZc=; b=IrCZU3K9/p8q1NwtLgR14Q030vKdcTr9yMGJW/4AjUMjrWMRUBPKR8cPcgDMY3olHU OzAv5P5TJR5Ih946MXKOg/38jWW7TxwMSJ/FGNm3kPTE9CfMY4bsvgEAFG17NfEDoMiY aB+gXg2IdxsA3tRalBjqLKa7f9fWCJk03sQfh361K3mnA5GB6SLTBtFEDB4S2fpksWl9 gH4pzISJK0Kk2imTugdDnbYv5jPlpQPbsF7NOlJMrl6k15v1MKIKBXXYCP/2H6u4mOv2 2pq5brI98b0lNmNu4ugzQkknyxLa68HF9amDWazlF3WvVbuCbHdMYcaRZYEEztXLe1w3 qKjw== MIME-Version: 1.0 X-Received: by 10.152.29.38 with SMTP id g6mr2274370lah.54.1405050863117; Thu, 10 Jul 2014 20:54:23 -0700 (PDT) Received: by 10.112.236.135 with HTTP; Thu, 10 Jul 2014 20:54:23 -0700 (PDT) In-Reply-To: References: Date: Fri, 11 Jul 2014 13:54:23 +1000 Message-ID: Subject: Re: [VOTE] Release Apache Curator version 2.6.0 From: Cameron McKenzie To: dev@curator.apache.org Content-Type: multipart/alternative; boundary=089e0158c004966c3904fde2e5f1 X-Virus-Checked: Checked by ClamAV on apache.org --089e0158c004966c3904fde2e5f1 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Right, it appears that no one wants to change their vote, so I guess the RC can be released. The fixes Mike's suggested can be picked up in 2.7.0 I will send out a formal email and release in the next day. cheers On Thu, Jul 10, 2014 at 8:20 AM, Cameron McKenzie wrote: > Thanks Mike, > So, currently we have: > 3 binding +1 > 1 non binding +1 > 1 non binding -1 > > Do any of the binding voters want to change their votes based on Mike's > assessment of the RC? > cheers > > > > > On Thu, Jul 10, 2014 at 8:08 AM, Mike Drob wrote: > >> Cam, >> >> As the person who called the vote, you will need to go through the email >> thread and tally up all the counts. Generally, an ASF project needs >> majority approval from committers to release, that is at least 3 +1s and >> more +1s than -1s. It would be a good idea to verify this against the >> Curator bylaws. If the vote passes, then you release, if the vote does n= ot >> pass then you build a new RC. >> >> My vote is non-binding, which means it doesn't count. I'd petition bindi= ng >> voters to -1 for the reasons I enumerated, but the decision is ultimatel= y >> out of my hands. >> >> Mike >> >> >> On Wed, Jul 9, 2014 at 5:02 PM, Cameron McKenzie >> wrote: >> >> > So where does this leave us guys? Presumably the release needs to be >> > rebuilt? >> > cheers >> > Cam >> > >> > >> > On Thu, Jul 10, 2014 at 2:57 AM, Jay Zarfoss >> wrote: >> > >> > > +1 >> > > >> > > Clean build runs fine / all test pass on my Mac. >> > > >> > > >> > > On Wed, Jul 9, 2014 at 8:45 AM, Jordan Zimmerman < >> > > jordan@jordanzimmerman.com >> > > > wrote: >> > > >> > > > Please open an issue in Jira for this one. That will touch every >> file. >> > > > >> > > > -JZ >> > > > >> > > > >> > > > On July 9, 2014 at 10:44:07 AM, Mike Drob (madrob@cloudera.com) >> wrote: >> > > > >> > > > Just noticed another issue - Appendix A needs to be updated in all >> of >> > the >> > > > LICENCE files. Instead of providing a year and copyright owner, we >> > should >> > > > use placeholder values, like in >> > > > http://www.apache.org/licenses/LICENSE-2.0.txt >> > > > >> > > > APPENDIX: How to apply the Apache License to your work. >> > > > >> > > > To apply the Apache License to your work, attach the followi= ng >> > > > boilerplate notice, with the fields enclosed by brackets "[]= " >> > > > replaced with your own identifying information. (Don't inclu= de >> > > > the brackets!) The text should be enclosed in the appropria= te >> > > > comment syntax for the file format. We also recommend that a >> > > > file or class name and description of purpose be included on >> the >> > > > same "printed page" as the copyright notice for easier >> > > > identification within third-party archives. >> > > > >> > > > Copyright [yyyy] [name of copyright owner] >> > > > >> > > > >> > > > >> > > > On Wed, Jul 9, 2014 at 10:34 AM, Mike Drob >> > wrote: >> > > > Just expand the header comment match what resides in the pom.xml >> > > > >> > > > Tomcat is a fine reference - >> > > > >> http://svn.apache.org/repos/asf/tomcat/site/trunk/docs/doap_Tomcat.rdf >> > > > >> > > > >> > > > On Wed, Jul 9, 2014 at 10:27 AM, Jordan Zimmerman < >> > > > jordan@jordanzimmerman.com> wrote: >> > > > * I=E2=80=99ve removed the dependency-reduced-pom.xml from the pro= ject. That >> > was >> > > a >> > > > mistake. >> > > > * I=E2=80=99ve updated the license plugins to ignore .json and .th= rift >> > > > >> > > > doap.rdf >> > > > What=E2=80=99s incomplete? >> > > > >> > > > It doesn't have _any_ licensing information, just a copyright >> > assertion. >> > > > Check out LEGAL-114 and LEGAL-172 for a long history. >> > > > That doap file has been that way for a while. Curator=E2=80=99s do= ap file >> looks >> > > > just like ZooKeeper=E2=80=99s. Without having to dive into the min= utia of >> those >> > > > docs, can you describe what needs to be added? Or, can you point t= o >> > > another >> > > > project=E2=80=99s DOAP file that is better? >> > > > >> > > > -JZ >> > > > >> > > > On July 9, 2014 at 10:20:56 AM, Mike Drob (madrob@cloudera.com) >> wrote: >> > > > >> > > > >> > > > >> > > > >> > > > On Wed, Jul 9, 2014 at 10:08 AM, Jordan Zimmerman < >> > > > jordan@jordanzimmerman.com> wrote: >> > > > * curator-x-rpc/dependency-reduced-pom.xml >> > > > This shouldn=E2=80=99t be part of the project >> > > > >> > > > Then it shouldn't be in the source or the release tag. :) >> > > > * curator-x-rpc/src/test/resources/configuration/connections.json >> > > > JSON doesn=E2=80=99t support comments so there=E2=80=99s no way to= have license >> > headers. >> > > > When I run =E2=80=9Cmvn license:check=E2=80=9D it comes back clean= (with some >> warnings >> > > > about unknown files). >> > > > >> > > > Yea, I'm not sure there's anything to be done for these, which is >> > > > unfortunate. >> > > > doap.rdf >> > > > What=E2=80=99s incomplete? >> > > > >> > > > It doesn't have _any_ licensing information, just a copyright >> > assertion. >> > > > Check out LEGAL-114 and LEGAL-172 for a long history. >> > > > >> > > > >> > > > >> > > > >> > > > On July 9, 2014 at 10:02:40 AM, Mike Drob (madrob@cloudera.com) >> wrote: >> > > > >> > > > -1 (non-binding) >> > > > >> > > > Verified signature and hashes, all good. >> > > > >> > > > The git tag does not match the source zip: >> > > > * zip contains a file called .test.swp >> > > > * zip differs from tag on curator-x-rpc/dependency-reduced-pom.xml >> > > (brought >> > > > up in the other thread) >> > > > >> > > > Missing licence headers: >> > > > * curator-x-rpc/pom.xml >> > > > * curator-x-rpc/dependency-reduced-pom.xml >> > > > * curator-x-rpc/src/main/thrift/curator.thrift >> > > > * curator-x-rpc/src/main/resources/curator/help.txt >> > > > * curator-x-rpc/src/test/resources/configuration/connections.json >> > > > * curator-x-rpc/src/test/resources/configuration/logging.json >> > > > * curator-x-rpc/src/test/resources/configuration/simple.json >> > > > * curator-x-rpc/src/test/resources/configuration/test.json >> > > > >> > > > Incomplete licence header: >> > > > * doap.rdf >> > > > >> > > > Attempted to build from source: mvn clean verify >> > > > * Compiled cleanly! >> > > > * Had a test failure in TestDistributedDelayQueue.testSorting() >> (fixed >> > in >> > > > master) >> > > > * Had a intermittent test failure in >> > > > TestDistributedQueue.testCustomExecutor() (will investigate furthe= r) >> > > > >> > > > >> > > > On Wed, Jul 9, 2014 at 6:47 AM, Eric Tschetter >> > > wrote: >> > > > >> > > > > +1 >> > > > > >> > > > > On Wednesday, July 9, 2014, Ashish >> wrote: >> > > > > >> > > > > > +1 (non-binding) >> > > > > > >> > > > > > Ran the build. Works fine and all test cases passed >> > > > > > >> > > > > > >> > > > > > On Wed, Jul 9, 2014 at 5:31 AM, Jordan Zimmerman < >> > > > > > jordan@jordanzimmerman.com >> > > > > > > wrote: >> > > > > > >> > > > > > > Validated keys >> > > > > > > >> > > > > > > +1 >> > > > > > > >> > > > > > > >> > > > > > > On July 8, 2014 at 5:08:37 PM, Cameron McKenzie ( >> > > > > mckenzie.cam@gmail.com >> > > > > > ) >> > > > > > > wrote: >> > > > > > > >> > > > > > > Hello, >> > > > > > > >> > > > > > > This is the vote for Apache Curator version 2.6.0 >> > > > > > > >> > > > > > > *** Please download, test and vote within approx. 72 hours >> > > > > > > >> > > > > > > Note that we are voting upon the source (tag) and binaries a= re >> > > > > > > provided for convenience. >> > > > > > > >> > > > > > > Link to release >> > > > > > > notes: >> > > > > > > >> > > > > > >> > > > > >> > > > >> > > >> > >> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=3D12314= 425&version=3D12327098 >> > > > > > > >> > > > > > > Staging repo: >> > https://dist.apache.org/repos/dist/dev/curator/2.6.0/ >> > > > > > > >> > > > > > > Binary artifacts: >> > > > > > > >> > > > > >> > > >> https://repository.apache.org/content/repositories/orgapachecurator-1004 >> > > > > > > >> > > > > > > >> > > > > > > The tag to be voted >> > > > > > > upon: >> > > > > > > >> > > > > > >> > > > > >> > > > >> > > >> > >> https://git-wip-us.apache.org/repos/asf?p=3Dcurator.git;a=3Dtag;h=3Dapac= he-curator-2.6.0 >> > > > > > > >> > > > > > > Curator's KEYS file containing PGP keys we use to sign the >> > > > > > > release:http://www.apache.org/dist/curator/KEYS >> > > > > > > >> > > > > > > [ ] +1 approve >> > > > > > > [ ] +0 no opinion >> > > > > > > [ ] -1 disapprove (and reason why) >> > > > > > > >> > > > > > > >> > > > > > > cheers >> > > > > > > >> > > > > > > Cam >> > > > > > > >> > > > > > >> > > > > > >> > > > > > >> > > > > > -- >> > > > > > thanks >> > > > > > ashish >> > > > > > >> > > > > > Blog: http://www.ashishpaliwal.com/blog >> > > > > > My Photo Galleries: http://www.pbase.com/ashishpaliwal >> > > > > > >> > > > > >> > > > >> > > > >> > > > >> > > > >> > > >> > >> > > --089e0158c004966c3904fde2e5f1--