Skip to content

feat: add docker compose include directive support#8850

Open
TupiC wants to merge 1 commit intocoollabsio:nextfrom
TupiC:fix/docker-compose-include-support
Open

feat: add docker compose include directive support#8850
TupiC wants to merge 1 commit intocoollabsio:nextfrom
TupiC:fix/docker-compose-include-support

Conversation

@TupiC
Copy link

@TupiC TupiC commented Mar 10, 2026

Changes

  • added support for "include" directive in docker compose files. they get resolved and replaced in the used files
  • made use of git show to read the compose file from the object store: no longer need sparse-checkout, working tree or git version check since git show is available in all git versions
  • i also added some unit tests for correct parsing and resolving

Issues

Category

  • Bug fix
  • Improvement
  • New feature
  • Adding new one click service
  • Fixing or updating existing one click service

Preview

suppose we have the following docker-compose.yml
image

the frontend/docker-compose.yml looks like this:
image

the backend/docker-compose.yml looks like this:
image

the include paths would get resolved and the correct directives get injected in the main docker compose which looks like this (raw):
image

AI Assistance

  • AI was NOT used to create this PR
  • AI was used (please describe below)

If AI was used:

  • Tools used: Cursor
  • How extensively: unit tests + verifying the changes made in this PR

Testing

  1. create a new resource in coolify (select public git repo)
  2. use this git url https://github.com/TupiC/coolify-include-test/ (test repo for this PR)
  3. set correct docker-compose.yml path
  4. verify the include paths get replaced with actual docker compose contents (also for deployable compose)
  5. deploy

also added unit tests for parsing and injecting

Contributor Agreement

Important

  • I have read and understood the contributor guidelines. If I have failed to follow any guideline, I understand that this PR may be closed without review.
  • I have searched existing issues and pull requests (including closed ones) to ensure this isn't a duplicate.
  • I have tested all the changes thoroughly with a local development instance of Coolify and I am confident that they will work as expected when a maintainer tests them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant