devika
devika copied to clipboard
[Improve] Improved Docker build
This commit provides an improved Docker build
- Adds build caching
- Adds and configures Miniconda environment with GPU driver support.
- Adds the ability to sync local and Docker environments with Conda and
environment.yml. - Adds name to
devika-backend-engineservice image. - Adds hostnames additionally to service names to simplify container communication by shorter and more convenient host names.
- Implements various environment variables for optimization.
- Patches source files at the image build stage for seamless Docker support
- Replace UI package manager with Yarn, which does not require additional installation
- Replaces base UI image with
node:18.20.2-bullseye, which provides the node out-of-the-box. - Replaces base image of backend service to
python:3.11.9-bookwormwith Python 3.11 preinstalled.
remove the lock file
also how much each the size of the containers?
Thanks for the prompt reply, @ARajgor!
Preparing another commit with hotfixes. I knew you'd ask ) All containers that use CUDA are large because CUDA is large. I coded from the view that size matters at the release stage, while the current one seems to be an active development, so I did not add any cleanup code and added caching, which speeds up rebuilds, even if I suspect that the price is image size or other disk resources. I know how to optimize images for production later.
devika-backend-engine-1 | Traceback (most recent call last):
devika-backend-engine-1 | File "
I'm not sure if this is related to this specific PR.
Original sample.config.toml must exist. Check if working directory contains it in container
- Removes bun lock file
- Fixes missing playwright browser error on task completion
@ARajgor docker-compose build should run smoothly now
you didn't remove the yarn.lock
yarn.lock must be present in repo for various reasons:
https://classic.yarnpkg.com/lang/en/docs/yarn-lock/#toc-check-into-source-control
https://classic.yarnpkg.com/blog/2016/11/24/lockfiles-for-all/
should-i-commit-the-yarn-lock-file-and-what-is-it-for
devika is using bun as it default package manager. bun.lockb is already included in the repo.
@nalaso, the reasoning is mentioned in the PR description:
"Replace UI package manager with Yarn, which does not require additional installation"
I don't care about specific PM. Feel free to revert bun.lock and npm install bun back.
how about using oven/bun? also the app can work with just Bun
No problem about Bun. This is based on my project, which uses Yarn as a package manager. Feel free to replace it with Bun or anything.
On Thu, May 2, 2024 at 11:45 AM Polygons1 @.***> wrote:
how about using oven/bun? also the app can work with just Bun
— Reply to this email directly, view it on GitHub https://github.com/stitionai/devika/pull/514#issuecomment-2089920148, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMYTLKG2NQ7EJALMLG54TKTZAH4J5AVCNFSM6AAAAABG7RHPVSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBZHEZDAMJUHA . You are receiving this because you authored the thread.Message ID: @.***>