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

compose and deploy are two separate commands #30

Open
rabbah opened this issue Mar 13, 2019 · 2 comments
Open

compose and deploy are two separate commands #30

rabbah opened this issue Mar 13, 2019 · 2 comments
Labels
enhancement New feature or request

Comments

@rabbah
Copy link
Member

rabbah commented Mar 13, 2019

compose --deploy in the past allowed one to conveniently lower the composition and deploy it in one command. Now there are two binaries compose and deploy.

what's the rationale for separating them completely (that is, why remove --deploy from compose which could delegate to deploy)?

@tardieu
Copy link
Contributor

tardieu commented Mar 27, 2019

Once upon a time, we wanted to separate the part of composer that depends on a connection to an OpenWhisk deployment from the part that does not, hence the split between the two commands. There was also a theory about integration with the OpenWhisk CLI.

I agree that from a user perspective a single command or CLI integration would make more sense. On my long-term todo list...

@tardieu tardieu added the enhancement New feature or request label Mar 27, 2019
@dgrove-oss
Copy link
Member

ran into this when update travis2slack to composer 0.11.0. Not having a single command makes the scripting slightly more annoying since one has to deal with a tmp file just to get the output from compose as input to deploy.

Another alternative would be to support getting the json input to deploy from stdin. Then something like: compose foo.js | deploy - foo could work.

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

No branches or pull requests

3 participants