dubeejw commented on issue #2847: Treat action code as attachments for
created/updated actions
URL:
https://github.com/apache/incubator-openwhisk/pull/2847#issuecomment-355719589
Pushed changes to use ansible/files/runtimes.json since the runtimes were
removed from
mrutkows opened a new pull request #685: WIP: Test
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/685
This is an automated message from the Apache Git Service.
To respond to the message, please log on
rabbah commented on issue #3151: invoker should not reuse warm container of a
deleted action
URL:
https://github.com/apache/incubator-openwhisk/issues/3151#issuecomment-355710182
This is I think a duplicate of
https://github.com/apache/incubator-openwhisk/issues/3120
tysonnorris opened a new issue #3151: invoker should not reuse warm container
of a deleted action
URL: https://github.com/apache/incubator-openwhisk/issues/3151
## Environment details:
* local deployment, vagrant, native ubuntu, Mac OS, Bluemix, ...
* version of
jonpspri commented on issue #184: Create tarball for Nginx container in
OpenWhisk as part of release deploy script
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/184#issuecomment-355691228
Good question marks. I'm going to come at them sideways first, because I
want to be
dgrove-oss commented on issue #140: use runtimesManifest to drive invoker
'docker pull'
URL:
https://github.com/apache/incubator-openwhisk-deploy-kube/pull/140#issuecomment-355686217
@rabbah - ready for review & merge
This
dgrove-oss commented on issue #141: exit job's init.sh with error if subcommand
fails
URL:
https://github.com/apache/incubator-openwhisk-deploy-kube/pull/141#issuecomment-355686148
@rabbah - ready for review & merge
This is
dgrove-oss opened a new pull request #141: exit job's init.sh with error if
subcommand fails
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/141
Fixes #136
This is an automated message from the Apache Git
dubeejw commented on issue #2077: Add NodeJS 8 initial support for Serverless
OpenWhisk
URL:
https://github.com/apache/incubator-openwhisk/issues/2077#issuecomment-355677755
@csantanapr, can this be closed?
This is an
dubeejw commented on issue #2966: WIP: Path parameter support for API GW
URL:
https://github.com/apache/incubator-openwhisk/pull/2966#issuecomment-355672259
@mdeuser, any more comments?
This is an automated message from the
dubeejw commented on issue #2538: vagrant "hello" script fails
URL:
https://github.com/apache/incubator-openwhisk/issues/2538#issuecomment-355667908
@csantanapr, is this still valid?
This is an automated message from the
dubeejw closed issue #2706: Syntax error in tasks/patchKernel.yml?
URL: https://github.com/apache/incubator-openwhisk/issues/2706
This is an automated message from the Apache Git Service.
To respond to the message, please
dgrove-oss opened a new pull request #140: use runtimesManifest to drive
invoker 'docker pull'
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/140
Replace hard-wired list of images to pull from invoker.yml
by an initContainer that interprets runtimesManifest to
jessealva commented on issue #170: WIP: Initial code checkin for path parameter
support
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/170#issuecomment-355665112
@dubeejw was still waiting on apache/incubator-openwhisk#2966 saw that the
stuff i was working on there was
bwmcadams commented on issue #2956: Introduce separate Akka dispatchers for
CouchDB and Kafka Clients
URL:
https://github.com/apache/incubator-openwhisk/pull/2956#issuecomment-355663811
@dubeejw the intent is still there; I have to check the upstream changes
though. Will update.
dubeejw closed pull request #3046: remove invoker.instances from
whisk.properties template
URL: https://github.com/apache/incubator-openwhisk/pull/3046
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
dubeejw closed issue #2867: Does "invoker.instances" still have any usages?
URL: https://github.com/apache/incubator-openwhisk/issues/2867
This is an automated message from the Apache Git Service.
To respond to the message,
dubeejw commented on issue #2956: Introduce separate Akka dispatchers for
CouchDB and Kafka Clients
URL:
https://github.com/apache/incubator-openwhisk/pull/2956#issuecomment-355660504
@bwmcadams, is this PR still valid?
dubeejw closed issue #3034: Loglevel should be configurable via ansible
URL: https://github.com/apache/incubator-openwhisk/issues/3034
This is an automated message from the Apache Git Service.
To respond to the message,
dubeejw commented on issue #3034: Loglevel should be configurable via ansible
URL:
https://github.com/apache/incubator-openwhisk/issues/3034#issuecomment-355659408
Closing as fixed.
This is an automated message from the
dubeejw closed issue #2983: 'Getting Started' guide assumes I have installed
the CLI
URL: https://github.com/apache/incubator-openwhisk/issues/2983
This is an automated message from the Apache Git Service.
To respond to the
dubeejw commented on issue #2983: 'Getting Started' guide assumes I have
installed the CLI
URL:
https://github.com/apache/incubator-openwhisk/issues/2983#issuecomment-355656477
Closing as fixed.
This is an automated message
rabbah closed issue #3120: WhiskAction and ExecutableWhiskAction equality
ignores DocRevision
URL: https://github.com/apache/incubator-openwhisk/issues/3120
This is an automated message from the Apache Git Service.
To
houshengbo commented on issue #1: Set up the Travis infrastructure
URL:
https://github.com/apache/incubator-openwhisk-release/pull/1#issuecomment-355656069
@mrutkows That is the problem. We should either get rid of the time limit,
or use other build environment.
dubeejw commented on issue #3120: WhiskAction and ExecutableWhiskAction
equality ignores DocRevision
URL:
https://github.com/apache/incubator-openwhisk/issues/3120#issuecomment-355654391
@rabbah, can this one be closed?
dubeejw commented on issue #2795: enable concurrent activation processing
URL:
https://github.com/apache/incubator-openwhisk/pull/2795#issuecomment-355651243
@tysonnorris, is this PR still valid?
This is an automated message
houshengbo commented on issue #184: Create tarball for Nginx container in
OpenWhisk as part of release deploy script
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/184#issuecomment-355650644
@jonpspri If I understand it correctly, we are about to use content.json to
save the
rabbah commented on issue #2972: "package update" can create a package with a
reserved name
URL:
https://github.com/apache/incubator-openwhisk/issues/2972#issuecomment-355650046
Fixed by https://github.com/apache/incubator-openwhisk/pull/3147.
rabbah closed issue #2972: "package update" can create a package with a
reserved name
URL: https://github.com/apache/incubator-openwhisk/issues/2972
This is an automated message from the Apache Git Service.
To respond to
rabbah closed pull request #3150: singleton to hold PureConfig keys
URL: https://github.com/apache/incubator-openwhisk/pull/3150
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
dubeejw commented on issue #2941: Add api gateway test that uses custom package
for action
URL:
https://github.com/apache/incubator-openwhisk/pull/2941#issuecomment-355648561
@jsanda, Travis failed with `:tests:checkTestScalafmt FAILED`. Can you
please run `gradle scalafmtAll` and then
jonpspri commented on a change in pull request #184: Create tarball for Nginx
container in OpenWhisk as part of release deploy script
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/184#discussion_r159958452
##
File path: config/content.json
##
@@ -0,0 +1,49
mrutkows opened a new pull request #684: WIP: Update WhiskClient.go to use i18n
IDs.
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/684
This is an automated message from the Apache Git Service.
To respond
dubeejw commented on issue #38: Add test cases regarding the action
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/38#issuecomment-355640825
@houshengbo, is this PR still valid?
This is an automated message from
dubeejw commented on issue #170: WIP: Initial code checkin for path parameter
support
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/170#issuecomment-355639993
@jessealva, what's the status of this PR?
This is
dubeejw commented on issue #46: Add retry support to every request
URL:
https://github.com/apache/incubator-openwhisk-client-go/pull/46#issuecomment-355639666
@houshengbo, what's the status of this PR?
This is an automated
dubeejw closed pull request #53: Fix typos/grammar in trace source file.
URL: https://github.com/apache/incubator-openwhisk-client-go/pull/53
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
dubeejw commented on a change in pull request #184: Create tarball for Nginx
container in OpenWhisk as part of release deploy script
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/184#discussion_r159955704
##
File path: config/content.json
##
@@ -0,0 +1,49
dubeejw commented on a change in pull request #184: Create tarball for Nginx
container in OpenWhisk as part of release deploy script
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/184#discussion_r159952947
##
File path: config/content.json
##
@@ -0,0 +1,49
dubeejw commented on issue #185: Move routemgmt src and install scripts from
main openwhisk repo
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/185#issuecomment-355634935
@csantanapr, please review.
This is an
dubeejw commented on issue #185: Move routemgmt src and install scripts from
main openwhisk repo
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/185#issuecomment-355634605
After conversations with @mdeuser and @csantanapr it sounds like we are
going to host the route
dubeejw closed pull request #3149: Cleanup test assets between unit tests using
afterEach
URL: https://github.com/apache/incubator-openwhisk/pull/3149
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
dubeejw commented on issue #3146: Use cache invalidation policy based on access
time
URL:
https://github.com/apache/incubator-openwhisk/pull/3146#issuecomment-355444697
PG3 1605 ?
This is an automated message from the
dgrove-oss commented on issue #3150: singleton to hold PureConfig keys
URL:
https://github.com/apache/incubator-openwhisk/pull/3150#issuecomment-355630216
PG2 2591 ? -- 1 test failure that can't be due to this change.
This
dubeejw closed pull request #3144: Env variable instead of parameter
URL: https://github.com/apache/incubator-openwhisk/pull/3144
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
dgrove-oss commented on a change in pull request #3148: Make memory limits as
config variables
URL:
https://github.com/apache/incubator-openwhisk/pull/3148#discussion_r159946732
##
File path: ansible/roles/controller/tasks/deploy.yml
##
@@ -82,6 +82,10 @@
dgrove-oss commented on issue #3148: Make memory limits as config variables
URL:
https://github.com/apache/incubator-openwhisk/pull/3148#issuecomment-355628015
I think the problem is that you didn't add the default values to
application.conf. You need to add them in the "whisk" stanza
dubeejw closed pull request #3147: Disallow create and update of package with
reserved names.
URL: https://github.com/apache/incubator-openwhisk/pull/3147
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
dubeejw commented on issue #3146: Use cache invalidation policy based on access
time
URL:
https://github.com/apache/incubator-openwhisk/pull/3146#issuecomment-355626124
PG2 2589 ?
This is an automated message from the
mrutkows commented on issue #1: Set up the Travis infrastructure
URL:
https://github.com/apache/incubator-openwhisk-release/pull/1#issuecomment-355625670
Apparently, you cannot wait for 65 minutes...
```
$ ./tools/travis/setup.sh
hello openwhisk release
$ travis_wait 66
jthomas commented on issue #1414: Activation API Response Should Include
Pagination Details
URL:
https://github.com/apache/incubator-openwhisk/issues/1414#issuecomment-355614184
This has been [an issue for me again in another
duynguyen commented on issue #3148: Make memory limits as config variables
URL:
https://github.com/apache/incubator-openwhisk/pull/3148#issuecomment-355612829
@dgrove-oss afaics the tests are failing because those pureconfig values are
not set in test setup. Do you have any pointer for
dgrove-oss commented on a change in pull request #3148: Make memory limits as
config variables
URL:
https://github.com/apache/incubator-openwhisk/pull/3148#discussion_r159924292
##
File path: common/scala/src/main/scala/whisk/core/entity/MemoryLimit.scala
##
@@ -41,12
houshengbo closed pull request #679: Use proper IDs when referencing i18n
strings. (Base changes + phase 1 conversions)
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/679
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is
mrutkows commented on issue #679: Use proper IDs when referencing i18n strings.
(Base changes + phase 1 conversions)
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/679#issuecomment-355599054
So far, this exercise has uncovered missing translations, translations not
mrutkows commented on issue #679: Use proper IDs when referencing i18n strings.
(Base changes + phase 1 conversions)
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/679#issuecomment-355599054
So far, this exercise has uncovered missing translations, translations not
mcdan commented on issue #77: Fixes a few linux issues with the makefile
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/77#issuecomment-355598961
This depends on PR #74 and #79 and should fix the build.
mrutkows commented on issue #679: Use proper IDs when referencing i18n strings.
(Base changes + phase 1 conversions)
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/679#issuecomment-355598278
This PR lays the groundwork for the change and includes about 1/3 of the
dgrove-oss commented on issue #3150: singleton to hold PureConfig keys
URL:
https://github.com/apache/incubator-openwhisk/pull/3150#issuecomment-355597576
PG2 2591 ?
This is an automated message from the Apache Git Service.
dgrove-oss opened a new pull request #3150: singleton to hold PureConfig keys
URL: https://github.com/apache/incubator-openwhisk/pull/3150
As suggested by @rabbah in review of another PR, introduce
a singleton to hold the various String keys used to load
config data via PureConfig to
jonpspri commented on a change in pull request #184: Create tarball for Nginx
container in OpenWhisk as part of release deploy script
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/184#discussion_r159913734
##
File path: config/content.json
##
@@ -0,0 +1,49
beemarie commented on issue #3144: Env variable instead of parameter
URL:
https://github.com/apache/incubator-openwhisk/pull/3144#issuecomment-355589997
? PG2:2583
This is an automated message from the Apache Git
beemarie commented on issue #3144: Env variable instead of parameter
URL:
https://github.com/apache/incubator-openwhisk/pull/3144#issuecomment-355589997
PG:
https://openwhisk-jenkins.swg-devops.com:8443/view/Playgrounds/job/Playground2/2583/
mcdan opened a new pull request #79: Fix #78 - update docker compose version
URL: https://github.com/apache/incubator-openwhisk-devtools/pull/79
This is an automated message from the Apache Git Service.
To respond to the
mcdan opened a new issue #78: Docker Compose Version variable is incorrect.
URL: https://github.com/apache/incubator-openwhisk-devtools/issues/78
Travis build is broken due to not being able to download the docker-compose.
The version number in the env variable is 1.13 but the version
mrutkows opened a new issue #683: wskdeply cobra command help missing i18n
translations
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/683
Missing _short_ and _long_ utility descriptions in root.go:
```
var RootCmd = {
Use: "wskdeploy",
houshengbo opened a new pull request #1: Set up the Travis infrastructure
URL: https://github.com/apache/incubator-openwhisk-release/pull/1
This is an automated message from the Apache Git Service.
To respond to the message,
mrutkows opened a new issue #682: Add missing i18n descriptions for Cobra
commands in root.go
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/682
Many of the commands were added without adding to the i18n string resource
file.
```
// TODO() i18n
duynguyen commented on issue #72: Updated version of PR-15
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/72#issuecomment-355516839
re. CouchDB snapshot: afaics this image works:
https://hub.docker.com/r/openwhisk/kube-couchdb/
I can work on a separate PR to update
duynguyen commented on a change in pull request #70: Use init.sh instead of
direct command
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/70#discussion_r159841476
##
File path: docker-compose/docker-compose.yml
##
@@ -38,7 +38,7 @@ services:
# WHISK
70 matches
Mail list logo
