Skip to content

feat/staging prepare - #18

Merged
wailbentafat merged 6 commits into
mainfrom
Depresso/prepare
Mar 18, 2026
Merged

feat/staging prepare#18
wailbentafat merged 6 commits into
mainfrom
Depresso/prepare

Conversation

@Tyjfre-j

Copy link
Copy Markdown
Collaborator

Summary

  • Added containerized FastAPI service to the main Docker Compose stack.
  • Introduced a root Dockerfile for building and running the API with uv + uvicorn.
  • Added .dockerignore to reduce build context and prevent local/cache files from being included.
  • Updated NATS service command to use authentication flags from environment variables.
  • Removed the obsolete compose version key.

How to test

  • Start all services:
    docker compose up --build -d
  • Check running containers:
    docker compose ps
  • View FastAPI logs:
    docker compose logs -f fastapi
  • Stop all services:
    docker compose down

…date docker-compose.yml to include fastapi service
Comment thread docker-compose.yml

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

create env for staging tht have the hosts of staging network (instead of local host we use service name ) and sone touch the .env we let it for the dev environement

Comment thread Dockerfile

ENV PYTHONDONTWRITEBYTECODE=1
ENV PYTHONUNBUFFERED=1

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add another dockerfile tht run the migration on postgress in the staging or do it in this dockerfile

Comment thread .env.staging.example Outdated
# =REDACTED
# Additional required variables
# =REDACTED
# ...add any other staging-specific variables here...

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

??????

Comment thread .env.staging.example Outdated
# Redis
# =REDACTED
REDIS_PORT=REDACTED
REDIS_HOST=REDACTED

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?????

Comment thread .env.staging.example Outdated

# =REDACTED
# MinIO
# =REDACTED

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?????

Comment thread docker-compose.yml Outdated
command: ["uv", "run", "alembic", "upgrade", "head"]
networks:
- multi_network

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove this we dont need this

@wailbentafat wailbentafat left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

za3im

@wailbentafat
wailbentafat merged commit da7529d into main Mar 18, 2026
1 check passed
@Tyjfre-j
Tyjfre-j deleted the Depresso/prepare branch April 11, 2026 20:44
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.

2 participants