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 |
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 |
> -----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> >> > --- >>
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
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(-)