Fix docker development setup after switch to puma #8396
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
single_process_mode
, which means it now requires a redis instance for development setups. This wasn't the case yet in our docker setup, so it failed to start because it didn't find redis. I added a redis container to our docker-compose setup.listen
configuration for development to only listen on localhost. While this is a good default for normal development setups to not expose a port, it also broke our docker setup because the port now couldn't be forwarded outside of the docker container anymore. So I change this back to the old default when used with docker.docker-compose
command outside of thediaspora-dev
script, so I'm able to run normaldocker-compose
commands directly without needing to manually set the correct environment variable that are required by ourdocker-compose.yml
.For the configurations for
redis
andlisten
, I overwrite the values set in thediaspora.toml
as environment variables in thedocker-compose.yml
, as otherwise everybody would need to ensure that the configure the correct settings themselves in theirdiaspora.toml
. That way it ensures that you can't break the docker-setup with a wrong setting, but can still set different settings in thediaspora.toml
when running without docker.