-
Notifications
You must be signed in to change notification settings - Fork 15.4k
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
Prebuild a dev container image to improve build time #6547
Comments
Hi @bamurtaugh, I have tried to run dev container multiple times both on codespaces and local machine(windows). But It always ends up giving this error. Also attaching the creation.log file for your reference executor failed running [/bin/sh -c chmod -R 0755 /tmp/dev-container-features/poetry_1 && cd /tmp/dev-container-features/poetry_1 && chmod +x ./devcontainer-features-install.sh && ./devcontainer-features-install.sh]: exit code: 1 |
I am having the same issue trying to build docker-compose dev container. it fails every time. I can attach vscode to an already running container, but that is about it. I can dump logs, but ultimately I am just getting the same error as the user above. I can't find any other logging related to the failure |
Hi, @bamurtaugh! I'm Dosu, and I'm here to help the LangChain team manage their backlog. I wanted to let you know that we are marking this issue as stale. From what I understand, this issue is a feature request to pre-build a dev container image for faster build times and improved configuration. You mentioned that you are open to feedback and collaboration on hosting a pre-built image. There are also two comments from other users who are experiencing an error when trying to build the dev container using docker-compose. They have provided logs and described the error they are encountering. Before we proceed, we would like to confirm if this issue is still relevant to the latest version of the LangChain repository. If it is, please let us know by commenting on this issue. Otherwise, feel free to close the issue yourself, or the issue will be automatically closed in 7 days. Thank you for your understanding and cooperation. We appreciate your contribution to the LangChain project! Best regards, |
Feature request
Thanks so much for merging the PR to update the dev container in this repo #6189!
While the dev container now builds and runs successfully, it can take some time to build. One recommendation is for the LangChain team to pre-build an image.
Motivation
We recommend pre-building images with the tools you need rather than creating and building a container image each time you open your project in a dev container. Using pre-built images will result in a faster container startup, simpler configuration, and allows you to pin to a specific version of tools to improve supply-chain security and avoid potential breaks. You can automate pre-building your image by scheduling the build using a DevOps or continuous integration (CI) service like GitHub Actions.
There's further info in our docs: https://containers.dev/implementors/reference/#prebuilding.
Your contribution
We're more than happy to answer any questions and would love to hear feedback if you're interested in hosting a pre-built image! On the dev container team side, we're also looking to even better document pre-building: devcontainers/spec#261, which should help as a reference for scenarios like this too.
The text was updated successfully, but these errors were encountered: