Skip to content
This repository has been archived by the owner on Sep 25, 2020. It is now read-only.

Load config parameters from environment variables via cc.env() #33

Open
lxe opened this issue Aug 6, 2015 · 4 comments
Open

Load config parameters from environment variables via cc.env() #33

lxe opened this issue Aug 6, 2015 · 4 comments

Comments

@lxe
Copy link
Contributor

lxe commented Aug 6, 2015

Add ability to load config from environment variables, like in https://github.com/dominictarr/config-chain example.

@Raynos
Copy link
Contributor

Raynos commented Aug 6, 2015

This is avoided by design. favor command line flags over environment variables.

@lxe
Copy link
Contributor Author

lxe commented Aug 6, 2015

Also CC's env loader doesn't do nested config... Also why do we favor command line parameters over environment variables?

@lxe
Copy link
Contributor Author

lxe commented Aug 27, 2015

@Raynos the cons...

hard to use correctly because they're visible (delete-able, set-able) from anywhere. If I install a new program that relies on environmental variables, are they going to stomp on my existing ones? Did I inadvertently screw up my environmental variables when I was monkeying around yesterday?

...disappear completely when you use docker. In fact I think all runtime config can be replaced with env variables and passed into the docker containers by the orchestrator.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants