csantanapr closed pull request #35: add webhook for slack notification
URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/35
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
ningyougang commented on issue #3413: Remove deprecated loadbalancer.
URL:
https://github.com/apache/incubator-openwhisk/pull/3413#issuecomment-376363899
Any progress on this patch?
This is an automated message from the
sciabarracom commented on issue #5: The go proxy
URL:
https://github.com/apache/incubator-openwhisk-runtime-go/pull/5#issuecomment-376077970
I removed the exec and exec.zip replacing it with a build script - however
tests must be run in an environment including go now... that should not
markusthoemmes commented on a change in pull request #3421: Add ElasticSearch
Log Store
URL:
https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r177029555
##
File path:
common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchLogStore.scala
markusthoemmes commented on a change in pull request #3421: Add ElasticSearch
Log Store
URL:
https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r177029293
##
File path:
common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchLogStore.scala
markusthoemmes commented on a change in pull request #3421: Add ElasticSearch
Log Store
URL:
https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r177029936
##
File path: common/scala/src/main/scala/whisk/http/PoolingRestClient.scala
##
@@ -55,16 +63,18 @@
markusthoemmes commented on a change in pull request #3421: Add ElasticSearch
Log Store
URL:
https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r177028472
##
File path:
common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchLogStore.scala
markusthoemmes commented on a change in pull request #3485: Action time limit
test cleanup
URL:
https://github.com/apache/incubator-openwhisk/pull/3485#discussion_r177022846
##
File path: tests/src/test/scala/whisk/core/limits/ActionLimitsTests.scala
##
@@ -63,37 +60,154
markusthoemmes commented on a change in pull request #3485: Action time limit
test cleanup
URL:
https://github.com/apache/incubator-openwhisk/pull/3485#discussion_r177022985
##
File path: tests/src/test/scala/whisk/core/limits/ActionLimitsTests.scala
##
@@ -63,37 +60,154
markusthoemmes commented on a change in pull request #3421: Add ElasticSearch
Log Store
URL:
https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r176049724
##
File path:
common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchRestClient.scala
markusthoemmes commented on a change in pull request #3421: Add ElasticSearch
Log Store
URL:
https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r177028529
##
File path:
common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchLogStore.scala
markusthoemmes commented on a change in pull request #3421: Add ElasticSearch
Log Store
URL:
https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r176050241
##
File path:
common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchLogStore.scala
markusthoemmes commented on a change in pull request #3421: Add ElasticSearch
Log Store
URL:
https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r177029998
##
File path:
tests/src/test/scala/whisk/core/containerpool/logging/ElasticSearchLogStoreTests.scala
sven-lange-last commented on a change in pull request #3485: Action time limit
test cleanup
URL:
https://github.com/apache/incubator-openwhisk/pull/3485#discussion_r177048403
##
File path: tests/src/test/scala/whisk/core/limits/ActionLimitsTests.scala
##
@@ -63,37
chetanmeh commented on issue #3249: #3243 - Use gradle scoverage to collect
test coverage stats
URL:
https://github.com/apache/incubator-openwhisk/pull/3249#issuecomment-376165014
Blocked on #3490
This is an automated
rabbah commented on a change in pull request #3412: Changes to runtimes
variables in support of retiring 'whisk/*' runtimes
URL:
https://github.com/apache/incubator-openwhisk/pull/3412#discussion_r177070688
##
File path: ansible/templates/whisk.properties.j2
##
@@ -33,7
rabbah closed pull request #3457: Ansible 2.4 deprecates ‘include’ in favor of
‘import_*’ and ‘include_tasks’
URL: https://github.com/apache/incubator-openwhisk/pull/3457
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
krook opened a new issue #818: FYI: Changing latest wskdeploy download URL
broke some travis jobs
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/818
FYI, if anyone had jobs (like I did, now fixed) that pulls the latest
tarball at the expected latest URL, the name
sven-lange-last commented on issue #3485: Action time limit test cleanup
URL:
https://github.com/apache/incubator-openwhisk/pull/3485#issuecomment-376139014
* Addressed all review comments.
* PG 3 / 2059 running.
This is
rabbah commented on issue #2971: OW Controller should be more lenient about the
data type of the statusCode value passed to it from the openwhisk action
URL:
https://github.com/apache/incubator-openwhisk/issues/2971#issuecomment-376142269
Closed by
rabbah closed pull request #3473: Support string based status code from actions.
URL: https://github.com/apache/incubator-openwhisk/pull/3473
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
rabbah closed issue #2971: OW Controller should be more lenient about the data
type of the statusCode value passed to it from the openwhisk action
URL: https://github.com/apache/incubator-openwhisk/issues/2971
This is an
jonpspri commented on a change in pull request #3412: Changes to runtimes
variables in support of retiring 'whisk/*' runtimes
URL:
https://github.com/apache/incubator-openwhisk/pull/3412#discussion_r177070402
##
File path: ansible/templates/whisk.properties.j2
##
@@
jonpspri commented on a change in pull request #3442: Add -o option to
overwrite target files
URL:
https://github.com/apache/incubator-openwhisk/pull/3442#discussion_r177066581
##
File path: ansible/roles/kafka/tasks/deploy.yml
##
@@ -79,7 +79,7 @@
env: "{{
jonpspri commented on a change in pull request #3442: Add -o option to
overwrite target files
URL:
https://github.com/apache/incubator-openwhisk/pull/3442#discussion_r177066435
##
File path: ansible/roles/cli/tasks/deploy.yml
##
@@ -69,5 +69,5 @@
# Use old good unzip
rabbah commented on a change in pull request #3340: Default activation response
to empty JsObject.
URL:
https://github.com/apache/incubator-openwhisk/pull/3340#discussion_r177068675
##
File path: common/scala/src/main/scala/whisk/core/entity/WhiskActivation.scala
##
@@
sven-lange-last commented on a change in pull request #3485: Action time limit
test cleanup
URL:
https://github.com/apache/incubator-openwhisk/pull/3485#discussion_r177050029
##
File path: tests/src/test/scala/whisk/core/limits/MaxActionDurationTests.scala
##
@@ -42,38
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-376146865
@dgrove-oss Hmm... so maybe we should eventually factor out an
invoker-helper that runs in the
jasonpet commented on issue #141: Add a membership management using the
discovery
URL:
https://github.com/apache/incubator-openwhisk-package-alarms/pull/141#issuecomment-376230255
@rabbah, @dubee , @csantanapr - someone with commit rights (which I do not
have) will need to review this
rabbah closed pull request #3442: Add -o option to overwrite target files
URL: https://github.com/apache/incubator-openwhisk/pull/3442
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
rabbah commented on issue #3490: Move controller config to reference.conf
URL:
https://github.com/apache/incubator-openwhisk/pull/3490#issuecomment-376166068
LGTM but defer to @markusthoemmes and @vvraskin
This is an
jasonpet commented on issue #141: Add a membership management using the
discovery
URL:
https://github.com/apache/incubator-openwhisk-package-alarms/pull/141#issuecomment-376229816
@dgrove-oss - will this udp discovery work for kubernetes? I know we will
be moving to kubernetes soon.
drcariel opened a new issue #259: For Ease of Maintenance, Update and Decide
Where these Tests Should Live
URL: https://github.com/apache/incubator-openwhisk-cli/issues/259
There is currently a level of manual syncing going on between duplicate test
files that exist in both
houshengbo opened a new pull request #71: Implement the versioning for
openwhisk release
URL: https://github.com/apache/incubator-openwhisk-release/pull/71
This is an automated message from the Apache Git Service.
To
jasonpet commented on issue #141: Add a membership management using the
discovery
URL:
https://github.com/apache/incubator-openwhisk-package-alarms/pull/141#issuecomment-376227890
@KeonHee - I think the issue I am having has to do with the fact that this
requires udp for the discovery.
houshengbo opened a new pull request #70: Add the link to license compliace to
README page
URL: https://github.com/apache/incubator-openwhisk-release/pull/70
Closes: #68
This is an automated message from the Apache Git
jonpspri closed pull request #3489: Switch 'scala' image to use OpenJDK 8
URL: https://github.com/apache/incubator-openwhisk/pull/3489
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
jonpspri commented on issue #3489: Switch 'scala' image to use OpenJDK 8
URL:
https://github.com/apache/incubator-openwhisk/pull/3489#issuecomment-376201314
Closing for now. It appears the original issue was resolved.
This
houshengbo commented on issue #61: tag repos with version after first release
with openwhisk@1.0.0
URL:
https://github.com/apache/incubator-openwhisk-release/issues/61#issuecomment-376194896
@csantanapr
Based on what have now in config.json, we are able to define a major and a
minor
houshengbo commented on issue #47: How can we generate the changlog and release
note for each release
URL:
https://github.com/apache/incubator-openwhisk-release/issues/47#issuecomment-376200349
We have already got a script to generate the release note:
houshengbo opened a new issue #9: Package the binaries of each repo in need
URL: https://github.com/apache/incubator-openwhisk-release/issues/9
We need to publish the binaries for the openwhisk projects, which are able
to provide the binaries.
ScottChapman commented on issue #819: wsk deploy not picking up new target
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/819#issuecomment-376190885
And then it magically worked... Here is the entire transcript:
> SECMacbookPro:Weather scottchapman$ wskdeploy
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-376200018
@mrutkows I had a PR in the openwhisk-apigateway repo to replace and add the
license
ScottChapman opened a new issue #819: wsk deploy not picking up new target
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/819
This has actually happened to me a couple of times. I will be working in a
space, wskdeploy working fine. Then I change the target with `bx
houshengbo commented on issue #61: tag repos with version after first release
with openwhisk@1.0.0
URL:
https://github.com/apache/incubator-openwhisk-release/issues/61#issuecomment-376194896
@csantanapr
Based on what have now in config.json, we are able to define a major and a
minor
houshengbo closed issue #21: Step 7: Upload the artifacts into staging
repository for release vote
URL: https://github.com/apache/incubator-openwhisk-release/issues/21
This is an automated message from the Apache Git
houshengbo commented on issue #21: Step 7: Upload the artifacts into staging
repository for release vote
URL:
https://github.com/apache/incubator-openwhisk-release/issues/21#issuecomment-376201174
This issue can be closed for now, because the code structure to upload the
artifacts into
csantanapr opened a new issue #111: 3.14.0 release
URL: https://github.com/apache/incubator-openwhisk-client-js/issues/111
## release notes
### 3.14.0 (2018.03.26)
- Add support to count entities with list (#109 )
## process
- `npm version patch` to bump
csantanapr opened a new pull request #112: update package.json and d.ts for
3.14
URL: https://github.com/apache/incubator-openwhisk-client-js/pull/112
This is an automated message from the Apache Git Service.
To respond to
csantanapr closed issue #29: swift 4.x add support throw for dict sync main
signature and handle as error field in out json
URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/29
This is an automated
csantanapr commented on issue #29: swift 4.x add support throw for dict sync
main signature and handle as error field in out json
URL:
https://github.com/apache/incubator-openwhisk-runtime-swift/issues/29#issuecomment-376289930
This is already implemented
csantanapr opened a new issue #36: Fix codable escaping for result handler
URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/36
The code will not compile if the main function calls another function that
escapes.
For example the following code:
```swift
jonpspri commented on issue #3467: Remove actionControllers tests from
incubator-openwhisk repo
URL:
https://github.com/apache/incubator-openwhisk/pull/3467#issuecomment-376336913
@csantanapr @rabbah Please review. Moving ActionContainer and
ResourceHelpers into a
rabbah commented on a change in pull request #3467: Remove actionControllers
tests from incubator-openwhisk repo
URL:
https://github.com/apache/incubator-openwhisk/pull/3467#discussion_r177259895
##
File path:
tests/src/test/scala/actionContainers/ActionProxyContainerTests.scala
rabbah commented on issue #3467: Remove actionControllers tests from
incubator-openwhisk repo
URL:
https://github.com/apache/incubator-openwhisk/pull/3467#issuecomment-376338579
actually I think
tests/src/test/scala/actionContainers/ActionProxyContainerTests.scala can be
made more
jonpspri commented on issue #28: Create a multi-architecture Docker image for
the runtime
URL:
https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/28#issuecomment-376332907
Re-based and removed superfluous changes to testing processes.
csantanapr closed pull request #71: Implement the versioning for openwhisk
release
URL: https://github.com/apache/incubator-openwhisk-release/pull/71
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
codecov-io commented on issue #112: update package.json and d.ts for 3.14
URL:
https://github.com/apache/incubator-openwhisk-client-js/pull/112#issuecomment-376303049
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk-client-js/pull/112?src=pr=h1)
Report
> Merging
jonpspri opened a new issue #3492: Add a utility main class in common to dump
the config as JSON
URL: https://github.com/apache/incubator-openwhisk/issues/3492
Create a utility main class in the 'common' library to be available in
controller and invoker JARs. The class should be in Java
jonpspri opened a new issue #3493: Validate whether 'runtimes.manifest' is
still needed by tests
URL: https://github.com/apache/incubator-openwhisk/issues/3493
The 'runtimes.manifest' variable in 'whisk.properties' and, therefore,
'whisk.properties.j2' may no longer be needed. If it is
dgrove-oss opened a new pull request #264: document purpose of encrypted URL
URL: https://github.com/apache/incubator-openwhisk-catalog/pull/264
Should have comment explaining what the encrypted URL is doing. The PR also
serves as a test that the hook is working as expected.
pritidesai closed issue #68: Add link to license_compliance.md under main
project README
URL: https://github.com/apache/incubator-openwhisk-release/issues/68
This is an automated message from the Apache Git Service.
To
pritidesai closed pull request #70: Add the link to license compliace to README
page
URL: https://github.com/apache/incubator-openwhisk-release/pull/70
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
63 matches
Mail list logo
