This repository has been archived by the owner on Nov 17, 2023. It is now read-only.
[v1.x] use centos7 base image for aarch64 build #20392
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Hey @mseth10 , Thanks for submitting the PR
CI supported jobs: [windows-cpu, clang, edge, miscellaneous, unix-cpu, centos-cpu, unix-gpu, windows-gpu, centos-gpu, website, sanity] Note: |
mseth10
force-pushed
the
centos7-aarch64
branch
8 times, most recently
from
ccc57fa
to
c28212e
Compare
mseth10
changed the title
mseth10
added
pr-awaiting-testing
mseth10
added
pr-work-in-progress
|
@mxnet-bot run ci [centos-cpu, centos-gpu, clang, edge, unix-cpu, unix-gpu, miscellaneous] |
|
Jenkins CI successfully triggered : [centos-gpu, unix-cpu, clang, centos-cpu, edge, miscellaneous, unix-gpu] |
mseth10
added
pr-awaiting-testing
mseth10
added
pr-awaiting-merge
mseth10
added
ARM
CD
and removed
pr-awaiting-merge
waytrue17
pushed a commit
to waytrue17/incubator-mxnet
that referenced
this pull request
Jul 7, 2021
* add centos7 aarch64 build * enable rh packages * update openssl curl version * install rhel7 armpl * find armpl cmake * test mkldnn build * test ninja * use armpl compiled with gcc-8 * enable py38 for unittests * check previously failing tests * create symlink for ninja * fix cd pypi * fix cd docker * update ssl for mac
6 tasks
mseth10
added a commit
that referenced
this pull request
Jul 7, 2021
…0423) * [v1.x] use centos7 base image for aarch64 build (#20392) * add centos7 aarch64 build * enable rh packages * update openssl curl version * install rhel7 armpl * find armpl cmake * test mkldnn build * test ninja * use armpl compiled with gcc-8 * enable py38 for unittests * check previously failing tests * create symlink for ninja * fix cd pypi * fix cd docker * update ssl for mac * [v1.x] Update CD pipeline to build with centos7 (#20409) Co-authored-by: Wei Chu <[email protected]> Co-authored-by: Manu Seth <[email protected]> Co-authored-by: Wei Chu <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR changes the base docker image used by CD pipeline from
Ubuntu 18toCentOS 7. This relaxes the systemglibcrequirement of the nightly wheel from>=2.27to>=2.17, thus allowing it to run on allmanylinux2014systems.We install
devtoolset-10andrh-python38packages from CentOS SCLo RH repository, which givesgcc/g++/gfortran-10.2allowing us to use-moutline-atomicsC/C++ build flag, andnumpy-v1.17.3which helps pass unittests which were failing earlier #20289.We continue to use ArmPL built with
gcc-8as ArmPL built withgcc-10requires somegfortran-10symbols that are not present indevtoolset-10as it is a patched version.The changes have been tested on a test CD pipeline and it passes for all flavors: https://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/restricted-mxnet-cd%2Fsethman-test-cd-pipeline/detail/sethman-test-cd-pipeline/18/pipeline/30