[GitHub] daisy-ycguo opened a new issue #417: New output messages from wskdeply

2017-08-29 Thread git
daisy-ycguo opened a new issue #417: New output messages from wskdeply URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/417 Do we really want to print these messages ? Or they are debug messages which should be removed ? ``` $ ./wskdeploy -i -m ~/workspace/owbp

[GitHub] lzbj commented on issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml

2017-08-29 Thread git
lzbj commented on issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/415#issuecomment-325892772 I have already changed the yaml file in my repo to .yaml, did I miss some steps to reprod

[GitHub] lzbj commented on issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml

2017-08-29 Thread git
lzbj commented on issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/415#issuecomment-325892537 @pritidesai , I can't reproduce this issue, could you help confirm. I forked your repo

[GitHub] lzbj commented on issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml

2017-08-29 Thread git
lzbj commented on issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/415#issuecomment-325892537 @pritidesai , I can't reproduce this issue, could you help confirm. I forked your repo

[GitHub] lzbj commented on issue #405: Unit test of TestParseManifestForSingleLineParams failing

2017-08-29 Thread git
lzbj commented on issue #405: Unit test of TestParseManifestForSingleLineParams failing URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/405#issuecomment-325886806 @pritidesai is our UT expert, please could you help take a look @pritidesai ---

[GitHub] lzbj closed issue #416: wskdeploy doesn't work with latest code merges.

2017-08-29 Thread git
lzbj closed issue #416: wskdeploy doesn't work with latest code merges. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/416 This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] lzbj commented on issue #416: wskdeploy doesn't work with latest code merges.

2017-08-29 Thread git
lzbj commented on issue #416: wskdeploy doesn't work with latest code merges. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/416#issuecomment-325884830 ok, got it, I didn't update my cli lib, yes it works now, close it. Thanks a lot @daisy-ycguo . -

[GitHub] cmdhema opened a new issue #2673: How can I restart OpenWhisk?

2017-08-29 Thread git
cmdhema opened a new issue #2673: How can I restart OpenWhisk? URL: https://github.com/apache/incubator-openwhisk/issues/2673 Hello I write codes on controller and invoker and others. And then, I did rebuild and deploy and successfully. After that, I invoke action with wsk cli

[GitHub] daisy-ycguo commented on issue #416: wskdeploy doesn't work with latest code merges.

2017-08-29 Thread git
daisy-ycguo commented on issue #416: wskdeploy doesn't work with latest code merges. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/416#issuecomment-325872468 @lzbj I updated to latest wskclient, and I can build now. I deployed a cloudant-trigger test case, it wor

[GitHub] daisy-ycguo commented on issue #416: wskdeploy doesn't work with latest code merges.

2017-08-29 Thread git
daisy-ycguo commented on issue #416: wskdeploy doesn't work with latest code merges. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/416#issuecomment-325872468 @lzbj I updated to latest wskclient, and I can build now. I deployed a cloudant-trigger test case, it wor

[GitHub] lzbj commented on issue #416: wskdeploy doesn't work with latest code merges.

2017-08-29 Thread git
lzbj commented on issue #416: wskdeploy doesn't work with latest code merges. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/416#issuecomment-325871561 @daisy-ycguo, said she can't build with latest codes. --

[GitHub] lzbj opened a new issue #416: wskdeploy doesn't work with latest code merges.

2017-08-29 Thread git
lzbj opened a new issue #416: wskdeploy doesn't work with latest code merges. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/416 is anyone face the issue. when I run wskdeploy -m /path/to/manifest, I got a error. 2017/08/30 11:44:26 Unmarshal Openwhisk info from in

[GitHub] daisy-ycguo commented on issue #399: Error out when env. variables from manifest/deployment are not defined

2017-08-29 Thread git
daisy-ycguo commented on issue #399: Error out when env. variables from manifest/deployment are not defined URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/399#issuecomment-325864360 I'm going to give warning msg when env variables are not set. Are you fine with it, @p

