[GitHub] csantanapr closed pull request #173: check if we should disable trigger for status codes on error object

csantanapr closed pull request #173: check if we should disable trigger for status codes on error object URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/173 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] csantanapr closed pull request #190: check if we should disable trigger for status codes on error object

csantanapr closed pull request #190: check if we should disable trigger for status codes on error object URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/190 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] dgrove-oss opened a new issue #398: Document all entries in values.yaml

dgrove-oss opened a new issue #398: Document all entries in values.yaml URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/398 We should uniformly document the meaning of every entry in values.yaml. One possible way to do this is to adopt the values-metadata.yaml

[GitHub] dgrove-oss closed issue #339: Controller and invoker stuck after reboot the cluster

dgrove-oss closed issue #339: Controller and invoker stuck after reboot the cluster URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/339 This is an automated message from the Apache Git Service. To

[GitHub] jasonpet opened a new pull request #190: check if we should disable trigger for status codes on error object

jasonpet opened a new pull request #190: check if we should disable trigger for status codes on error object URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/190 Currently a trigger disable check is only done for status codes on response object. With this update it

[GitHub] jasonpet opened a new pull request #173: check if we should disable trigger for status codes on error object

jasonpet opened a new pull request #173: check if we should disable trigger for status codes on error object URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/173 Currently a trigger disable check is only done for status codes on response object. With this update it

[GitHub] daisy-ycguo closed pull request #394: version bump all components to Dec 13 tags.

daisy-ycguo closed pull request #394: version bump all components to Dec 13 tags. URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/394 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] codecov-io commented on issue #4185: recreate http client on resume()

codecov-io commented on issue #4185: recreate http client on resume() URL: https://github.com/apache/incubator-openwhisk/pull/4185#issuecomment-448069584 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4185?src=pr=h1) Report > Merging

[GitHub] jasonpet closed pull request #68: bump Go to 1.11.4

jasonpet closed pull request #68: bump Go to 1.11.4 URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/68 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] csantanapr opened a new pull request #68: bump Go to 1.11.4

csantanapr opened a new pull request #68: bump Go to 1.11.4 URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/68 Fixes #67 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] asteed opened a new pull request #190: Fix support for `wsk sdk install docker`

asteed opened a new pull request #190: Fix support for `wsk sdk install docker` URL: https://github.com/apache/incubator-openwhisk-devtools/pull/190 * Add `/blackbox.tar.gz` location to apigateway config This is an automated

[GitHub] jasonpet closed issue #67: Update to Go 1.11.4 released on Dec. 14 2018

jasonpet closed issue #67: Update to Go 1.11.4 released on Dec. 14 2018 URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/67 This is an automated message from the Apache Git Service. To respond to the

[GitHub] csantanapr closed pull request #188: Add PHP 7.3 runtime to the docker-compose manifest

csantanapr closed pull request #188: Add PHP 7.3 runtime to the docker-compose manifest URL: https://github.com/apache/incubator-openwhisk-devtools/pull/188 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] csantanapr closed issue #4174: Add PHP 7.3 runtime support to default deployments

csantanapr closed issue #4174: Add PHP 7.3 runtime support to default deployments URL: https://github.com/apache/incubator-openwhisk/issues/4174 This is an automated message from the Apache Git Service. To respond to the

[GitHub] csantanapr closed pull request #51: Update PHP 7.1.25 and 7.2.13

csantanapr closed pull request #51: Update PHP 7.1.25 and 7.2.13 URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/51 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] csantanapr closed issue #48: update php 7.1 to 7.1.25

csantanapr closed issue #48: update php 7.1 to 7.1.25 URL: https://github.com/apache/incubator-openwhisk-runtime-php/issues/48 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] csantanapr closed issue #49: update PHP 7.2 to 7.2.13

