From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id AA250A09E9; Fri, 11 Dec 2020 21:07:49 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 18B85AC9C; Fri, 11 Dec 2020 21:07:48 +0100 (CET) Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id D43F9AC9A for ; Fri, 11 Dec 2020 21:07:44 +0100 (CET) IronPort-SDR: +1ue0V7AcuXtgQf4uHxOx7/tHIXUCGnm5ssRGL8JI7XgIaW3ppSjdOE2HDUch7/TelZY2I/iT5 YydUYA5nUBkw== X-IronPort-AV: E=McAfee;i="6000,8403,9832"; a="161539999" X-IronPort-AV: E=Sophos;i="5.78,412,1599548400"; d="scan'208";a="161539999" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Dec 2020 12:07:43 -0800 IronPort-SDR: QaFPOkKF/jvcWWikxu8ka5yQTEQn4dOEf4dQXZ9645QJvYo4PSezn0MBquvBOdt1mGIgkWdw8N kBpwXrTPajIA== X-IronPort-AV: E=Sophos;i="5.78,412,1599548400"; d="scan'208";a="409434668" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.213.200.9]) ([10.213.200.9]) by orsmga001-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Dec 2020 12:07:42 -0800 To: David Marchand , dev@dpdk.org Cc: aconole@redhat.com, Michael Santana , Thomas Monjalon References: <20201124215700.12126-1-david.marchand@redhat.com> <20201204173622.12053-1-david.marchand@redhat.com> From: Ferruh Yigit Message-ID: <0da994ab-da6f-2b07-d58c-210ae1b9f1ce@intel.com> Date: Fri, 11 Dec 2020 20:07:38 +0000 MIME-Version: 1.0 In-Reply-To: <20201204173622.12053-1-david.marchand@redhat.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH v2 1/2] ci: hook to GitHub Actions X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 12/4/2020 5:36 PM, David Marchand wrote: > With the recent changes in terms of free access to the Travis CI, let's > offer an alternative with GitHub Actions. > Running jobs on ARM is not supported unless using external runners, so > this commit only adds builds for x86_64 and cross compiling for i386 and > aarch64. > > Differences with the Travis CI integration: > - Error logs are not dumped to the console when something goes wrong. > Instead, they are gathered in a "catch-all" step and attached as > artifacts. > - A cache entry is stored once and for all, but if no cache is found you > can inherit from the default branch cache. The cache is 5GB large, for > the whole git repository. > - The maximum retention of logs and artifacts is 3 months. > - /home/runner is world writable, so a workaround has been added for > starting dpdk processes. > - Ilya, working on OVS GHA support, noticed that jobs can run with > processors that don't have the same capabilities. For DPDK, this > impacts the ccache content since everything was built with > -march=native so far, and we will end up with binaries that can't run > in a later build. The problem has not been seen in Travis CI (?) but > it is safer to use a fixed "-Dmachine=default" in any case. > - Scheduling jobs is part of the configuration and takes the form of a > crontab. A build is scheduled every Monday at 0:00 (UTC) to provide a > default ccache for the week (useful for the ovsrobot). > > Signed-off-by: David Marchand > --- > Changelog since v1: > - changed shell variables value in CI scripts and Travis configuration > (s/=[^\$]*1/=\1true), this makes it easier for GHA, > - forced compilation as 'default' to avoid random unit tests issues in > GHA, > - scheduled a run per week on Monday at 0:00 UTC, > - updated the ccache key: > - no need to depend on the default-library parameter since this > parameter only impacts the linking of dpdk binaries, > - the week when the cache is generated is added so that jobs in > other branches can benefit from a recent cache (mimicking what we had > for the robot in Travis), > - realigned documentation generation with what is done in Travis: > generating the doc in all jobs was a waste of resources, > For series, Tested-by: Ferruh Yigit Confirmed that ABI check script is detecting issues, in the absence of the Travis checks I am for having this alternative.