* [dpdk-dev] [RFC PATCH v1 1/2] ci: remove x86 Travis jobs
2021-02-17 9:42 [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits Juraj Linkeš
@ 2021-02-17 9:42 ` Juraj Linkeš
2021-02-17 10:39 ` Thomas Monjalon
2021-02-17 15:47 ` Aaron Conole
2021-02-17 9:42 ` [dpdk-dev] [RFC PATCH v1 2/2] ci: update arm64 Travis jobs to Graviton2 Juraj Linkeš
` (3 subsequent siblings)
4 siblings, 2 replies; 22+ messages in thread
From: Juraj Linkeš @ 2021-02-17 9:42 UTC (permalink / raw)
To: thomas, david.marchand, aconole, maicolgabriel
Cc: dev, Ruifeng.Wang, Honnappa.Nagarahalli, Juraj Linkeš
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(-)
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
--
2.20.1
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [dpdk-dev] [RFC PATCH v1 1/2] ci: remove x86 Travis jobs
2021-02-17 9:42 ` [dpdk-dev] [RFC PATCH v1 1/2] ci: remove x86 Travis jobs Juraj Linkeš
@ 2021-02-17 10:39 ` Thomas Monjalon
2021-02-17 15:44 ` Aaron Conole
2021-02-17 15:47 ` Aaron Conole
1 sibling, 1 reply; 22+ messages in thread
From: Thomas Monjalon @ 2021-02-17 10:39 UTC (permalink / raw)
To: Juraj Linkeš
Cc: david.marchand, aconole, maicolgabriel, dev, Ruifeng.Wang,
Honnappa.Nagarahalli, Juraj Linkeš
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.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [dpdk-dev] [RFC PATCH v1 1/2] ci: remove x86 Travis jobs
2021-02-17 10:39 ` Thomas Monjalon
@ 2021-02-17 15:44 ` Aaron Conole
2021-02-17 17:18 ` Honnappa Nagarahalli
0 siblings, 1 reply; 22+ messages in thread
From: Aaron Conole @ 2021-02-17 15:44 UTC (permalink / raw)
To: Thomas Monjalon
Cc: Juraj Linkeš,
david.marchand, maicolgabriel, dev, Ruifeng.Wang,
Honnappa.Nagarahalli
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?
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [dpdk-dev] [RFC PATCH v1 1/2] ci: remove x86 Travis jobs
2021-02-17 15:44 ` Aaron Conole
@ 2021-02-17 17:18 ` Honnappa Nagarahalli
0 siblings, 0 replies; 22+ messages in thread
From: Honnappa Nagarahalli @ 2021-02-17 17:18 UTC (permalink / raw)
To: Aaron Conole, thomas
Cc: Juraj Linkeš,
david.marchand, maicolgabriel, dev, Ruifeng Wang, nd,
Honnappa Nagarahalli, nd
<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.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [dpdk-dev] [RFC PATCH v1 1/2] ci: remove x86 Travis jobs
2021-02-17 9:42 ` [dpdk-dev] [RFC PATCH v1 1/2] ci: remove x86 Travis jobs Juraj Linkeš
2021-02-17 10:39 ` Thomas Monjalon
@ 2021-02-17 15:47 ` Aaron Conole
2021-02-17 17:19 ` Honnappa Nagarahalli
1 sibling, 1 reply; 22+ messages in thread
From: Aaron Conole @ 2021-02-17 15:47 UTC (permalink / raw)
To: Juraj Linkeš
Cc: thomas, david.marchand, maicolgabriel, dev, Ruifeng.Wang,
Honnappa.Nagarahalli
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.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [dpdk-dev] [RFC PATCH v1 1/2] ci: remove x86 Travis jobs
2021-02-17 15:47 ` Aaron Conole
@ 2021-02-17 17:19 ` Honnappa Nagarahalli
0 siblings, 0 replies; 22+ messages in thread
From: Honnappa Nagarahalli @ 2021-02-17 17:19 UTC (permalink / raw)
To: Aaron Conole, Juraj Linkeš
Cc: thomas, david.marchand, maicolgabriel, dev, Ruifeng Wang, nd,
Honnappa Nagarahalli, nd
<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
^ permalink raw reply [flat|nested] 22+ messages in thread
* [dpdk-dev] [RFC PATCH v1 2/2] ci: update arm64 Travis jobs to Graviton2
2021-02-17 9:42 [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits Juraj Linkeš
2021-02-17 9:42 ` [dpdk-dev] [RFC PATCH v1 1/2] ci: remove x86 Travis jobs Juraj Linkeš
@ 2021-02-17 9:42 ` Juraj Linkeš
2021-02-17 10:43 ` [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits Ferruh Yigit
` (2 subsequent siblings)
4 siblings, 0 replies; 22+ messages in thread
From: Juraj Linkeš @ 2021-02-17 9:42 UTC (permalink / raw)
To: thomas, david.marchand, aconole, maicolgabriel
Cc: dev, Ruifeng.Wang, Honnappa.Nagarahalli, Juraj Linkeš
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(-)
diff --git a/.travis.yml b/.travis.yml
index 899e0cd605..fb3913ed20 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -35,10 +35,10 @@ jobs:
include:
# 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
@@ -47,7 +47,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:
@@ -55,7 +58,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:
@@ -64,10 +70,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
--
2.20.1
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits
2021-02-17 9:42 [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits Juraj Linkeš
2021-02-17 9:42 ` [dpdk-dev] [RFC PATCH v1 1/2] ci: remove x86 Travis jobs Juraj Linkeš
2021-02-17 9:42 ` [dpdk-dev] [RFC PATCH v1 2/2] ci: update arm64 Travis jobs to Graviton2 Juraj Linkeš
@ 2021-02-17 10:43 ` Ferruh Yigit
2021-02-17 15:46 ` Aaron Conole
2021-02-18 2:40 ` Honnappa Nagarahalli
2021-02-19 7:34 ` [dpdk-dev] [PATCH v2] ci: update arm64 Travis jobs to Graviton2 Juraj Linkeš
4 siblings, 1 reply; 22+ messages in thread
From: Ferruh Yigit @ 2021-02-17 10:43 UTC (permalink / raw)
To: Juraj Linkeš, thomas, david.marchand, aconole, maicolgabriel
Cc: dev, Ruifeng.Wang, Honnappa.Nagarahalli
On 2/17/2021 9:42 AM, Juraj Linkeš wrote:
> Since Travis added credits we need to make some changes to the Travis
> CI. Arm has a credit agreement with Travis for arm jobs, so remove x86
> jobs. Also, migrate all arm jobs to the latest environment.
>
So this is becoming supporting a CI infrastructure within DPDK for a specific
vendor, do we want this?
And can't this Travis credit/support be expanded for the whole project, so
everyone in the community benefit from it?
> Juraj Linkeš (2):
> ci: remove x86 Travis jobs
> ci: update arm64 Travis jobs to Graviton2
>
> .travis.yml | 113 ++++++++--------------------------------------------
> 1 file changed, 16 insertions(+), 97 deletions(-)
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits
2021-02-17 10:43 ` [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits Ferruh Yigit
@ 2021-02-17 15:46 ` Aaron Conole
2021-02-17 16:08 ` Honnappa Nagarahalli
0 siblings, 1 reply; 22+ messages in thread
From: Aaron Conole @ 2021-02-17 15:46 UTC (permalink / raw)
To: Ferruh Yigit
Cc: Juraj Linkeš,
thomas, david.marchand, maicolgabriel, dev, Ruifeng.Wang,
Honnappa.Nagarahalli
Ferruh Yigit <ferruh.yigit@intel.com> writes:
> On 2/17/2021 9:42 AM, Juraj Linkeš wrote:
>> Since Travis added credits we need to make some changes to the Travis
>> CI. Arm has a credit agreement with Travis for arm jobs, so remove x86
>> jobs. Also, migrate all arm jobs to the latest environment.
>>
>
> So this is becoming supporting a CI infrastructure within DPDK for a
> specific vendor, do we want this?
>
> And can't this Travis credit/support be expanded for the whole
> project, so everyone in the community benefit from it?
I agree - it would be best to have it working universally.
>> Juraj Linkeš (2):
>> ci: remove x86 Travis jobs
>> ci: update arm64 Travis jobs to Graviton2
>>
>> .travis.yml | 113 ++++++++--------------------------------------------
>> 1 file changed, 16 insertions(+), 97 deletions(-)
>>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits
2021-02-17 15:46 ` Aaron Conole
@ 2021-02-17 16:08 ` Honnappa Nagarahalli
0 siblings, 0 replies; 22+ messages in thread
From: Honnappa Nagarahalli @ 2021-02-17 16:08 UTC (permalink / raw)
To: Aaron Conole, Ferruh Yigit
Cc: Juraj Linkeš,
thomas, david.marchand, maicolgabriel, dev, Ruifeng Wang, nd,
Honnappa Nagarahalli, nd
<snip>
> Subject: Re: [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after
> credits
>
> Ferruh Yigit <ferruh.yigit@intel.com> writes:
>
> > On 2/17/2021 9:42 AM, Juraj Linkeš wrote:
> >> Since Travis added credits we need to make some changes to the Travis
> >> CI. Arm has a credit agreement with Travis for arm jobs, so remove
> >> x86 jobs. Also, migrate all arm jobs to the latest environment.
> >>
> >
> > So this is becoming supporting a CI infrastructure within DPDK for a
> > specific vendor, do we want this?
> >
> > And can't this Travis credit/support be expanded for the whole
> > project, so everyone in the community benefit from it?
>
> I agree - it would be best to have it working universally.
My understanding is that the credits are not associated with any architecture.
>
> >> Juraj Linkeš (2):
> >> ci: remove x86 Travis jobs
> >> ci: update arm64 Travis jobs to Graviton2
> >>
> >> .travis.yml | 113 ++++++++--------------------------------------------
> >> 1 file changed, 16 insertions(+), 97 deletions(-)
> >>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits
2021-02-17 9:42 [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits Juraj Linkeš
` (2 preceding siblings ...)
2021-02-17 10:43 ` [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits Ferruh Yigit
@ 2021-02-18 2:40 ` Honnappa Nagarahalli
2021-02-18 7:50 ` Juraj Linkeš
2021-02-19 7:34 ` [dpdk-dev] [PATCH v2] ci: update arm64 Travis jobs to Graviton2 Juraj Linkeš
4 siblings, 1 reply; 22+ messages in thread
From: Honnappa Nagarahalli @ 2021-02-18 2:40 UTC (permalink / raw)
To: Juraj Linkeš, thomas, david.marchand, aconole, maicolgabriel
Cc: dev, Ruifeng Wang, nd, Honnappa Nagarahalli, nd
<snip>
> Subject: [RFC PATCH v1 0/2] Travis CI jobs update after credits
>
> Since Travis added credits we need to make some changes to the Travis CI. Arm
> has a credit agreement with Travis for arm jobs, so remove x86 jobs. Also,
I think there is a mis-understanding here. The credits are for the ovsrobot account and apply to all architectures.
> migrate all arm jobs to the latest environment.
>
> Juraj Linkeš (2):
> ci: remove x86 Travis jobs
> ci: update arm64 Travis jobs to Graviton2
>
> .travis.yml | 113 ++++++++--------------------------------------------
> 1 file changed, 16 insertions(+), 97 deletions(-)
>
> --
> 2.20.1
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits
2021-02-18 2:40 ` Honnappa Nagarahalli
@ 2021-02-18 7:50 ` Juraj Linkeš
0 siblings, 0 replies; 22+ messages in thread
From: Juraj Linkeš @ 2021-02-18 7:50 UTC (permalink / raw)
To: Honnappa Nagarahalli, thomas, david.marchand, aconole, maicolgabriel
Cc: dev, Ruifeng Wang, nd, nd
> -----Original Message-----
> From: Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>
> Sent: Thursday, February 18, 2021 3:40 AM
> To: Juraj Linkeš <juraj.linkes@pantheon.tech>; thomas@monjalon.net;
> david.marchand@redhat.com; aconole@redhat.com;
> maicolgabriel@hotmail.com
> Cc: dev@dpdk.org; Ruifeng Wang <Ruifeng.Wang@arm.com>; nd
> <nd@arm.com>; Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>; nd
> <nd@arm.com>
> Subject: RE: [RFC PATCH v1 0/2] Travis CI jobs update after credits
>
> <snip>
>
> > Subject: [RFC PATCH v1 0/2] Travis CI jobs update after credits
> >
> > Since Travis added credits we need to make some changes to the Travis
> > CI. Arm has a credit agreement with Travis for arm jobs, so remove x86
> > jobs. Also,
> I think there is a mis-understanding here. The credits are for the ovsrobot
> account and apply to all architectures.
>
This is likely my mistake. I thought there would only be enough credits to run just arm jobs. It seems there are enough credits to run everything, so I'll drop this patch.
> > migrate all arm jobs to the latest environment.
> >
> > Juraj Linkeš (2):
> > ci: remove x86 Travis jobs
> > ci: update arm64 Travis jobs to Graviton2
> >
> > .travis.yml | 113
> > ++++++++--------------------------------------------
> > 1 file changed, 16 insertions(+), 97 deletions(-)
> >
> > --
> > 2.20.1
^ permalink raw reply [flat|nested] 22+ messages in thread
* [dpdk-dev] [PATCH v2] ci: update arm64 Travis jobs to Graviton2
2021-02-17 9:42 [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits Juraj Linkeš
` (3 preceding siblings ...)
2021-02-18 2:40 ` Honnappa Nagarahalli
@ 2021-02-19 7:34 ` Juraj Linkeš
2021-02-19 9:13 ` Ruifeng Wang
` (2 more replies)
4 siblings, 3 replies; 22+ messages in thread
From: Juraj Linkeš @ 2021-02-19 7:34 UTC (permalink / raw)
To: thomas, david.marchand, aconole, maicolgabriel
Cc: dev, Ruifeng.Wang, Honnappa.Nagarahalli, Juraj Linkeš
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(-)
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
--
2.20.1
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [dpdk-dev] [PATCH v2] ci: update arm64 Travis jobs to Graviton2
2021-02-19 7:34 ` [dpdk-dev] [PATCH v2] ci: update arm64 Travis jobs to Graviton2 Juraj Linkeš
@ 2021-02-19 9:13 ` Ruifeng Wang
2021-02-19 13:27 ` Aaron Conole
2021-02-23 8:28 ` [dpdk-dev] [PATCH v3] " Juraj Linkeš
2 siblings, 0 replies; 22+ messages in thread
From: Ruifeng Wang @ 2021-02-19 9:13 UTC (permalink / raw)
To: Juraj Linkeš, thomas, david.marchand, aconole, maicolgabriel
Cc: dev, Honnappa Nagarahalli, nd
> -----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(-)
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [dpdk-dev] [PATCH v2] ci: update arm64 Travis jobs to Graviton2
2021-02-19 7:34 ` [dpdk-dev] [PATCH v2] ci: update arm64 Travis jobs to Graviton2 Juraj Linkeš
2021-02-19 9:13 ` Ruifeng Wang
@ 2021-02-19 13:27 ` Aaron Conole
2021-02-22 14:49 ` Juraj Linkeš
2021-02-23 8:28 ` [dpdk-dev] [PATCH v3] " Juraj Linkeš
2 siblings, 1 reply; 22+ messages in thread
From: Aaron Conole @ 2021-02-19 13:27 UTC (permalink / raw)
To: Juraj Linkeš
Cc: thomas, david.marchand, maicolgabriel, dev, Ruifeng.Wang,
Honnappa.Nagarahalli
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>
> ---
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [dpdk-dev] [PATCH v2] ci: update arm64 Travis jobs to Graviton2
2021-02-19 13:27 ` Aaron Conole
@ 2021-02-22 14:49 ` Juraj Linkeš
2021-02-22 19:37 ` Aaron Conole
0 siblings, 1 reply; 22+ messages in thread
From: Juraj Linkeš @ 2021-02-22 14:49 UTC (permalink / raw)
To: Aaron Conole
Cc: thomas, david.marchand, maicolgabriel, dev, Ruifeng.Wang,
Honnappa.Nagarahalli
> -----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>
> > ---
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [dpdk-dev] [PATCH v2] ci: update arm64 Travis jobs to Graviton2
2021-02-22 14:49 ` Juraj Linkeš
@ 2021-02-22 19:37 ` Aaron Conole
0 siblings, 0 replies; 22+ messages in thread
From: Aaron Conole @ 2021-02-22 19:37 UTC (permalink / raw)
To: Juraj Linkeš
Cc: thomas, david.marchand, maicolgabriel, dev, Ruifeng.Wang,
Honnappa.Nagarahalli
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>
>> > ---
>>
^ permalink raw reply [flat|nested] 22+ messages in thread
* [dpdk-dev] [PATCH v3] ci: update arm64 Travis jobs to Graviton2
2021-02-19 7:34 ` [dpdk-dev] [PATCH v2] ci: update arm64 Travis jobs to Graviton2 Juraj Linkeš
2021-02-19 9:13 ` Ruifeng Wang
2021-02-19 13:27 ` Aaron Conole
@ 2021-02-23 8:28 ` Juraj Linkeš
2021-02-23 9:02 ` Ruifeng Wang
` (2 more replies)
2 siblings, 3 replies; 22+ messages in thread
From: Juraj Linkeš @ 2021-02-23 8:28 UTC (permalink / raw)
To: thomas, david.marchand, aconole, maicolgabriel
Cc: dev, Ruifeng.Wang, Honnappa.Nagarahalli, Juraj Linkeš
Use only the newer Graviton2 environment in Travis CI instead of using
the older platform, which has intermittent issues:
1. collect2: fatal error: ld terminated with signal 9 [Killed]
2. ticketlock_autotest sometimes times out
These failures hint at resource availability issues in container
environments. The Graviton2 environment is using VMs and these failures
are not observed in it.
Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
---
.travis.yml | 26 ++++++++++++++++----------
1 file changed, 16 insertions(+), 10 deletions(-)
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
--
2.20.1
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [dpdk-dev] [PATCH v3] ci: update arm64 Travis jobs to Graviton2
2021-02-23 8:28 ` [dpdk-dev] [PATCH v3] " Juraj Linkeš
@ 2021-02-23 9:02 ` Ruifeng Wang
2021-02-23 14:38 ` Aaron Conole
2021-03-02 12:39 ` David Marchand
2 siblings, 0 replies; 22+ messages in thread
From: Ruifeng Wang @ 2021-02-23 9:02 UTC (permalink / raw)
To: Juraj Linkeš, thomas, david.marchand, aconole, maicolgabriel
Cc: dev, Honnappa Nagarahalli, nd
> -----Original Message-----
> From: Juraj Linkeš <juraj.linkes@pantheon.tech>
> Sent: Tuesday, February 23, 2021 4:28 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 v3] ci: update arm64 Travis jobs to Graviton2
>
> Use only the newer Graviton2 environment in Travis CI instead of using the
> older platform, which has intermittent issues:
> 1. collect2: fatal error: ld terminated with signal 9 [Killed] 2.
> ticketlock_autotest sometimes times out
>
> These failures hint at resource availability issues in container environments.
> The Graviton2 environment is using VMs and these failures are not observed
> in it.
>
> 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(-)
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [dpdk-dev] [PATCH v3] ci: update arm64 Travis jobs to Graviton2
2021-02-23 8:28 ` [dpdk-dev] [PATCH v3] " Juraj Linkeš
2021-02-23 9:02 ` Ruifeng Wang
@ 2021-02-23 14:38 ` Aaron Conole
2021-03-02 12:39 ` David Marchand
2 siblings, 0 replies; 22+ messages in thread
From: Aaron Conole @ 2021-02-23 14:38 UTC (permalink / raw)
To: Juraj Linkeš
Cc: thomas, david.marchand, maicolgabriel, dev, Ruifeng.Wang,
Honnappa.Nagarahalli
Juraj Linkeš <juraj.linkes@pantheon.tech> writes:
> Use only the newer Graviton2 environment in Travis CI instead of using
> the older platform, which has intermittent issues:
> 1. collect2: fatal error: ld terminated with signal 9 [Killed]
> 2. ticketlock_autotest sometimes times out
>
> These failures hint at resource availability issues in container
> environments. The Graviton2 environment is using VMs and these failures
> are not observed in it.
>
> Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
> ---
Acked-by: Aaron Conole <aconole@redhat.com>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [dpdk-dev] [PATCH v3] ci: update arm64 Travis jobs to Graviton2
2021-02-23 8:28 ` [dpdk-dev] [PATCH v3] " Juraj Linkeš
2021-02-23 9:02 ` Ruifeng Wang
2021-02-23 14:38 ` Aaron Conole
@ 2021-03-02 12:39 ` David Marchand
2 siblings, 0 replies; 22+ messages in thread
From: David Marchand @ 2021-03-02 12:39 UTC (permalink / raw)
To: Juraj Linkeš
Cc: Thomas Monjalon, Aaron Conole, Michael Santana, dev,
Ruifeng Wang (Arm Technology China),
Honnappa Nagarahalli
On Tue, Feb 23, 2021 at 9:28 AM Juraj Linkeš <juraj.linkes@pantheon.tech> wrote:
>
> Use only the newer Graviton2 environment in Travis CI instead of using
> the older platform, which has intermittent issues:
> 1. collect2: fatal error: ld terminated with signal 9 [Killed]
> 2. ticketlock_autotest sometimes times out
>
> These failures hint at resource availability issues in container
> environments. The Graviton2 environment is using VMs and these failures
> are not observed in it.
>
> Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
Reviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>
Acked-by: Aaron Conole <aconole@redhat.com>
Applied, thanks Juraj.
--
David Marchand
^ permalink raw reply [flat|nested] 22+ messages in thread