-
Notifications
You must be signed in to change notification settings - Fork 66
[rocm6.5_internal_testing] remove centos.stream dockerfile and move contents into dockerfile #2044
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: rocm6.5_internal_testing
Are you sure you want to change the base?
[rocm6.5_internal_testing] remove centos.stream dockerfile and move contents into dockerfile #2044
Conversation
Jenkins build for 2dd188e54c25f121e400edf121ba0d22030f0e85 commit finished as FAILURE Detected error during base docker image building:
|
.ci/docker/centos-rocm/Dockerfile
Outdated
RUN bash ./install_amdsmi.sh | ||
RUN rm install_amdsmi.sh | ||
|
||
ENV ROCM_PATH /opt/rocm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dont know why this is added only in centOS dockerfiles, dont remember the history. Try build by removing it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pruthvistony It's not only in CentOS Dockerfile, it was actually ported from Ubuntu Dockerfile to enable triton build: https://github.com/pytorch/pytorch/blame/47d34261e06e2416e7a1e7d51a3d428e4ea51f9d/.ci/docker/ubuntu-rocm/Dockerfile#L63
Jenkins build for 2dd188e54c25f121e400edf121ba0d22030f0e85 commit finished as FAILURE |
2dd188e
to
b72033a
Compare
Jenkins build for 81ae6254f808a4cb43ceabd310ff7f2672d8e6c6 commit is in progress |
rocm6.5_internal_testing move contents of centos stream dockerfile into dockerfile
Validation: http://rocm-ci.amd.com/job/mainline-framework-pytorch-ci-phantom/111/