[v2] ci: update arm64 Travis jobs to Graviton2
Checks
Commit Message
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
> -----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(-)
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>
> ---
> -----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>
> > ---
>
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>
>> > ---
>>
@@ -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