[RFC,v1,1/2] ci: remove x86 Travis jobs

Message ID 1613554953-14736-2-git-send-email-juraj.linkes@pantheon.tech (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers
Series Travis CI jobs update after credits |

Checks

Context Check Description
ci/checkpatch success coding style OK

Commit Message

Juraj Linkeš Feb. 17, 2021, 9:42 a.m. UTC
  Remove x86 jobs for which we don't have Travis credits. Leave arm64 jobs
for which we do have the credits.

Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
---
 .travis.yml | 87 -----------------------------------------------------
 1 file changed, 87 deletions(-)
  

Comments

Thomas Monjalon Feb. 17, 2021, 10:39 a.m. UTC | #1
17/02/2021 10:42, Juraj Linkeš:
> Remove x86 jobs for which we don't have Travis credits. Leave arm64 jobs
> for which we do have the credits.

We are missing some references.
  
Aaron Conole Feb. 17, 2021, 3:44 p.m. UTC | #2
Thomas Monjalon <thomas@monjalon.net> writes:

> 17/02/2021 10:42, Juraj Linkeš:
>> Remove x86 jobs for which we don't have Travis credits. Leave arm64 jobs
>> for which we do have the credits.
>
> We are missing some references.

I do see that the travis job is still running, and does include the x86
builds as well... Is there a specific problem to be resolved?
  
Aaron Conole Feb. 17, 2021, 3:47 p.m. UTC | #3
Juraj Linkeš <juraj.linkes@pantheon.tech> writes:

> Remove x86 jobs for which we don't have Travis credits. Leave arm64 jobs
> for which we do have the credits.
>
> Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
> ---

Individual developers will still derive benefit from those jobs, yes?  I
thought it was possible to use travis still for individual developer
accounts.  If that's true, then we shouldn't remove these jobs.
  
Honnappa Nagarahalli Feb. 17, 2021, 5:18 p.m. UTC | #4
<snip>

> 
> Thomas Monjalon <thomas@monjalon.net> writes:
> 
> > 17/02/2021 10:42, Juraj Linkeš:
> >> Remove x86 jobs for which we don't have Travis credits. Leave arm64
> >> jobs for which we do have the credits.
> >
> > We are missing some references.
> 
> I do see that the travis job is still running, and does include the x86 builds as
> well... Is there a specific problem to be resolved?
Agree, I do not see a particular problem to be resolved.
  
Honnappa Nagarahalli Feb. 17, 2021, 5:19 p.m. UTC | #5
<snip>

> 
> Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
> 
> > Remove x86 jobs for which we don't have Travis credits. Leave arm64
> > jobs for which we do have the credits.
> >
> > Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
> > ---
> 
> Individual developers will still derive benefit from those jobs, yes?  I thought
> it was possible to use travis still for individual developer accounts.  If that's
> true, then we shouldn't remove these jobs.
+1
  

Patch

diff --git a/.travis.yml b/.travis.yml
index 6a2181299f..899e0cd605 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -17,21 +17,9 @@  addons:
       - [libnuma-dev, python3-setuptools, python3-wheel, python3-pip, python3-pyelftools, ninja-build]
       - [libbsd-dev, libpcap-dev, libibverbs-dev, libcrypto++-dev, libfdt-dev, libjansson-dev]
 
-_aarch64_packages: &aarch64_packages
-  - *required_packages
-  - [gcc-aarch64-linux-gnu, libc6-dev-arm64-cross, pkg-config-aarch64-linux-gnu]
-
-_aarch64_clang_packages: &aarch64_clang_packages
-  - *required_packages
-  - [libgcc-7-dev-arm64-cross, libatomic1-arm64-cross, libc6-dev-arm64-cross, pkg-config-aarch64-linux-gnu]
-
 _libabigail_build_packages: &libabigail_build_packages
   - [autoconf, automake, libtool, pkg-config, libxml2-dev, libdw-dev]
 
-_build_32b_packages: &build_32b_packages
-  - *required_packages
-  - [gcc-multilib]
-
 _doc_packages: &doc_packages
   - [doxygen, graphviz, python3-sphinx]
 
@@ -45,81 +33,6 @@  env:
 
 jobs:
   include:
-  # x86_64 gcc jobs
-  - env: DEF_LIB="static"
-    arch: amd64
-    compiler: gcc
-  - env: DEF_LIB="shared" RUN_TESTS=true
-    arch: amd64
-    compiler: gcc
-  - env: DEF_LIB="shared" BUILD_DOCS=true
-    arch: amd64
-    compiler: gcc
-    addons:
-      apt:
-        packages:
-          - *required_packages
-          - *doc_packages
-  - env: DEF_LIB="shared" ABI_CHECKS=true
-    arch: amd64
-    compiler: gcc
-    addons:
-      apt:
-        packages:
-          - *required_packages
-          - *libabigail_build_packages
-  # x86_64 clang jobs
-  - env: DEF_LIB="static"
-    arch: amd64
-    compiler: clang
-  - env: DEF_LIB="shared" RUN_TESTS=true
-    arch: amd64
-    compiler: clang
-  - env: DEF_LIB="shared" BUILD_DOCS=true
-    arch: amd64
-    compiler: clang
-    addons:
-      apt:
-        packages:
-          - *required_packages
-          - *doc_packages
-  # x86_64 cross-compiling 32-bits jobs
-  - env: DEF_LIB="static" BUILD_32BIT=true
-    arch: amd64
-    compiler: gcc
-    addons:
-      apt:
-        packages:
-          - *build_32b_packages
-  # x86_64 cross-compiling aarch64 jobs
-  - env: DEF_LIB="static" AARCH64=true
-    arch: amd64
-    compiler: gcc
-    addons:
-      apt:
-        packages:
-          - *aarch64_packages
-  - env: DEF_LIB="shared" AARCH64=true
-    arch: amd64
-    compiler: gcc
-    addons:
-      apt:
-        packages:
-          - *aarch64_packages
-  - env: DEF_LIB="static" AARCH64=true
-    arch: amd64
-    compiler: clang
-    addons:
-      apt:
-        packages:
-          - *aarch64_clang_packages
-  - env: DEF_LIB="shared" AARCH64=true
-    arch: amd64
-    compiler: clang
-    addons:
-      apt:
-        packages:
-          - *aarch64_clang_packages
   # aarch64 gcc jobs
   - env: DEF_LIB="static"
     arch: arm64