Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docker compose override #106

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Jul 15, 2023

Closes #105

Joomlatools server certainly seems to provide for override files:
https://github.com/joomlatools/joomlatools-server/blob/master/.gitignore#L9

And the Joomlatools Control also can look for these files if provided in the correct format:
https://github.com/joomlatools/joomlatools-server/blob/master/.gitignore#L9
https://github.com/joomlatools/joomlatools-server/blob/master/bin/jtctl#L149-L158

Can't for the life of me, with the current code how to provide the list of docker compose files, that would be exploded into an array here:
https://github.com/joomlatools/joomlatools-server/blob/master/bin/jtctl#L149

So have modified for testing.

Testing

  1. `mv docker-compose.override-sample.yml docker-compose.override.yml
  2. Modify it to extend the base configuration
  3. export SERVER_FILES="docker-compose.yml docker-compose.override.yml"
  4. sh bin/jtctl start
  5. Double check the configuration via docker compose config

yiendos added 3 commits July 15, 2023 11:33
- could work out how the string would be split into an array
- modified the creation of the array so that spaces would be delimeter instead
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.

docker compose override
1 participant