[GitHub] daisy-ycguo closed issue #414: Reporting warning information when the env variables in yaml files are not set

2017-08-29 Thread git
daisy-ycguo closed issue #414: Reporting warning information when the env variables in yaml files are not set URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/414 This is an automated message from the Apa

[GitHub] daisy-ycguo commented on issue #414: Reporting warning information when the env variables in yaml files are not set

2017-08-29 Thread git
daisy-ycguo commented on issue #414: Reporting warning information when the env variables in yaml files are not set URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/414#issuecomment-325864248 Duplicated with #399. Close this one. ---

[GitHub] pritidesai opened a new issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml

2017-08-29 Thread git
pritidesai opened a new issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/415 wskdeploy does not support dependent packages which has manifest.yaml file. It only deploys dependent packa

[GitHub] daisy-ycguo opened a new issue #414: Reporting warning errors when the environment values in yaml files are not set

2017-08-29 Thread git
daisy-ycguo opened a new issue #414: Reporting warning errors when the environment values in yaml files are not set URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/414 This is an automated message from t

[GitHub] mrutkows commented on issue #393: Deployment.yaml files all use "package" key, spec. has "packages"

2017-08-29 Thread git
mrutkows commented on issue #393: Deployment.yaml files all use "package" key, spec. has "packages" URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/393#issuecomment-325861796 @pritidesai can you please look at changing this globally; see if we can allow (in some deprec

[GitHub] mrutkows commented on issue #398: Concurrent issues when running integration tests accessing the same resources

2017-08-29 Thread git
mrutkows commented on issue #398: Concurrent issues when running integration tests accessing the same resources URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/398#issuecomment-325861245 Vincent mitigated this issue by naming each test (package) with a different name t

[GitHub] daisy-ycguo commented on issue #391: Wskdeploy sometimes tries to create sequence before action

2017-08-29 Thread git
daisy-ycguo commented on issue #391: Wskdeploy sometimes tries to create sequence before action URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/391#issuecomment-325861256 Fixed with PR #396. This is an a

[GitHub] daisy-ycguo closed issue #391: Wskdeploy sometimes tries to create sequence before action

2017-08-29 Thread git
daisy-ycguo closed issue #391: Wskdeploy sometimes tries to create sequence before action URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/391 This is an automated message from the Apache Git Service. To

[GitHub] mrutkows closed issue #398: Concurrent issues when running integration tests accessing the same resources

2017-08-29 Thread git
mrutkows closed issue #398: Concurrent issues when running integration tests accessing the same resources URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/398 This is an automated message from the Apache

[GitHub] mrutkows commented on issue #360: Validate integration tests are running as part of Travis build

2017-08-29 Thread git
mrutkows commented on issue #360: Validate integration tests are running as part of Travis build URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/360#issuecomment-325860875 All integration tests are not running again, we will use individual issues marked high priority t

[GitHub] mrutkows closed issue #360: Validate integration tests are running as part of Travis build

2017-08-29 Thread git
mrutkows closed issue #360: Validate integration tests are running as part of Travis build URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/360 This is an automated message from the Apache Git Service. To

[GitHub] mrutkows commented on issue #305: Wondering why triggers are updated with overwrite set to false

2017-08-29 Thread git
mrutkows commented on issue #305: Wondering why triggers are updated with overwrite set to false URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/305#issuecomment-325860621 Fixed in https://github.com/apache/incubator-openwhisk-wskdeploy/pull/377 --

[GitHub] mrutkows closed issue #305: Wondering why triggers are updated with overwrite set to false

2017-08-29 Thread git
mrutkows closed issue #305: Wondering why triggers are updated with overwrite set to false URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/305 This is an automated message from the Apache Git Service. To

[GitHub] mrutkows commented on issue #306: Implicit Runtime for Java is set to nodejs:default

2017-08-29 Thread git
mrutkows commented on issue #306: Implicit Runtime for Java is set to nodejs:default URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/306#issuecomment-325860434 Fixed in https://github.com/apache/incubator-openwhisk-wskdeploy/pull/366 --

