From dev-return-48388-archive-asf-public=cust-asf.ponee.io@couchdb.apache.org Wed Feb 27 21:26:35 2019 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 BF5A4180608 for ; Wed, 27 Feb 2019 22:26:34 +0100 (CET) Received: (qmail 19657 invoked by uid 500); 27 Feb 2019 21:26:33 -0000 Mailing-List: contact dev-help@couchdb.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@couchdb.apache.org Delivered-To: mailing list dev@couchdb.apache.org Received: (qmail 19643 invoked by uid 99); 27 Feb 2019 21:26:33 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd1-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 27 Feb 2019 21:26:33 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd1-us-west.apache.org (ASF Mail Server at spamd1-us-west.apache.org) with ESMTP id B21F1C8FF7 for ; Wed, 27 Feb 2019 21:26:32 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd1-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 0.501 X-Spam-Level: X-Spam-Status: No, score=0.501 tagged_above=-999 required=6.31 tests=[HEADER_FROM_DIFFERENT_DOMAINS=0.001, KAM_NUMSUBJECT=0.5, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=disabled Received: from mx1-lw-eu.apache.org ([10.40.0.8]) by localhost (spamd1-us-west.apache.org [10.40.0.7]) (amavisd-new, port 10024) with ESMTP id G5g4LjLrtMKt for ; Wed, 27 Feb 2019 21:26:30 +0000 (UTC) Received: from smtp.justsomehost.net (smtp.justsomehost.net [204.11.51.157]) by mx1-lw-eu.apache.org (ASF Mail Server at mx1-lw-eu.apache.org) with ESMTPS id 7C4C0610ED for ; Wed, 27 Feb 2019 21:26:29 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp.justsomehost.net (Postfix) with ESMTP id 24E6B58009B for ; Wed, 27 Feb 2019 16:26:27 -0500 (EST) Received: from smtp.justsomehost.net ([127.0.0.1]) by localhost (smtp.justsomehost.net [127.0.0.1]) (amavisd-new, port 10032) with ESMTP id b9o_NBOwBft2 for ; Wed, 27 Feb 2019 16:26:26 -0500 (EST) Received: from localhost (localhost [127.0.0.1]) by smtp.justsomehost.net (Postfix) with ESMTP id 66328580186 for ; Wed, 27 Feb 2019 16:26:26 -0500 (EST) X-Virus-Scanned: amavisd-new at smtp.justsomehost.net Received: from smtp.justsomehost.net ([127.0.0.1]) by localhost (smtp.justsomehost.net [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id XoQfHzhVZkfy for ; Wed, 27 Feb 2019 16:26:26 -0500 (EST) Received: from smtp.justsomehost.net (smtp.justsomehost.net [204.11.51.157]) by smtp.justsomehost.net (Postfix) with ESMTP id 4AEF958009B for ; Wed, 27 Feb 2019 16:26:26 -0500 (EST) Date: Wed, 27 Feb 2019 16:26:26 -0500 (EST) From: Joan Touzet Reply-To: wohali@apache.org To: dev@couchdb.apache.org Message-ID: <806132352.2539.1551302787738.JavaMail.Joan@BRAIN> In-Reply-To: References: <1550586614.2948676.1660696904.052F9C19@webmail.messagingengine.com> <6A181EFC-3874-40BE-9947-F48FF474AEB7@apache.org> <282357197.1451.1550700114772.JavaMail.Joan@BRAIN> <2F26917F-2C4F-4463-BABA-BD81AF7404B7@apache.org> Subject: Re: [VOTE] Release Apache CouchDB 2.3.1 RC2 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Originating-IP: [204.11.51.157] X-Mailer: Zimbra 8.8.9_GA_3026 (Zimbra Desktop/7.3.1_13063_Windows) Thread-Topic: Release Apache CouchDB 2.3.1 RC2 Thread-Index: Icw/qWbvv0mFkfjUYYQFcwYLqkuCQA== Based on discussion with Russell Branca (chewbranca) in IRC, we need to abort this RC vote as he is effectively voting -1. Here's the full transcript of our discussion: ------------------------ 16:06 <+Wohali> chewbranca: you there? are you seeing these eunit context setup errors in 2.3.0 as well as the 2.3.1 RC and master? 16:06 <+Wohali> I don't want to hold up 2.3.1 over something that was a pre-existing condition, but if it's something that changed between 2.3.0 and 2.3.1/master, we need to fix it 16:07 Wohali: well the fundamental issue right now is test suite failures don't fail the build, which IMO should be fixed before any further builds 16:08 I've been using this diff locally, which fails the `make eunit` check upon an eunit failure: https://gist.github.com/chewbranca/65d2969ac191a5dfaf87172ace18d2ee 16:08 not sure that's the best approach, but we need something like that 16:08 <+Wohali> What I'm asking is: do you think this should block the release of 2.3.1? 16:08 <+Wohali> By all means PR that to master and let's get shit in gear 16:08 <+Wohali> I'm trying to work out when this problem started occurring, though. 16:09 yes, should definitely block any further releases, because unless someone is manually inspecting the eunit output, then we could have test failures bubbling through 16:11 in theory this particular issue was introduced 26 days ago with the change to running individual eunit tests: https://github.com/apache/couchdb/commit/20bbfbf972ad1f822e2ef1edfb3d47f2cec3f639 16:11 so this is probably a new thing, but we've definitely had issues with eunit over the years 16:12 Wohali: I can make a quick PR with the diff I pasted above and then we should be good to go IMO, but it wouldn't hurt to see if there's a more proper way to do that in a Makefile than just `|| exit 1` 16:16 <+Wohali> chewbranca: are you 100% sure that context setup failures mean the tests are actually failing? They seem to be running and passing even after that. I'm too unfamiliar to know for sure. 16:17 <+Wohali> chewbranca: that change you linked isn't in 2.3.1. 16:17 context setup failure means that setting up a series of eunit test generators failed and those tests aren't being executed 16:17 <+Wohali> ok. 16:18 those will fail if you do `|| exit 1`, but they continue running today because we don't exit on the individual eunit runs 16:18 <+Wohali> 2.3.1 has a critical fix for buffer sizes that we need to get out there. WOuld you accept me manually reviewing the output of 2.3.1's test suite to ensure no context setup failures? 16:18 <+Wohali> then we make this a blocker for 2.4.0? 16:18 what I linked above is just a diff that I've been using locally because I wanted the suite to fail, and it works 16:19 Wohali: IMO let's just add that diff and then if folks know a more proper Makefile approach to doing that type of thing then they can fix it later 16:19 <+Wohali> to both 2.3.1 and master? And to Makefile.Win I presume? ;) Then we'll have to cancel the current RC and re-spin. ... 16:25 https://github.com/apache/couchdb/pull/1951 ------------------------ ----- Original Message ----- > From: "Dave Cottlehuber" > To: dev@couchdb.apache.org > Sent: Monday, February 25, 2019 6:10:05 AM > Subject: Re: [VOTE] Release Apache CouchDB 2.3.1 RC2 > > On Mon, 25 Feb 2019, at 10:56, Dave Cottlehuber wrote: > > On Thu, 21 Feb 2019, at 06:27, Jan Lehnardt wrote: > > > > FreeBSD 12.0-RELEASE-p3 amd64 + OTP 21.2.6 custom > > > > - OK sigs and checksums > > - OK release > > - fauxton verify is happy > > - make check fails with the C.UTF-8 issues Joan has mentioned > > previously > > > > belated +1 from me > > > > BTW the port will be a bit delayed this time as I need to bump OTP > > version and that usually has a bit of ports tree shakeout. My patch > > for > > that is https://reviews.freebsd.org/D18820 > > I forgot to mention that the tarball has the annoying -RC2 suffix in > filenames, which makes the downstream packaging diffs fiddly. I have > that unfinished PR https://github.com/apache/couchdb/pull/1927 > hopefully to fix that for next time. > > A+ > Dave >