[Devops] Add Blender source to Docker images#21793
[Devops] Add Blender source to Docker images#21793sarnex wants to merge 2 commits intointel:syclfrom
Conversation
Signed-off-by: Nick Sarnie <nick.sarnie@intel.com>
There was a problem hiding this comment.
Does Blender local build also take ~1Gb of space? Would it be better to create a separate image based on ubuntu build / intel_drivers images to avoid slowing down pre-commit etc.?
UPD copilot says there is some caching on self-hosted runners, so probably it's not an issue
|
Thanks for the review. Yeah let me just create a separate image, thanks. |
Signed-off-by: Nick Sarnie <nick.sarnie@intel.com>
|
@KornevNikita Latest commit should address feedback, thanks! |
KornevNikita
left a comment
There was a problem hiding this comment.
LGTM with minor comments, although my approval is not required to merge:)
| # as the ones above were not yet pushed. | ||
| build_and_push_blender_images: | ||
| needs: build_and_push_driver_images | ||
| name: "Containers" |
There was a problem hiding this comment.
Looks like we have 3 jobs named "Containers" here. Could we use some specific name?
|
|
||
| USER sycl | ||
|
|
||
| ENTRYPOINT ["/bin/bash", "/drivers_entrypoint.sh"] |
There was a problem hiding this comment.
Do we still need this given this doesn't work? (or was it fixed?)
| set -e | ||
| set -o pipefail | ||
|
|
||
| apt update && apt install -yqq git-lfs |
There was a problem hiding this comment.
Probably we should clean-up this at the end to reduce the image size.
I am working to add a CI workflow to build/run Blender on Linux, we already have one for Windows.
Preparing the Blender source is a bit complicated and it's a large download (~1GB) so let's just add it to a new Blender Docker image so running the CI job will be fast.