Mailing-List: contact dev-help@openwhisk.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@openwhisk.apache.org
MIME-Version: 1.0
References: <0900B1C6-1DE9-4B6F-ACF2-DB8EC524D78D@ge.com> <OFADE16082.A2AED42F-ON0025814C.0064B684@LocalDomain>
 <OF54B0FB47.02DAC4A4-ON0025814E.00101F9A-4825814E.001115E9@notes.na.collabserv.com>
 <CAHYtTq=KqLrFHsWAfr+gXvXDfpa3tVqSvfJ16=wCZDeXpq3OrA@mail.gmail.com> <CAHYtTqk7OX-XC6vDK95by+MVkR+qYVWmBVMUZYQLH7v5fUo_Yw@mail.gmail.com>
In-Reply-To: <CAHYtTqk7OX-XC6vDK95by+MVkR+qYVWmBVMUZYQLH7v5fUo_Yw@mail.gmail.com>
From: Carlos Santana <csantana23@gmail.com>
Date: Wed, 22 Nov 2017 04:00:05 +0000
Message-ID: <CAHYtTqkfs2cBiup1V3dB_9nc27EvC6uTPAZ2RvzRnCLd5=e-gA@mail.gmail.com>
Subject: Re: OpenWhisk CLI separation from openwhisk repo: status and help
To: dev@openwhisk.apache.org
Content-Type: multipart/alternative; boundary="001a114d881874c61e055e8a5b60"
archived-at: Wed, 22 Nov 2017 04:00:23 -0000

--001a114d881874c61e055e8a5b60
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

As promised the go CLI source code and all issues have being moved to the
openwhisk-cli repo https://github.com/apache/incubator-openwhisk-cl
<https://github.com/apache/incubator-openwhisk-cli>i


Main repo:
merged https://github.com/apache/incubator-openwhisk/pull/2979

Dependencies:
merged https://github.com/apache/incubator-openwhisk-runtime-swift/pull/7
merged https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/12
merged https://github.com/apache/incubator-openwhisk-runtime-java/pull/4
merged https://github.com/apache/incubator-openwhisk-runtime-python/pull/2
merged https://github.com/apache/incubator-openwhisk-runtime-docker/pull/1
merged https://github.com/apache/incubator-openwhisk-runtime-php/pull/1
merged https://github.com/apache/incubator-openwhisk-apigateway/pull/263

CLI issue tracker: https://github.com/apache/incubator-openwhisk-cli/issues


Now the fun part, if you or know a friend that knows goLang or want to
learn goLang we have 51 open issues for the CLI !!!

We need help, send them to the issue tracker.

One thing we would do is that for new code, bug fixes, that the test cases
are written in goLang instead of Scala.
And of course if you want to port some scala tests to goLang I know some go
guys would have a smile.

Thanks again Vincent for the work on getting this done.


--Carlos


On Fri, Nov 17, 2017 at 2:58 PM Carlos Santana <csantana23@gmail.com> wrote=
:

