openwhisk-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Matt Rutkowski <mrutkow...@apache.org>
Subject Re: PR Submitted to Integrate wskdeploy into wsk CLI
Date Fri, 14 Dec 2018 20:49:25 GMT
James,

Yes, we will need to update "https://github.com/apache/incubator-openwhisk-cli/blob/master/build.gradle"
which has the Go lang. deps. with the correct hash we want to pickup from the wskdeploy repo.
treating it as any other package deps.

-Matt

On 2018/12/14 14:24:24, Carlos Santana <csantana23@gmail.com> wrote: 
> +1 Is great to get to the point of consolidating under a single CLI
> 
> 
> On Fri, Dec 14, 2018 at 1:52 AM James Thomas <jthomas.uk@gmail.com> wrote:
> 
> > This is excellent Priti! It will make it much easier for developers to use
> > the tool and increase adoption having it availble through a single CLI.
> >
> > Will you be manually managing updating the `wsk` repo with new `wskdeploy`
> > deps for each release?
> >
> > On Wed, 12 Dec 2018 at 20:55, Priti Desai <pdesai@us.ibm.com> wrote:
> >
> > > Hi,
> > >
> > > I have made an attempt to integrate wskdeploy into wsk CLI by following a
> > > simple source/GitHub dependency approach. I have a PR here, please send
> > me
> > > your comments/feedback:
> > >
> > > https://github.com/apache/incubator-openwhisk-cli/pull/394
> > >
> > > Here is the list of changes and how tos:
> > >
> > > 1. Introducing wsk project subcommand which is equivalent to wskdeploy.
> > > 2. To invoke deployments, run wsk CLI with: wsk project deploy --manifest
> > > manifest.yaml. This is equivalent of running wskdeploy --manifest
> > > manifest.yaml or wskdeploy -m manifest.yaml
> > > 3. Undeploy/remove OpenWhisk assets with CLI; wsk project undeploy
> > > --manifest manifest.yaml. This is equivalent of running wskdeploy
> > undeploy
> > > --manifest manifest.yaml or wskdeploy undeploy -m manifest.yaml
> > > 4. Similary run sync/export with wsk project sync and wsk project export
> > > 5. wsk project deploy | undeploy | sync | export does not support short
> > > hand flags including -m, -d, -p unlike wskdeploy. The reason behind this
> > > deprecation is wsk CLI already has some short hand flags like -d for
> > debug
> > > which can not be overloaded.
> > >
> > > Please let me know if you any questions/comments. I would like to this
> > > feature go live if there is no objection.
> > >
> > > Cheers
> > > Priti
> > >
> > >
> >
> > --
> > Regards,
> > James Thomas
> >
> 
> 
> -- 
> Carlos Santana
> <csantana23@gmail.com>
> 

Mime
View raw message