diff mbox series

[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
Delegated to: Thomas Monjalon
Headers show
Series [v2] ci: update arm64 Travis jobs to Graviton2 | expand

Checks

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

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>
>> > ---
>>
diff mbox series

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