csantanapr closed pull request #1: Update Travis to use remote as the cli
installation mode
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/1
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the
jthomas commented on a change in pull request #3: support large arguments
URL:
https://github.com/apache/incubator-openwhisk-runtime-docker/pull/3#discussion_r164169939
##
File path: core/actionProxy/actionproxy.py
##
@@ -127,18 +127,29 @@ def error(msg):
try:
jthomas commented on a change in pull request #3: support large arguments
URL:
https://github.com/apache/incubator-openwhisk-runtime-docker/pull/3#discussion_r164169939
##
File path: core/actionProxy/actionproxy.py
##
@@ -127,18 +127,29 @@ def error(msg):
try:
jthomas commented on issue #3: support large arguments
URL:
https://github.com/apache/incubator-openwhisk-runtime-docker/pull/3#issuecomment-360848476
LGTM.
This is an automated message from the Apache Git Service.
To
RSulzmann opened a new pull request #3: support large arguments
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/3
There is a limit of the argument size in a command line (MAX_ARG_STRLEN =
131071 bytes) .
To support larger sizes, the input to the action code is
rabbah commented on issue #3: support large arguments
URL:
https://github.com/apache/incubator-openwhisk-runtime-docker/pull/3#issuecomment-360211242
Closes https://github.com/apache/incubator-openwhisk/issues/2542.
@jthomas can you review.
rabbah commented on a change in pull request #3: support large arguments
URL:
https://github.com/apache/incubator-openwhisk-runtime-docker/pull/3#discussion_r163620453
##
File path: core/actionProxy/actionproxy.py
##
@@ -127,18 +127,29 @@ def error(msg):
try:
rabbah commented on issue #3: support large arguments
URL:
https://github.com/apache/incubator-openwhisk-runtime-docker/pull/3#issuecomment-360211763
We will need to add a test. You can use this example, which I did for swift
to test the change.
csantanapr commented on issue #4: fix travis CI/CD
URL:
https://github.com/apache/incubator-openwhisk-runtime-docker/issues/4#issuecomment-360924634
Currently broken, it should be easy to fix in the
csantanapr opened a new issue #8: in certain environments with ipv6 disable
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/issues/8
>Not being specific here causes python3 to try an ipv6 address first which
fails on kernels/OS which do not contain the ipv6 module.
csantanapr opened a new pull request #12: adds jq perl zip git
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/12
This is an automated message from the Apache Git Service.
To respond to the message,
csantanapr commented on issue #5: Add perl to dockerskeleton
URL:
https://github.com/apache/incubator-openwhisk-runtime-docker/issues/5#issuecomment-360924902
cc @rabbah
This is an automated message from the Apache Git
csantanapr opened a new issue #7: input larger than 128KB not working
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/issues/7
Fix by #3
This is an automated message from the Apache Git Service.
To respond
csantanapr opened a new issue #9: add git utility to docker actions
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/issues/9
Would be useful to have git utility in dockerskeleton
Allows users to build devops actions where they can git clone and
build/compile/package
csantanapr opened a new pull request #11: Issue 4 fix travis build
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/11
This is an automated message from the Apache Git Service.
To respond to the
rabbah closed issue #4: fix travis CI/CD
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/issues/4
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub
rabbah closed pull request #11: fix travis build
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/11
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
csantanapr commented on issue #11: fix travis build
URL:
https://github.com/apache/incubator-openwhisk-runtime-docker/pull/11#issuecomment-360980736
Could someone merge this?
??
This is an automated message from the
csantanapr commented on issue #3: support large arguments
URL:
https://github.com/apache/incubator-openwhisk-runtime-docker/pull/3#issuecomment-362906189
Yep you read my mind ?
This is an automated message from the Apache
rabbah commented on a change in pull request #12: adds jq perl zip git
URL:
https://github.com/apache/incubator-openwhisk-runtime-docker/pull/12#discussion_r165839302
##
File path: core/actionProxy/Dockerfile
##
@@ -2,11 +2,8 @@
FROM python:3.6.1-alpine
# Upgrade and
csantanapr commented on a change in pull request #3: support large arguments
URL:
https://github.com/apache/incubator-openwhisk-runtime-docker/pull/3#discussion_r165844124
##
File path: core/actionProxy/actionproxy.py
##
@@ -127,18 +128,29 @@ def error(msg):
csantanapr commented on a change in pull request #12: adds jq perl zip git
URL:
https://github.com/apache/incubator-openwhisk-runtime-docker/pull/12#discussion_r165847397
##
File path: core/actionProxy/Dockerfile
##
@@ -2,11 +2,8 @@
FROM python:3.6.1-alpine
# Upgrade
csantanapr closed issue #10: add zip to dockerskeleton
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/issues/10
This is an automated message from the Apache Git Service.
To respond to the message, please
csantanapr opened a new pull request #14: refactor large input tests to include
perl and python
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/14
This is an automated message from the Apache Git
csantanapr closed issue #6: Add jq utility
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/issues/6
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub
csantanapr closed issue #9: add git utility to docker actions
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/issues/9
This is an automated message from the Apache Git Service.
To respond to the message,
csantanapr closed issue #7: input larger than 128KB not working
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/issues/7
This is an automated message from the Apache Git Service.
To respond to the message,
csantanapr closed issue #8: in certain environments with ipv6 disable
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/issues/8
This is an automated message from the Apache Git Service.
To respond to the
csantanapr closed issue #5: Add perl to dockerskeleton
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/issues/5
This is an automated message from the Apache Git Service.
To respond to the message, please
csantanapr closed pull request #2: Create socket on specific ipv4 address.
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/2
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:
mrutkows commented on issue #1: Migration of Red Hat's existing work to the new
ASF Incubator Repo
URL:
https://github.com/apache/incubator-openwhisk-deploy-openshift/pull/1#issuecomment-404603414
@bwmcadams and thanks for preserving the Git history
bwmcadams opened a new pull request #1: Migration of Red Hat's existing work to
the new ASF Incubator Repo
URL: https://github.com/apache/incubator-openwhisk-deploy-openshift/pull/1
This comprises the work Red Hat has done thus far on making OpenShift and
OpenWhisk work together. It
dgrove-oss commented on issue #1: Initial import of the Composer code
URL:
https://github.com/apache/incubator-openwhisk-composer/pull/1#issuecomment-426771696
Scancode is clean:
```
daves-mbp:incubator-openwhisk-utilities dgrove$ ./scancode/scanCode.py
--config
tardieu commented on issue #2: more boilerplate: fix DISCLAIMER; add
CONTRIBUTING
URL:
https://github.com/apache/incubator-openwhisk-composer/pull/2#issuecomment-426781083
I propose `openwhisk-composer` for the npm package name.
tardieu opened a new pull request #1: Initial import of the Composer code
URL: https://github.com/apache/incubator-openwhisk-composer/pull/1
IBM is donating the implementation of Composer (previously available
https://github.com/ibm-functions/composer at under the Apache 2 license) to the
dgrove-oss opened a new pull request #2: more boilerplate: fix DISCLAIMER; add
CONTRIBUTING
URL: https://github.com/apache/incubator-openwhisk-composer/pull/2
This is an automated message from the Apache Git Service.
To
dgrove-oss closed pull request #1: Initial import of the Composer code
URL: https://github.com/apache/incubator-openwhisk-composer/pull/1
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
dgrove-oss commented on issue #1: Initial import of the Composer code
URL:
https://github.com/apache/incubator-openwhisk-composer/pull/1#issuecomment-426771838
This is the contribution from IBM that is documented in
https://issues.apache.org/jira/browse/INCUBATOR-222
dgrove-oss commented on issue #2: more boilerplate: fix DISCLAIMER; add
CONTRIBUTING
URL:
https://github.com/apache/incubator-openwhisk-composer/pull/2#issuecomment-426780383
that's the name of the old NPM module. The thought was to leave the npm
module intact for a short time until we
rabbah commented on issue #2: more boilerplate: fix DISCLAIMER; add CONTRIBUTING
URL:
https://github.com/apache/incubator-openwhisk-composer/pull/2#issuecomment-426778537
can you fix the ibm mention in the README?
This is an
rabbah edited a comment on issue #2: more boilerplate: fix DISCLAIMER; add
CONTRIBUTING
URL:
https://github.com/apache/incubator-openwhisk-composer/pull/2#issuecomment-426778537
can you fix the ibm mention in the README?
`@ibm-functions/composer`
bruceadams opened a new pull request #1: Proposed changes to action_loop
URL: https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1
The key change here is using more of the power of Serde for parsing the
input.
There are several other, bigger picture things that should
sciabarracom merged pull request #1: Proposed changes to action_loop
URL: https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1
This is an automated message from the Apache Git Service.
To respond to the message,
sciabarracom commented on issue #1: Proposed changes to action_loop
URL:
https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1#issuecomment-470587428
I am asking Roberto Diaz opinion as he is the original author of the Go
ActionLoop
bruceadams edited a comment on issue #1: Proposed changes to action_loop
URL:
https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1#issuecomment-470714058
Ah, yup. Right there in the documentation
https://doc.rust-lang.org/cargo/guide/cargo-toml-vs-cargo-lock.html
> If
sciabarracom commented on issue #1: Proposed changes to action_loop
URL:
https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1#issuecomment-470717971
Is it possible to get a fix also for the example?
This is an
rdiaz82 commented on issue #1: Proposed changes to action_loop
URL:
https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1#issuecomment-470720792
I would like to see this method signature `fn main(input_data: Value) ->
Result` I have tested locally and it works nice
rdiaz82 commented on issue #1: Proposed changes to action_loop
URL:
https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1#issuecomment-470724708
we can create one specific for that, what about `{"result": "error
generating output response"}` or something like that
bruceadams commented on issue #1: Proposed changes to action_loop
URL:
https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1#issuecomment-470714058
Ah, yup. Right there in the documentation
https://doc.rust-lang.org/cargo/guide/cargo-toml-vs-cargo-lock.html
> If you’re
bruceadams edited a comment on issue #1: Proposed changes to action_loop
URL:
https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1#issuecomment-470712862
My understanding it that Rust projects that produce a binary _should_ check
`Cargo.lock` into source control and Rust
rdiaz82 commented on issue #1: Proposed changes to action_loop
URL:
https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1#issuecomment-470725225
I think that we should move this conversation to other place, to not spam
the PR. I think that we can merge it and continue from
sciabarracom commented on issue #1: Proposed changes to action_loop
URL:
https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1#issuecomment-470725579
Ok give me the time to check if it builds and I will merge
rdiaz82 commented on a change in pull request #1: Proposed changes to
action_loop
URL:
https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1#discussion_r263592618
##
File path: rust1.32/src/action_loop/src/main.rs
##
@@ -1,49 +1,46 @@
-extern crate
sciabarracom commented on issue #1: Proposed changes to action_loop
URL:
https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1#issuecomment-470722586
Which JSON the mainloop will emit if the result is an error?
bruceadams commented on a change in pull request #1: Proposed changes to
action_loop
URL:
https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1#discussion_r263588112
##
File path: rust1.32/src/action_loop/src/main.rs
##
@@ -1,49 +1,46 @@
-extern crate
rdiaz82 commented on a change in pull request #1: Proposed changes to
action_loop
URL:
https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1#discussion_r263540710
##
File path: rust1.32/src/action_loop/src/main.rs
##
@@ -1,49 +1,46 @@
-extern crate
bruceadams commented on a change in pull request #1: Proposed changes to
action_loop
URL:
https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1#discussion_r263590593
##
File path: rust1.32/src/action_loop/src/main.rs
##
@@ -1,49 +1,46 @@
-extern crate
jthomas opened a new issue #1: Provider should be installable using Helm
URL: https://github.com/apache/incubator-openwhisk-pluggable-provider/issues/1
Follow same approach to other project components and allow providers to be
installed with Helm charts.
jthomas opened a new issue #2: Trigger manager should return failures for
trigger firing
URL: https://github.com/apache/incubator-openwhisk-pluggable-provider/issues/2
https://github.com/apache/incubator-openwhisk-pluggable-provider/blob/master/provider/lib/triggers_manager.js#L202-L240
markusthoemmes commented on a change in pull request #3: Fixing numerous issues
discovered whilst developing new event source
URL:
https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/3#discussion_r290255887
##
File path: provider/lib/triggers_manager.js
jthomas commented on a change in pull request #3: Fixing numerous issues
discovered whilst developing new event source
URL:
https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/3#discussion_r290292293
##
File path: provider/lib/triggers_manager.js
##
@@
markusthoemmes commented on a change in pull request #3: Fixing numerous issues
discovered whilst developing new event source
URL:
https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/3#discussion_r290255720
##
File path: actions/event-actions/changesWebAction.js
jthomas commented on a change in pull request #3: Fixing numerous issues
discovered whilst developing new event source
URL:
https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/3#discussion_r290289829
##
File path: actions/event-actions/changesWebAction.js
markusthoemmes commented on a change in pull request #3: Fixing numerous issues
discovered whilst developing new event source
URL:
https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/3#discussion_r290293704
##
File path: actions/event-actions/changesWebAction.js
jthomas commented on issue #1: Provider should be installable using Helm
URL:
https://github.com/apache/incubator-openwhisk-pluggable-provider/issues/1#issuecomment-499146909
Work for this has started in
https://github.com/apache/incubator-openwhisk-deploy-kube/pull/474
jthomas opened a new issue #7: Set up Travis to publish images to Docker Hub
URL: https://github.com/apache/incubator-openwhisk-pluggable-provider/issues/7
This pluggable provider should be made available as an official image on DH.
jthomas opened a new pull request #6: Removing Cloudant references
URL: https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/6
Continuing work to remove old references to Cloudant from the original fork.
This includes
- Adding new systems tests for generic
jthomas merged pull request #6: Removing Cloudant references
URL: https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/6
This is an automated message from the Apache Git Service.
To respond to the message,
jthomas closed issue #4: Re-factor tests for new pluggable provider
URL: https://github.com/apache/incubator-openwhisk-pluggable-provider/issues/4
This is an automated message from the Apache Git Service.
To respond to the
jthomas opened a new issue #5: Build is broken due to scancode issues.
URL: https://github.com/apache/incubator-openwhisk-pluggable-provider/issues/5
https://travis-ci.org/apache/incubator-openwhisk-pluggable-provider/builds
jthomas commented on a change in pull request #8: Fixing scancode issues.
URL:
https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/8#discussion_r293856268
##
File path: .travis.yml
##
@@ -1,4 +1,5 @@
-# A Travis CI configuration file.
Review comment:
jthomas opened a new pull request #8: Fixing scancode issues.
URL: https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/8
See
https://travis-ci.org/apache/incubator-openwhisk-pluggable-provider/builds/535290433?utm_source=github_status_medium=notification
for issues.
rabbah commented on a change in pull request #8: Fixing scancode issues.
URL:
https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/8#discussion_r293854937
##
File path: gradle/wrapper/gradle-wrapper.properties
##
@@ -1,3 +1,6 @@
+# Licensed to the Apache
rabbah commented on a change in pull request #8: Fixing scancode issues.
URL:
https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/8#discussion_r293854656
##
File path: .travis.yml
##
@@ -1,4 +1,5 @@
-# A Travis CI configuration file.
Review comment:
mrutkows commented on a change in pull request #8: Fixing scancode issues.
URL:
https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/8#discussion_r294308645
##
File path: gradle/wrapper/gradle-wrapper.properties
##
@@ -1,3 +1,6 @@
+# Licensed to the
jthomas commented on issue #7: Set up Travis to publish images to Docker Hub
URL:
https://github.com/apache/incubator-openwhisk-pluggable-provider/issues/7#issuecomment-505382910
The build process is broken
jthomas commented on a change in pull request #8: Fixing scancode issues.
URL:
https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/8#discussion_r294325716
##
File path: gradle/wrapper/gradle-wrapper.properties
##
@@ -1,3 +1,6 @@
+# Licensed to the Apache
jthomas commented on a change in pull request #8: Fixing scancode issues.
URL:
https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/8#discussion_r294325898
##
File path: .travis.yml
##
@@ -1,4 +1,5 @@
-# A Travis CI configuration file.
Review comment:
jthomas merged pull request #8: Fixing scancode issues.
URL: https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/8
This is an automated message from the Apache Git Service.
To respond to the message, please
jthomas opened a new pull request #3: Fixing numerous issues discovered whilst
developing new event source
URL: https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/3
Whilst developing a new event source
(https://github.com/jthomas/openwhisk-amqp-trigger-feed) for the
jthomas closed issue #2: Trigger manager should return failures for trigger
firing
URL: https://github.com/apache/incubator-openwhisk-pluggable-provider/issues/2
This is an automated message from the Apache Git Service.
To
jthomas opened a new issue #4: Re-factor tests for new pluggable provider
URL: https://github.com/apache/incubator-openwhisk-pluggable-provider/issues/4
- Existing tests have been copied wholesale from original Cloudant project.
These needs re-working for this project.
jthomas opened a new pull request #10: Fixing WSK_CLI definition.
URL: https://github.com/apache/openwhisk-pluggable-provider/pull/10
Needed to resolve https://github.com/apache/openwhisk-deploy-kube/pull/474
This is an
jthomas opened a new pull request #9: Fix invalid Travis credentials and change
tag to nightly
URL: https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/9
This is an automated message from the Apache Git
jthomas merged pull request #9: Fix invalid Travis credentials and change tag
to nightly
URL: https://github.com/apache/incubator-openwhisk-pluggable-provider/pull/9
This is an automated message from the Apache Git Service.
jthomas closed issue #5: Build is broken due to scancode issues.
URL: https://github.com/apache/incubator-openwhisk-pluggable-provider/issues/5
This is an automated message from the Apache Git Service.
To respond to the
rabbah commented on a change in pull request #2:
URL:
https://github.com/apache/openwhisk-runtime-deno/pull/2#discussion_r486587116
##
File path: .travis.yml
##
@@ -0,0 +1,61 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license
rabbah commented on pull request #2:
URL:
https://github.com/apache/openwhisk-runtime-deno/pull/2#issuecomment-706631629
@satyarohith let's avoid the "loop" part of the naming in the docker images
and class names - it's historical and no longer necessary to make that
distinction. Since
rabbah opened a new pull request #1:
URL: https://github.com/apache/openwhisk-runtime-deno/pull/1
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
rabbah merged pull request #1:
URL: https://github.com/apache/openwhisk-runtime-deno/pull/1
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above
satyarohith opened a new pull request #2:
URL: https://github.com/apache/openwhisk-runtime-deno/pull/2
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
agnithamohan opened a new issue #12:
URL: https://github.com/apache/openwhisk-pluggable-provider/issues/12
Hello,
I've deployed openwhisk to Kubernetes by following
[openwhisk-deploy-kube](https://github.com/apache/openwhisk-deploy-kube) using
kind and the helm charts. I'm using
rabbah merged pull request #11:
URL: https://github.com/apache/openwhisk-pluggable-provider/pull/11
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
jbampton opened a new pull request #11:
URL: https://github.com/apache/openwhisk-pluggable-provider/pull/11
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
rabbah opened a new pull request #4:
URL: https://github.com/apache/openwhisk-runtime-deno/pull/4
This add some ancillary files (License, contribution guide, gradle) and
splits them out of #3 to reduce the initial runtime PR.
rabbah opened a new pull request #5:
URL: https://github.com/apache/openwhisk-runtime-deno/pull/5
This PR reduces the change set to introduce the Deno runtime implementation
to the salient code.
This is an automated
rabbah commented on a change in pull request #3:
URL:
https://github.com/apache/openwhisk-runtime-deno/pull/3#discussion_r584954875
##
File path: .travis.yml
##
@@ -0,0 +1,42 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license
rabbah closed pull request #3:
URL: https://github.com/apache/openwhisk-runtime-deno/pull/3
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above
rabbah commented on pull request #3:
URL:
https://github.com/apache/openwhisk-runtime-deno/pull/3#issuecomment-788201501
Replaced by two smaller PRs #4 and #5.
This is an automated message from the Apache Git Service.
To
rabbah commented on a change in pull request #5:
URL:
https://github.com/apache/openwhisk-runtime-deno/pull/5#discussion_r585110269
##
File path: .travis.yml
##
@@ -25,3 +25,18 @@ before_install:
install: true
script:
- "./tools/travis/scan.sh && ./tools/travis/build.sh &&
1 - 100 of 110 matches
Mail list logo