feat(docker): verify full Docker platform stack - #9
Open
RXVEN-1907 wants to merge 2 commits into
Open
Conversation
- Fix docker-compose.yml: remove conflicting replicas, add health checks for all services - Create missing docker config files: prometheus.yml, grafana-datasources.yml, grafana-dashboards.yml, init-db.sql - Add openre-api binary with main.rs for server startup and database migrations - Add worker command to CLI (openre worker start) with job handler support - Add AnalysisJobHandler in openre-api for processing analysis jobs - Fix compilation errors: add tracing-subscriber, fix Config::load(), error handling - All binaries build successfully, tests pass for modified crates
…fix error handling - Fix worker/worker-ai healthchecks: init_telemetry now starts Prometheus HTTP server on port 9090 - Improve AnalysisJobHandler: avoid OOM by streaming file check, add file size lookup, add proper TODO for analysis pipeline - Fix error handling: add From<String> for CliError, preserve ApiError type through From impl - Add get_size method to ObjectStore for file size queries - Fix Telemetry::new() to be async and call init_telemetry properly - Use state.telemetry.metrics for worker metrics instead of creating separate registry
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR verifies the full Docker platform stack comes up correctly by fixing all Docker-related issues.
Changes
Docker Compose
docker-compose.yml: removed conflictingreplicas: 2from worker service (conflicted withcontainer_name)curl http://localhost:8080/healthcurl http://localhost:9090/metricscurl http://localhost/healthwget --spider http://localhost:9090/-/healthycurl http://localhost:3000/api/healthNew Docker Configuration Files
docker/prometheus.yml- Prometheus scrape config for all servicesdocker/grafana-datasources.yml- Grafana Prometheus datasourcedocker/grafana-dashboards.yml- Grafana dashboard provisioningdocker/init-db.sql- PostgreSQL initialization scriptAPI Binary
[[bin]]section tocrates/openre-api/Cargo.tomlcrates/openre-api/src/main.rswith:Worker Support
workercommand to CLI (openre worker start)crates/openre-cli/src/commands/worker.rsAnalysisJobHandlerincrates/openre-api/src/workers.rsfor processing analysis jobsCompilation Fixes
tracing-subscriberdependency to openre-apiConfig::load()usage (no path argument needed)CliErrorvariantstracingdependency to openre-cliVerification
openre-api,openre(CLI)Test Endpoints (after docker compose up)
curl http://localhost:8080/healthcurl http://localhost:3000curl http://localhost:9090/-/healthycurl http://localhost:3001/api/health