>
> Dear OpenWhiskers
>
> Want to first congratulate on Vincent on taking on the this major task on
> separating the CLI from the openwhisk core repo, which required many
> changes and a test framework based on REST API to replace the test cases
> that were using the CLI.
>
> During the last 5 months Vincent have also being manually syncing the CLI
> code and related scala tests cases to the openwhiks-cli [1] and the
> openwhisk-client-go [2] repositories.
>
> Now what's left is to remove the CLI source code from the main repo,
> Vincent have open PR #29179 [3], and I will merge the PR by silent
> consensus if there are no objections in 72 hours, we'll merge the PR #297=
9.
>
> After the merge I will start moving the CLI open issues from the core rep=
o
> to the openwhisk-cli repo [1].
>
> We are also updating the runtimes and apigateway repos to adjust to the
> change.
>
> [1] https://github.com/apache/incubator-openwhisk-cli
> [2] https://github.com/apache/incubator-openwhisk-client-go
> [3] https://github.com/apache/incubator-openwhisk/pull/2979
>
>
> On Fri, Jun 30, 2017 at 7:29 PM Carlos Santana <csantana23@gmail.com>
> wrote:
>
>> This is great progress on CLI, Vincent
>>
>> It's very important to do it in this way to keep quality on both backend
>> API and CLI
>>
>> It's going be a bit annoying to keep duplicates code and tests between
>> repos
>>
>> But it's key to have tests on the REST API independent of the client
>> side, we have a few client side libraries for the API golang, JavaScript=
,
>> python, etc.
>>
>> We need the help from folks on best practices and implementation for the
>> two go packages base library and CLI/cobra
>>
>> -- Carlos
>>
>> On Wed, Jun 28, 2017 at 10:58 PM David ZL Liu <david.liu@cn.ibm.com>
>> wrote:
>>
>>> Hi Vincent,
>>>
>>> Very great job you have done, amazing!
>>>
>>> yes, please count me one and don't hesitate to assign me issues to
>>> investigate if you think I can help. It's my pleasure.
>>>
>>>
>>> Kind Regards,
>>> dliu
>>>
>>>
>>>
>>>
>>> From:   "Vincent S Hou" <shou@us.ibm.com>
>>> To:     dev@openwhisk.apache.org
>>> Date:   2017-06-28 =E4=B8=8A=E5=8D=88 02:20
>>> Subject:        OpenWhisk CLI separation from openwhisk repo: status an=
d
>>> help
>>>
>>>
>>>
>>>
>>> Hi openwhiskers,
>>>
>>> OpenWhisk CLI is currently on the stage to separate from the main
>>> openwhisk
>>> repo for better modularity. We have already established two new
>>> repositories
>>> to respectively host the the Go client code and the Go CLI code:
>>>
>>>    OpenWhisk Client Go:
>>>    https://github.com/apache/incubator-openwhisk-client-go, originating
>>>    from OpenWhisk Go Whisk(
>>>
>>>
>>> https://github.com/apache/incubator-openwhisk/tree/master/tools/cli/go-=
whisk
>>>
>>>    )
>>>    OpenWhisk CLI: https://github.com/apache/incubator-openwhisk-cli,
>>>    originating from OpenWhisk Go Whisk CLI(
>>>
>>>
>>> https://github.com/apache/incubator-openwhisk/tree/master/tools/cli/go-=
whisk-cli
>>>
>>>    )
>>>
>>> Current status:
>>> All the source code has been copied and adjusted in the new repos. Scal=
a
>>> tests related to CLI have been moved into cli repo. Travis CIs for both
>>> repos are running
>>> the test cases in Scala to make sure the code works. The source code of
>>> each repo has already synchronized with openwhisk repo, and is able to =
do
>>> further sync-up. However, there is no integration test in openwhisk
>>> client
>>> go, and only a few integration tests implemented in Go for openwhisk cl=
i.
>>>
>>> Most urgent work items to do next:
>>>
>>>    OpenWhisk Client Go: Unit tests for all the Go files, and Integratio=
n
>>>    tests against openwhisk service in Go.
>>>    Issues opened:
>>> https://github.com/apache/incubator-openwhisk-client-go/issues/17,
>>> https://github.com/apache/incubator-openwhisk-client-go/issues/14
>>>
>>>    OpenWhisk Cli: Integration tests against openwhisk service in Go.
>>> (Unit
>>>    tests remain to be discussed, since this repo is implemented in cobr=
a
>>>    package and calls openwhisk client go.)
>>>    Issues opened:
>>> https://github.com/apache/incubator-openwhisk-cli/issues/17
>>>
>>>    OpenWhisk: We need to reimplement the Scala tests by calling the RES=
T
>>>    APIs instead of calling the binaries, which means the basic Wsk clas=
s
>>>    needs to be rewritten:
>>>
>>> https://github.com/apache/incubator-openwhisk/blob/master/tests/src/tes=
t/scala/common/Wsk.scala
>>>
>>> , and invoked by all the other tests when accessing the openwhisk
>>> services.
>>> The current Wsk.scala is calling the CLI binary, and we need to call th=
e
>>> rest api to order to departure the CLI.
>>> Issues opened:
>>> https://github.com/apache/incubator-openwhisk/issues/2430
>>>
>>> We are moving forward in this direction now. Anyone interested in the
>>> contribution to this piece of work? Welcome to reach out to me.
>>>
>>>
>>> Best wishes.
>>> Vincent Hou
>>>
>>>
>>>
>>>
>>>

--001a114d881874c61e055e8a5b60--