Return-Path: X-Original-To: apmail-incubator-general-archive@www.apache.org Delivered-To: apmail-incubator-general-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 173C2181F3 for ; Wed, 15 Jul 2015 01:51:40 +0000 (UTC) Received: (qmail 28369 invoked by uid 500); 15 Jul 2015 01:51:39 -0000 Delivered-To: apmail-incubator-general-archive@incubator.apache.org Received: (qmail 28152 invoked by uid 500); 15 Jul 2015 01:51:39 -0000 Mailing-List: contact general-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: general@incubator.apache.org Delivered-To: mailing list general@incubator.apache.org Received: (qmail 28141 invoked by uid 99); 15 Jul 2015 01:51:39 -0000 Received: from mail-relay.apache.org (HELO mail-relay.apache.org) (140.211.11.15) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 15 Jul 2015 01:51:39 +0000 Received: from mail-lb0-f176.google.com (mail-lb0-f176.google.com [209.85.217.176]) by mail-relay.apache.org (ASF Mail Server at mail-relay.apache.org) with ESMTPSA id B1CEC1A05D8 for ; Wed, 15 Jul 2015 01:51:38 +0000 (UTC) Received: by lbbyj8 with SMTP id yj8so16230761lbb.0 for ; Tue, 14 Jul 2015 18:51:36 -0700 (PDT) MIME-Version: 1.0 X-Received: by 10.112.30.176 with SMTP id t16mr1451234lbh.30.1436925096692; Tue, 14 Jul 2015 18:51:36 -0700 (PDT) Reply-To: jfarrell@apache.org Received: by 10.112.34.105 with HTTP; Tue, 14 Jul 2015 18:51:36 -0700 (PDT) In-Reply-To: References: Date: Tue, 14 Jul 2015 21:51:36 -0400 Message-ID: Subject: Re: Podling request: Gerrit From: Jake Farrell To: "general@incubator.apache.org" Cc: David Nalley Content-Type: multipart/alternative; boundary=001a1133b7c8f5127d051ae03186 --001a1133b7c8f5127d051ae03186 Content-Type: text/plain; charset=UTF-8 Hey Till All commits must occur against the ASF codebase and can not be mirrored over from a different location (this includes Github, Gerrit, etc.). This is a mandate that has come from the board level and is not a negotiable item. Gerrit has been discussed many times before and each time the answer has been that without people to help maintain the service and until Gerrit can be used as only a review tool and not require controlling the primary codebase and resulting merge that it will not be an viable option for the ASF's use. We are working to see what we can do to help ease some of the pain with pull requests, but at this time the tooling available is as stated previously in this thread. Infra is always happy to take suggestions and any ideas and we are always looking for volunteers that are interested in lending a hand and helping improve our ecosystem for all projects. If you have any questions please let me know -Jake On Tue, Jul 14, 2015 at 8:08 PM, Till Westmann wrote: > > On 14 Jul 2015, at 15:31, David Nalley wrote: > > On Tue, Jul 14, 2015 at 1:14 AM, Ian Maxon wrote: >> >> We use Gerrit as >>> a tool to do code reviews and to organize the commits, as well as to >>> facilitate easy testing. However that's all it's used for- we still >>> clone from repositories that come downstream from ASF, not the other >>> way around. I'd be interested to understand how this would be >>> considered any different than what is done with Github Pull Requests. >>> >>> >> So GH PR have a subtle distinction (at least in the way that they are >> handled at the ASF). Projects can't merge pull requests into the repo >> at github. Non-committers see a workflow that is the Github workflow, >> because that's very familiar, and lowers the barrier to contribution. >> Committers, however, have a very different workflow than the folks who >> typically review and close pull requests on github. They have to take >> the patch [1], and merge it into the canonical repository at the ASF, >> which then appears in the github repository because of the mirror >> process. This stops the problem of diverging codebases that you are >> currently experiencing, calls to rewrite history to align the ASF repo >> with the external repo, etc. >> > > As Ian indicated AsterixDB's process also requires manual interaction of > a committer. The current steps are now documented on the website [2]. > > There are some other problems, that aren't necessarily as worrisome, >> but should be something to consider. First, you're relying on a third >> party to provide that resource. That's not inherently a problem, but >> we have a number of examples of projects using external tools and >> those being shut down or phased out which causes tremendous disruption >> to projects. It's also at the old project's home, which might cause >> some folks to question whether the project is truly independent, or >> not. >> > > In my view Gerrit is "just" a tool that the AsterixDB community chose > to keep when starting the incubation process. It is is non-essential and > has been used by developers from different organizations before the > incubation started. But I think that its use was and is very beneficial > to the project. > When we started incubation it seemed to us, that keeping the existing > tool would be a good idea as it > a) allows for a smoother transition and > b) would not put additional requirements on the ASF infrastructure. > > However, I do agree that a shut down of the service (which seems very > unlikely at the current point in time) could be a disruption to the > project. So it might be better to run this tool on the ASF > infrastructure. > Should we pursue this? > Or is it acceptable to keep the tool on external hardware for now? > Or do you see fundamental issues with AsterixDB's use of Gerrit? > > Thanks, > Till > > [1] >> https://patch-diff.githubusercontent.com/raw/apache/airavata/pull/18.patch >> > > [2] https://asterixdb.incubator.apache.org/pushing.html > > --------------------------------------------------------------------- > To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org > For additional commands, e-mail: general-help@incubator.apache.org > > --001a1133b7c8f5127d051ae03186--