[GitHub] mrutkows closed issue #306: Implicit Runtime for Java is set to nodejs:default

2017-08-29 Thread git
mrutkows closed issue #306: Implicit Runtime for Java is set to nodejs:default URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/306 This is an automated message from the Apache Git Service. To respond to t

[GitHub] mrutkows closed issue #373: Load the credentials the same way as client go does

2017-08-29 Thread git
mrutkows closed issue #373: Load the credentials the same way as client go does URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/373 This is an automated message from the Apache Git Service. To respond to

[GitHub] mrutkows commented on issue #392: Load the credentials the same way as client go does

2017-08-29 Thread git
mrutkows commented on issue #392: Load the credentials the same way as client go does URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/392#issuecomment-325859539 Fixed with https://github.com/apache/incubator-openwhisk-wskdeploy/pull/392 -

[GitHub] mrutkows commented on issue #373: Load the credentials the same way as client go does

2017-08-29 Thread git
mrutkows commented on issue #373: Load the credentials the same way as client go does URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/373#issuecomment-325859624 Fixed with https://github.com/apache/incubator-openwhisk-wskdeploy/pull/392 ---

[GitHub] mrutkows commented on issue #373: Load the credentials the same way as client go does

2017-08-29 Thread git
mrutkows commented on issue #373: Load the credentials the same way as client go does URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/373#issuecomment-325859624 https://github.com/apache/incubator-openwhisk-wskdeploy/pull/392 --

[GitHub] mrutkows commented on issue #392: Load the credentials the same way as client go does

2017-08-29 Thread git
mrutkows commented on issue #392: Load the credentials the same way as client go does URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/392#issuecomment-325859539 Fixed with https://github.com/apache/incubator-openwhisk-wskdeploy/pull/392 -

[GitHub] mrutkows closed pull request #396: Deploy sequences separately other than combining with actions

2017-08-29 Thread git
mrutkows closed pull request #396: Deploy sequences separately other than combining with actions URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/396 This is an automated message from the Apache Git Service

[GitHub] daisy-ycguo opened a new issue #413: Create enough sequence tests to make sure actions are created before sequences

2017-08-29 Thread git
daisy-ycguo opened a new issue #413: Create enough sequence tests to make sure actions are created before sequences URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/413 This is an automated message from t

[GitHub] daisy-ycguo closed issue #411: typo in error message

2017-08-29 Thread git
daisy-ycguo closed issue #411: typo in error message URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/411 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] daisy-ycguo commented on issue #411: typo in error message

2017-08-29 Thread git
daisy-ycguo commented on issue #411: typo in error message URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/411#issuecomment-325856282 Fixed and closed. This is an automated message from the Apache Git Se

[GitHub] mrutkows closed pull request #412: fix some simple typos

2017-08-29 Thread git
mrutkows closed pull request #412: fix some simple typos URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/412 This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] lzbj commented on issue #366: Support more runtime types from bluemix host info.

2017-08-29 Thread git
lzbj commented on issue #366: Support more runtime types from bluemix host info. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/366#issuecomment-325855014 sorry, my bad, I thought i was belong to runtimes, so @pritidesai , please help fix this and I'll merged your PR asa

[GitHub] daisy-ycguo opened a new pull request #412: fix some simple typos

2017-08-29 Thread git
daisy-ycguo opened a new pull request #412: fix some simple typos URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/412 Address issue #411 This is an automated message from the Apache Git Service. To respond t

[GitHub] houshengbo commented on issue #405: Unit test of TestParseManifestForSingleLineParams failing

2017-08-29 Thread git
houshengbo commented on issue #405: Unit test of TestParseManifestForSingleLineParams failing URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/405#issuecomment-325854461 @daisy-ycguo It is valid in mac. I tried ubuntu as well, there is no issue for linux.

[GitHub] daisy-ycguo commented on issue #405: Unit test of TestParseManifestForSingleLineParams failing

