From patchwork Wed Feb 19 19:41:31 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: David Marchand X-Patchwork-Id: 65928 X-Patchwork-Delegate: thomas@monjalon.net Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 08A03A0555; Wed, 19 Feb 2020 20:42:22 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 149761BFAD; Wed, 19 Feb 2020 20:42:00 +0100 (CET) Received: from us-smtp-delivery-1.mimecast.com (us-smtp-2.mimecast.com [205.139.110.61]) by dpdk.org (Postfix) with ESMTP id 2ACB11BF9F for ; Wed, 19 Feb 2020 20:41:57 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1582141316; 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=FiWRqDDGsssEX6XU3vJcZCr5MgeEamsfC2auAxPcPjU=; b=QUJf6PsoP/gG5nQJZKsdpgr7ljopnmDgFK8xyl3efk/v6pvlqmmOSMgii4lPbWQGM28/Aj eYQQZU/viYag0PHpns5wjkK4I45E0O+VdAJOI7U1grHM5pUjOb7ECPcH/47ru0nlF2s3MN b3UPcovfv/SZaURJZNmCcM7KsC/x2vc= 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-38-Qd7LhIu-M06Mpbsxmm_pxA-1; Wed, 19 Feb 2020 14:41:52 -0500 X-MC-Unique: Qd7LhIu-M06Mpbsxmm_pxA-1 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id A68AE477; Wed, 19 Feb 2020 19:41:51 +0000 (UTC) Received: from dmarchan.remote.csb (ovpn-204-34.brq.redhat.com [10.40.204.34]) by smtp.corp.redhat.com (Postfix) with ESMTP id 629841001B09; Wed, 19 Feb 2020 19:41:50 +0000 (UTC) From: David Marchand To: thomas@monjalon.net, aconole@redhat.com Cc: dev@dpdk.org, Michael Santana Date: Wed, 19 Feb 2020 20:41:31 +0100 Message-Id: <20200219194131.29417-5-david.marchand@redhat.com> In-Reply-To: <20200219194131.29417-1-david.marchand@redhat.com> References: <20200219194131.29417-1-david.marchand@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Subject: [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" 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 Acked-by: Thomas Monjalon Acked-by: Aaron Conole --- .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, ninja-build] + - [libbsd-dev, libpcap-dev, libibverbs-dev, libcrypto++-dev, libfdt-dev, libjansson4] _aarch64_packages: &aarch64_packages - *required_packages - [gcc-aarch64-linux-gnu, libc6-dev-arm64-cross, pkg-config-aarch64-linux-gnu] -_extra_packages: &extra_packages - - *required_packages - - [libbsd-dev, libpcap-dev, libibverbs-dev, libcrypto++-dev, libfdt-dev, libjansson4] - _libabigail_build_packages: &libabigail_build_packages - [autoconf, automake, libtool, pkg-config, libxml2-dev, libdw-dev] @@ -43,66 +40,39 @@ jobs: - env: DEF_LIB="static" arch: amd64 compiler: gcc - - env: DEF_LIB="shared" - arch: amd64 - compiler: gcc - env: DEF_LIB="shared" RUN_TESTS=1 arch: amd64 compiler: gcc - - env: DEF_LIB="static" EXTRA_PACKAGES=1 - arch: amd64 - compiler: gcc - addons: - apt: - packages: - - *extra_packages - - env: DEF_LIB="shared" EXTRA_PACKAGES=1 RUN_TESTS=1 BUILD_DOCS=1 + - env: DEF_LIB="shared" BUILD_DOCS=1 arch: amd64 compiler: gcc addons: apt: packages: - - *extra_packages + - *required_packages - *doc_packages - - env: DEF_LIB="shared" EXTRA_PACKAGES=1 ABI_CHECKS=1 + - env: DEF_LIB="shared" ABI_CHECKS=1 arch: amd64 compiler: gcc addons: apt: packages: - - *extra_packages + - *required_packages - *libabigail_build_packages # x86_64 clang jobs - env: DEF_LIB="static" arch: amd64 compiler: clang - - env: DEF_LIB="shared" - arch: amd64 - compiler: clang - env: DEF_LIB="shared" RUN_TESTS=1 arch: amd64 compiler: clang - - env: DEF_LIB="static" EXTRA_PACKAGES=1 - arch: amd64 - compiler: clang - addons: - apt: - packages: - - *extra_packages - - env: DEF_LIB="shared" EXTRA_PACKAGES=1 - arch: amd64 - compiler: clang - addons: - apt: - packages: - - *extra_packages - - env: DEF_LIB="shared" EXTRA_PACKAGES=1 RUN_TESTS=1 BUILD_DOCS=1 + - env: DEF_LIB="shared" BUILD_DOCS=1 arch: amd64 compiler: clang addons: apt: packages: - - *extra_packages + - *required_packages - *doc_packages # x86_64 cross-compiling 32-bits jobs - env: DEF_LIB="static" BUILD_32BIT=1 @@ -131,9 +101,6 @@ jobs: - env: DEF_LIB="static" arch: arm64 compiler: gcc - - env: DEF_LIB="shared" - arch: arm64 - compiler: gcc - env: DEF_LIB="shared" BUILD_DOCS=1 arch: arm64 compiler: gcc @@ -142,13 +109,13 @@ jobs: packages: - *required_packages - *doc_packages - - env: DEF_LIB="shared" EXTRA_PACKAGES=1 ABI_CHECKS=1 + - env: DEF_LIB="shared" ABI_CHECKS=1 arch: arm64 compiler: gcc addons: apt: packages: - - *extra_packages + - *required_packages - *libabigail_build_packages # aarch64 clang jobs - env: DEF_LIB="static"