Skip to content
This repository was archived by the owner on Nov 17, 2023. It is now read-only.

[v1.x] Update CD pipeline to build with centos7 - #20409

Merged
leezu merged 18 commits into
apache:v1.xfrom
waytrue17:debug
Jul 6, 2021
Merged

[v1.x] Update CD pipeline to build with centos7#20409
leezu merged 18 commits into
apache:v1.xfrom
waytrue17:debug

Conversation

@waytrue17

@waytrue17 waytrue17 commented Jun 30, 2021

Copy link
Copy Markdown
Contributor

Description

Change the base docker image used by CD pipeline from Ubuntu 14.04 to CentOS 7. Unify CentOS docker images by using docker-compose.

Checklist

Essentials

  • PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
  • Changes are complete (i.e. I finished coding on this PR)
  • All changes have test coverage
  • Code is well-documented

Changes

  • Feature1, tests, (and when applicable, API doc)
  • Feature2, tests, (and when applicable, API doc)

Comments

  • If this change is a backward incompatible change, why must this change be made.
  • Interesting edge cases to note here

@mxnet-bot

Copy link
Copy Markdown

Hey @waytrue17 , Thanks for submitting the PR
All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands:

  • To trigger all jobs: @mxnet-bot run ci [all]
  • To trigger specific jobs: @mxnet-bot run ci [job1, job2]

CI supported jobs: [website, unix-gpu, unix-cpu, centos-cpu, clang, sanity, windows-cpu, windows-gpu, miscellaneous, edge, centos-gpu]


Note:
Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin.
All CI tests must pass before the PR can be merged.

@mseth10 mseth10 added pr-awaiting-testing PR is reviewed and waiting CI build and test pr-work-in-progress PR is still work in progress and removed pr-awaiting-testing PR is reviewed and waiting CI build and test labels Jun 30, 2021
@waytrue17 waytrue17 changed the title [v1.x] Update CD pipeline to build with centos7 [WIP][v1.x] Update CD pipeline to build with centos7 Jul 1, 2021
@waytrue17 waytrue17 changed the title [WIP][v1.x] Update CD pipeline to build with centos7 [v1.x] Update CD pipeline to build with centos7 Jul 1, 2021
@mseth10 mseth10 added pr-awaiting-testing PR is reviewed and waiting CI build and test pr-awaiting-review PR is waiting for code review pr-work-in-progress PR is still work in progress and removed pr-work-in-progress PR is still work in progress pr-awaiting-testing PR is reviewed and waiting CI build and test pr-awaiting-review PR is waiting for code review labels Jul 1, 2021

@mseth10 mseth10 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@mseth10 mseth10 added pr-awaiting-merge Review and CI is complete. Ready to Merge and removed pr-awaiting-testing PR is reviewed and waiting CI build and test labels Jul 6, 2021
@leezu
leezu merged commit 4aed576 into apache:v1.x Jul 6, 2021
@waytrue17
waytrue17 deleted the debug branch July 7, 2021 20:17
waytrue17 added a commit to waytrue17/incubator-mxnet that referenced this pull request Jul 7, 2021
Co-authored-by: Wei Chu <weichu@amazon.com>
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 <weichu@amazon.com>

Co-authored-by: Manu Seth <22492939+mseth10@users.noreply.github.com>
Co-authored-by: Wei Chu <weichu@amazon.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

pr-awaiting-merge Review and CI is complete. Ready to Merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants