Skip to content
This repository was archived by the owner on Apr 24, 2026. It is now read-only.

fix: add restart: unless-stopped to vtm containers#3

Merged
jaewilson07 merged 1 commit into
mainfrom
fix/vtm-restart-policy
Apr 20, 2026
Merged

fix: add restart: unless-stopped to vtm containers#3
jaewilson07 merged 1 commit into
mainfrom
fix/vtm-restart-policy

Conversation

@jaewilson07

Copy link
Copy Markdown
Owner

Summary

  • Add restart: unless-stopped to both vtm-api and ollama services
  • Containers now recover automatically after crashes or Docker restarts
  • Pairs with Docker Desktop "Start on login" setting for always-on local GPU access

Test plan

  • docker compose up -d — confirm both containers start
  • docker restart vtm-api — confirm it comes back up automatically
  • Reboot machine — confirm containers restart with Docker Desktop

🤖 Generated with Claude Code

Ensures containers recover automatically after crashes or Docker
restarts, supporting Docker Desktop auto-start on Windows login.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 20, 2026 05:05
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds Docker Compose restart policies so the local VTM stack is more resilient to crashes and Docker restarts.

Changes:

  • Set restart: unless-stopped for the ollama service.
  • Set restart: unless-stopped for the api service.

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

@jaewilson07
jaewilson07 merged commit 2e7b1b3 into main Apr 20, 2026
6 checks passed
@jaewilson07
jaewilson07 deleted the fix/vtm-restart-policy branch April 20, 2026 19:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants