Skip to content

feat: Adding Container Build Option to Crew AI #609

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

jland-redhat
Copy link

@jland-redhat jland-redhat commented May 20, 2025

Description

This PR introduces container files and build instructions for the CrewAI sample. This aims to simplify the setup process for users more familiar with Docker than Python environments.

I believe expanding containerization to other components could be beneficial, but I wanted to start with this single sample to gauge interest and ensure this direction aligns with project goals before investing more time into it.

Testing Plan

I was able to build the server and connect to it with the demo application locally.

Screenshot From 2025-05-20 16-32-22
Screenshot From 2025-05-20 16-37-49

Thank you for opening a Pull Request!
Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • [X ] Follow the CONTRIBUTING Guide.
  • [x ] Make your Pull Request title in the https://www.conventionalcommits.org/ specification.
  • [ x] Ensure the tests and linter pass (Run nox -s format from the repository root to format)
  • [x ] Appropriate docs were updated (if necessary)

@jland-redhat jland-redhat requested a review from a team as a code owner May 20, 2025 20:45
@r3motecontrol
Copy link

r3motecontrol commented May 21, 2025

I had a hacky version for Langgraph. I switched to your version by changing the WORKDIR and it worked fine.

@jland-redhat
Copy link
Author

I had a hacky version for Langgraph. I switched to your version by changing the WORKDIR and it worked fine.

Ha ha I feel like this is still kinda hacky. But seemed like the best option considering the mono-repo setup.

Realistically we could probably make it a cleaner smaller image with like a multi-build, but I figured since this is a sample and first pass it was best to keep it simple.

@jland-redhat
Copy link
Author

@DJ-os Hey sorry to ping you directly, but was wondering if there is anyway I can ping to try to ask for a PR review?

@KunjShah01
Copy link

CAN I work on it @DJ-os

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants