Skip to content

Evaluator Scheduler - Automated Deployment from GitHub#827

Open
JersyJ wants to merge 3 commits intomrlvsb:masterfrom
JersyJ:evaluator-scheduler-automated-deployment
Open

Evaluator Scheduler - Automated Deployment from GitHub#827
JersyJ wants to merge 3 commits intomrlvsb:masterfrom
JersyJ:evaluator-scheduler-automated-deployment

Conversation

@JersyJ
Copy link
Contributor

@JersyJ JersyJ commented Feb 9, 2026

  • Add Docker HEALTHCHECK-based health check fallback when healthcheck_url is not provided, and introduce configurable health check timeout (global + per-request override).
  • Extend deployment_service/deploy.py to accept --health-check-timeout and include service_name in the GitHub Job Summary titles.
  • Update docs and CI workflow to describe/use the new health-check behavior and deploy an additional evaluator_scheduler service.

Copilot AI review requested due to automatic review settings February 9, 2026 15:48
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the deployment service and GitHub Actions workflow to support automated deployments with improved health-check behavior (HTTP or Docker HEALTHCHECK) and configurable timeouts, plus clearer GitHub job summaries per service.

Changes:

  • Add Docker HEALTHCHECK-based health check fallback when healthcheck_url is not provided, and introduce configurable health check timeout (global + per-request override).
  • Extend deployment_service/deploy.py to accept --health-check-timeout and include service_name in the GitHub Job Summary titles.
  • Update docs and CI workflow to describe/use the new health-check behavior and deploy an additional evaluator_scheduler service.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
docs/docs/02-developers-guide/02-deployment.mdx Documents Docker-healthcheck fallback and timeout override behavior.
deployment_service/tests/test_deployment.py Updates health-check assertions and adds Docker-healthcheck test cases.
deployment_service/deploy.py Adds --health-check-timeout, passes it to server, and improves summary titles with service name.
deployment_service/app/models.py Makes healthcheck_url optional and adds health_check_timeout to the request schema.
deployment_service/app/main.py Wires optional healthcheck_url and timeout into DeploymentManager.
deployment_service/app/deployment.py Implements HTTP vs Docker health-check paths and uses configurable timeout.
deployment_service/app/config.py Adds global health_check_timeout setting.
.github/workflows/ci.yml Adds evaluator scheduler deployment invocation in CI deploy job.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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