From dev-return-2950-archive-asf-public=cust-asf.ponee.io@openwhisk.apache.org Fri Dec 14 21:49:28 2018 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx-eu-01.ponee.io (Postfix) with SMTP id F2659180645 for ; Fri, 14 Dec 2018 21:49:27 +0100 (CET) Received: (qmail 20075 invoked by uid 500); 14 Dec 2018 20:49:27 -0000 Mailing-List: contact dev-help@openwhisk.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@openwhisk.apache.org Delivered-To: mailing list dev@openwhisk.apache.org Received: (qmail 20064 invoked by uid 99); 14 Dec 2018 20:49:27 -0000 Received: from ui-eu-01.ponee.io (HELO localhost) (176.9.59.70) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 14 Dec 2018 20:49:27 +0000 x-ponymail-agent: PonyMail Composer/0.3 Content-Type: text/plain; charset=utf-8 References: From: Matt Rutkowski X-Mailer: LuaSocket 3.0-rc1 To: In-Reply-To: Subject: Re: PR Submitted to Integrate wskdeploy into wsk CLI MIME-Version: 1.0 Date: Fri, 14 Dec 2018 20:49:25 -0000 x-ponymail-sender: 0fa86a36c3825c8358b8c23bd6394831c78e1a14 Message-ID: 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 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 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 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 > >