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

[BACKPORT]Update Ubuntu images used on CI to 20.04 (#19588) #19935

Closed
wants to merge 1 commit into from
Closed

[BACKPORT]Update Ubuntu images used on CI to 20.04 (#19588) #19935

wants to merge 1 commit into from

Conversation

access2rohit
Copy link
Contributor

  • Update Ubuntu images used on CI to 20.04

This helps ensure MXNet to work well on recent Linux distributions (while ensuring it continues to work well on ancient distributions based on the CentOS7 CI pipeline)

  • Preserve Ubuntu 18.04 images for TensorRT pipeline as NVidia failed to make TensorRT available for Ubuntu 20.04

  • Temporarily disable NVML on CI

[2020-12-03T18:33:10.380Z] OSError: /work/mxnet/python/mxnet/../../build/libmxnet.so: undefined symbol: nvmlDeviceGetComputeRunningProcesses_v2

backport #19588 as a part of effort #19911

@leezu
Update Ubuntu images used on CI to 20.04 (apache#19588)
67a211e 
* Update Ubuntu images used on CI to 20.04

This helps ensure MXNet to work well on recent Linux distributions (while ensuring it continues to work well on ancient distributions based on the CentOS7 CI pipeline)

* Preserve Ubuntu 18.04 images for TensorRT pipeline as NVidia failed to make TensorRT available for Ubuntu 20.04

* Temporarily disable NVML on CI

[2020-12-03T18:33:10.380Z] OSError: /work/mxnet/python/mxnet/../../build/libmxnet.so: undefined symbol: nvmlDeviceGetComputeRunningProcesses_v2
@mxnet-bot
Copy link

Hey @access2rohit , 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, miscellaneous, unix-gpu, centos-cpu, windows-gpu, unix-cpu, sanity, windows-cpu, centos-gpu, edge, clang]

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.

@access2rohit
Copy link
Contributor Author

@leezu @josephevans please review

@lanking520 lanking520 added the pr-awaiting-testing PR is reviewed and waiting CI build and test label Feb 19, 2021
@access2rohit access2rohit mentioned this pull request Feb 19, 2021
Closed
13 tasks
@lanking520 lanking520 added 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 Feb 19, 2021
@access2rohit
Copy link
Contributor Author

Not required..... Closing

@leezu
Copy link
Contributor

leezu commented Feb 22, 2021

You may still want to update some of the containers that are EOL, for example https://github.com/access2rohit/incubator-mxnet/blob/u20_ci_update_bp/ci/docker/Dockerfile.build.ubuntu_base_cpu

You can choose to adopt the same approach as on master or just keep the structure as on v1.x. But the goal may still be to avoid using EOL unmaintained base images?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@aaronmarkham aaronmarkham Awaiting requested review from aaronmarkham aaronmarkham is a code owner

@marcoabreu marcoabreu Awaiting requested review from marcoabreu marcoabreu is a code owner

@szha szha Awaiting requested review from szha szha is a code owner

Assignees
No one assigned
Labels
pr-work-in-progress PR is still work in progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@access2rohit @mxnet-bot @leezu @lanking520