csantanapr closed issue #49: update PHP 7.2 to 7.2.13 URL: https://github.com/apache/incubator-openwhisk-runtime-php/issues/49 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] csantanapr closed pull request #4182: Add PHP 7.3 runtime

csantanapr closed pull request #4182: Add PHP 7.3 runtime URL: https://github.com/apache/incubator-openwhisk/pull/4182 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] dgrove-oss commented on issue #391: synchronize with upstream configuration files

dgrove-oss commented on issue #391: synchronize with upstream configuration files URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/391#issuecomment-447947743 I agree, but I think maybe we can do it via values-metadata.yaml. I opened #398 to track.

[GitHub] dgrove-oss edited a comment on issue #391: synchronize with upstream configuration files

dgrove-oss edited a comment on issue #391: synchronize with upstream configuration files URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/391#issuecomment-447947743 I agree, but I think maybe we can do it via values-metadata.yaml and only have entries in

[GitHub] dgrove-oss commented on issue #394: version bump all components to Dec 13 tags.

dgrove-oss commented on issue #394: version bump all components to Dec 13 tags. URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/394#issuecomment-447934521 ready to merge -- I rebased to pick up php 7.3.

[GitHub] csantanapr commented on issue #4174: Add PHP 7.3 runtime support to default deployments

csantanapr commented on issue #4174: Add PHP 7.3 runtime support to default deployments URL: https://github.com/apache/incubator-openwhisk/issues/4174#issuecomment-447934653 All PRs have being merged, Thanks ❤️ @akrabat 

[GitHub] pritidesai commented on issue #394: Integrating wskdeploy into wsk CLI

pritidesai commented on issue #394: Integrating wskdeploy into wsk CLI URL: https://github.com/apache/incubator-openwhisk-cli/pull/394#issuecomment-447956534 thank you everyone for the comments, its ready to merge, I need +2 on this PR 

[GitHub] tysonnorris closed pull request #4183: Ensure cache gets properly updated with concurrent access for action with attachments

tysonnorris closed pull request #4183: Ensure cache gets properly updated with concurrent access for action with attachments URL: https://github.com/apache/incubator-openwhisk/pull/4183 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is

[GitHub] tysonnorris opened a new pull request #4186: Track activation counts in ContainerPool (not ContainerProxy)

tysonnorris opened a new pull request #4186: Track activation counts in ContainerPool (not ContainerProxy) URL: https://github.com/apache/incubator-openwhisk/pull/4186 Move concurrent activation count tracking from ContainerProxy -> ContainerPool, so that prewam/cold -> warm can be

[GitHub] rabbah commented on issue #394: Integrating wskdeploy into wsk CLI

rabbah commented on issue #394: Integrating wskdeploy into wsk CLI URL: https://github.com/apache/incubator-openwhisk-cli/pull/394#issuecomment-447959571 @pritidesai please give me a chance to review. This is an automated

[GitHub] tysonnorris closed pull request #4173: use latest mesos-actor

tysonnorris closed pull request #4173: use latest mesos-actor URL: https://github.com/apache/incubator-openwhisk/pull/4173 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] saurabh1284 commented on issue #104: "wsk api list" is not working

saurabh1284 commented on issue #104: "wsk api list" is not working URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/104#issuecomment-448125551 @dgrove-oss , I tried fresh installation and same issue persist. -> wsk api list -i error: Unable to obtain the

[GitHub] markusthoemmes commented on a change in pull request #4185: recreate http client on resume()

markusthoemmes commented on a change in pull request #4185: recreate http client on resume() URL: https://github.com/apache/incubator-openwhisk/pull/4185#discussion_r242426665 ## File path: common/scala/src/main/scala/org/apache/openwhisk/core/containerpool/Container.scala

[GitHub] markusthoemmes commented on a change in pull request #4185: recreate http client on resume()

markusthoemmes commented on a change in pull request #4185: recreate http client on resume() URL: https://github.com/apache/incubator-openwhisk/pull/4185#discussion_r242426212 ## File path:

