Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@zilto
Copy link
Collaborator

@zilto zilto commented Sep 12, 2025

dlt uses several containers in its test setup. Most define the policy restart: unless-stopped in their docker-compose.

This means these containers are started when booting the developer's machine and run most of the time in the background. This has a noticeable impact on CPU during intensive usage. The developer is required to manually spin down containers each time via docker container stop $(docker ps -q -a)

Changes

  • set policy restart: "on-failure" which doesn't start containers on boot.
  • remove the version: field; this has been deprecated and ignored for some time
  • set container_name with a dlt_test_ prefix to facilitate container management and prevent spawning duplicate containers

Future steps

  • some docker compose use docker networks and named volumes; those can be harder to cleanup. Would be useful to minimize resource usage of dev machines or provide cleanup scripts

@zilto zilto requested a review from rudolfix September 12, 2025 13:49
@zilto zilto self-assigned this Sep 12, 2025
@netlify
Copy link

netlify bot commented Sep 12, 2025

Deploy Preview for dlt-hub-docs canceled.

Name Link
🔨 Latest commit 279e4a3
🔍 Latest deploy log https://app.netlify.com/projects/dlt-hub-docs/deploys/68c45dd2ea007d000844be91

@zilto zilto requested a review from sh-rp September 14, 2025 21:59
@sh-rp
Copy link
Collaborator

sh-rp commented Sep 15, 2025

@zilto thanks! This PR makes total sense.

@sh-rp sh-rp merged commit b428bfa into devel Sep 15, 2025
67 checks passed
@sh-rp sh-rp deleted the fix/disable-docker-autorestart branch September 15, 2025 10:50
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.

3 participants