2017-08-29 Thread git
daisy-ycguo commented on issue #405: Unit test of TestParseManifestForSingleLineParams failing URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/405#issuecomment-325853751 @lzbj @houshengbo interesting issue. I run under ubuntu and it works fine. ---

[GitHub] houshengbo commented on issue #405: Unit test of TestParseManifestForSingleLineParams failing

2017-08-29 Thread git
houshengbo commented on issue #405: Unit test of TestParseManifestForSingleLineParams failing URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/405#issuecomment-325853514 @lzbj Line 25 and Line 26 put '' and "" as the values. It seems the parser does not read them correc

[GitHub] lzbj commented on issue #405: Unit test of TestParseManifestForSingleLineParams failing

2017-08-29 Thread git
lzbj commented on issue #405: Unit test of TestParseManifestForSingleLineParams failing URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/405#issuecomment-325853042 @houshengbo , correct, so I doubt we hit a yaml lib bug or even a golang bug perhaps, need to dig deeper i

[GitHub] rabbah opened a new issue #2672: Action attachments are not cached and end up repeatedly fetched even if not needed

2017-08-29 Thread git
rabbah opened a new issue #2672: Action attachments are not cached and end up repeatedly fetched even if not needed URL: https://github.com/apache/incubator-openwhisk/issues/2672 Actions which have an inlined `exec` (all but Java actions today) currently get cached in their entirety, but f

[GitHub] lzbj closed pull request #410: Update the README with more instructions of building binaries

2017-08-29 Thread git
lzbj closed pull request #410: Update the README with more instructions of building binaries URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/410 This is an automated message from the Apache Git Service. To

[GitHub] daisy-ycguo commented on issue #396: Deploy sequences separately other than combining with actions

2017-08-29 Thread git
daisy-ycguo commented on issue #396: Deploy sequences separately other than combining with actions URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/396#issuecomment-325851955 @pritidesai If you run it with `wskdeploy -i`, you will see the difference.

[GitHub] ningyougang commented on issue #2662: Add judge condition for invoker deployment

2017-08-29 Thread git
ningyougang commented on issue #2662: Add judge condition for invoker deployment URL: https://github.com/apache/incubator-openwhisk/pull/2662#issuecomment-325849967 @markusthoemmes ,what do you think about this? This is an a

[GitHub] pritidesai opened a new issue #411: typo in error message

2017-08-29 Thread git
pritidesai opened a new issue #411: typo in error message URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/411 ``` log.Println("the runtime it not supported by Openwhisk platform.") ``` should be ``` log.Println("the runtime is not supported by Open

[GitHub] pritidesai commented on issue #396: Deploy sequences separately other than combining with actions

2017-08-29 Thread git
pritidesai commented on issue #396: Deploy sequences separately other than combining with actions URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/396#issuecomment-325817448 @daisy-ycguo we have to make sure that wskdeploy deploys all actions prior to deploying sequences.

[GitHub] pritidesai commented on issue #396: Deploy sequences separately other than combining with actions

2017-08-29 Thread git
pritidesai commented on issue #396: Deploy sequences separately other than combining with actions URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/396#issuecomment-325817448 @daisy-ycguo we have to make sure that wskdeploy deploys all actions prior to deploying sequences.

[GitHub] mdeuser commented on issue #99: WIP Changes required to add service bindings to the Bluemix plugin

2017-08-29 Thread git
mdeuser commented on issue #99: WIP Changes required to add service bindings to the Bluemix plugin URL: https://github.com/apache/incubator-openwhisk-cli/pull/99#issuecomment-325817245 @jessealva oh.. also, you will want to submit open cli PRs against the apache/incubator-openwhisk repo i

[GitHub] codecov-io commented on issue #63: style: add .editorconfig, eslint and reformat js file

2017-08-29 Thread git
codecov-io commented on issue #63: style: add .editorconfig, eslint and reformat js file URL: https://github.com/apache/incubator-openwhisk-client-js/pull/63#issuecomment-320281397 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk-client-js/pull/63?src=pr&el=h1) Report > Me