[GitHub] markusthoemmes commented on a change in pull request #4185: recreate http client on resume()

markusthoemmes commented on a change in pull request #4185: recreate http client on resume() URL: https://github.com/apache/incubator-openwhisk/pull/4185#discussion_r242426229 ## File path:

[GitHub] csantanapr opened a new pull request #51: Update php 71 25 72 13

csantanapr opened a new pull request #51: Update php 71 25 72 13 URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/51 update PHP 7.1 to 7.1.25 Fixes #48 update PHP 7.2 to 7.2.13 Fixes #49 This is an

[GitHub] akrabat commented on issue #51: Update PHP 7.1.25 and 7.2.13

akrabat commented on issue #51: Update PHP 7.1.25 and 7.2.13 URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/51#issuecomment-447914084 Need to update the CHANGELOGs too. This is an automated message from

[GitHub] akrabat commented on issue #188: Add PHP 7.3 runtime to the docker-compose manifest

akrabat commented on issue #188: Add PHP 7.3 runtime to the docker-compose manifest URL: https://github.com/apache/incubator-openwhisk-devtools/pull/188#issuecomment-447915412 > minor nit, need to add image to Makefile for docker-pull-full Done.

[GitHub] akrabat edited a comment on issue #188: Add PHP 7.3 runtime to the docker-compose manifest

akrabat edited a comment on issue #188: Add PHP 7.3 runtime to the docker-compose manifest URL: https://github.com/apache/incubator-openwhisk-devtools/pull/188#issuecomment-447915412 > minor nit, need to add image to Makefile for docker-pull-full Done.

[GitHub] csantanapr closed pull request #189: Update Makefile with php 7.3 image

csantanapr closed pull request #189: Update Makefile with php 7.3 image URL: https://github.com/apache/incubator-openwhisk-devtools/pull/189 This is an automated message from the Apache Git Service. To respond to the

[GitHub] csantanapr commented on issue #189: Update Makefile with php 7.3 image

csantanapr commented on issue #189: Update Makefile with php 7.3 image URL: https://github.com/apache/incubator-openwhisk-devtools/pull/189#issuecomment-447915953 covered by #188 This is an automated message from the Apache

[GitHub] codecov-io commented on issue #4182: Add PHP 7.3 runtime

codecov-io commented on issue #4182: Add PHP 7.3 runtime URL: https://github.com/apache/incubator-openwhisk/pull/4182#issuecomment-447926093 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4182?src=pr=h1) Report > Merging

[GitHub] tysonnorris opened a new pull request #4185: recreate http client on resume()

tysonnorris opened a new pull request #4185: recreate http client on resume() URL: https://github.com/apache/incubator-openwhisk/pull/4185 In testing concurrent actions (multiple activations in same container) we noticed that when a container is used after `Container.suspend()`, the

[GitHub] chetanmeh opened a new pull request #4183: Ensure cache gets properly updated with concurrent access for action with attachments

chetanmeh opened a new pull request #4183: Ensure cache gets properly updated with concurrent access for action with attachments URL: https://github.com/apache/incubator-openwhisk/pull/4183 @tysonnorris notices with concurrent action invocation scenario that same attachment is being

[GitHub] daisy-ycguo commented on issue #391: synchronize with upstream configuration files

daisy-ycguo commented on issue #391: synchronize with upstream configuration files URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/391#issuecomment-447821161 @dgrove-oss I merged this PR. I think we'd better add documentation about limitations and kafka

[GitHub] daisy-ycguo closed pull request #391: synchronize with upstream configuration files

daisy-ycguo closed pull request #391: synchronize with upstream configuration files URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/391 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] daisy-ycguo edited a comment on issue #391: synchronize with upstream configuration files

daisy-ycguo edited a comment on issue #391: synchronize with upstream configuration files URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/391#issuecomment-447821161 @dgrove-oss I merged this PR. I think we'd better add documents about limitations and kafka

