Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add an environment variable function #30

Closed
andrioni opened this issue Sep 9, 2015 · 5 comments
Closed

Add an environment variable function #30

andrioni opened this issue Sep 9, 2015 · 5 comments
Assignees
Milestone

Comments

@andrioni
Copy link
Contributor

andrioni commented Sep 9, 2015

It'd be nice to be able to support environment variables directly: it helps with the testing and reduces the chances of someone committing credentials :)

@alexanderdean
Copy link
Member

Neat idea!

@alexanderdean alexanderdean changed the title Support environment variables inside playbooks Add an environment variable function Sep 14, 2015
@alexanderdean
Copy link
Member

This is less important given #35, but still nice

@jbeemster
Copy link
Member

The priority here being:

  • Load Env vars into variables map
  • If present in playbook use playbook value
  • If present in -var use that value

Is that right @alexanderdean ?

@alexanderdean
Copy link
Member

Can't we just have a systemEnv("foo") function that checks to see if there is an environment variable available called "foo"?

@jbeemster
Copy link
Member

We good as well yeah! Does seem a bit saner than loading all of the environment variables into the application..

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

No branches or pull requests

3 participants