-
Notifications
You must be signed in to change notification settings - Fork 3.6k
CI: Update base-ipu docker image
#14057
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
Conversation
base-ipu docker imagebase-ipu docker image
|
I'll keep this PR as draft so that we don't merge it before verifying it in the IPU CI. |
14f1339 to
93ffd57
Compare
This PR is basically ready, but I'd like to check in advance if the docker image built from this dockerfile works in our CI... Could anyone help with merging this PR? |
|
Hi, @akihironitta, what do you need help with? I'm here 🙂 |
|
For the note, the only task left here is to validate the change in the IPU server:
|
base-ipu docker imagebase-ipu docker image
base-ipu docker imagebase-ipu docker image
|
@akihironitta blocked in title does not do anything :) |
|
@akihironitta is this still relevant? Otherwise feel free to close it :) |
base-ipu docker imagebase-ipu docker image
|
Closing this PR since #14934 fixed the original issue which this PR was trying to resolve. |
What does this PR do?
>100minto<10min)base-cudadockerfile.Does your PR introduce any breaking changes? If yes, please list them.
None
Before submitting
PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:
Did you have fun?
Make sure you had fun coding 🙃
cc @carmocca @akihironitta @Borda