[GitHub] markusthoemmes commented on a change in pull request #4183: Ensure cache gets properly updated with concurrent access for action with attachments

markusthoemmes commented on a change in pull request #4183: Ensure cache gets properly updated with concurrent access for action with attachments URL: https://github.com/apache/incubator-openwhisk/pull/4183#discussion_r242122269 ## File path:

[GitHub] markusthoemmes commented on a change in pull request #4183: Ensure cache gets properly updated with concurrent access for action with attachments

markusthoemmes commented on a change in pull request #4183: Ensure cache gets properly updated with concurrent access for action with attachments URL: https://github.com/apache/incubator-openwhisk/pull/4183#discussion_r242120019 ## File path:

[GitHub] csantanapr closed pull request #396: Update the pollSince time based on latest activation start time

csantanapr closed pull request #396: Update the pollSince time based on latest activation start time URL: https://github.com/apache/incubator-openwhisk-cli/pull/396 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] mhenke1 commented on issue #4163: Correct rejection behavior in the multi authentication case

mhenke1 commented on issue #4163: Correct rejection behavior in the multi authentication case URL: https://github.com/apache/incubator-openwhisk/pull/4163#issuecomment-447851741 @ddragosd validations = our code that implements MiscDirectives.validate (mostly checks on a valid

[GitHub] mhenke1 edited a comment on issue #4163: Correct rejection behavior in the multi authentication case

mhenke1 edited a comment on issue #4163: Correct rejection behavior in the multi authentication case URL: https://github.com/apache/incubator-openwhisk/pull/4163#issuecomment-447851741 @ddragosd validations = our code that implements MiscDirectives.validate (mostly checks on a

[GitHub] codecov-io commented on issue #4163: Correct rejection behavior in the multi authentication case

codecov-io commented on issue #4163: Correct rejection behavior in the multi authentication case URL: https://github.com/apache/incubator-openwhisk/pull/4163#issuecomment-447857820 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4163?src=pr=h1) Report > Merging

[GitHub] codecov-io edited a comment on issue #4163: Correct rejection behavior in the multi authentication case

codecov-io edited a comment on issue #4163: Correct rejection behavior in the multi authentication case URL: https://github.com/apache/incubator-openwhisk/pull/4163#issuecomment-447857820 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4163?src=pr=h1) Report >

[GitHub] dgrove-oss closed pull request #397: Add PHP 7.3 runtime

dgrove-oss closed pull request #397: Add PHP 7.3 runtime URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/397 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] csantanapr opened a new issue #48: update php 7.1 to 7.1.25

csantanapr opened a new issue #48: update php 7.1 to 7.1.25 URL: https://github.com/apache/incubator-openwhisk-runtime-php/issues/48 Pick up latest patch release http://us3.php.net/ChangeLog-7.php#7.1.25 This is an

[GitHub] csantanapr opened a new issue #49: update PHP 7.2 to 7.2.13

csantanapr opened a new issue #49: update PHP 7.2 to 7.2.13 URL: https://github.com/apache/incubator-openwhisk-runtime-php/issues/49 pick up latest patch release http://us3.php.net/ChangeLog-7.php#7.2.13 This is an

[GitHub] akrabat opened a new issue #50: Deprecate PHP 7.1 runtime

akrabat opened a new issue #50: Deprecate PHP 7.1 runtime URL: https://github.com/apache/incubator-openwhisk-runtime-php/issues/50 With PHP 7.1. out of active support now, we should deprecate it now as it'll be out of security fixes within a year.

[GitHub] csantanapr commented on issue #50: Deprecate PHP 7.1 runtime

csantanapr commented on issue #50: Deprecate PHP 7.1 runtime URL: https://github.com/apache/incubator-openwhisk-runtime-php/issues/50#issuecomment-447908184 Yep I agree This is an automated message from the Apache Git

