From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ernst.netinsight.se (ernst.netinsight.se [194.16.221.21]) by dpdk.org (Postfix) with SMTP id 8F2E2378B for ; Thu, 28 May 2015 08:54:43 +0200 (CEST) Received: from [10.100.1.152] (unverified [10.100.1.152]) by ernst.netinsight.se (EMWAC SMTPRS 0.83) with SMTP id ; Thu, 28 May 2015 08:54:39 +0200 Message-ID: <5566BBAF.1040408@netinsight.net> Date: Thu, 28 May 2015 08:54:39 +0200 From: =?windows-1252?Q?Simon_K=E5gstr=F6m?= User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: dev@dpdk.org References: <55662DAF.3000909@redhat.com> In-Reply-To: <55662DAF.3000909@redhat.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] DPDK: Proposal for a patch patch-test integration tree X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 28 May 2015 06:54:43 -0000 I spot a can of worms to be opened here :-) On 2015-05-27 22:48, Thomas F Herbert wrote: > Work Flow and Process: > > All patches will be taken from from public submissions to dpdk-dev.org > scraped from dpdk patchwork. Patches will be applied to the patch-test > tree and tested against HEAD as they are received. The feedback from the > testing will be provided to the community. The patch-test tree will > periodically be git pull'ed from dpdk. > > Longer term goal: > > Initially, the patches will be applied along with some simple smoke > tests. The longer term goal is to automate this process, apply more > extensive tests and post the results in dpdk patchwork, > http://dpdk.org/dev/patchwork/project/dpdk/list/ which would have an > accompanying mailing list for distribution of a results summary of the > tests. Actually, github and services such as travis-ci and coveralls already provide this functionality (with very little setup). So when someone sends a pull request, the continuous integration service travis-ci will notice it, and start a build and (possibly) run a test suite on the code - with the patches applied. If code coverage is collected in the process [1], it's uploaded to the coveralls site. Both travis-ci and coveralls will add a note to the pull request saying something like "Build failed with this patch, be careful" or "Build OK, everything is fine" and "Coverage decreased with 5% with this patch" etc etc. Of course, github provides an API so it's entirely possible to add your own continuous integration support with the same functionality as travis-ci (customized for DPDK). So before venturing into implementing something like this, I think the DPDK project should at least consider the existing alternatives. And with that, I close the can of worms again. I hope no worms were hurt in the process! :-) // Simon [1] https://github.com/SimonKagstrom/kcov - yes my personal project