Conversation
Deploying mxtoai-backend with
|
| Latest commit: |
b7773c4
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://57e5f987.mxtoai-backend.pages.dev |
| Branch Preview URL: | https://docker.mxtoai-backend.pages.dev |
docker-compose.yml
Outdated
| RABBITMQ_DEFAULT_USER: guest | ||
| RABBITMQ_DEFAULT_PASS: guest | ||
| restart: unless-stopped |
There was a problem hiding this comment.
let's not hardcode this, it should be wired from .env file
docker-compose.yml
Outdated
| image: redis:7-alpine | ||
| container_name: redis | ||
| ports: | ||
| - "6379:6379" |
There was a problem hiding this comment.
let's add auth, env variable for REDIS_PASSWORD or something
docker-compose.yml
Outdated
| - "8000:8000" | ||
| environment: | ||
| - REDIS_URL=redis://redis:6379/0 | ||
| - RABBITMQ_URL=amqp://guest:guest@rabbitmq:5672/ |
There was a problem hiding this comment.
username and passwords should be wired from .env
docker/api_server.dockerfile
Outdated
| RUN poetry config virtualenvs.create false && poetry install --no-root --no-interaction --no-ansi | ||
|
|
||
| # Copy the rest of the application | ||
| COPY . . |
There was a problem hiding this comment.
this is too permissive, let's copy only the modules that are needed i.e. /mxtoai
docker/api_server.dockerfile
Outdated
| COPY . . | ||
|
|
||
| # Expose API port (change as needed) | ||
| EXPOSE 8000 |
There was a problem hiding this comment.
let's not expose it here, should be done in docker ocmpose
docker/api_server.dockerfile
Outdated
| EXPOSE 8000 | ||
|
|
||
| # Run the API | ||
| CMD ["poetry", "run", "python", "run_api.py"] |
There was a problem hiding this comment.
We need to run it via uvicorn, for example this is the command we use in prod uvicorn mxtoai.api:app --host 0.0.0.0 --port 9292 --workers 4
docker/api_server.dockerfile
Outdated
| WORKDIR /app | ||
|
|
||
| # Install Poetry | ||
| ENV POETRY_VERSION=2.1.3 |
There was a problem hiding this comment.
let's not fix the poetry version, this can be omitted
docker/worker.dockerfile
Outdated
| WORKDIR /app | ||
|
|
||
| # Install Poetry | ||
| ENV POETRY_VERSION=2.1.3 |
There was a problem hiding this comment.
same as pervious comment
docker/worker.dockerfile
Outdated
| RUN poetry config virtualenvs.create false && poetry install --no-root --no-interaction --no-ansi | ||
|
|
||
| # Copy rest of the worker code | ||
| COPY . . |
There was a problem hiding this comment.
same as pervious comment
docker-compose.yml
Outdated
| volumes: | ||
| - .:/app |
There was a problem hiding this comment.
why do we need this?
docker-compose.yml
Outdated
| volumes: | ||
| - .:/app |
There was a problem hiding this comment.
same, this looks very permissive, why do we need this
There was a problem hiding this comment.
I think we only need this for model-config, nothing else, maybe put model-config under credentials directory and mount as volume.
Add linkedin data api tools via RapidAPI
Fix output formatting, have factcheck to include additional summary, …
Fix bold marker being confused for list item
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Add PDF export handle and tool
Fix other domain validation issue
* Add independent search tools and guidelines * Fix bug with brave search
Formatting fixes and run ruff
Skip processing AWS system emails
3 participants
Setup docker to work with mxtoai. Did a local test. The apis did startup, but couldn't tests every handle yet.
Ollama can be made optional by editing compose or by manually specifying the services while running compose up. Ollama is over 2.5 gigs to better to not include unless needed.