Skip to content
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

Utility to run commands in docker containers as isolated execution environment. #9766

Conversation

piotr-grodek-dsai
Copy link

DRAFT

Description: It is a wrapper around docker python library, allowing to pull/build docker images with ability to run and execute command in a container. It is a low-level building block which allows to isolate execution and constrain resources - should allow to improve security a bit after integration with things like PythonREPL.

Dependencies: docker

Twitter handle: @deepsense_ai

WIP: For now I think integration tests are required. I run tests outside langchain so I need yet to see if they pass & can run on CI.

@vercel
Copy link

vercel bot commented Aug 25, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview Sep 20, 2023 7:04am

@dosubot dosubot bot added the 🤖:enhancement A large net-new component, integration, or chain. Use sparingly. The largest features label Aug 25, 2023
@piotr-grodek-dsai piotr-grodek-dsai force-pushed the deepsense/isolated_env_execution branch from 059a1d9 to a145f9c Compare August 28, 2023 06:53
@piotr-grodek-dsai piotr-grodek-dsai force-pushed the deepsense/isolated_env_execution branch from a145f9c to 98e528f Compare August 30, 2023 10:15
@baskaryan
Copy link
Collaborator

cc @nfcampos @obi1kenobi

@piotr-grodek-dsai piotr-grodek-dsai force-pushed the deepsense/isolated_env_execution branch from 9f594f4 to 88455be Compare September 20, 2023 07:04
@leo-gan
Copy link
Collaborator

leo-gan commented Sep 21, 2023

@piotr-grodek-dsai If this PR is not needed anymore, could you, please, let me know or close this PR?

@piotr-grodek-dsai
Copy link
Author

Yes, we can close it - more complete version is under review and as agreed with Numo part of the code will live as python package.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖:enhancement A large net-new component, integration, or chain. Use sparingly. The largest features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants