cbickel opened a new pull request #3260: Enhance the RestResult with more
information to debug failing tests e?
URL: https://github.com/apache/incubator-openwhisk/pull/3260
?asier.
I had to debug a failing test case. But most information from the RestResult
was hidden. Only the info
markusthoemmes commented on a change in pull request #3260: Enhance the
RestResult with more information to debug failing tests e?
URL:
https://github.com/apache/incubator-openwhisk/pull/3260#discussion_r166878476
##
File path: tests/src/test/scala/common/rest/WskRest.scala
##
mhaack commented on issue #3241: Impossible the see usage of a shared package
URL:
https://github.com/apache/incubator-openwhisk/issues/3241#issuecomment-364059971
I think of a list of namespaces that have bindings to my public packages.
The intention is to get at least a list of namespace
mhaack commented on issue #3241: Impossible the see usage of a shared package
URL:
https://github.com/apache/incubator-openwhisk/issues/3241#issuecomment-364059971
I think that's a list of namespaces that have bindings to my public
packages. The intention is to get at least a list of names
rabbah commented on a change in pull request #3240: Add a loadbalancer with
local state and horizontal invoker sharding.
URL:
https://github.com/apache/incubator-openwhisk/pull/3240#discussion_r166901098
##
File path:
core/controller/src/main/scala/whisk/core/loadBalancer/Sharding
rabbah commented on a change in pull request #3240: Add a loadbalancer with
local state and horizontal invoker sharding.
URL:
https://github.com/apache/incubator-openwhisk/pull/3240#discussion_r166899101
##
File path: common/scala/src/main/scala/whisk/common/ForcableSemaphore.scala
rabbah commented on a change in pull request #3240: Add a loadbalancer with
local state and horizontal invoker sharding.
URL:
https://github.com/apache/incubator-openwhisk/pull/3240#discussion_r166897507
##
File path: common/scala/src/main/scala/whisk/common/ForcableSemaphore.scala
rabbah commented on a change in pull request #3240: Add a loadbalancer with
local state and horizontal invoker sharding.
URL:
https://github.com/apache/incubator-openwhisk/pull/3240#discussion_r166900059
##
File path:
core/controller/src/main/scala/whisk/core/loadBalancer/Sharding
rabbah commented on issue #3241: Impossible the see usage of a shared package
URL:
https://github.com/apache/incubator-openwhisk/issues/3241#issuecomment-364082032
For this kind of thing why not use versioning; it would be a better
approach. You could add a version to the package name (ver
daisy-ycguo commented on issue #709: Add Apache License
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/709#issuecomment-364091569
I added more Apache license before those source files which are missing
headers. Below files are exceptions which are tracked by `.rat-exclud
mhaack commented on issue #3241: Impossible the see usage of a shared package
URL:
https://github.com/apache/incubator-openwhisk/issues/3241#issuecomment-364093548
I agree on the package versioning, that is and idea we are currently looking
into. Copying is out of scope, if some copies an
rabbah commented on issue #3241: Impossible the see usage of a shared package
URL:
https://github.com/apache/incubator-openwhisk/issues/3241#issuecomment-364099128
If the goal is to prevent breaking changes, then versioning is the preferred
route.
We could provide a count of references
rabbah commented on issue #3241: Impossible the see usage of a shared package
URL:
https://github.com/apache/incubator-openwhisk/issues/3241#issuecomment-364100745
A related thought - today when you invoke an action through a binding, the
resolution to the package happens fairly early on.
rabbah commented on issue #3241: Impossible the see usage of a shared package
URL:
https://github.com/apache/incubator-openwhisk/issues/3241#issuecomment-364100745
A related thought - today when you invoke an action through a binding, the
resolution to the package happens fairly early on.
rabbah commented on issue #3241: Impossible the see usage of a shared package
URL:
https://github.com/apache/incubator-openwhisk/issues/3241#issuecomment-364100745
A related thought - today when you invoke an action through a binding, the
resolution to the package happens fairly early on.
rabbah commented on issue #3241: Impossible the see usage of a shared package
URL:
https://github.com/apache/incubator-openwhisk/issues/3241#issuecomment-364100745
A related thought - today when you invoke an action through a binding, the
resolution to the package happens fairly early on.
dubeejw commented on a change in pull request #3254: Do not disable rule when
it is updated
URL:
https://github.com/apache/incubator-openwhisk/pull/3254#discussion_r166935522
##
File path: core/controller/src/main/scala/whisk/core/controller/Rules.scala
##
@@ -313,7 +314,
markusthoemmes commented on a change in pull request #3240: Add a loadbalancer
with local state and horizontal invoker sharding.
URL:
https://github.com/apache/incubator-openwhisk/pull/3240#discussion_r166954536
##
File path: common/scala/src/main/scala/whisk/common/ForcableSemapho
tardieu opened a new pull request #3261: Pin docker engine version on ubuntu
URL: https://github.com/apache/incubator-openwhisk/pull/3261
We require a specific docker engine version on ubuntu. We should pin the
package so that it does not get automatically updated when running system-wide
cbickel closed pull request #3206: Refactor controller role to use variables
for hostname and host index
URL: https://github.com/apache/incubator-openwhisk/pull/3206
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sak
mrutkows commented on issue #716: Some source files are missing Apache license
headers
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/716#issuecomment-364156898
@daisy-ycguo
So other Apache projects have ASF headers for:
- .md
- .yaml (config files)
b
jasonpet commented on a change in pull request #86: Do not hard code API host
of push notification service
URL:
https://github.com/apache/incubator-openwhisk-package-pushnotifications/pull/86#discussion_r166982594
##
File path: packages/actions/sendMessage.js
##
@@ -426,1
michaelmarth commented on issue #716: Some source files are missing Apache
license headers
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/716#issuecomment-364165889
@bdelacretaz do you happen to have input on the above?
---
houshengbo opened a new pull request #720: Add the support to generate the
"latest" tag automatically
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/720
This is an automated message from the Apache Git Ser
jasonpet opened a new pull request #126: optionally limit cron fields to 5
instead of 6
URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/126
This allows vendors to limit the finest granularity to 1 minute instead of 1
second.
-
csantanapr closed pull request #720: Add the support to generate the "latest"
tag automatically
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/720
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sak
mrutkows opened a new pull request #23: WIP: Add support for a minified ASF
license that uses # hash prefix.
URL: https://github.com/apache/incubator-openwhisk-utilities/pull/23
This is an automated message from the Apache G
mdeuser opened a new pull request #3262: Handle trigger activations with
inactive rules
URL: https://github.com/apache/incubator-openwhisk/pull/3262
- log trigger activation entry for disabled rules (but at least one active
rule is needed to create a trigger activation record)
- if trig
csantanapr closed pull request #86: Do not hard code API host of push
notification service
URL:
https://github.com/apache/incubator-openwhisk-package-pushnotifications/pull/86
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below
csantanapr commented on issue #84: make travis more efficient
URL:
https://github.com/apache/incubator-openwhisk-package-pushnotifications/pull/84#issuecomment-364203880
@drcariel abandoned PR? Can I close it?
This is an auto
lionelvillard commented on issue #705: Unable to use ENV VARS in input JSON
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/705#issuecomment-364205529
Looking at k8s and helm, templating is based on go templates (`{{ .username
}}`). Maybe we could have the same for wskd
rabbah commented on a change in pull request #3262: Handle trigger activations
with inactive rules
URL:
https://github.com/apache/incubator-openwhisk/pull/3262#discussion_r167026849
##
File path: core/controller/src/main/scala/whisk/core/controller/Triggers.scala
##
@@ -3
rabbah commented on a change in pull request #3262: Handle trigger activations
with inactive rules
URL:
https://github.com/apache/incubator-openwhisk/pull/3262#discussion_r167027116
##
File path: core/controller/src/main/scala/whisk/core/controller/Triggers.scala
##
@@ -3
rabbah commented on a change in pull request #3262: Handle trigger activations
with inactive rules
URL:
https://github.com/apache/incubator-openwhisk/pull/3262#discussion_r167026849
##
File path: core/controller/src/main/scala/whisk/core/controller/Triggers.scala
##
@@ -3
ScottChapman commented on issue #705: Unable to use ENV VARS in input JSON
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/705#issuecomment-364207364
I think the issue isn't what syntax is used for injecting env vars, the
issue is that right now the text associated with
mdeuser commented on a change in pull request #3262: Handle trigger activations
with inactive rules
URL:
https://github.com/apache/incubator-openwhisk/pull/3262#discussion_r167031517
##
File path: core/controller/src/main/scala/whisk/core/controller/Triggers.scala
##
@@ -
rabbah commented on a change in pull request #3256: Refactor some bits of the
triggers API.
URL:
https://github.com/apache/incubator-openwhisk/pull/3256#discussion_r167034113
##
File path: core/controller/src/main/scala/whisk/core/controller/Triggers.scala
##
@@ -305,59 +
rabbah commented on a change in pull request #3256: Refactor some bits of the
triggers API.
URL:
https://github.com/apache/incubator-openwhisk/pull/3256#discussion_r167033685
##
File path: core/controller/src/main/scala/whisk/core/controller/Triggers.scala
##
@@ -305,59 +
rabbah commented on a change in pull request #3262: Handle trigger activations
with inactive rules
URL:
https://github.com/apache/incubator-openwhisk/pull/3262#discussion_r167034972
##
File path: core/controller/src/main/scala/whisk/core/controller/Triggers.scala
##
@@ -1
rabbah closed pull request #3261: Pin docker engine version on ubuntu
URL: https://github.com/apache/incubator-openwhisk/pull/3261
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 forei
houshengbo opened a new pull request #721: Comment out the integration tests
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/721
This is an automated message from the Apache Git Service.
To respond to the me
mrutkows closed pull request #721: Comment out the integration tests
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/721
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
pritidesai commented on issue #708: redeploy manifest with rule leaves as
disables
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/708#issuecomment-364230166
I am noticing the same behavior with `wsk` with this sequence of operation:
```
wsk rule create R1 ...
houshengbo opened a new pull request #722: Add the correct api key
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/722
This is an automated message from the Apache Git Service.
To respond to the message, ple
mrutkows opened a new issue #27: Please list Stage 1 release plan includes
"source TAR only" and list included repos.
URL: https://github.com/apache/incubator-openwhisk-release/issues/27
On the main README.md, we need to clearly say what is in Stage 1 release.
That is, it includes so
mrutkows closed pull request #722: Add the correct api key
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/722
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
pritidesai commented on a change in pull request #715: Adding support for
export managed project back to manifest
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/715#discussion_r167052402
##
File path: cmd/export.go
##
@@ -0,0 +1,266 @@
+/*
+ * Licensed
pritidesai commented on a change in pull request #715: Adding support for
export managed project back to manifest
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/715#discussion_r167048358
##
File path: cmd/export.go
##
@@ -0,0 +1,266 @@
+/*
+ * Licensed
houshengbo opened a new pull request #723: Add the integration test back to the
Travis build
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/723
This is an automated message from the Apache Git Service.
To
sciabarracom commented on issue #19: support go actions
URL:
https://github.com/apache/incubator-openwhisk/issues/19#issuecomment-364242226
I am looking to the implementation of the nodejs runner. I wonder if the
right implementation of a go action should not be instead of requiring an htt
sciabarracom commented on issue #19: support go actions
URL:
https://github.com/apache/incubator-openwhisk/issues/19#issuecomment-364242226
I am looking to the implementation of the nodejs runner. I wonder if the
right implementation of a go action should not be instead of requiring an htt
markusthoemmes commented on a change in pull request #3256: Refactor some bits
of the triggers API.
URL:
https://github.com/apache/incubator-openwhisk/pull/3256#discussion_r167062270
##
File path: core/controller/src/main/scala/whisk/core/controller/Triggers.scala
##
@@ -
markusthoemmes commented on a change in pull request #3256: Refactor some bits
of the triggers API.
URL:
https://github.com/apache/incubator-openwhisk/pull/3256#discussion_r167062629
##
File path: core/controller/src/main/scala/whisk/core/controller/Triggers.scala
##
@@ -
markusthoemmes commented on a change in pull request #3262: Handle trigger
activations with inactive rules
URL:
https://github.com/apache/incubator-openwhisk/pull/3262#discussion_r167063098
##
File path: core/controller/src/main/scala/whisk/core/controller/Triggers.scala
#
rabbah commented on a change in pull request #3256: Refactor some bits of the
triggers API.
URL:
https://github.com/apache/incubator-openwhisk/pull/3256#discussion_r167063189
##
File path: core/controller/src/main/scala/whisk/core/controller/Triggers.scala
##
@@ -305,59 +
rabbah commented on a change in pull request #3256: Refactor some bits of the
triggers API.
URL:
https://github.com/apache/incubator-openwhisk/pull/3256#discussion_r167062871
##
File path: core/controller/src/main/scala/whisk/core/controller/Triggers.scala
##
@@ -305,59 +
mrutkows closed pull request #723: Add the integration test back to the Travis
build
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/723
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 proven
rabbah commented on issue #3262: Handle trigger activations with inactive rules
URL:
https://github.com/apache/incubator-openwhisk/pull/3262#issuecomment-364245062
@mdeuser can you take a look at #3256 I think we should get that one in
first as it will benefit the enhancement in this PR.
mdeuser commented on a change in pull request #3256: Refactor some bits of the
triggers API.
URL:
https://github.com/apache/incubator-openwhisk/pull/3256#discussion_r167068356
##
File path: core/controller/src/main/scala/whisk/core/controller/Triggers.scala
##
@@ -372,58
markusthoemmes commented on a change in pull request #3256: Refactor some bits
of the triggers API.
URL:
https://github.com/apache/incubator-openwhisk/pull/3256#discussion_r167069427
##
File path: core/controller/src/main/scala/whisk/core/controller/Triggers.scala
##
@@ -
mrutkows closed issue #711: Action: Web (export): Support web-custom-options
annotation
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/711
This is an automated message from the Apache Git Service.
To res
mrutkows closed pull request #719: Adding examples of web action annotations
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/719
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
mrutkows closed issue #712: Action: Web (action): Support the
require-whisk-auth annotation
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/712
This is an automated message from the Apache Git Service.
To
mrutkows closed issue #710: Actions: Web (export): Support "final" annotation
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/710
This is an automated message from the Apache Git Service.
To respond to the
houshengbo commented on issue #173: Added test cases for quoted strings.
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/173#issuecomment-364265724
@jessealva please take a look at the issue:
https://github.com/apache/incubator-openwhisk-cli/issues/221 Travis build is
not happ
houshengbo closed pull request #61: Adding placeholder for apigw access token
URL: https://github.com/apache/incubator-openwhisk-client-go/pull/61
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 opened a new issue #15: This is a test issue "ET Phone HOME !!"
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/issues/15
This is a test issue to see if the email notifications are going to issues@
and not dev@ for the Apache OpenWhisk mailing lists.
--
csantanapr commented on issue #15: This is a test issue "ET Phone HOME !!"
URL:
https://github.com/apache/incubator-openwhisk-runtime-docker/issues/15#issuecomment-364282792
problem fixed and INFRA ticket closed
https://issues.apache.org/jira/browse/INFRA-15998
---
csantanapr closed issue #15: This is a test issue "ET Phone HOME !!"
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/issues/15
This is an automated message from the Apache Git Service.
To respond to the mess
pritidesai commented on issue #681: proper handling of 'default' package name
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/681#issuecomment-364292402
After talking to Matt and scoping this feature of creating OW entities under
`namespace/` rather than `namespace/`, i
pritidesai commented on issue #681: proper handling of 'default' package name
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/681#issuecomment-364292402
After talking to Matt and scoping this feature of creating OW entities under
`namespace/` rather than `namespace/`, i
mrutkows commented on issue #709: Add Apache License
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/709#issuecomment-364294439
@daisy-ycguo well done!
This is an automated message from the Apache Git Servic
mrutkows commented on issue #709: Add Apache License
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/709#issuecomment-364294635
@daisy-ycguo next time please label the issue "review" and add me as
reviewer (or who you feel is appropriate) or I do not know it is "ready" or
ScottChapman commented on issue #681: proper handling of 'default' package name
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/681#issuecomment-364303316
I can live with that.
This is an automated message
daisy-ycguo commented on issue #716: Some source files are missing Apache
license headers
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/716#issuecomment-364309959
@mrutkows Please refer to
[.rat-excludes](https://github.com/apache/incubator-openwhisk-wskdeploy/blob/m
daisy-ycguo commented on issue #716: Some source files are missing Apache
license headers
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/716#issuecomment-364310279
@mrutkows Thank you for merging PR #709. This issue has been fixed. Now
wskdeploy can pass the license h
csantanapr commented on issue #681: proper handling of 'default' package name
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/681#issuecomment-364311287
I like the solution ?default? is already reserved package name because of
web actions uri
+1
---
daisy-ycguo commented on issue #716: Some source files are missing Apache
license headers
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/716#issuecomment-364309959
@mrutkows As said in [Apache release policy
document](http://www.apache.org/legal/src-headers.html#faq-
daisy-ycguo commented on issue #716: Some source files are missing Apache
license headers
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/716#issuecomment-364309959
@mrutkows As said in [Apache release policy
document](http://www.apache.org/legal/src-headers.html#faq-
daisy-ycguo commented on issue #716: Some source files are missing Apache
license headers
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/716#issuecomment-364309959
@mrutkows As said in [Apache release policy
document](http://www.apache.org/legal/src-headers.html#faq-
daisy-ycguo commented on issue #716: Some source files are missing Apache
license headers
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/716#issuecomment-364309959
@mrutkows As mentioned in the section "WHAT FILES IN AN APACHE RELEASE DO
NOT REQUIRE A LICENSE HEADER?
houshengbo closed pull request #23: Add support for a minified ASF license that
uses # hash prefix.
URL: https://github.com/apache/incubator-openwhisk-utilities/pull/23
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the
82 matches
Mail list logo