[Openvpn-devel] [PATCH v2] travis-ci: add arm64, s390x builds.
From: Ilya Shipitsin as described on https://docs.travis-ci.com/user/multi-cpu-architectures travis-ci now supports amd64, ppcle, arm64, s390 architectures. Add arm64 and s390x. travis-ci images were upgraded to bionic. "sudo" is deprecated, let us remove it, also "matrix" is deprecated in favour of "jobs". LD_LIBRARY_PATH was replaced by using "rpath" in LDFLAGS, which is more elegant way of linking. also, dependencies were upgraded to the latest versions. travis_wait was added for long openssl builds. cmocka was added to linux and osx builds. --- v2 resolved travis_wait output redirection issue, now it works as expected. I had to specify "names" for jobs, without names travis puts secure variable as job name .travis.yml | 87 +-- .travis/build-check.sh| 10 + .travis/build-deps.sh | 10 +++-- .travis/run-build-deps.sh | 10 - 4 files changed, 62 insertions(+), 55 deletions(-) delete mode 100755 .travis/run-build-deps.sh diff --git a/.travis.yml b/.travis.yml index 40296d87..925d09ea 100644 --- a/.travis.yml +++ b/.travis.yml @@ -1,5 +1,4 @@ -sudo: required -dist: xenial +dist: bionic os: linux @@ -11,86 +10,111 @@ env: - PREFIX="${HOME}/opt" - TAP_WINDOWS_VERSION=9.23.3 - LZO_VERSION=2.10 -- PKCS11_HELPER_VERSION=1.25.1 -- MBEDTLS_VERSION=2.16.1 +- PKCS11_HELPER_VERSION=1.26 +- MBEDTLS_VERSION=2.16.4 - MBEDTLS_CFLAGS="-I${PREFIX}/include" - MBEDTLS_LIBS="-L${PREFIX}/lib -lmbedtls -lmbedx509 -lmbedcrypto" -- OPENSSL_VERSION=1.0.2s +- OPENSSL_VERSION=1.0.2u - OPENSSL_CFLAGS="-I${PREFIX}/include" - OPENSSL_LIBS="-L${PREFIX}/lib -lssl -lcrypto" # The next declaration is the encrypted COVERITY_SCAN_TOKEN, created # via the "travis encrypt" command using the project repo's public key - secure: "l9mSnEW4LJqjxftH5i1NdDaYfGmQB1mPXnSB3DXnsjzkCWZ+yJLfBemfQ0tx/wS7chBYxqUaUIMT0hw4zJVp/LANFJo2vfh//ymTS6h0uApRY1ofg9Pp1BFcV1laG6/u8pwSZ2EBy/GhCd3DS436oE8sYBRaFM9FU62L/oeQBfJ7r4ID/0eB1b8bqlbD4paty9MHui2P8EZJwR+KAD84prtfpZOcrSMxPh9OUhJxzxUvvVoP4s4+lZ5Kgg1bBQ3yzKGDqe8VOgK2BWCEuezqhMMc8oeKmAe7CUkoz5gsGYH++k3I9XzP9Z4xeJKoQnC/82qi4xkJmlaOxdionej9bHIcjfRt7D8j1J0U+wOj4p8VrDy7yHaxuN2fi0K5MGa/CaXQSrkna8dePniCng+xQ2MY/zxuRX2gA6xPNLUyQLU9LqIug7wj4z84Hk9iWib4L20MoPjeEo+vAUNq8FtjOPxMuHNpv4iGGx6kgJm7RXl5vC5hxfK6MprrnYe2U5Mcd8jpzagKBaKHL3zV2FxX9k0jRO9Mccz7M2WnaV0MQ6zcngzTN4+s0kCjhfGKd2F2ANT2Gkhj3Me36eNHfuE0dBbvYCMh4b3Mgd7b/OuXwQWdJ8PjJ1WHXjSOw5sHw1suaV6cEO2Meyz5j1tOkyOi0M9QF+LFenQ9vLH4sBCww8U=" -matrix: +jobs: include: -- env: +- name: cl + env: - SSLLIB="openssl" - OPENSSL_VERSION="1.1.1d" - P7Z="c:\Program Files\7-Zip\7z.exe" - CC="cl" os: windows compiler: cl -- env: SSLLIB="openssl" RUN_COVERITY="1" +- name: Coverity scan + env: SSLLIB="openssl" RUN_COVERITY="1" os: linux compiler: gcc -- env: SSLLIB="openssl" OPENSSL_VERSION="1.0.1u" +- name: gcc | openssl-1.0.1u + env: SSLLIB="openssl" OPENSSL_VERSION="1.0.1u" os: linux compiler: gcc -- env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1c" +- name: gcc | openssl-1.1.1d + env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d" os: linux + arch: amd64 compiler: gcc -- env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1c" LABEL="linux-ppc64le" - os: linux-ppc64le +- name: gcc | openssl-1.1.1d + env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d" + os: linux + arch: ppc64le + compiler: gcc +- name: gcc | openssl-1.1.1d + env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d" + os: linux + arch: arm64 + compiler: gcc +- name: gcc | openssl-1.1.1d + env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d" + os: linux + arch: s390x compiler: gcc -- env: SSLLIB="openssl" EXTRA_CONFIG="--enable-iproute2" +- name: gcc | openssl-1.0.2u | iproute2 + env: SSLLIB="openssl" EXTRA_CONFIG="--enable-iproute2" os: linux compiler: gcc -- env: SSLLIB="openssl" CFLAGS="-fsanitize=address" CC=clang-9 +- name: clang+asan | openssl-1.0.2u + env: SSLLIB="openssl" CFLAGS="-fsanitize=address" CC=clang-9 os: linux compiler: clang -- env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1c" CC=clang-9 +- name: clang | openssl-1.1.1d + env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d" CC=clang-9 os: linux compiler: clang -- env: SSLLIB="mbedtls" +- name: gcc | mbedtls + env: SSLLIB="mbedtls" os: linux compiler: gcc -- env: SSLLIB="mbedtls" CFLAGS="-fsanitize=address" CC=clang-9 +- name: clang+asan | mbedtls + env: SSLLIB="mbedtls" CFLAGS="-fsanitize=address" CC=clang-9 os: linux compiler: clang -- env: SSLLIB="openssl" +- name: clang | openssl-1.0.2u + env: SSLLIB="openssl" os: osx compiler: clang -- env:
Re: [Openvpn-devel] [PATCH v2] travis-ci: add arm64, s390x builds.
Am 03.02.20 um 10:59 schrieb Илья Шипицин: > > > пн, 3 февр. 2020 г. в 14:51, Steffan Karger > mailto:steffan.kar...@foxcrypto.com>>: > > On 03-02-2020 09:04, Илья Шипицин wrote: > > also, ARM64 builds are flaky. maybe we should add them as > allow_failures. > > What is flaky about the ARM64 builds? Is it our build? Is it the travis > infra? > > > in travis-ci terms "our" failure means "build failed", travis-ci infra > means "build errored" > > they run arm64 on linaro cloud, it fails (sometimes) on infra level (I > guess cloud is overloaded or like that). I did not seen yet tests failure > Hm. If we ignore the failure then the test is almost worthless since I think most people only look at the failures if there is an error. If we build and ignore it, we can also ignore it completely. Just my 2c. Arne signature.asc Description: OpenPGP digital signature ___ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
Re: [Openvpn-devel] [PATCH v2] travis-ci: add arm64, s390x builds.
пн, 3 февр. 2020 г. в 14:51, Steffan Karger : > On 03-02-2020 09:04, Илья Шипицин wrote: > > also, ARM64 builds are flaky. maybe we should add them as allow_failures. > > What is flaky about the ARM64 builds? Is it our build? Is it the travis > infra? > in travis-ci terms "our" failure means "build failed", travis-ci infra means "build errored" they run arm64 on linaro cloud, it fails (sometimes) on infra level (I guess cloud is overloaded or like that). I did not seen yet tests failure > > -Steffan > > > ___ > Openvpn-devel mailing list > Openvpn-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/openvpn-devel > ___ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
Re: [Openvpn-devel] [PATCH v2] travis-ci: add arm64, s390x builds.
On 03-02-2020 09:04, Илья Шипицин wrote: > also, ARM64 builds are flaky. maybe we should add them as allow_failures. What is flaky about the ARM64 builds? Is it our build? Is it the travis infra? -Steffan ___ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
Re: [Openvpn-devel] [PATCH v2] travis-ci: add arm64, s390x builds.
also, ARM64 builds are flaky. maybe we should add them as allow_failures. пн, 3 февр. 2020 г. в 12:49, Илья Шипицин : > https://travis-ci.org/chipitsine/openvpn/jobs/645173717#L62 > > I expected it to be 30 minutes wait. > > пн, 3 февр. 2020 г. в 12:48, Илья Шипицин : > >> >> >> пн, 3 февр. 2020 г. в 12:40, Lev Stipakov : >> >>> Hi, >>> >>> Could you provide a link to a travis build with your changes? >>> >> >> >> https://travis-ci.org/chipitsine/openvpn/builds/645173716 >> >> >> there's at least issue regarding "travis_wait 30", as I can see, windows >> builds waits for 10 minutes. >> >> I'll fix it in "v3" (after all reviews) >> >>> >>> -- >>> -Lev >>> >> ___ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
Re: [Openvpn-devel] [PATCH v2] travis-ci: add arm64, s390x builds.
https://travis-ci.org/chipitsine/openvpn/jobs/645173717#L62 I expected it to be 30 minutes wait. пн, 3 февр. 2020 г. в 12:48, Илья Шипицин : > > > пн, 3 февр. 2020 г. в 12:40, Lev Stipakov : > >> Hi, >> >> Could you provide a link to a travis build with your changes? >> > > > https://travis-ci.org/chipitsine/openvpn/builds/645173716 > > > there's at least issue regarding "travis_wait 30", as I can see, windows > builds waits for 10 minutes. > > I'll fix it in "v3" (after all reviews) > >> >> -- >> -Lev >> > ___ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
Re: [Openvpn-devel] [PATCH v2] travis-ci: add arm64, s390x builds.
пн, 3 февр. 2020 г. в 12:40, Lev Stipakov : > Hi, > > Could you provide a link to a travis build with your changes? > https://travis-ci.org/chipitsine/openvpn/builds/645173716 there's at least issue regarding "travis_wait 30", as I can see, windows builds waits for 10 minutes. I'll fix it in "v3" (after all reviews) > > -- > -Lev > ___ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
Re: [Openvpn-devel] [PATCH v2] travis-ci: add arm64, s390x builds.
Hi, Could you provide a link to a travis build with your changes? -- -Lev ___ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
[Openvpn-devel] [PATCH v2] travis-ci: add arm64, s390x builds.
From: Ilya Shipitsin as described on https://docs.travis-ci.com/user/multi-cpu-architectures travis-ci now supports amd64, ppcle, arm64, s390 architectures. Add arm64 and s390x. travis-ci images were upgraded to bionic. "sudo" is deprecated, let us remove it, also "matrix" is deprecated in favour of "jobs". LD_LIBRARY_PATH was replaced by using "rpath" in LDFLAGS, which is more elegant way of linking. also, dependencies were upgraded to the latest versions. travis_wait was added for long openssl builds. cmocka was added to linux and osx builds. --- v2 rebased against proper commit .travis.yml | 42 +-- .travis/build-check.sh| 10 +- .travis/build-deps.sh | 10 ++ .travis/run-build-deps.sh | 10 -- 4 files changed, 30 insertions(+), 42 deletions(-) delete mode 100755 .travis/run-build-deps.sh diff --git a/.travis.yml b/.travis.yml index 40296d87..9f9db954 100644 --- a/.travis.yml +++ b/.travis.yml @@ -1,5 +1,4 @@ -sudo: required -dist: xenial +dist: bionic os: linux @@ -11,18 +10,18 @@ env: - PREFIX="${HOME}/opt" - TAP_WINDOWS_VERSION=9.23.3 - LZO_VERSION=2.10 -- PKCS11_HELPER_VERSION=1.25.1 -- MBEDTLS_VERSION=2.16.1 +- PKCS11_HELPER_VERSION=1.26 +- MBEDTLS_VERSION=2.16.4 - MBEDTLS_CFLAGS="-I${PREFIX}/include" - MBEDTLS_LIBS="-L${PREFIX}/lib -lmbedtls -lmbedx509 -lmbedcrypto" -- OPENSSL_VERSION=1.0.2s +- OPENSSL_VERSION=1.0.2u - OPENSSL_CFLAGS="-I${PREFIX}/include" - OPENSSL_LIBS="-L${PREFIX}/lib -lssl -lcrypto" # The next declaration is the encrypted COVERITY_SCAN_TOKEN, created # via the "travis encrypt" command using the project repo's public key - secure: "l9mSnEW4LJqjxftH5i1NdDaYfGmQB1mPXnSB3DXnsjzkCWZ+yJLfBemfQ0tx/wS7chBYxqUaUIMT0hw4zJVp/LANFJo2vfh//ymTS6h0uApRY1ofg9Pp1BFcV1laG6/u8pwSZ2EBy/GhCd3DS436oE8sYBRaFM9FU62L/oeQBfJ7r4ID/0eB1b8bqlbD4paty9MHui2P8EZJwR+KAD84prtfpZOcrSMxPh9OUhJxzxUvvVoP4s4+lZ5Kgg1bBQ3yzKGDqe8VOgK2BWCEuezqhMMc8oeKmAe7CUkoz5gsGYH++k3I9XzP9Z4xeJKoQnC/82qi4xkJmlaOxdionej9bHIcjfRt7D8j1J0U+wOj4p8VrDy7yHaxuN2fi0K5MGa/CaXQSrkna8dePniCng+xQ2MY/zxuRX2gA6xPNLUyQLU9LqIug7wj4z84Hk9iWib4L20MoPjeEo+vAUNq8FtjOPxMuHNpv4iGGx6kgJm7RXl5vC5hxfK6MprrnYe2U5Mcd8jpzagKBaKHL3zV2FxX9k0jRO9Mccz7M2WnaV0MQ6zcngzTN4+s0kCjhfGKd2F2ANT2Gkhj3Me36eNHfuE0dBbvYCMh4b3Mgd7b/OuXwQWdJ8PjJ1WHXjSOw5sHw1suaV6cEO2Meyz5j1tOkyOi0M9QF+LFenQ9vLH4sBCww8U=" -matrix: +jobs: include: - env: - SSLLIB="openssl" @@ -37,11 +36,21 @@ matrix: - env: SSLLIB="openssl" OPENSSL_VERSION="1.0.1u" os: linux compiler: gcc -- env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1c" +- env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d" os: linux + arch: amd64 compiler: gcc -- env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1c" LABEL="linux-ppc64le" - os: linux-ppc64le +- env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d" + os: linux + arch: ppc64le + compiler: gcc +- env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d" + os: linux + arch: arm64 + compiler: gcc +- env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d" + os: linux + arch: s390x compiler: gcc - env: SSLLIB="openssl" EXTRA_CONFIG="--enable-iproute2" os: linux @@ -49,7 +58,7 @@ matrix: - env: SSLLIB="openssl" CFLAGS="-fsanitize=address" CC=clang-9 os: linux compiler: clang -- env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1c" CC=clang-9 +- env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d" CC=clang-9 os: linux compiler: clang - env: SSLLIB="mbedtls" @@ -67,7 +76,7 @@ matrix: - env: SSLLIB="openssl" CHOST=x86_64-w64-mingw32 OPENSSL_VERSION="1.0.1u" os: linux compiler: ": Win64 build only" -- env: SSLLIB="openssl" CHOST=x86_64-w64-mingw32 OPENSSL_VERSION="1.1.1c" +- env: SSLLIB="openssl" CHOST=x86_64-w64-mingw32 OPENSSL_VERSION="1.1.1d" os: linux compiler: ": Win64 build only" - env: SSLLIB="openssl" CHOST=i686-w64-mingw32 @@ -82,15 +91,11 @@ matrix: addons: apt: -sources: - - ubuntu-toolchain-r-test - - sourceline: 'deb https://apt.llvm.org/xenial/ llvm-toolchain-xenial-9 main' -key_url: 'https://apt.llvm.org/llvm-snapshot.gpg.key' update: true -packages: [ liblzo2-dev, libpam0g-dev, liblz4-dev, linux-libc-dev, man2html, mingw-w64 ] +packages: [ liblzo2-dev, libpam0g-dev, liblz4-dev, linux-libc-dev, man2html, mingw-w64, clang-9, libcmocka-dev ] homebrew: update: true -packages: [ lzo ] +packages: [ lzo, lz4, cmocka ] cache: directories: @@ -99,9 +104,8 @@ cache: - ${HOME}/Library/Caches/Homebrew install: - - if [ "$TRAVIS_OS_NAME" = "linux" ] && [ "$TRAVIS_CPU_ARCH" != "ppc64le" ]; then sudo apt-get install clang-9; fi - if [ ! -z "${CHOST}" ]; then unset CC; fi - - .travis/run-build-deps.sh + - travis_wait 30 .travis/build-deps.sh >