[GitHub] mdeuser commented on issue #99: WIP Changes required to add service bindings to the Bluemix plugin

2017-08-29 Thread git
mdeuser commented on issue #99: WIP Changes required to add service bindings to the Bluemix plugin URL: https://github.com/apache/incubator-openwhisk-cli/pull/99#issuecomment-325816295 @jessealva - in general, when the pr is merged, no additional technical debt is expected. in this case,

[GitHub] houshengbo closed issue #395: Deprecation of baseUrl

2017-08-29 Thread git
houshengbo closed issue #395: Deprecation of baseUrl URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/395 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] houshengbo commented on issue #395: Deprecation of baseUrl

2017-08-29 Thread git
houshengbo commented on issue #395: Deprecation of baseUrl URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/395#issuecomment-325813855 This is already fixed. This is an automated message from the Apache Gi

[GitHub] ert78gb commented on issue #63: style: add .editorconfig, eslint and reformat js file

2017-08-29 Thread git
ert78gb commented on issue #63: style: add .editorconfig, eslint and reformat js file URL: https://github.com/apache/incubator-openwhisk-client-js/pull/63#issuecomment-325813494 @csantanapr - added `npm run lint` to travis.yml - added pre-commit hook to the project and add a few wor

[GitHub] tleyden commented on issue #25: Not adding "api" to the URL

2017-08-29 Thread git
tleyden commented on issue #25: Not adding "api" to the URL URL: https://github.com/apache/incubator-openwhisk-client-go/issues/25#issuecomment-325812063 > Instead, credentials are now able to load in .wskprops Does it use that environment variable to find out which `.wskprops` to l

[GitHub] tleyden commented on issue #25: Not adding "api" to the URL

2017-08-29 Thread git
tleyden commented on issue #25: Not adding "api" to the URL URL: https://github.com/apache/incubator-openwhisk-client-go/issues/25#issuecomment-325811802 Cool, I'll pull the latest code and retest this This is an automated

[GitHub] mrutkows closed pull request #408: Make the config file able to work for .wskprops

2017-08-29 Thread git
mrutkows closed pull request #408: Make the config file able to work for .wskprops URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/408 This is an automated message from the Apache Git Service. To respond t

[GitHub] pritidesai closed pull request #406: fixing david's runtimetests

2017-08-29 Thread git
pritidesai closed pull request #406: fixing david's runtimetests URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/406 This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] pritidesai commented on issue #406: fixing david's runtimetests

2017-08-29 Thread git
pritidesai commented on issue #406: fixing david's runtimetests URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/406#issuecomment-325806759 All of these changes are incorporated into Vincent's PR #408 This

[GitHub] pritidesai commented on issue #410: Update the README with more instructions of building binaries

2017-08-29 Thread git
pritidesai commented on issue #410: Update the README with more instructions of building binaries URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/410#issuecomment-325805303 @houshengbo please make sure to incorporate Zach's comments from issue #400

[GitHub] houshengbo opened a new pull request #410: Update the README with more instructions of building binaries

2017-08-29 Thread git
houshengbo opened a new pull request #410: Update the README with more instructions of building binaries URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/410 This is an automated message from the Apache Git

[GitHub] houshengbo commented on issue #405: Unit test of TestParseManifestForSingleLineParams failing

2017-08-29 Thread git
houshengbo commented on issue #405: Unit test of TestParseManifestForSingleLineParams failing URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/405#issuecomment-325782197 @lzbj Strangely travis does not report this error, but it really exists when I run make test locally

[GitHub] pritidesai commented on issue #408: Make the config file able to work for .wskprops

2017-08-29 Thread git
pritidesai commented on issue #408: Make the config file able to work for .wskprops URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/408#issuecomment-325794301 LGTM This is an automated message from the Apa

[GitHub] qabbasi opened a new pull request #2671: Ignore IntelliJ '.iml' project file

