From owner-svn-src-head@freebsd.org Wed Jun 20 21:28:18 2018 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id E5EFD100D6F2; Wed, 20 Jun 2018 21:28:17 +0000 (UTC) (envelope-from hackagadget@gmail.com) Received: from mail-ot0-x243.google.com (mail-ot0-x243.google.com [IPv6:2607:f8b0:4003:c0f::243]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 53B1075D20; Wed, 20 Jun 2018 21:28:17 +0000 (UTC) (envelope-from hackagadget@gmail.com) Received: by mail-ot0-x243.google.com with SMTP id d19-v6so1164824oti.8; Wed, 20 Jun 2018 14:28:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=Vqd+77iwXqqbaOVEm7nm4isJRl1ymIOPF3kQXz7NHmU=; b=Vfj0l3BgICSVJneeAJHcCgNom5bB/fRN5QHjblm7lYBmyJWCMo0gdglV5rcipMywp8 yB49D5RQn7/OCmPzxKlHCmYkhcz9OY9P0UcBnvLJ8ySF7U0q9FLawQuZyo7R/fxC5xg4 uJUwPNXLawitP6hVvxtkkcAFIWd2C++GDWTglctekP5Ejh0efbqHqpZZ/1FJEgtKP1A/ dGZJLgX/wkh0eEqrfNmXX+pnVlxoGBUqF0r3WWjqgr1qctA+OFOHVYeppMUK/9QmIvAK dEPVEhlN/jGTv5mGBwG5GE2BawuVSd2EaatirZPHTTP95AWPs7w6rYhtzMiYTp8d8OEm +Gbg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=Vqd+77iwXqqbaOVEm7nm4isJRl1ymIOPF3kQXz7NHmU=; b=gyrPhGOZi8tBHbigvhoCOD2WhAq/ATK6rKcLK8bZqYis8yPddbJXmLpsM19FQWStN1 84NHqd2TS/V8Jz81TEUFKlBDerK3le8yPgIx6pGotQjOzmZjm+I2+OsVa4Rwm4J4Vkfe vBK7pj+p8TsHIPK+uTOmXhK64nxN/3rILALNq5uu0niMfaS3OjLytj0A354RFCvTKNkV Ju0WJw2wqpldh2mZART2wzOVXLLoNVmX87KcG0swStrWlyMeLfvdc9OI2MpL3WrHQ6qV wL3zQYS/BlWbkn6sZxzKQvkfuJE/p//Tuf01a4QuVgdWX7G8sIkZ5KZODi3AVCB/EppK fDzA== X-Gm-Message-State: APt69E22YNYDvFHTw4f3sge4SDbjRRaCtar9tgjcEAgF64gYNOmaF+3M IHNlWLG2q63cQITiau94o8ZVjiEX/nqMksr5Xc/M9J+rYg4= X-Google-Smtp-Source: ADUXVKLE+9JxilGoTINI3cQj4yA9iFOUjqpPNd/0KOp8V0iK/6u5vf0VvCkEHLT36+Pg82sbgShKp+416kRxiJdmnKE= X-Received: by 2002:a9d:5190:: with SMTP id y16-v6mr14867764otg.252.1529530096533; Wed, 20 Jun 2018 14:28:16 -0700 (PDT) MIME-Version: 1.0 Received: by 2002:a9d:1918:0:0:0:0:0 with HTTP; Wed, 20 Jun 2018 14:28:15 -0700 (PDT) In-Reply-To: References: <201806200108.w5K18sIR050132@repo.freebsd.org> <96021.1529475664@kaos.jnpr.net> <17033.1529508519@kaos.jnpr.net> <1529510299.24573.5.camel@freebsd.org> From: Stephen Kiernan Date: Wed, 20 Jun 2018 14:28:15 -0700 Message-ID: Subject: Re: svn commit: r335402 - head/sbin/veriexecctl To: cem@freebsd.org Cc: Jonathan Anderson , svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.26 X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 20 Jun 2018 21:28:18 -0000 (Apologies to cem@ for the duplicate in his inbox, Gmail decided to not reply-all in my reply.) On Wed, Jun 20, 2018 at 1:07 PM, Conrad Meyer wrote: > Hi Jon, > > On Wed, Jun 20, 2018 at 11:58 AM, Jonathan Anderson > wrote: > > On 20 Jun 2018, at 15:32, Jonathan T. Looney wrote: > >> My reasoning > >> was fairly simple: when a review has been open for over a year with no > >> action, it seems like the submitter should be able to commit it without > >> waiting for more review, if they are confident in their change. I stand > by > >> that (and, in fact, would substitute something shorter for "over a > year"). > > For this as a question of general process, I think "it depends." For > minor fixes? Of course! Less than one year. For minor improvements > or enhancements? Also yes, also probably less than 1yr. For large > features like this, I think the answer is more nuanced. > > Sometimes being in a review for a year means the reviewers are > overloaded and don't have time. In that case, maybe they shouldn't > block progress. Sometimes being in review for a year just means the > right reviewers haven't been found. I think that was more the case > here. > > I would suggest anyone submitting a large feature and not getting > feedback in a reasonable timeframe to solicit feedback from a wider > audience much sooner than one year, and also indicate intention to > merge the unreviewed change with some time window (1-2 weeks?) on > giving feedback or at least asking for more time to review. > That was the problem. I soliticted feedback many times over at least 3 years. The first set was in my GitHub branch and I had discussed with various people starting at BSDCan 3 or 4 years ago. I had even sent an early version of the changes to rwatson (whom I had previously worked with at TIS/NAI/McAfee) and was provided some feedback based on a cursory look over of the code, which was integrated into the branch. At MeetBSD 2016, I also discussed the code with some people and that was when it was suggested that I create an account of Phabricator and put the code up for review. Some members of core and/or the Foundation had suggested who to talk to get reviews and I had also spoken with rwatson again to see if he had any suggestions. Once the code had landed in review, I also talked with people during vBSDcon last year in an effort to try to find out who else should put eyes on it. I even talked about it during one of the BoF sessions and pointed out both my GitHub branch and the set of reviews on Phabricator, which is when there started to get some additional feedback and/or watchers on the review. Also, after I had received my commit bit and the reviews were still collecting dust in Phabricator, on numberous occassions I solicited feedback or suggestions on who to contact on the IRC channel. I'm guessing that people had not looked at the reviews before suggesting who to talk to or they did not realize the areas of interest that the changes might cover. I think this experience illustrates the need for some better way for someone contributing to be able to find out who the correct parties are that need to be contacted. It was 3+ year struggle for me to find people to put eyes on the code and it was not just a fire it off and wait forever thing. One can only listen to crickets in response for requests for help for so long. -Steve