[GitHub] csantanapr commented on a change in pull request #4182: Add PHP 7.3 runtime

csantanapr commented on a change in pull request #4182: Add PHP 7.3 runtime URL: https://github.com/apache/incubator-openwhisk/pull/4182#discussion_r242221354 ## File path: docs/actions-php.md ## @@ -101,8 +106,12 @@ The PHP runtime will automatically include Composer's

[GitHub] rabbah commented on issue #37: php 7.1 runtime does not handle large input correctly

rabbah commented on issue #37: php 7.1 runtime does not handle large input correctly URL: https://github.com/apache/incubator-openwhisk-runtime-php/issues/37#issuecomment-447910337 Closing as won’t fix. See #50 This is an

[GitHub] rabbah closed issue #37: php 7.1 runtime does not handle large input correctly

rabbah closed issue #37: php 7.1 runtime does not handle large input correctly URL: https://github.com/apache/incubator-openwhisk-runtime-php/issues/37 This is an automated message from the Apache Git Service. To respond to

[GitHub] rabbah closed pull request #4167: Add tid from header to RestResult in tests.

rabbah closed pull request #4167: Add tid from header to RestResult in tests. URL: https://github.com/apache/incubator-openwhisk/pull/4167 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] alibanaei opened a new issue #4184: ./gradlew :core:controller:compileScala does not working

alibanaei opened a new issue #4184: ./gradlew :core:controller:compileScala does not working URL: https://github.com/apache/incubator-openwhisk/issues/4184 I have installed openwhisk on local vm on ubuntu 14.04, then I want to change the source code and build it. I have changed the

[GitHub] codecov-io commented on issue #4167: Add tid from header to RestResult in tests.

codecov-io commented on issue #4167: Add tid from header to RestResult in tests. URL: https://github.com/apache/incubator-openwhisk/pull/4167#issuecomment-447888272 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4167?src=pr=h1) Report > Merging

[GitHub] csantanapr commented on issue #188: update Node.js version to 8.14.0

csantanapr commented on issue #188: update Node.js version to 8.14.0 URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/188#issuecomment-447889612 Why not nodejs10? This is an automated message from the

[GitHub] cbickel commented on a change in pull request #4167: Add tid from header to RestResult in tests.

cbickel commented on a change in pull request #4167: Add tid from header to RestResult in tests. URL: https://github.com/apache/incubator-openwhisk/pull/4167#discussion_r242205357 ## File path: tests/src/test/scala/common/rest/WskRestOperations.scala ## @@ -1319,6

[GitHub] rabbah commented on issue #4184: ./gradlew :core:controller:compileScala is not working

rabbah commented on issue #4184: ./gradlew :core:controller:compileScala is not working URL: https://github.com/apache/incubator-openwhisk/issues/4184#issuecomment-447912350 Did you redeploy the container for the controller? There’s a helper script in bin/wskdev to rebuild and redeploy

[GitHub] csantanapr opened a new pull request #189: Update Makefile with php 7.3 image

csantanapr opened a new pull request #189: Update Makefile with php 7.3 image URL: https://github.com/apache/incubator-openwhisk-devtools/pull/189 This is an automated message from the Apache Git Service. To respond to the

[GitHub] csantanapr closed pull request #357: Update for PHP 7.3 runtime

csantanapr closed pull request #357: Update for PHP 7.3 runtime URL: https://github.com/apache/incubator-openwhisk-website/pull/357 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] csantanapr closed pull request #1021: Add PHP 7.3 runtime

csantanapr closed pull request #1021: Add PHP 7.3 runtime URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/1021 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] codecov-io commented on issue #4183: Ensure cache gets properly updated with concurrent access for action with attachments

codecov-io commented on issue #4183: Ensure cache gets properly updated with concurrent access for action with attachments URL: https://github.com/apache/incubator-openwhisk/pull/4183#issuecomment-447893931 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4183?src=pr=h1)