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 14190A0555; Wed, 19 Feb 2020 22:39:53 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 875F2397D; Wed, 19 Feb 2020 22:39:52 +0100 (CET) Received: from us-smtp-1.mimecast.com (us-smtp-delivery-1.mimecast.com [207.211.31.120]) by dpdk.org (Postfix) with ESMTP id 3E2453B5 for ; Wed, 19 Feb 2020 22:39:50 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1582148389; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=t2RA90mWYXW5OcWpBHG5h5DWmmOqNH+2fjsyg+mn4NM=; b=Gh8yWxspPNwdXiAeLE5SyvAI1jj+VD93wHAwXXATzZ0YaA3j9rZWMym19g6kBMUNr4GG3n btHQAoOAShiebrqZuZ4JZc4dspVmtFx0azGLId5XuLJ/eoz/PH3buU3O/kX4jgivTRRYzp AA8Ad4DysEqad11oa2j1K08rb9z9k18= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-305-nj9FhYCRPqmBrNpnGd3xog-1; Wed, 19 Feb 2020 16:39:41 -0500 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 4FA85800D50; Wed, 19 Feb 2020 21:39:40 +0000 (UTC) Received: from dhcp-25.97.bos.redhat.com (unknown [10.18.25.126]) by smtp.corp.redhat.com (Postfix) with ESMTPS id D11578ECF3; Wed, 19 Feb 2020 21:39:36 +0000 (UTC) From: Aaron Conole To: David Marchand Cc: thomas@monjalon.net, dev@dpdk.org, Michael Santana , Bruce Richardson References: <20200219194131.29417-1-david.marchand@redhat.com> <20200219194131.29417-5-david.marchand@redhat.com> Date: Wed, 19 Feb 2020 16:39:36 -0500 In-Reply-To: <20200219194131.29417-5-david.marchand@redhat.com> (David Marchand's message of "Wed, 19 Feb 2020 20:41:31 +0100") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux) MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-MC-Unique: nj9FhYCRPqmBrNpnGd3xog-1 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain Content-Transfer-Encoding: quoted-printable Subject: Re: [dpdk-dev] [PATCH 4/4] ci: reorganise Travis jobs 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" David Marchand writes: > Let's prune the jobs list to limit the amount of time spent by the robot > in Travis. > > Since meson enables automatically the relevant components, there is not > much gain in testing with extra_packages vs required_packages only. > > For a given arch/compiler/env combination, compilation is first tested > in all jobs that run tests or build the docs or run the ABI checks. > In the same context, for jobs that accumulates running tests, building > the docs etc..., those steps are independent and can be split to save > some cpu on Travis. > > With this, we go down from 21 to 15 jobs. > > Note: this patch requires a flush of the existing caches in Travis. > > Signed-off-by: David Marchand > --- In general, I think the idea with required vs. extra was to have a build that did the minimum required, and one that did all the packages (to allow a minimum vs. full DPDK). At least, that's from http://mails.dpdk.org/archives/dev/2019-January/124007.html Not sure if that's still something anyone cares about. > .travis.yml | 51 +++++++++------------------------------------------ > 1 file changed, 9 insertions(+), 42 deletions(-) > > diff --git a/.travis.yml b/.travis.yml > index c174d5f30..b64a81bd0 100644 > --- a/.travis.yml > +++ b/.travis.yml > @@ -15,15 +15,12 @@ addons: > update: true > packages: &required_packages > - [libnuma-dev, python3-setuptools, python3-wheel, python3-pip, ni= nja-build] > + - [libbsd-dev, libpcap-dev, libibverbs-dev, libcrypto++-dev, libfd= t-dev, libjansson4] > > _aarch64_packages: &aarch64_packages > - *required_packages > - [gcc-aarch64-linux-gnu, libc6-dev-arm64-cross, pkg-config-aarch64-li= nux-gnu] > =20 > -_extra_packages: &extra_packages > - - *required_packages > - - [libbsd-dev, libpcap-dev, libibverbs-dev, libcrypto++-dev, libfdt-de= v, libjansson4] > - > _libabigail_build_packages: &libabigail_build_packages > - [autoconf, automake, libtool, pkg-config, libxml2-dev, libdw-dev] > =20 > @@ -43,66 +40,39 @@ jobs: > - env: DEF_LIB=3D"static" > arch: amd64 > compiler: gcc > - - env: DEF_LIB=3D"shared" > - arch: amd64 > - compiler: gcc > - env: DEF_LIB=3D"shared" RUN_TESTS=3D1 > arch: amd64 > compiler: gcc > - - env: DEF_LIB=3D"static" EXTRA_PACKAGES=3D1 > - arch: amd64 > - compiler: gcc > - addons: > - apt: > - packages: > - - *extra_packages > - - env: DEF_LIB=3D"shared" EXTRA_PACKAGES=3D1 RUN_TESTS=3D1 BUILD_DOCS= =3D1 > + - env: DEF_LIB=3D"shared" BUILD_DOCS=3D1 > arch: amd64 > compiler: gcc > addons: > apt: > packages: > - - *extra_packages > + - *required_packages > - *doc_packages > - - env: DEF_LIB=3D"shared" EXTRA_PACKAGES=3D1 ABI_CHECKS=3D1 > + - env: DEF_LIB=3D"shared" ABI_CHECKS=3D1 > arch: amd64 > compiler: gcc > addons: > apt: > packages: > - - *extra_packages > + - *required_packages > - *libabigail_build_packages > # x86_64 clang jobs > - env: DEF_LIB=3D"static" > arch: amd64 > compiler: clang > - - env: DEF_LIB=3D"shared" > - arch: amd64 > - compiler: clang > - env: DEF_LIB=3D"shared" RUN_TESTS=3D1 > arch: amd64 > compiler: clang > - - env: DEF_LIB=3D"static" EXTRA_PACKAGES=3D1 > - arch: amd64 > - compiler: clang > - addons: > - apt: > - packages: > - - *extra_packages > - - env: DEF_LIB=3D"shared" EXTRA_PACKAGES=3D1 > - arch: amd64 > - compiler: clang > - addons: > - apt: > - packages: > - - *extra_packages > - - env: DEF_LIB=3D"shared" EXTRA_PACKAGES=3D1 RUN_TESTS=3D1 BUILD_DOCS= =3D1 > + - env: DEF_LIB=3D"shared" BUILD_DOCS=3D1 > arch: amd64 > compiler: clang > addons: > apt: > packages: > - - *extra_packages > + - *required_packages > - *doc_packages > # x86_64 cross-compiling 32-bits jobs > - env: DEF_LIB=3D"static" BUILD_32BIT=3D1 > @@ -131,9 +101,6 @@ jobs: > - env: DEF_LIB=3D"static" > arch: arm64 > compiler: gcc > - - env: DEF_LIB=3D"shared" > - arch: arm64 > - compiler: gcc > - env: DEF_LIB=3D"shared" BUILD_DOCS=3D1 > arch: arm64 > compiler: gcc > @@ -142,13 +109,13 @@ jobs: > packages: > - *required_packages > - *doc_packages > - - env: DEF_LIB=3D"shared" EXTRA_PACKAGES=3D1 ABI_CHECKS=3D1 > + - env: DEF_LIB=3D"shared" ABI_CHECKS=3D1 > arch: arm64 > compiler: gcc > addons: > apt: > packages: > - - *extra_packages > + - *required_packages > - *libabigail_build_packages > # aarch64 clang jobs > - env: DEF_LIB=3D"static"