Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [eclipse-pmc] Gerrit submit strategies proposal
  • From: "Becker, Matthias" <ma.becker@xxxxxxx>
  • Date: Wed, 2 Oct 2019 07:42:20 +0000
  • Accept-language: en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=sap.com; dmarc=pass action=none header.from=sap.com; dkim=pass header.d=sap.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=veE8+KzURazfmoA8VlSreE49uVSt3NgoH4EYi9g0GNs=; b=EdYAfpdlg8ZFHJTw2QOv+xqtU8F4ZARf7/sHOS8dmuKF4WA8nX0O/qewTvYOM5eQKpYEJCCw4WTQ/VBA3aPvGqIlPP2dTkb8A8jC5s+qTvmqlYnMB+A4flbeJZ2R8KDjhk2Qd6dtMMrNl7mBgjVHhk3efeiM8SKz1AZlnmrwv8vTx+6+gKc0RigteRx9NsnTyKl6HkPUIolSUoiwZkmZp2dvbLpeDKIJmXTQKEyZcNaGfBWMNRln5Q7RjHcszhEjZgObyKmLDhROauA/9jrWaPXepc5+bdfflTGBO0jCTwKkXXtKRmk33yk1Nl//qcgrXxoGgv41ITMfVbrS75WtOg==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=TxMQ+diwhlQCZ1iLxj5wj3Wt1t7WTn4pDV3HEQOp/ZkEsuGw76qiOgejlnF2A63CZRhXpq5/X7DyVyrTWw5Xhchmp8ObxIx9fXFDrTV9h/1DkC2anjmvCBK+VDOOc0stfodedF89x1NBnAxkOuR9I0JJlwxDrdIjgHLSkSYbXt7Q/bZdIg95jL92ISyksXwcw+zjheHiM7winSu0DVfPFeUTkjqoriNPijBIx2ecI0x9GMExMnO522Hdp6k50mPASPiaXelQJkyQCo70DdpLKtrjOI6QkYXNpw7amwD6jc/0b0ZT9dgBCUnaDZbjMyX60x1WmRQLBcxLtQ3CWpUGDQ==
  • Delivered-to: eclipse-pmc@xxxxxxxxxxx
  • List-archive: <https://www.eclipse.org/mailman/private/eclipse-pmc>
  • List-help: <mailto:eclipse-pmc-request@eclipse.org?subject=help>
  • List-subscribe: <https://www.eclipse.org/mailman/listinfo/eclipse-pmc>, <mailto:eclipse-pmc-request@eclipse.org?subject=subscribe>
  • List-unsubscribe: <https://www.eclipse.org/mailman/options/eclipse-pmc>, <mailto:eclipse-pmc-request@eclipse.org?subject=unsubscribe>
  • Thread-index: AQHVeETI0FYbqtEaIU6nfAf/XDOQGqdFnmGAgAFNjICAAC7oAA==
  • Thread-topic: [eclipse-pmc] Gerrit submit strategies proposal
  • User-agent: Microsoft-MacOutlook/10.1d.0.190908
+1 too.
In my development projects here at SAP we use Gerrit with "merge if necessary" without issue.
I assume that the platform.ui gerrit builds job do a rebase on master...

On 02.10.19, 08:55, "eclipse-pmc-bounces@xxxxxxxxxxx on behalf of Karsten Thoms" <eclipse-pmc-bounces@xxxxxxxxxxx on behalf of karsten.thoms@xxxxxxxxx> wrote:

    Yes, PLEASE, it is PITA to rebase over and over again. Especially for long running builds as in platform-ui. This is wasting committers' time and frustrates contributors.
    
    +1
    
    > Am 01.10.2019 um 13:00 schrieb Lars Vogel <lars.vogel@xxxxxxxxxxx>:
    > 
    > +1 I don't think that the rebased changes are reviewed again and
    > again, committers just have to wait a long time to get a merge window
    > and that is demotivating.
    > 
    > Now that our committer and contributor community activity has hugely
    > increased, we should remove that demotivating approach.
    > 
    > On Tue, Oct 1, 2019 at 12:40 PM Aleksandar Kurtakov <akurtako@xxxxxxxxxx> wrote:
    >> 
    >> We are seeing an increating contributions and our gerrit verification jobs start to suffer due to our usage of "Fast forward only" strategy. Reality though is that no one rebases his change multiple times and wait for gerrit again just because another commit went in, I've tried and in platform.ui I gave up on 3rd rebase and just pushed it right after rebase. But even this generated one more totally useless gerrit build.
    >> So my proposal is to change our strategy to "Rebase if Necessary"  - I don't see any downside using it and multiple benefits - free a lot of time on our builders, reduce interruptions for contributors and so on.
    >> Let's discuss today on PMC call
    >> 
    >> Strategies explained: https://gerrit-review.googlesource.com/Documentation/concept-changes.html#submit-strategies
    >> 
    >> --
    >> Alexander Kurtakov
    >> Red Hat Eclipse Team
    >> _______________________________________________
    >> eclipse-pmc mailing list
    >> eclipse-pmc@xxxxxxxxxxx
    >> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
    >> https://www.eclipse.org/mailman/listinfo/eclipse-pmc
    > 
    > 
    > 
    > -- 
    > Eclipse Platform project co-lead
    > CEO vogella GmbH
    > 
    > Haindaalwisch 17a, 22395 Hamburg
    > Amtsgericht Hamburg: HRB 127058
    > Geschäftsführer: Lars Vogel, Jennifer Nerlich de Vogel
    > USt-IdNr.: DE284122352
    > Fax (040) 5247 6322, Email: lars.vogel@xxxxxxxxxxx, Web: http://www.vogella.com
    > _______________________________________________
    > eclipse-pmc mailing list
    > eclipse-pmc@xxxxxxxxxxx
    > To change your delivery options, retrieve your password, or unsubscribe from this list, visit
    > https://www.eclipse.org/mailman/listinfo/eclipse-pmc
    
    _______________________________________________
    eclipse-pmc mailing list
    eclipse-pmc@xxxxxxxxxxx
    To change your delivery options, retrieve your password, or unsubscribe from this list, visit
    https://www.eclipse.org/mailman/listinfo/eclipse-pmc

Back to the top