dgrove-oss closed pull request #46: update travis script to fail earlier
URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/46
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 pull request #46: update travis script to fail earlier
URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/46
This is an automated message from the Apache Git Service.
To respond to
csantanapr opened a new issue #45: Update Travis to stop build earlier
URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/45
Travis should be updated to be similar as the other repos
csantanapr commented on issue #42: Add support for Kotlin actions
URL:
https://github.com/apache/incubator-openwhisk-runtime-java/issues/42#issuecomment-385128486
We already started work on Kotlin runtime, work under construction once is
stable IBM will contribute to Apache OpenWhisk
lmoran opened a new issue #121: API Gateway does not start with docker-compose:
openwhisk-server-cert.pem file missing
URL: https://github.com/apache/incubator-openwhisk-devtools/issues/121
When I try "make quick-start" all the containers are built and started, but
the API
codecov-io commented on issue #3504: use alpine-java base image (alpine 3.7 +
oracle 8u162b12_server-jre)
URL:
https://github.com/apache/incubator-openwhisk/pull/3504#issuecomment-385123620
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3504?src=pr=h1)
Report
>
csantanapr commented on issue #3504: use alpine-java base image (alpine 3.7 +
oracle 8u162b12_server-jre)
URL:
https://github.com/apache/incubator-openwhisk/pull/3504#issuecomment-385120184
My guess is yes to add the variables
mrutkows closed pull request #75: Add the licenses of the dependencies into
LICENSE
URL: https://github.com/apache/incubator-openwhisk-client-go/pull/75
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 commented on issue #97: Document LICENSE file Exclusions for
incubator-openwhisk-cli
URL:
https://github.com/apache/incubator-openwhisk-release/issues/97#issuecomment-385118212
@daisy-ycguo PR https://github.com/apache/incubator-openwhisk-cli/pull/283
has been merged however,
mrutkows commented on issue #97: Document LICENSE file Exclusions for
incubator-openwhisk-cli
URL:
https://github.com/apache/incubator-openwhisk-release/issues/97#issuecomment-385118025
PR
https://github.com/apache/incubator-openwhisk-cli/pull/271#pullrequestreview-116107376
has been
mrutkows closed pull request #271: Add the licenses of the dependencies into
LICENSE
URL: https://github.com/apache/incubator-openwhisk-cli/pull/271
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:
tysonnorris commented on issue #3504: use alpine-java base image (alpine 3.7 +
oracle 8u162b12_server-jre)
URL:
https://github.com/apache/incubator-openwhisk/pull/3504#issuecomment-385117618
nice sleuthing!
One other question: should I add these back in?
```
ENV LANG
mrutkows commented on issue #222: A list of source files missing Apache license
headers
URL:
https://github.com/apache/incubator-openwhisk-cli/issues/222#issuecomment-385116807
@daisy-ycguo The files under these "tests" may need headers as they are not
performance related. PR 283 only
mrutkows commented on issue #222: A list of source files missing Apache license
headers
URL:
https://github.com/apache/incubator-openwhisk-cli/issues/222#issuecomment-385116807
@daisy-ycguo The files under these "tests" may need headers as they are not
performance related.
daisy-ycguo opened a new issue #222: A list of source files missing Apache
license headers
URL: https://github.com/apache/incubator-openwhisk-cli/issues/222
Following [Apache license header
guideline](http://www.apache.org/legal/src-headers.html), **all human-readable
Apache-developed
csantanapr commented on issue #3504: use alpine-java base image (alpine 3.7 +
oracle 8u162b12_server-jre)
URL:
https://github.com/apache/incubator-openwhisk/pull/3504#issuecomment-385116582
It's available I used a script to list all tags
csantanapr commented on issue #3504: use alpine-java base image (alpine 3.7 +
oracle 8u162b12_server-jre)
URL:
https://github.com/apache/incubator-openwhisk/pull/3504#issuecomment-385116582
It's available I used a script to list all tags
tysonnorris commented on issue #3504: use alpine-java base image (alpine 3.7 +
oracle 8u162b12_server-jre)
URL:
https://github.com/apache/incubator-openwhisk/pull/3504#issuecomment-385116073
No specific criteria, other than a particular tag that is not reused for
different versions of
csantanapr commented on issue #3504: use alpine-java base image (alpine 3.7 +
oracle 8u162b12_server-jre)
URL:
https://github.com/apache/incubator-openwhisk/pull/3504#issuecomment-385115465
What specific criteria are you looking for the specific version of jdk 8 and
alpine?
csantanapr commented on issue #3504: use alpine-java base image (alpine 3.7 +
oracle 8u162b12_server-jre)
URL:
https://github.com/apache/incubator-openwhisk/pull/3504#issuecomment-385115246
@tysonnorris
>@csantanapr the adoptopenjdk image you pointed to was openjdk8-openj9 - I
assume
mrutkows commented on issue #3264: A list of source files miss Apache license
headers
URL:
https://github.com/apache/incubator-openwhisk/issues/3264#issuecomment-385111832
@daisy-ycguo it appears after merging 3565 (ansible/*) and 3566 that only
the following files remain:
```
mrutkows commented on issue #3264: A list of source files miss Apache license
headers
URL:
https://github.com/apache/incubator-openwhisk/issues/3264#issuecomment-385111832
@daisy-ycguo it appears after merging 3565 (ansible/*) and 3566 that only
the following files remain:
```
mrutkows commented on issue #131: license_compliance.md excludes "Gradle files"
but rationale not sound
URL:
https://github.com/apache/incubator-openwhisk-release/issues/131#issuecomment-385113966
and "settings.gradle" as well.
mrutkows commented on issue #3550: Add the licenses of the dependencies into
LICENSE
URL:
https://github.com/apache/incubator-openwhisk/pull/3550#issuecomment-385112858
Partially addresses issues in issue
https://github.com/apache/incubator-openwhisk-release/issues/93
tysonnorris commented on issue #3504: use alpine-java base image (alpine 3.7 +
oracle 8u162b12_server-jre)
URL:
https://github.com/apache/incubator-openwhisk/pull/3504#issuecomment-385112773
@csantanapr the adoptopenjdk image you pointed to was `openjdk8-openj9` - I
assume that is a j9
mrutkows commented on issue #3264: A list of source files miss Apache license
headers
URL:
https://github.com/apache/incubator-openwhisk/issues/3264#issuecomment-385111832
@daisy-ycguo it appears after merging 3565 (ansible/*) and 3566 that only
the following files remain:
```
csantanapr commented on issue #3504: use alpine-java base image (alpine 3.7 +
oracle 8u162b12_server-jre)
URL:
https://github.com/apache/incubator-openwhisk/pull/3504#issuecomment-385111319
The image I pointed is not J9, it's actually openjdk from the adoptopenjdk
project.
mrutkows opened a new issue #131: license_compliance.md excludes "Gradle files"
but rationale not sound
URL: https://github.com/apache/incubator-openwhisk-release/issues/131
In []() the exclusion table allows for "Gradle files" not having ASF
headers; however, the rationale is simple "not
mrutkows opened a new issue #130: Revisit LICENSE.tx vs. LICENSE or document
decision
URL: https://github.com/apache/incubator-openwhisk-release/issues/130
.txt is allowed by Apache policy on LICENSE and NOTICE files.
http://www.apache.org/dev/apply-license.html
Perhaps we
mrutkows opened a new issue #129: "data files" that need ASF license not well
described
URL: https://github.com/apache/incubator-openwhisk-release/issues/129
The
[license_compliance.md](https://github.com/apache/incubator-openwhisk-release/blob/master/docs/license_compliance.md)
file
mrutkows opened a new pull request #128: Update README and
license_compliance.md to be more accurate/informative
URL: https://github.com/apache/incubator-openwhisk-release/pull/128
This is an automated message from the
pritidesai closed pull request #278: Update Travis to use Scandcode with the
ASF Release configuration.
URL: https://github.com/apache/incubator-openwhisk-cli/pull/278
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the
eweiter opened a new issue #9: Add ability to push master tag to Docker after
Travis build
URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/9
Update the Travis build to push a tag "master" to Docker when the master
branch is updated.
This will make it easier to pull
eweiter opened a new issue #16: Update php runtime to push master tag to Docker
after Travis builds
URL: https://github.com/apache/incubator-openwhisk-runtime-php/issues/16
Update the Travis build to push a tag "master" to Docker when the master
branch is updated.
This will make it
eweiter opened a new issue #18: Update python runtime to push a master tag from
Travis build
URL: https://github.com/apache/incubator-openwhisk-runtime-python/issues/18
Update the Travis build to push a tag "master" when the master branch is
updated.
This will make it easier to pull
eweiter opened a new issue #30: update docker runtime to push master tag from
Travis build
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/issues/30
Update the Travis build to push a tag "master" when a the master branch is
updated.
This will make it easier to pull
eweiter opened a new issue #49: Update swift runtime to push master tag from
Travis build
URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/49
Update the Travis build to push a tag "master" when a the master branch is
updated.
This will make it easier to pull the
eweiter opened a new issue #44: Update Node runtime to push master tag from
Travis build
URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/issues/44
Update the Travis build to push a tag "master" when a the master branch is
updated.
This will make it easier to pull the
pritidesai commented on a change in pull request #278: Update Travis to use
Scandcode with the ASF Release configuration.
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/278#discussion_r184744550
##
File path: tools/travis/test_openwhisk.sh
##
@@ -41,9 +41,17
csantanapr closed pull request #3569: Use scanCode from utilities repo. using
ASF Release configuration.
URL: https://github.com/apache/incubator-openwhisk/pull/3569
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the
jasonpet closed pull request #266: Health test improvements
URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/266
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
csantanapr closed pull request #3589: bump java from 1.0.1 -> 1.1.0
URL: https://github.com/apache/incubator-openwhisk/pull/3589
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
eweiter commented on issue #3589: bump java from 1.0.1 -> 1.1.0
URL:
https://github.com/apache/incubator-openwhisk/pull/3589#issuecomment-384971388
PG4 Build 1620 passed
This is an automated message from the Apache Git
dgrove-oss commented on issue #3591: Move kafka/zookeeper hosts to pureconfig
based configuration.
URL:
https://github.com/apache/incubator-openwhisk/pull/3591#issuecomment-385006900
change makes sense, but will need compensating commit in at least
deploy-kube to change the environment
abaruni opened a new pull request #266: Health test improvements
URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/266
log timestamp of produced message. ensure consumer created before producing
message by polling health endpoint for uuid
jasonpet commented on issue #146: Add worker to logging
URL:
https://github.com/apache/incubator-openwhisk-package-alarms/pull/146#issuecomment-384991599
I would recommend you put this information in the message if you need it. I
personally name my container name in a way that gives me
mrutkows commented on issue #880: wskdeploy export produces empty file
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/880#issuecomment-384991180
@brunogirin can you supply the input manifest YAML file you sent as input
(as well as any other command line options or
jasonpet commented on issue #146: Add worker to logging
URL:
https://github.com/apache/incubator-openwhisk-package-alarms/pull/146#issuecomment-384990421
Please see the expected format comment:
jasonpet commented on issue #146: Add worker to logging
URL:
https://github.com/apache/incubator-openwhisk-package-alarms/pull/146#issuecomment-384989625
this breaks the kibana query for the `component` field. with this change it
is now set to None instead of alarmsTrigger.
eweiter commented on issue #3589: bump java from 1.0.1 -> 1.1.0
URL:
https://github.com/apache/incubator-openwhisk/pull/3589#issuecomment-384988762
I have hopefully addressed the comments left above.
This is an
houshengbo commented on issue #288: Apache release does not allow to have
private license as the header
URL:
https://github.com/apache/incubator-openwhisk-apigateway/issues/288#issuecomment-384987747
@ddragosd Could you look at this issue?
We should not keep any copyright information
codecov-io commented on issue #3552: Emit activation metadata to Kafka
URL:
https://github.com/apache/incubator-openwhisk/pull/3552#issuecomment-384703979
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3552?src=pr=h1)
Report
> Merging
jasonpet commented on issue #147: No alarm when openwhisk runs on different port
URL:
https://github.com/apache/incubator-openwhisk-package-alarms/issues/147#issuecomment-384977478
this issue was resolved by PR #148
This is
jasonpet closed issue #147: No alarm when openwhisk runs on different port
URL: https://github.com/apache/incubator-openwhisk-package-alarms/issues/147
This is an automated message from the Apache Git Service.
To respond to
csantanapr closed pull request #170: allow apihost port to be configurable
URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/170
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
csantanapr closed pull request #148: allow apihost port to be configurable
URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/148
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:
csantanapr commented on issue #3589: bump java from 1.0.1 -> 1.1.0
URL:
https://github.com/apache/incubator-openwhisk/pull/3589#issuecomment-384976043
@eweiter
1. Could you add a description to first PR comment
2. Add diff git url using github compare url
jasonpet commented on issue #148: allow apihost port to be configurable
URL:
https://github.com/apache/incubator-openwhisk-package-alarms/pull/148#issuecomment-384975502
resolves issue #147
This is an automated message from
jasonpet opened a new pull request #170: allow apihost port to be configurable
URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/170
Do not hardcode port to 443 which is the default anyways. If not 443 it
should be appended to end of ROUTER_HOST environment var.
jasonpet opened a new pull request #148: allow apihost port to be configurable
URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/148
Do not hardcode port to 443 which is the default anyways. If not 443 it
should be appended to end of ROUTER_HOST environment var.
eweiter commented on issue #3589: bump java from 1.0.1 -> 1.1.0
URL:
https://github.com/apache/incubator-openwhisk/pull/3589#issuecomment-384971388
PG4 Build #1620 passed
This is an automated message from the Apache Git
codecov-io commented on issue #3592: Make limit overcommit relative to the
actual cluster size.
URL:
https://github.com/apache/incubator-openwhisk/pull/3592#issuecomment-384964071
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3592?src=pr=h1)
Report
> Merging
markusthoemmes opened a new pull request #3592: Make limit overcommit relative
to the actual cluster size.
URL: https://github.com/apache/incubator-openwhisk/pull/3592
The overcommit of limits needs to be relative to the actual size of the
cluster to appropriately scale those limits.
markusthoemmes opened a new pull request #3591: Move kafka/zookeeper hosts to
pureconfig based configuration.
URL: https://github.com/apache/incubator-openwhisk/pull/3591
`KAFKA_HOSTS` being in WhiskConfig forces many classes to thread through the
loaded config object.
## My
codecov-io commented on issue #3562: ArtifactStore implementation for CosmosDB
URL:
https://github.com/apache/incubator-openwhisk/pull/3562#issuecomment-384942423
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3562?src=pr=h1)
Report
> Merging
codecov-io commented on issue #3590: Collect test reports as part of log
collection
URL:
https://github.com/apache/incubator-openwhisk/pull/3590#issuecomment-384934946
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3590?src=pr=h1)
Report
> Merging
vvraskin commented on issue #3552: Emit activation metadata to Kafka
URL:
https://github.com/apache/incubator-openwhisk/pull/3552#issuecomment-384934396
PG3/2181 is :white_check_mark:
This is an automated message from the
chetanmeh commented on a change in pull request #3562: ArtifactStore
implementation for CosmosDB
URL:
https://github.com/apache/incubator-openwhisk/pull/3562#discussion_r184652297
##
File path:
common/scala/src/main/scala/whisk/core/database/cosmosdb/CosmosDBSupport.scala
fmaschler commented on issue #147: No alarm when openwhisk runs on different
port
URL:
https://github.com/apache/incubator-openwhisk-package-alarms/issues/147#issuecomment-384912243
As said before I adjusted the `ROUTER_HOST` variable for the alarmtrigger
container. When I attach to it
chetanmeh opened a new pull request #3590: Collect test reports as part of log
collection
URL: https://github.com/apache/incubator-openwhisk/pull/3590
Modifies the log collection playbook to also include test reports (html,
xml) generated by Gradle. Some of the data in these reports can
chetanmeh commented on a change in pull request #3562: ArtifactStore
implementation for CosmosDB
URL:
https://github.com/apache/incubator-openwhisk/pull/3562#discussion_r184629230
##
File path:
chetanmeh commented on a change in pull request #3562: ArtifactStore
implementation for CosmosDB
URL:
https://github.com/apache/incubator-openwhisk/pull/3562#discussion_r184626235
##
File path:
chetanmeh commented on a change in pull request #3562: ArtifactStore
implementation for CosmosDB
URL:
https://github.com/apache/incubator-openwhisk/pull/3562#discussion_r184625896
##
File path: common/scala/build.gradle
##
@@ -47,6 +47,10 @@ dependencies {
compile
codecov-io commented on issue #3552: Emit activation metadata to Kafka
URL:
https://github.com/apache/incubator-openwhisk/pull/3552#issuecomment-384703979
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3552?src=pr=h1)
Report
> Merging
codecov-io commented on issue #3552: Emit activation metadata to Kafka
URL:
https://github.com/apache/incubator-openwhisk/pull/3552#issuecomment-384703979
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3552?src=pr=h1)
Report
> Merging
markusthoemmes commented on issue #3589: bump java from 1.0.1 -> 1.1.0
URL:
https://github.com/apache/incubator-openwhisk/pull/3589#issuecomment-384877033
A description on why you're bumping this and what the changes are would be
great.
markusthoemmes commented on a change in pull request #2795: enable concurrent
activation processing
URL:
https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r184601638
##
File path:
core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala
markusthoemmes commented on a change in pull request #2795: enable concurrent
activation processing
URL:
https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r184601074
##
File path: performance/preparation/create-async.sh
##
@@ -0,0 +1,17 @@
+#!/bin/sh
markusthoemmes commented on a change in pull request #2795: enable concurrent
activation processing
URL:
https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r184601181
##
File path: performance/wrk_tests/throughput-async.sh
##
@@ -0,0 +1,31 @@
+#!/bin/sh
markusthoemmes commented on a change in pull request #2795: enable concurrent
activation processing
URL:
https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r184601872
##
File path:
core/invoker/src/main/scala/whisk/core/containerpool/ContainerPool.scala
markusthoemmes commented on issue #3504: use alpine-java base image (alpine 3.7
+ oracle 8u162b12_server-jre)
URL:
https://github.com/apache/incubator-openwhisk/pull/3504#issuecomment-384873181
Fram what I've read about J9 it's a different JVM. That would mean we switch
from Hotspot to
81 matches
Mail list logo