2017-08-29 Thread git
qabbasi opened a new pull request #2671: Ignore IntelliJ '.iml' project file URL: https://github.com/apache/incubator-openwhisk/pull/2671 This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] houshengbo commented on issue #405: Unit test of TestParseManifestForSingleLineParams failing

2017-08-29 Thread git
houshengbo commented on issue #405: Unit test of TestParseManifestForSingleLineParams failing URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/405#issuecomment-325782197 @lzbj Strangely travis does not report this error, but it really exists when I run make test locally

[GitHub] pritidesai opened a new issue #409: Undeploy doesn't work with manifest.yml

2017-08-29 Thread git
pritidesai opened a new issue #409: Undeploy doesn't work with manifest.yml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/409 We can have deployment done with `manifest.yml` but when you try to undeploy, it fails with: ``` wskdeploy.Path is wskdeploy ==> E

[GitHub] houshengbo closed issue #25: Not adding "api" to the URL

2017-08-29 Thread git
houshengbo closed issue #25: Not adding "api" to the URL URL: https://github.com/apache/incubator-openwhisk-client-go/issues/25 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] houshengbo commented on issue #25: Not adding "api" to the URL

2017-08-29 Thread git
houshengbo commented on issue #25: Not adding "api" to the URL URL: https://github.com/apache/incubator-openwhisk-client-go/issues/25#issuecomment-325781332 @tleyden I have fixed this issue. Feel free to reopen it if it still valid. ---

[GitHub] mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135871039 ## File path: tools/cli/go-whisk-cli/commands/util.go ## @@ -866,6 +873,44 @@ func readFile(

[GitHub] mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135878011 ## File path: tools/cli/go-whisk-cli/commands/action.go ## @@ -908,29 +1019,29 @@ func init(

[GitHub] mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135869150 ## File path: tools/cli/go-whisk-cli/commands/action.go ## @@ -489,6 +492,87 @@ func getExec

[GitHub] mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135874611 ## File path: tools/cli/go-whisk-cli/commands/util.go ## @@ -866,6 +873,44 @@ func readFile(

[GitHub] mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135871798 ## File path: tools/cli/go-whisk-cli/commands/util.go ## @@ -866,6 +873,44 @@ func readFile(

[GitHub] mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135875181 ## File path: tools/cli/go-whisk-cli/commands/util.go ## @@ -866,6 +873,44 @@ func readFile(

[GitHub] mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135876203 ## File path: tools/cli/go-whisk-cli/commands/util.go ## @@ -866,6 +873,44 @@ func readFile(

[GitHub] mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135878561 ## File path: tools/cli/go-whisk-cli/commands/action.go ## @@ -870,6 +970,17 @@ func printAc

[GitHub] mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135873280 ## File path: tools/cli/go-whisk-cli/commands/util.go ## @@ -866,6 +873,44 @@ func readFile(

[GitHub] mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
mdeuser commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135869038 ## File path: tools/cli/go-whisk-cli/commands/action.go ## @@ -489,6 +492,87 @@ func getExec

[GitHub] mdeuser commented on a change in pull request #2591: User-Agent CLI/version header

2017-08-29 Thread git
mdeuser commented on a change in pull request #2591: User-Agent CLI/version header URL: https://github.com/apache/incubator-openwhisk/pull/2591#discussion_r135885573 ## File path: tools/cli/go-whisk-cli/commands/commands.go ## @@ -60,6 +60,7 @@ func setupClientConfig(cmd

[GitHub] pritidesai commented on issue #408: Make the config file able to work for .wskprops

2017-08-29 Thread git
pritidesai commented on issue #408: Make the config file able to work for .wskprops URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/408#issuecomment-325756305 WSK CLI stores the properties set in ~/.wskprops which looks like: ``` cat ~/.wskprops AUTH=849

[GitHub] houshengbo closed pull request #407: adding back jaraction integration test

2017-08-29 Thread git
houshengbo closed pull request #407: adding back jaraction integration test URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/407 This is an automated message from the Apache Git Service. To respond to the me

[GitHub] houshengbo commented on a change in pull request #406: fixing david's runtimetests

