[v2] ci: update arm64 Travis jobs to Graviton2

Message ID 1613720062-19647-1-git-send-email-juraj.linkes@pantheon.tech (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers
Series [v2] ci: update arm64 Travis jobs to Graviton2 |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/travis-robot fail travis build: failed
ci/github-robot success github build: passed
ci/iol-testing success Testing PASS

Commit Message

Juraj Linkeš Feb. 19, 2021, 7:34 a.m. UTC
  Instead of using the older platform, use only the newer Graviton2
environment in Travis CI.

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

Comments

Ruifeng Wang Feb. 19, 2021, 9:13 a.m. UTC | #1
> -----Original Message-----
> From: Juraj Linkeš <juraj.linkes@pantheon.tech>
> Sent: Friday, February 19, 2021 3:34 PM
> To: thomas@monjalon.net; david.marchand@redhat.com;
> aconole@redhat.com; maicolgabriel@hotmail.com
> Cc: dev@dpdk.org; Ruifeng Wang <Ruifeng.Wang@arm.com>; Honnappa
> Nagarahalli <Honnappa.Nagarahalli@arm.com>; Juraj Linkeš
> <juraj.linkes@pantheon.tech>
> Subject: [PATCH v2] ci: update arm64 Travis jobs to Graviton2
> 
> Instead of using the older platform, use only the newer Graviton2
> environment in Travis CI.
> 
> Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>

Reviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>
> ---
>  .travis.yml | 26 ++++++++++++++++----------
>  1 file changed, 16 insertions(+), 10 deletions(-)
  
Aaron Conole Feb. 19, 2021, 1:27 p.m. UTC | #2
Juraj Linkeš <juraj.linkes@pantheon.tech> writes:

> Instead of using the older platform, use only the newer Graviton2
> environment in Travis CI.

Can you expand on this a bit.  Why should we need to use the newer
environment?  What problems will occur by using the older environment?

I'm not against the change, but I don't understand it.

> Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
> ---
  
Juraj Linkeš Feb. 22, 2021, 2:49 p.m. UTC | #3
> -----Original Message-----
> From: Aaron Conole <aconole@redhat.com>
> Sent: Friday, February 19, 2021 2:27 PM
> To: Juraj Linkeš <juraj.linkes@pantheon.tech>
> Cc: thomas@monjalon.net; david.marchand@redhat.com;
> maicolgabriel@hotmail.com; dev@dpdk.org; Ruifeng.Wang@arm.com;
> Honnappa.Nagarahalli@arm.com
> Subject: Re: [PATCH v2] ci: update arm64 Travis jobs to Graviton2
> 
> Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
> 
> > Instead of using the older platform, use only the newer Graviton2
> > environment in Travis CI.
> 
> Can you expand on this a bit.  Why should we need to use the newer
> environment?  What problems will occur by using the older environment?
> 

We're seeing intermittent issues with the environment.

An example run https://travis-ci.com/github/ovsrobot/dpdk/builds/217641063

Issue 1:
collect2: fatal error: ld terminated with signal 9 [Killed]

This suggest an issue with resources.

Issue 2:
ticketlock_autotest sometimes times out. This could be due variable resource availability in a container environment, but may not be.

There could be other issues, I haven't analysed all of the logs, but there are enough to make us want to switch to Graviton2 for all jobs.

We're don't see either issue with the Graviton2 environment (which uses VMs).

> I'm not against the change, but I don't understand it.
> 
> > Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
> > ---
>
  
Aaron Conole Feb. 22, 2021, 7:37 p.m. UTC | #4
Juraj Linkeš <juraj.linkes@pantheon.tech> writes:

>> -----Original Message-----
>> From: Aaron Conole <aconole@redhat.com>
>> Sent: Friday, February 19, 2021 2:27 PM
>> To: Juraj Linkeš <juraj.linkes@pantheon.tech>
>> Cc: thomas@monjalon.net; david.marchand@redhat.com;
>> maicolgabriel@hotmail.com; dev@dpdk.org; Ruifeng.Wang@arm.com;
>> Honnappa.Nagarahalli@arm.com
>> Subject: Re: [PATCH v2] ci: update arm64 Travis jobs to Graviton2
>> 
>> Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
>> 
>> > Instead of using the older platform, use only the newer Graviton2
>> > environment in Travis CI.
>> 
>> Can you expand on this a bit.  Why should we need to use the newer
>> environment?  What problems will occur by using the older environment?
>> 
>
> We're seeing intermittent issues with the environment.
>
> An example run https://travis-ci.com/github/ovsrobot/dpdk/builds/217641063
>
> Issue 1:
> collect2: fatal error: ld terminated with signal 9 [Killed]
>
> This suggest an issue with resources.
>
> Issue 2:
> ticketlock_autotest sometimes times out. This could be due variable
> resource availability in a container environment, but may not be.
>
> There could be other issues, I haven't analysed all of the logs, but
> there are enough to make us want to switch to Graviton2 for all jobs.
>
> We're don't see either issue with the Graviton2 environment (which uses VMs).

I guess it would be good to mention this motivation in the commit
message - otherwise looking at the history we won't understand why we
made this change.

Please post v2 mentioning resource availability issues with travis-ci
using the container infrastructure which don't seem to impact the VM
infrastructure.  I will ACK.

>> I'm not against the change, but I don't understand it.
>> 
>> > Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
>> > ---
>>
  

Patch

diff --git a/.travis.yml b/.travis.yml
index 6a2181299f..7ed982ac07 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -122,10 +122,10 @@  jobs:
           - *aarch64_clang_packages
   # aarch64 gcc jobs
   - env: DEF_LIB="static"
-    arch: arm64
-    compiler: gcc
-  - env: DEF_LIB="shared" RUN_TESTS=true
-    arch: arm64
+    dist: focal
+    arch: arm64-graviton2
+    virt: vm
+    group: edge
     compiler: gcc
   - env: DEF_LIB="shared" RUN_TESTS=true
     dist: focal
@@ -134,7 +134,10 @@  jobs:
     group: edge
     compiler: gcc
   - env: DEF_LIB="shared" BUILD_DOCS=true
-    arch: arm64
+    dist: focal
+    arch: arm64-graviton2
+    virt: vm
+    group: edge
     compiler: gcc
     addons:
       apt:
@@ -142,7 +145,10 @@  jobs:
           - *required_packages
           - *doc_packages
   - env: DEF_LIB="shared" ABI_CHECKS=true
-    arch: arm64
+    dist: focal
+    arch: arm64-graviton2
+    virt: vm
+    group: edge
     compiler: gcc
     addons:
       apt:
@@ -151,10 +157,10 @@  jobs:
           - *libabigail_build_packages
   # aarch64 clang jobs
   - env: DEF_LIB="static"
-    arch: arm64
-    compiler: clang
-  - env: DEF_LIB="shared" RUN_TESTS=true
-    arch: arm64
+    dist: focal
+    arch: arm64-graviton2
+    virt: vm
+    group: edge
     compiler: clang
   - env: DEF_LIB="shared" RUN_TESTS=true
     dist: focal