tz70s closed pull request #3469: Ensure clean up testspace under wskadmin test.
URL: https://github.com/apache/incubator-openwhisk/pull/3469
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
tz70s commented on issue #3469: Ensure clean up testspace under wskadmin test.
URL:
https://github.com/apache/incubator-openwhisk/pull/3469#issuecomment-374837104
Reopen this pr later, b.c. I need to figure out why the test got failed and
avoid disturbing upstream travis build.
mrutkows closed pull request #811: Deprecating WithinOpenWhisk flag
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/811
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
mrutkows closed issue #791: Crash when running with __OW_API_KEY and
__OW_API_HOST set
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/791
This is an automated message from the Apache Git Service.
To
csantanapr closed pull request #34: Remove swift:4.0, only swift:4.1
URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/34
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
csantanapr closed issue #30: Remove Swift 4.0, only support 4.1 and forward
URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/30
This is an automated message from the Apache Git Service.
To respond to
dubee closed pull request #3472: Remove email address
URL: https://github.com/apache/incubator-openwhisk/pull/3472
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 request
dubee closed pull request #250: Remove email addresses
URL: https://github.com/apache/incubator-openwhisk-cli/pull/250
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
dubee commented on issue #250: Remove email addresses
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/250#issuecomment-374769905
Travis failed because of intermittent test failure to be fixed in this PR
https://github.com/apache/incubator-openwhisk-cli/pull/247.
mdeuser commented on issue #3464: Add wsk action get to CLI docs
URL:
https://github.com/apache/incubator-openwhisk/pull/3464#issuecomment-374765536
@dubee changes look ok... although i think this section probably needs
additional rework to better integrate the web action aspects (web
pritidesai commented on issue #811: Deprecating WithinOpenWhisk flag
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/811#issuecomment-374763837
@beemarie `exec` deals with the set of env. variables specified in
execOptions, its nothing to do with `wskdeploy`. Its more
dubee commented on a change in pull request #247: Fix intermittent test failure
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/247#discussion_r175927899
##
File path: tests/src/test/scala/system/basic/WskBasicTests.scala
##
@@ -549,13 +549,18 @@ class
dubee closed pull request #249: Use proper license headers
URL: https://github.com/apache/incubator-openwhisk-cli/pull/249
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
houshengbo commented on issue #251: Add the support to specify the version of
the binaries
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/251#issuecomment-374760632
@jonpspri Please help me review this PR, thanks.
dubee closed pull request #248: Fix test to not compare namespace to subject
URL: https://github.com/apache/incubator-openwhisk-cli/pull/248
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
dgrove-oss commented on issue #167: Add webhook for slackbot notification of
TravisCI jobs
URL:
https://github.com/apache/incubator-openwhisk-deploy-kube/pull/167#issuecomment-374759296
changed as requested; ready to merge.
beemarie commented on issue #811: Deprecating WithinOpenWhisk flag
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/811#issuecomment-374756424
@pritidesai @mrutkows The code priti posted is how wskdeploy is being called
from the deploy package. The environment data
houshengbo opened a new pull request #251: Add the support to specify the
version of the binaries
URL: https://github.com/apache/incubator-openwhisk-cli/pull/251
This PR changes the task name from release to releaseBinaries in
order to make it consistent in the release process. It also
houshengbo commented on issue #63: Verify API Gateway legal NOTICE / CREDITS
files ack. licenses/contribs.
URL:
https://github.com/apache/incubator-openwhisk-release/issues/63#issuecomment-374753437
closes: https://github.com/apache/incubator-openwhisk-release/issues/9
houshengbo commented on issue #63: Verify API Gateway legal NOTICE / CREDITS
files ack. licenses/contribs.
URL:
https://github.com/apache/incubator-openwhisk-release/issues/63#issuecomment-374753437
closes: https://github.com/apache/incubator-openwhisk-release/issues/9
dubee opened a new pull request #3472: Remove email address
URL: https://github.com/apache/incubator-openwhisk/pull/3472
Removes an email address from API GW tests.
## Description
## Related issue and scope
- [ ] I opened an issue to propose and discuss this
dubee opened a new pull request #250: Remove email addresses
URL: https://github.com/apache/incubator-openwhisk-cli/pull/250
Scrub email address from a couple of files.
This is an automated message from the Apache Git
mdeuser commented on a change in pull request #247: Fix intermittent test
failure
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/247#discussion_r175916324
##
File path: tests/src/test/scala/system/basic/WskBasicTests.scala
##
@@ -549,13 +549,18 @@ class
dubee opened a new pull request #249: Use proper license headers
URL: https://github.com/apache/incubator-openwhisk-cli/pull/249
The wrong license headers were being used in a couple of files.
This is an automated message
pritidesai commented on issue #811: Deprecating WithinOpenWhisk flag
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/811#issuecomment-374748897
@mrutkows based on the `exec` command from [package
pritidesai commented on issue #811: Deprecating WithinOpenWhisk flag
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/811#issuecomment-374748897
@mrutkows based on the `exec` command from [package
dubee opened a new pull request #248: Fix test to not compare namespace to
subject
URL: https://github.com/apache/incubator-openwhisk-cli/pull/248
`create a trigger, and fire a trigger to get its individual fields from an
activation` directly compares a namespace to a subject which will
mdeuser commented on a change in pull request #3464: Add wsk action get to CLI
docs
URL:
https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175910182
##
File path: docs/actions.md
##
@@ -31,6 +31,7 @@ Learn how to create, invoke, and debug actions in
mdeuser commented on a change in pull request #3464: Add wsk action get to CLI
docs
URL:
https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175907988
##
File path: docs/actions.md
##
@@ -1219,6 +1184,71 @@ This command starts a polling loop that
mdeuser commented on a change in pull request #244: Add --web-secure option to
action create/update
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175903104
##
File path: commands/action.go
##
@@ -442,14 +457,109 @@ func parseAction(cmd
dubee opened a new pull request #247: Fix intermittent test failure
URL: https://github.com/apache/incubator-openwhisk-cli/pull/247
`create, and get a trigger summary` fails intermittently as the returned
annotations array is not always in the expected order.
mdeuser commented on a change in pull request #244: Add --web-secure option to
action create/update
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175904374
##
File path: commands/action.go
##
@@ -442,14 +457,109 @@ func parseAction(cmd
mrutkows commented on issue #791: Crash when running with __OW_API_KEY and
__OW_API_HOST set
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/791#issuecomment-374736872
@pritidesai it seems that package-deploy does not use them, but did ask for
Belinda to affirm
mrutkows commented on issue #811: Deprecating WithinOpenWhisk flag
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/811#issuecomment-374736355
@beemarie Can you confirm that you no longer need to pass in arguments via a
JSON structure when running 'wskdeploy' as an
mdeuser commented on a change in pull request #244: Add --web-secure option to
action create/update
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175903104
##
File path: commands/action.go
##
@@ -442,14 +457,109 @@ func parseAction(cmd
mdeuser commented on issue #246: Fix Go regex for path parameter parsing
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/246#issuecomment-374731987
@drcariel -- would you mind reviewing
This is an automated
mrutkows commented on issue #807: Deprecating Interactive Mode and introducing
Preview
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/807#issuecomment-374729320
@csantanapr today!
This is an automated
mrutkows closed pull request #807: Deprecating Interactive Mode and introducing
Preview
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/807
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
mrutkows closed issue #693: whiskclient_test.go:
"TestNewWhiskConfigInteractiveMode" incomplete/commented out
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/693
This is an automated message from the
mrutkows closed issue #758: Deprecate interactive mode in wskdeploy
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/758
This is an automated message from the Apache Git Service.
To respond to the message,
bwmcadams opened a new pull request #3471: Fixes #3470 - Changed hardcoded
timeouts for idle containers and pause grace to config entries
URL: https://github.com/apache/incubator-openwhisk/pull/3471
## Description
Previously, the `idleTimeout` and `pauseGrace` settings
bwmcadams opened a new issue #3470: Change ContainerProxy `unusedTimeout` and
`pauseGrace` from hardcoded defaults to config driven
URL: https://github.com/apache/incubator-openwhisk/issues/3470
## Steps to reproduce the issue:
1. Currently, `unusedTimeout` and `pauseGrace`
dubee commented on a change in pull request #3464: Add wsk action get to CLI
docs
URL:
https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175886022
##
File path: docs/actions.md
##
@@ -1219,6 +1184,71 @@ This command starts a polling loop that
dubee commented on a change in pull request #3464: Add wsk action get to CLI
docs
URL:
https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175882446
##
File path: docs/actions.md
##
@@ -1219,6 +1184,71 @@ This command starts a polling loop that
dubee commented on issue #3464: Add wsk action get to CLI docs
URL:
https://github.com/apache/incubator-openwhisk/pull/3464#issuecomment-374713064
@rabbah, @mdeuser, anything else?
This is an automated message from the
dubee commented on a change in pull request #3464: Add wsk action get to CLI
docs
URL:
https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175545924
##
File path: docs/actions.md
##
@@ -1219,6 +1184,71 @@ This command starts a polling loop that
dubee commented on a change in pull request #3464: Add wsk action get to CLI
docs
URL:
https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175876601
##
File path: docs/actions.md
##
@@ -31,6 +31,7 @@ Learn how to create, invoke, and debug actions in your
dubee commented on a change in pull request #3464: Add wsk action get to CLI
docs
URL:
https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175876068
##
File path: docs/actions.md
##
@@ -1219,6 +1184,71 @@ This command starts a polling loop that
dubee commented on a change in pull request #3464: Add wsk action get to CLI
docs
URL:
https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175876068
##
File path: docs/actions.md
##
@@ -1219,6 +1184,71 @@ This command starts a polling loop that
dubee commented on a change in pull request #3464: Add wsk action get to CLI
docs
URL:
https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175873830
##
File path: docs/actions.md
##
@@ -1219,6 +1184,71 @@ This command starts a polling loop that
dubee commented on a change in pull request #3464: Add wsk action get to CLI
docs
URL:
https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175865776
##
File path: docs/actions.md
##
@@ -1219,6 +1184,71 @@ This command starts a polling loop that
dubee commented on a change in pull request #3464: Add wsk action get to CLI
docs
URL:
https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175865776
##
File path: docs/actions.md
##
@@ -1219,6 +1184,71 @@ This command starts a polling loop that
dubee commented on a change in pull request #244: Add --web-secure option to
action create/update
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175850403
##
File path: tests/src/test/scala/whisk/core/cli/test/WskBasicUsageTests.scala
##
@@
dubee commented on a change in pull request #244: Add --web-secure option to
action create/update
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175849808
##
File path: commands/action.go
##
@@ -442,14 +457,109 @@ func parseAction(cmd
dubee commented on a change in pull request #244: Add --web-secure option to
action create/update
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175849353
##
File path: commands/action.go
##
@@ -442,14 +457,109 @@ func parseAction(cmd
dubee commented on a change in pull request #244: Add --web-secure option to
action create/update
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175841358
##
File path: commands/action.go
##
@@ -442,14 +457,109 @@ func parseAction(cmd
dubee commented on a change in pull request #244: Add --web-secure option to
action create/update
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175840729
##
File path: commands/action.go
##
@@ -442,14 +457,109 @@ func parseAction(cmd
dubee commented on a change in pull request #244: Add --web-secure option to
action create/update
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175848990
##
File path: commands/action.go
##
@@ -442,14 +457,109 @@ func parseAction(cmd
tz70s opened a new pull request #3469: Ensure clean up testspace under wskadmin
test.
URL: https://github.com/apache/incubator-openwhisk/pull/3469
Fix up testsuite `block and unblock a user respectively` fails when
`testspace` contains pre-existing users.
Signed-off-by: Tzu-Chiao
mdeuser commented on a change in pull request #3464: Add wsk action get to CLI
docs
URL:
https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175824240
##
File path: docs/actions.md
##
@@ -31,6 +31,7 @@ Learn how to create, invoke, and debug actions in
mdeuser commented on a change in pull request #3464: Add wsk action get to CLI
docs
URL:
https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175823953
##
File path: docs/actions.md
##
@@ -31,6 +31,7 @@ Learn how to create, invoke, and debug actions in
chetanmeh commented on issue #3412: Changes to runtimes variables in support of
retiring 'whisk/*' runtimes
URL:
https://github.com/apache/incubator-openwhisk/pull/3412#issuecomment-374636516
> Ideally in this scenery, one would use ‘shell:’ to invoke an entry point
in the invoker jar
jonpspri commented on issue #3412: Changes to runtimes variables in support of
retiring 'whisk/*' runtimes
URL:
https://github.com/apache/incubator-openwhisk/pull/3412#issuecomment-374627781
I was thinking about something along @chetanmeh ‘s way of thinking.
Although I’d rather go the
rabbah commented on a change in pull request #3435: CLI docs update and
markdown linting.
URL:
https://github.com/apache/incubator-openwhisk/pull/3435#discussion_r175794906
##
File path: ansible/README.md
##
@@ -172,24 +172,44 @@ Use `ansible-playbook -i environments/
jonpspri commented on a change in pull request #3435: CLI docs update and
markdown linting.
URL:
https://github.com/apache/incubator-openwhisk/pull/3435#discussion_r175794097
##
File path: ansible/README.md
##
@@ -172,24 +172,44 @@ Use `ansible-playbook -i environments/
jonpspri commented on issue #3467: Remove actionControllers tests from
incubator-openwhisk repo
URL:
https://github.com/apache/incubator-openwhisk/pull/3467#issuecomment-374624880
Maybe it can be, but should we move it into a different namespace from the
runtime-specific code (maybe also
tz70s opened a new issue #3468: Docker daemon error under ubuntu xenial
URL: https://github.com/apache/incubator-openwhisk/issues/3468
## Environment details:
Local deployment under ubuntu xenial.
## Steps to reproduce the issue:
```bash
# Equivalent steps in
dgrove-oss commented on issue #3412: Changes to runtimes variables in support
of retiring 'whisk/*' runtimes
URL:
https://github.com/apache/incubator-openwhisk/pull/3412#issuecomment-374607395
I'd like to have just one place to get these defaults. I like @chetanmeh's
suggestion of
rabbah commented on issue #3467: Remove actionControllers tests from
incubator-openwhisk repo
URL:
https://github.com/apache/incubator-openwhisk/pull/3467#issuecomment-374605804
We will need to designate a single place as the canonical root for the
common code. Maybe it can still be this
jonpspri opened a new pull request #3467: Remove actionControllers tests from
incubator-openwhisk repo
URL: https://github.com/apache/incubator-openwhisk/pull/3467
## Description
When the various action controllers were spun out into their own
repositories, they took their test
jonpspri commented on issue #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java
runtime for Java actions
URL:
https://github.com/apache/incubator-openwhisk-runtime-java/pull/24#issuecomment-374599948
@Param-S The same unicode test case as above. Hold tight — I seem to have
some classpath
chetanmeh commented on issue #3453: Introduce a AttachmentStore SPI
URL:
https://github.com/apache/incubator-openwhisk/pull/3453#issuecomment-374598806
> dont follow the need for the volatile attachments.
The updates to state are performed via stream stage and hence is single
chetanmeh commented on a change in pull request #3453: Introduce a
AttachmentStore SPI
URL:
https://github.com/apache/incubator-openwhisk/pull/3453#discussion_r175763836
##
File path:
common/scala/src/main/scala/whisk/core/database/AttachmentStore.scala
##
@@ -0,0 +1,51
csantanapr opened a new pull request #34: Remove swift:4.0, only swift:4.1
URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/34
Remove swift:4.0 as swift:4.1 is the only right now being used.
This is an
tz70s opened a new pull request #3466: Support alternative path for python in
whiskprops
URL: https://github.com/apache/incubator-openwhisk/pull/3466
By default, the path of python2.7 is located at `/usr/bin/`
instead of `/usr/local/bin/`. Since folks may desire to use homebrew
jonpspri commented on issue #3412: Changes to runtimes variables in support of
retiring 'whisk/*' runtimes
URL:
https://github.com/apache/incubator-openwhisk/pull/3412#issuecomment-374580802
Hi @dgrove-oss, I missed that memo, so forgive me if I’m catching up.
Currently there’s logic in
jonpspri commented on issue #3457: Ansible 2.4 deprecates ‘include’ in favor of
‘import_*’ and ‘include_tasks’
URL:
https://github.com/apache/incubator-openwhisk/pull/3457#issuecomment-374578440
Agreed re PG. WIll do it with @csantanapr later today.
dgrove-oss commented on issue #3412: Changes to runtimes variables in support
of retiring 'whisk/*' runtimes
URL:
https://github.com/apache/incubator-openwhisk/pull/3412#issuecomment-374577087
I thought we decided to move the misc defaults to pureconfig
dgrove-oss commented on issue #3412: Changes to runtimes variables in support
of retiring 'whisk/*' runtimes
URL:
https://github.com/apache/incubator-openwhisk/pull/3412#issuecomment-374577087
I thought we decided to move the misc defaults to pureconfig
dgrove-oss commented on issue #3412: Changes to runtimes variables in support
of retiring 'whisk/*' runtimes
URL:
https://github.com/apache/incubator-openwhisk/pull/3412#issuecomment-374577087
I thought we decided to move the misc defaults to pureconfig
mdeuser commented on a change in pull request #3464: Add wsk action get to CLI
docs
URL:
https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175737539
##
File path: docs/actions.md
##
@@ -1219,6 +1184,71 @@ This command starts a polling loop that
csantanapr commented on issue #3208: Source files for images in docs folder
URL:
https://github.com/apache/incubator-openwhisk/issues/3208#issuecomment-374567561
@shazron
Which open source tools could we use to edit PSD/EPS?
I’m +1 with draw.io
sciabarracom commented on issue #319: plugable model for adding action language
support
URL:
https://github.com/apache/incubator-openwhisk/issues/319#issuecomment-374524170
HI Imesh actually the "pluggable" support is already written. It lives in
imesh commented on issue #319: plugable model for adding action language support
URL:
https://github.com/apache/incubator-openwhisk/issues/319#issuecomment-374522746
Thanks @rabbah for the update and all your help on implementing it!
cbickel commented on a change in pull request #3342: #3168 Rate limit requests
to download CLI, Docker and iOS SDKs
URL:
https://github.com/apache/incubator-openwhisk/pull/3342#discussion_r175682262
##
File path: ansible/roles/nginx/templates/nginx.conf.j2
##
@@ -94,18
cbickel commented on issue #3342: #3168 Rate limit requests to download CLI,
Docker and iOS SDKs
URL:
https://github.com/apache/incubator-openwhisk/pull/3342#issuecomment-374511609
@mptap
Thanks a lot for your contribution. I like the idea of limiting the download
of some
cbickel commented on a change in pull request #3451: Fix several loadbalancer
bugs.
URL:
https://github.com/apache/incubator-openwhisk/pull/3451#discussion_r175671605
##
File path:
core/controller/src/main/scala/whisk/core/loadBalancer/ShardingContainerPoolBalancer.scala
cbickel commented on a change in pull request #3451: Fix several loadbalancer
bugs.
URL:
https://github.com/apache/incubator-openwhisk/pull/3451#discussion_r175671248
##
File path:
core/controller/src/main/scala/whisk/core/loadBalancer/ShardingContainerPoolBalancer.scala
88 matches
Mail list logo