2017-08-29 Thread git
houshengbo commented on a change in pull request #406: fixing david's runtimetests URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/406#discussion_r135875483 ## File path: tests/src/integration/runtimetests/runtimes_test.go ## @@ -28,11 +28,10 @@ import (

[GitHub] houshengbo commented on a change in pull request #406: fixing david's runtimetests

2017-08-29 Thread git
houshengbo commented on a change in pull request #406: fixing david's runtimetests URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/406#discussion_r135875616 ## File path: tests/src/integration/runtimetests/runtimes_test.go ## @@ -28,11 +28,10 @@ import (

[GitHub] houshengbo commented on a change in pull request #406: fixing david's runtimetests

2017-08-29 Thread git
houshengbo commented on a change in pull request #406: fixing david's runtimetests URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/406#discussion_r135875488 ## File path: tests/src/integration/runtimetests/runtimes_test.go ## @@ -28,11 +28,10 @@ import (

[GitHub] markusthoemmes commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
markusthoemmes commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135872664 ## File path: tools/cli/go-whisk-cli/wski18n/resources/en_US.all.json ## @@ -1522,4 +

[GitHub] markusthoemmes commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
markusthoemmes commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135872418 ## File path: tools/cli/go-whisk-cli/wski18n/resources/en_US.all.json ## @@ -1522,4 +

[GitHub] mrutkows closed pull request #33: Remove bluemix URL as the default URL for client go

2017-08-29 Thread git
mrutkows closed pull request #33: Remove bluemix URL as the default URL for client go URL: https://github.com/apache/incubator-openwhisk-client-go/pull/33 This is an automated message from the Apache Git Service. To respond

[GitHub] mrutkows closed pull request #34: Replace the path of wskprops' parent dir with the path of wskprops

2017-08-29 Thread git
mrutkows closed pull request #34: Replace the path of wskprops' parent dir with the path of wskprops URL: https://github.com/apache/incubator-openwhisk-client-go/pull/34 This is an automated message from the Apache Git Serv

[GitHub] rabbah commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
rabbah commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135870714 ## File path: tests/src/test/scala/whisk/core/cli/test/WskBasicUsageTests.scala ## @@ -826,6

[GitHub] rabbah commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
rabbah commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135870847 ## File path: tests/src/test/scala/whisk/core/cli/test/WskBasicUsageTests.scala ## @@ -826,6

[GitHub] rabbah commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
rabbah commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135870088 ## File path: tests/src/test/scala/common/Wsk.scala ## @@ -168,13 +168,17 @@ trait ListOrGetF

[GitHub] rabbah commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
rabbah commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135870847 ## File path: tests/src/test/scala/whisk/core/cli/test/WskBasicUsageTests.scala ## @@ -826,6

[GitHub] rabbah commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
rabbah commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135870188 ## File path: tests/src/test/scala/whisk/core/cli/test/WskBasicUsageTests.scala ## @@ -826,6

[GitHub] rabbah commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
rabbah commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135870088 ## File path: tests/src/test/scala/common/Wsk.scala ## @@ -168,13 +168,17 @@ trait ListOrGetF

[GitHub] rabbah commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
rabbah commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135870007 ## File path: tools/cli/go-whisk/whisk/action.go ## @@ -51,6 +51,7 @@ type Exec struct {

[GitHub] rabbah commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review)

2017-08-29 Thread git
rabbah commented on a change in pull request #2544: Allow CLI to Save Code from Action (Review) URL: https://github.com/apache/incubator-openwhisk/pull/2544#discussion_r135870714 ## File path: tests/src/test/scala/whisk/core/cli/test/WskBasicUsageTests.scala ## @@ -826,6

[GitHub] houshengbo opened a new pull request #408: Make the config file able to work for .wskprops

2017-08-29 Thread git
houshengbo opened a new pull request #408: Make the config file able to work for .wskprops URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/408 This is an automated message from the Apache Git Service. To r

  1   2   >