Skip to content

fix(compose): preserve raw remote mounts#4795

Open
yusoofsh wants to merge 1 commit into
Dokploy:canaryfrom
yusoofsh:fix/4794-preserve-compose-mounts
Open

fix(compose): preserve raw remote mounts#4795
yusoofsh wants to merge 1 commit into
Dokploy:canaryfrom
yusoofsh:fix/4794-preserve-compose-mounts

Conversation

@yusoofsh

Copy link
Copy Markdown
Contributor

Summary

  • use the saved raw compose source as the conversion source of truth
  • preserve tmpfs declarations and long-syntax named-volume mounts
  • add regression coverage for raw stack compose on remote servers

Fixes #4794

Verification

  • biome check packages/server/src/utils/docker/domain.ts apps/dokploy/__test__/compose/domain/raw-compose.test.ts (passes with one pre-existing @ts-ignore warning in domain.ts)
  • git diff --check
  • focused Vitest execution could not run locally because the dependency installer stopped at ERR_NUB_TRUST_DOWNGRADE for @octokit/webhooks@12.3.2; no trust policy was weakened

Use the saved raw compose source during conversion instead of reading a potentially stale remote materialization. Preserve tmpfs declarations and long-syntax volume mounts through preview and stack deployment.

Fixes Dokploy#4794

Co-Authored-By: Codex <noreply@openai.com>
@yusoofsh yusoofsh requested a review from Siumauricio as a code owner July 11, 2026 06:03
@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jul 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Raw stack compose drops tmpfs and additional long-syntax mounts on remote server

1 participant