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

Support env_file #44

Closed
dhulihan opened this issue May 23, 2016 · 3 comments
Closed

Support env_file #44

dhulihan opened this issue May 23, 2016 · 3 comments

Comments

@dhulihan
Copy link

This would be a killer feature for container-transform.

@dhulihan
Copy link
Author

dhulihan commented May 30, 2016

Nice! Good progress, but PR #50 omits env-file from ECS when provided (since ECS doesn't support env-file in task definitions, totally understandable). What do you think about reading in env-file and converting line entries to environment members?

@micahhausler
Copy link
Owner

I'd rather not enable unsupported features, add as few surprises as possible, and keep the tool simple to maintain and understand.

While it is a manual step, you could still script the dumping an env file to your compose:

cat <<EOF >app.env
TERM=xterm
CT_INPUT_TYPE=compose
CT_OUTPUT_TYPE=ecs
EOF
cat app.env | sed -e 's/=/: /g' >> docker-compose.yml

@dhulihan
Copy link
Author

No worries, thanks for the consideration and snippet.

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

No branches or pull requests

2 participants