Use ENV variables about semaphore HOT 6 CLOSED

Fair point, and shouldn't be too hard..
It will probably be done at the same time as the UI upgrade (feature/ui-work branch).

from semaphore.

👍

from semaphore.

IMO, this is only partly done. The default values for the environment variables only come into play in the presence of the config json file you are trying to remove. Default env vars should be present in the Dockerfile as an indicator for users who wish to override them.

For example

ENV REDIS_HOST 127.0.0.1
ENV REDIS_PORT 6379

from semaphore.

@bibby Check out https://github.com/ansible-semaphore/semaphore/blob/master/lib/config.js#L7

The program is runnable w/o configuration files.

from semaphore.

The line you have linked tests for the absence of environment variables. The defaults are then loaded from the configuration file. So in that way, no, it cannot run w/o config files.

Using ENV in the dockerfile with a default value means the required value is never absent, being either a user defined value or the predefined default.

from semaphore.

1. Tries to load configuration file
2. If not present, tests for some required env variables. If they're not present, the program cannot run
3. Load default configuration
4. (line 17-21) Overrides configuration from environment

from semaphore.