environment variables are ignored by default in `scripts/fedbiomed_environment`
In the script fedbiomed_environment
we provide the option to set
UPLOADS_URL
MQTT_BROKER
MQTT_BROKER_PORT
through environment variables. This is very nice, and is common practice.
However, these variables are ignored by default unless the user also sets the undocumented FEDBIOMED_NO_RESET
variable.
I personally found this confusing and it took me a long time to identify the problem. I would prefer the following policy:
- environment variables take priority over any other configuration
- we optionally allow a
FEDBIOMED_FORCE_RESET
env variable to force resetting the environment (hence unsetting those env variables) at the beginning of the script execution.