mirror of https://github.com/buster-so/buster.git
Refactor Docker Compose and API configuration for improved service management
- Consolidated Redis service into the main `docker-compose.yml`, removing the separate API Docker Compose file. - Added health checks for Redis and API services to ensure proper service readiness. - Updated API router to include a public health check endpoint. - Cleaned up the web Dockerfile by removing unnecessary environment variable copying. These changes enhance service orchestration and improve the reliability of the application during development.
This commit is contained in:
parent
176d5eb06b
commit
4679adf04a
|
@ -1,8 +0,0 @@
|
||||||
version: '3.8'
|
|
||||||
|
|
||||||
services:
|
|
||||||
redis:
|
|
||||||
image: redis
|
|
||||||
container_name: buster-redis
|
|
||||||
ports:
|
|
||||||
- "6379:6379"
|
|
|
@ -47,13 +47,14 @@ async fn main() {
|
||||||
}
|
}
|
||||||
|
|
||||||
let protected_router = Router::new().nest("/api/v1", routes::protected_router());
|
let protected_router = Router::new().nest("/api/v1", routes::protected_router());
|
||||||
|
let public_router = Router::new().route("/health", axum::routing::get(|| async { "OK" }));
|
||||||
|
|
||||||
let (shutdown_tx, _) = broadcast::channel::<()>(1);
|
let (shutdown_tx, _) = broadcast::channel::<()>(1);
|
||||||
let shutdown_tx = Arc::new(shutdown_tx);
|
let shutdown_tx = Arc::new(shutdown_tx);
|
||||||
|
|
||||||
let app = Router::new()
|
let app = Router::new()
|
||||||
.merge(protected_router)
|
.merge(protected_router)
|
||||||
// .merge(public_router)
|
.merge(public_router)
|
||||||
.layer(TraceLayer::new_for_http())
|
.layer(TraceLayer::new_for_http())
|
||||||
.layer(cors())
|
.layer(cors())
|
||||||
.layer(CompressionLayer::new())
|
.layer(CompressionLayer::new())
|
||||||
|
|
|
@ -1,30 +1,41 @@
|
||||||
|
include:
|
||||||
|
- supabase/docker-compose.yml
|
||||||
|
|
||||||
services:
|
services:
|
||||||
db:
|
redis:
|
||||||
image: supabase/postgres:15.1.0.117
|
image: redis
|
||||||
restart: always
|
container_name: buster-redis
|
||||||
environment:
|
|
||||||
POSTGRES_PASSWORD: ${POSTGRES_PASSWORD:-your-super-secret-password}
|
|
||||||
POSTGRES_USER: ${POSTGRES_USER:-postgres}
|
|
||||||
POSTGRES_DB: ${POSTGRES_DB:-buster}
|
|
||||||
volumes:
|
|
||||||
- db_data:/var/lib/postgresql/data
|
|
||||||
- ./migrations:/docker-entrypoint-initdb.d
|
|
||||||
ports:
|
ports:
|
||||||
- "5432:5432"
|
- "6379:6379"
|
||||||
|
healthcheck:
|
||||||
|
test: ["CMD", "redis-cli", "ping"]
|
||||||
|
interval: 1s
|
||||||
|
timeout: 3s
|
||||||
|
retries: 30
|
||||||
|
|
||||||
api:
|
api:
|
||||||
build:
|
build:
|
||||||
context: ./api
|
context: ./api
|
||||||
dockerfile: Dockerfile
|
dockerfile: Dockerfile
|
||||||
|
env_file:
|
||||||
|
- .env
|
||||||
ports:
|
ports:
|
||||||
- "3001:3001"
|
- "3001:3001"
|
||||||
environment:
|
healthcheck:
|
||||||
DATABASE_URL: postgresql://${POSTGRES_USER:-postgres}:${POSTGRES_PASSWORD:-your-super-secret-password}@db:5432/${POSTGRES_DB:-buster}
|
test: ["CMD", "curl", "-f", "http://localhost:3001/health"]
|
||||||
|
interval: 30s
|
||||||
|
timeout: 10s
|
||||||
|
retries: 3
|
||||||
depends_on:
|
depends_on:
|
||||||
- db
|
redis:
|
||||||
|
condition: service_healthy
|
||||||
|
db:
|
||||||
|
condition: service_healthy
|
||||||
|
|
||||||
web:
|
web:
|
||||||
build: ./web
|
build: ./web
|
||||||
|
env_file:
|
||||||
|
- .env
|
||||||
ports:
|
ports:
|
||||||
- "3000:3000"
|
- "3000:3000"
|
||||||
environment:
|
environment:
|
||||||
|
|
|
@ -0,0 +1,15 @@
|
||||||
|
#!/bin/bash
|
||||||
|
|
||||||
|
echo "Starting Supabase..."
|
||||||
|
cd supabase
|
||||||
|
docker compose up -d
|
||||||
|
|
||||||
|
echo "Waiting for Supabase to be healthy..."
|
||||||
|
until curl -s http://localhost:54321/rest/v1/ > /dev/null; do
|
||||||
|
echo "Waiting for Supabase..."
|
||||||
|
sleep 5
|
||||||
|
done
|
||||||
|
|
||||||
|
echo "Supabase is ready! Starting main services..."
|
||||||
|
cd ..
|
||||||
|
docker compose up
|
|
@ -6,11 +6,8 @@ WORKDIR /app
|
||||||
# Copy package files
|
# Copy package files
|
||||||
COPY package*.json ./
|
COPY package*.json ./
|
||||||
|
|
||||||
# Copy environment variables for build
|
|
||||||
COPY .env ./
|
|
||||||
COPY next.config.mjs ./
|
COPY next.config.mjs ./
|
||||||
|
|
||||||
|
|
||||||
# Install dependencies
|
# Install dependencies
|
||||||
RUN npm ci
|
RUN npm ci
|
||||||
|
|
||||||
|
@ -34,8 +31,6 @@ COPY --from=builder /app/.next ./.next
|
||||||
COPY --from=builder /app/node_modules ./node_modules
|
COPY --from=builder /app/node_modules ./node_modules
|
||||||
COPY --from=builder /app/package.json ./package.json
|
COPY --from=builder /app/package.json ./package.json
|
||||||
COPY --from=builder /app/next.config.mjs ./
|
COPY --from=builder /app/next.config.mjs ./
|
||||||
# Copy environment files
|
|
||||||
COPY --from=builder /app/.env ./
|
|
||||||
|
|
||||||
# Expose the port your app runs on
|
# Expose the port your app runs on
|
||||||
EXPOSE 3000
|
EXPOSE 3000
|
||||||
|
|
Loading…
Reference in New Issue