Skip to content
This repository has been archived by the owner on Feb 14, 2018. It is now read-only.

Document environment variable configuration in manifest #22

Open
gmr opened this issue Nov 3, 2014 · 2 comments
Open

Document environment variable configuration in manifest #22

gmr opened this issue Nov 3, 2014 · 2 comments

Comments

@gmr
Copy link
Contributor

gmr commented Nov 3, 2014

Pretty sure it's a list of strings:

["FOO=BAR", "BAZ=QUX"]

Double check and add example to documents.

@gmr gmr added the enhancement label Nov 3, 2014
@hunt3r
Copy link
Contributor

hunt3r commented Nov 3, 2014

We tried

env:
     - DEPLOY_LOCAL=1

This did not seem to work either, just FYI.

I believe it's related to these lines:

 if container.get 'env'
        config.Env = container.get 'env'
 if not env?
        config.Env = []

env is never assigned and there for will always be falsy, config.Env will always be assigned new array, which explains why we see VM_IPADDR in our environment vars.

The next line successfully assigned the variable by pushing into the list.

config.Env.push "VM_IPADDR=#{network.get 'ip'}"

@hunt3r
Copy link
Contributor

hunt3r commented Nov 3, 2014

PR Incoming.

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

No branches or pull requests

2 participants