Skip to content

Switch to docker compose ps for container validation in integration tests #1644

@coderabbitai

Description

@coderabbitai

Description

Currently, the integration test utilities in integration-tests/tests/utils/docker_utils.py use docker ps with name prefix matching to validate running containers (see list_running_containers_with_prefix()).

Proposed Enhancement

Switch to using docker compose ps with the project name specified. This approach would provide:

  1. Exact container list: Query the precise set of containers managed by the docker compose project, eliminating false positives from containers with similar name patterns
  2. Exit status validation: Check for containers that should have exited at certain stages of the package run, enabling more comprehensive lifecycle validation
  3. Better integration: Align with docker compose's native container management and project scoping

Context

Backlinks:

Metadata

Metadata

Labels

No labels
No labels

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions