feat(server): align Dockerfile entrypoint with spec (python -m cleveragents serve) #1088

Closed
opened 2026-03-21 04:13:43 +00:00 by brent.edwards · 5 comments
Member

Background

The specification mandates python -m cleveragents as the server entrypoint.
Currently Dockerfile.server uses python -m uvicorn directly because the
cleveragents serve CLI subcommand does not yet exist.

A TODO comment in Dockerfile.server documents this deviation.

Acceptance Criteria

  • Implement cleveragents serve CLI subcommand that starts the ASGI server
  • Update Dockerfile.server ENTRYPOINT to use python -m cleveragents serve
  • Update k8s/templates/deployment.yaml command/args to use the new entrypoint
  • Remove the TODO comment from Dockerfile.server

Dependencies

  • Depends on #928 (Kubernetes/Helm deployment configuration)

Metadata

  • Suggested commit message: feat(server): implement serve CLI subcommand and align Dockerfile entrypoint
  • Suggested branch name: feat/server-serve-entrypoint

Definition of Done

Dockerfile.server uses python -m cleveragents serve and all existing tests pass.

## Background The specification mandates `python -m cleveragents` as the server entrypoint. Currently `Dockerfile.server` uses `python -m uvicorn` directly because the `cleveragents serve` CLI subcommand does not yet exist. A TODO comment in `Dockerfile.server` documents this deviation. ## Acceptance Criteria - [ ] Implement `cleveragents serve` CLI subcommand that starts the ASGI server - [ ] Update `Dockerfile.server` ENTRYPOINT to use `python -m cleveragents serve` - [ ] Update `k8s/templates/deployment.yaml` command/args to use the new entrypoint - [ ] Remove the TODO comment from `Dockerfile.server` ## Dependencies - Depends on #928 (Kubernetes/Helm deployment configuration) ## Metadata - **Suggested commit message:** `feat(server): implement serve CLI subcommand and align Dockerfile entrypoint` - **Suggested branch name:** `feat/server-serve-entrypoint` ## Definition of Done `Dockerfile.server` uses `python -m cleveragents serve` and all existing tests pass.
freemo added this to the v3.6.0 milestone 2026-03-22 16:35:03 +00:00
Owner

Triage

Milestone: v3.6.0
Assignee: @brent.edwards
Labels: Type/Task, Priority/Medium, State/Unverified, Points/2, MoSCoW/Should have

Rationale: Aligning the Dockerfile entrypoint with the spec (python -m cleveragents serve) is important for deployment consistency and Helm chart compatibility. Placed in v3.6.0 since it is not blocking current development but should be resolved before broader deployment efforts. Estimated at 2 points — a small, focused Dockerfile change plus validation that the container starts correctly. Assigned to @brent.edwards who owns the infrastructure/DevOps work.

## Triage **Milestone:** v3.6.0 **Assignee:** @brent.edwards **Labels:** Type/Task, Priority/Medium, State/Unverified, Points/2, MoSCoW/Should have **Rationale:** Aligning the Dockerfile entrypoint with the spec (`python -m cleveragents serve`) is important for deployment consistency and Helm chart compatibility. Placed in v3.6.0 since it is not blocking current development but should be resolved before broader deployment efforts. Estimated at 2 points — a small, focused Dockerfile change plus validation that the container starts correctly. Assigned to @brent.edwards who owns the infrastructure/DevOps work.
Owner

Planning Agent — Discussion Review

Triage confirmed. v3.6.0, Priority/Medium, 2 points, assigned to @brent.edwards.

The spec mandates python -m cleveragents serve as the container entrypoint. This is a small, focused change that should be validated with a container start test. Placement in v3.6.0 is appropriate — not blocking current development but needed before broader deployment.

Note: Like #1089, this was extracted from the Helm deployment work (#928). Correct approach per CONTRIBUTING.md §Atomic Commits — infrastructure changes should be separate from deployment configuration.

No disputes.

## Planning Agent — Discussion Review Triage confirmed. v3.6.0, Priority/Medium, 2 points, assigned to @brent.edwards. The spec mandates `python -m cleveragents serve` as the container entrypoint. This is a small, focused change that should be validated with a container start test. Placement in v3.6.0 is appropriate — not blocking current development but needed before broader deployment. Note: Like #1089, this was extracted from the Helm deployment work (#928). Correct approach per CONTRIBUTING.md §Atomic Commits — infrastructure changes should be separate from deployment configuration. No disputes.
freemo self-assigned this 2026-04-02 06:13:58 +00:00
Owner

PR #1272 reviewed, approved, and merged.

The PR adds BDD test coverage verifying:

  • Dockerfile.server ENTRYPOINT uses python -m cleveragents (spec-mandated entrypoint)
  • Dockerfile.server CMD includes server serve subcommand tokens

The underlying implementation (server serve CLI command, Dockerfile ENTRYPOINT/CMD, and k8s deployment.yaml) was already in place from prior commits. This PR formalizes the verification with Behave scenarios.

PR #1272 reviewed, approved, and merged. The PR adds BDD test coverage verifying: - `Dockerfile.server` ENTRYPOINT uses `python -m cleveragents` (spec-mandated entrypoint) - `Dockerfile.server` CMD includes `server serve` subcommand tokens The underlying implementation (`server serve` CLI command, Dockerfile ENTRYPOINT/CMD, and k8s deployment.yaml) was already in place from prior commits. This PR formalizes the verification with Behave scenarios.
Owner

[Backlog Groomer - groomer-1] 📋 Label state mismatch. This issue has State/Unverified but PR #1272 (feat(server): implement serve CLI subcommand and align Dockerfile entrypoint) is open and references this issue. The state should be updated to State/In Review.

**[Backlog Groomer - groomer-1]** 📋 **Label state mismatch.** This issue has `State/Unverified` but PR #1272 (`feat(server): implement serve CLI subcommand and align Dockerfile entrypoint`) is open and references this issue. The state should be updated to `State/In Review`.
Owner

PR #1272 reviewed, approved, and merged.

The PR added BDD test coverage verifying:

  • Dockerfile.server ENTRYPOINT uses python -m cleveragents (spec-mandated form)
  • Dockerfile.server CMD includes server serve subcommand tokens

The server serve CLI subcommand, Dockerfile entrypoint, and k8s deployment configuration were already in place from prior work. This PR completed the acceptance criteria by adding formal test verification.

PR #1272 reviewed, approved, and merged. The PR added BDD test coverage verifying: - `Dockerfile.server` ENTRYPOINT uses `python -m cleveragents` (spec-mandated form) - `Dockerfile.server` CMD includes `server serve` subcommand tokens The `server serve` CLI subcommand, Dockerfile entrypoint, and k8s deployment configuration were already in place from prior work. This PR completed the acceptance criteria by adding formal test verification.
Sign in to join this conversation.
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
cleveragents/cleveragents-core#1088
No description provided.