codecov-io commented on issue #2941: Add api gateway test that uses custom
package for action
URL:
https://github.com/apache/incubator-openwhisk/pull/2941#issuecomment-396118743
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/2941?src=pr=h1)
Report
> Merging
codecov-io commented on issue #2941: Add api gateway test that uses custom
package for action
URL:
https://github.com/apache/incubator-openwhisk/pull/2941#issuecomment-396118743
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/2941?src=pr=h1)
Report
> Merging
codecov-io commented on issue #3751: Allow invoker pools to overlap for small N.
URL:
https://github.com/apache/incubator-openwhisk/pull/3751#issuecomment-396465146
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3751?src=pr=h1)
Report
> Merging
rabbah commented on issue #3693: new invokers cause load balancer to reallocate
black box invokers
URL:
https://github.com/apache/incubator-openwhisk/issues/3693#issuecomment-396461426
@Ahmed-elsayed-mahmoud this is the issue I mentioned on slack and the
accompanying PR #3751.
rabbah opened a new pull request #3751: Allow invoker pools to overlap for
small N.
URL: https://github.com/apache/incubator-openwhisk/pull/3751
## Description
For small N, allow the managed invokers to overlap with blackbox invokers.
As an example, for a blackbox fraction of
codecov-io commented on issue #3750: Fix arithmetic bug calculating blackbox
and managed invokers.
URL:
https://github.com/apache/incubator-openwhisk/pull/3750#issuecomment-396453489
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3750?src=pr=h1)
Report
> Merging
rabbah closed issue #3749: managed and blackbox invoker counts are wrong when
adding new invokers
URL: https://github.com/apache/incubator-openwhisk/issues/3749
This is an automated message from the Apache Git Service.
To
codecov-io commented on issue #3680: Adjust invoker playbook to pull docker
images when a prefix and tag is specified.
URL:
https://github.com/apache/incubator-openwhisk/pull/3680#issuecomment-390292471
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3680?src=pr=h1)
rabbah closed pull request #3750: Fix arithmetic bug calculating blackbox and
managed invokers.
URL: https://github.com/apache/incubator-openwhisk/pull/3750
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
rabbah opened a new pull request #3750: Fix arithmetic bug calculating blackbox
and managed invokers.
URL: https://github.com/apache/incubator-openwhisk/pull/3750
This patch fixes a bug computing the number of blackbox and managed invokers.
## Description
## Related
rabbah opened a new issue #3749: managed and blackbox invoker counts are wrong
when adding new invokers
URL: https://github.com/apache/incubator-openwhisk/issues/3749
If you deploy additional invokers, they will register with the load balancer
but will not get assigned additional load.
daisy-ycguo closed pull request #216: Stop testing non-Helm based deployments
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/216
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:
dgrove-oss opened a new pull request #217: add license header
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/217
license header is inside Go template comment markers ( {{/* ... */}}) to
keep it from appearing in end-user visible text (the usage message generated
from
dgrove-oss opened a new issue #46: scan code is not checking .txt files for
license
URL: https://github.com/apache/incubator-openwhisk-utilities/issues/46
In the openwhisk-kube-deploy project, the file home/templates/Notes.txt does
not have a license header. However, scancode with the
remore commented on a change in pull request #3725: Add Ruby2.5 runtime support
URL:
https://github.com/apache/incubator-openwhisk/pull/3725#discussion_r194578413
##
File path:
tests/src/test/scala/actionContainers/Ruby25ActionContainerTests.scala
##
@@ -0,0 +1,408 @@
remore commented on a change in pull request #3725: Add Ruby2.5 runtime support
URL:
https://github.com/apache/incubator-openwhisk/pull/3725#discussion_r194578413
##
File path:
tests/src/test/scala/actionContainers/Ruby25ActionContainerTests.scala
##
@@ -0,0 +1,408 @@
mrutkows commented on issue #181: Add openwhisk-kube-deploy to list of release
repos.
URL:
https://github.com/apache/incubator-openwhisk-release/pull/181#issuecomment-396400309
If we add Kube, do we add Mesos? At this point, if adding Kube is that
important for first release (cannot
mrutkows commented on issue #181: Add openwhisk-kube-deploy to list of release
repos.
URL:
https://github.com/apache/incubator-openwhisk-release/pull/181#issuecomment-396405440
@rabbah thanks Rodric, we will try.. will look at all the files tomorrow in
detail... and if we hit a snag
rabbah commented on issue #181: Add openwhisk-kube-deploy to list of release
repos.
URL:
https://github.com/apache/incubator-openwhisk-release/pull/181#issuecomment-396403119
Thanks for the explanation Matt. I think it’s worth trying quickly. If you
find it’s too much work and will delay
houshengbo closed pull request #181: Add openwhisk-kube-deploy to list of
release repos.
URL: https://github.com/apache/incubator-openwhisk-release/pull/181
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 #181: Add openwhisk-kube-deploy to list of release
repos.
URL:
https://github.com/apache/incubator-openwhisk-release/pull/181#issuecomment-396401713
@houshengbo @rabbah @dgrove-oss Vincent and I spoke and he will start adding
Kube to release automation. It
houshengbo closed issue #140: Key words like IBM, Adobe, etc should be removed
from the source code
URL: https://github.com/apache/incubator-openwhisk-release/issues/140
This is an automated message from the Apache Git
mrutkows commented on issue #181: Add openwhisk-kube-deploy to list of release
repos.
URL:
https://github.com/apache/incubator-openwhisk-release/pull/181#issuecomment-396400309
If we add Kube, do we add Mesos? At this point, if adding Kube is that
important for first release (cannot
houshengbo commented on issue #141: Some license files copied from client-go
are not exactly same as original one.
URL:
https://github.com/apache/incubator-openwhisk-release/issues/141#issuecomment-396399956
We have changed to Copyright 2015-2018 The Apache Software Foundation for
all
houshengbo closed issue #141: Some license files copied from client-go are not
exactly same as original one.
URL: https://github.com/apache/incubator-openwhisk-release/issues/141
This is an automated message from the Apache
mrutkows commented on issue #181: Add openwhisk-kube-deploy to list of release
repos.
URL:
https://github.com/apache/incubator-openwhisk-release/pull/181#issuecomment-396399275
@rabbah we had planned to submit the vote by EOD Wed. (or sooner) to the
"dev" list. I have no doubt that the
houshengbo closed issue #132: Document treatment of .conf files as part of
LICENSE header exclusions/inclusion
URL: https://github.com/apache/incubator-openwhisk-release/issues/132
This is an automated message from the
houshengbo closed issue #130: Revisit LICENSE.txt vs. LICENSE or document
decision
URL: https://github.com/apache/incubator-openwhisk-release/issues/130
This is an automated message from the Apache Git Service.
To respond
houshengbo commented on issue #132: Document treatment of .conf files as part
of LICENSE header exclusions/inclusion
URL:
https://github.com/apache/incubator-openwhisk-release/issues/132#issuecomment-396397474
We have added the headers to all the .conf files in api gateway repo.
houshengbo commented on issue #107: Do we need to add LICENSE header to binary
files?
URL:
https://github.com/apache/incubator-openwhisk-release/issues/107#issuecomment-396396031
We will revisit this issue for further binary releases.
houshengbo closed issue #93: Track LICENSE and NOTICE issues for
`incubator-openwhisk`
URL: https://github.com/apache/incubator-openwhisk-release/issues/93
This is an automated message from the Apache Git Service.
To
houshengbo closed issue #99: Track LICENSE and NOTICE issues for
`incubator-openwhisk-runtime-nodejs`
URL: https://github.com/apache/incubator-openwhisk-release/issues/99
This is an automated message from the Apache Git
houshengbo closed issue #100: Track LICENSE and NOTICE issues for
`incubator-openwhisk-runtime-swift`
URL: https://github.com/apache/incubator-openwhisk-release/issues/100
This is an automated message from the Apache Git
mrutkows closed issue #659: Add flag to suppress Warnings -warning false
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/659
This is an automated message from the Apache Git Service.
To respond to the
mrutkows closed issue #940: API section doesn't support path parameters
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/940
This is an automated message from the Apache Git Service.
To respond to the
mrutkows closed issue #724: API Gateway - Add support to specify response type
in manifest file
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/724
This is an automated message from the Apache Git
mrutkows closed issue #916: deploying manifest containing APIs with same
backend action fails
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/916
This is an automated message from the Apache Git Service.
mrutkows closed issue #944: API section doesn't support response-type
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/944
This is an automated message from the Apache Git Service.
To respond to the
mrutkows opened a new issue #101: Track LICENSE and NOTICE issues for
`incubator-openwhisk-runtime-docker`
URL: https://github.com/apache/incubator-openwhisk-release/issues/101
Please use this issue to track any issues surrounding either the
houshengbo closed issue #101: Track LICENSE and NOTICE issues for
`incubator-openwhisk-runtime-docker`
URL: https://github.com/apache/incubator-openwhisk-release/issues/101
This is an automated message from the Apache Git
houshengbo closed issue #101: Track LICENSE and NOTICE issues for
`incubator-openwhisk-runtime-docker`
URL: https://github.com/apache/incubator-openwhisk-release/issues/101
This is an automated message from the Apache Git
houshengbo closed issue #102: Track LICENSE and NOTICE issues for
`incubator-openwhisk-runtime-java`
URL: https://github.com/apache/incubator-openwhisk-release/issues/102
This is an automated message from the Apache Git
houshengbo closed issue #103: Track LICENSE and NOTICE issues for
`incubator-openwhisk-runtime-php`
URL: https://github.com/apache/incubator-openwhisk-release/issues/103
This is an automated message from the Apache Git
houshengbo closed issue #104: Track LICENSE and NOTICE issues for
`incubator-openwhisk-runtime-python`
URL: https://github.com/apache/incubator-openwhisk-release/issues/104
This is an automated message from the Apache Git
houshengbo commented on issue #104: Track LICENSE and NOTICE issues for
`incubator-openwhisk-runtime-python`
URL:
https://github.com/apache/incubator-openwhisk-release/issues/104#issuecomment-396392798
All PRs have been merged, so this issue can be closed.
houshengbo closed issue #120: Define format of LICENSE file
URL: https://github.com/apache/incubator-openwhisk-release/issues/120
This is an automated message from the Apache Git Service.
To respond to the message, please
houshengbo commented on issue #120: Define format of LICENSE file
URL:
https://github.com/apache/incubator-openwhisk-release/issues/120#issuecomment-396391239
@daisy-ycguo @mrutkows We can refer to what are under
houshengbo commented on issue #137: JDK license issue for the runtime Java
URL:
https://github.com/apache/incubator-openwhisk-release/issues/137#issuecomment-396390975
Revisit this issue for further binary releases.
This is
houshengbo commented on issue #122: Documentation requirements for LogBack
(default logger)
URL:
https://github.com/apache/incubator-openwhisk-release/issues/122#issuecomment-396390854
Revisit this issue for further binary releases.
houshengbo closed issue #87: Review all (12) NOTICE files for contents
URL: https://github.com/apache/incubator-openwhisk-release/issues/87
This is an automated message from the Apache Git Service.
To respond to the message,
houshengbo commented on issue #87: Review all (12) NOTICE files for contents
URL:
https://github.com/apache/incubator-openwhisk-release/issues/87#issuecomment-396390468
@mrutkows We are fine with the LICENSE and NOTICE files so far, since we
expect source code release only as the first
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-396390140
@mrutkows This issue has been resolved, since Adobe contributors have
changed the
houshengbo closed issue #63: Verify API Gateway legal NOTICE / CREDITS files
ack. licenses/contribs.
URL: https://github.com/apache/incubator-openwhisk-release/issues/63
This is an automated message from the Apache Git
houshengbo commented on issue #34: List BSD / MIT licenses in LICENSE.txt if
needed
URL:
https://github.com/apache/incubator-openwhisk-release/issues/34#issuecomment-396389844
For source code release, we do not list them. We will revisit this issue for
further binary releases.
houshengbo commented on issue #13: Step 5: Add NOTICE, LICENSE, and CHANGELOG
to each OpenWhisk project
URL:
https://github.com/apache/incubator-openwhisk-release/issues/13#issuecomment-396389506
The LICENSE and NOTICE files have been added to the 12 repos we are about to
release for
houshengbo commented on issue #158: Add ASF license to ISSUE_TEMPLATE.md under
.github
URL:
https://github.com/apache/incubator-openwhisk-release/issues/158#issuecomment-396388860
.github is excluded in the source code release.
houshengbo closed issue #159: Verify source code release TARs do not contain
"ignore" and project tooling files.
URL: https://github.com/apache/incubator-openwhisk-release/issues/159
This is an automated message from the
houshengbo commented on issue #159: Verify source code release TARs do not
contain "ignore" and project tooling files.
URL:
https://github.com/apache/incubator-openwhisk-release/issues/159#issuecomment-396388684
duplicate of the issue
houshengbo commented on issue #161: Verify we do not include any tooling dot
`.` configuration files in source release
URL:
https://github.com/apache/incubator-openwhisk-release/issues/161#issuecomment-396388343
@mrutkows .dockerignore has to stay, because we need to build the docker
houshengbo commented on issue #161: Verify we do not include any tooling dot
`.` configuration files in source release
URL:
https://github.com/apache/incubator-openwhisk-release/issues/161#issuecomment-396388343
@mrutkows .dockerignore has to stay, because we need to build the docker
houshengbo commented on issue #165: Do we need to list Lua modules (luarocks)
installed as part of build?
URL:
https://github.com/apache/incubator-openwhisk-release/issues/165#issuecomment-396386635
@mrutkows tests are excluded in source code release.
houshengbo closed issue #165: Do we need to list Lua modules (luarocks)
installed as part of build?
URL: https://github.com/apache/incubator-openwhisk-release/issues/165
This is an automated message from the Apache Git
houshengbo commented on issue #166: Assure `lua_modules` directory tree (test
deps.) is not part of source or binary release.
URL:
https://github.com/apache/incubator-openwhisk-release/issues/166#issuecomment-396386461
@mrutkows Tests folder is excluded, since there will not be any
houshengbo closed issue #166: Assure `lua_modules` directory tree (test deps.)
is not part of source or binary release.
URL: https://github.com/apache/incubator-openwhisk-release/issues/166
This is an automated message from
houshengbo closed issue #170: Change directory from 'license' to 'licenses' for
runtime-swift
URL: https://github.com/apache/incubator-openwhisk-release/issues/170
This is an automated message from the Apache Git Service.
houshengbo commented on issue #170: Change directory from 'license' to
'licenses' for runtime-swift
URL:
https://github.com/apache/incubator-openwhisk-release/issues/170#issuecomment-396386059
@mrutkows This issue is fixed.
houshengbo commented on issue #171: Automate checks to verify 'tests' (and
other files) are removed from source release TAR files
URL:
https://github.com/apache/incubator-openwhisk-release/issues/171#issuecomment-396385957
@mrutkows I confirm this issue is resolved.
houshengbo closed issue #171: Automate checks to verify 'tests' (and other
files) are removed from source release TAR files
URL: https://github.com/apache/incubator-openwhisk-release/issues/171
This is an automated message
houshengbo opened a new pull request #182: Exclude specification/archive and
specification/diagrams for wskdeploy
URL: https://github.com/apache/incubator-openwhisk-release/pull/182
Closes: #175
This is an automated message
houshengbo commented on issue #153: Change JDK used to build main OW to be
"pluggable" for building Source (release)
URL:
https://github.com/apache/incubator-openwhisk-release/issues/153#issuecomment-396383412
@mrutkows We have already resolved this issue, since both openwhisk and
houshengbo commented on issue #153: Change JDK used to build main OW to be
"pluggable" for building Source (release)
URL:
https://github.com/apache/incubator-openwhisk-release/issues/153#issuecomment-396383412
@mrutkows We have already resolved this issue, since both openwhisk and
houshengbo closed issue #153: Change JDK used to build main OW to be
"pluggable" for building Source (release)
URL: https://github.com/apache/incubator-openwhisk-release/issues/153
This is an automated message from the
houshengbo commented on issue #153: Change JDK used to build main OW to be
"pluggable" for building Source (release)
URL:
https://github.com/apache/incubator-openwhisk-release/issues/153#issuecomment-396383412
@mrutkows We have already resolved this issue, since both openwhisk and
houshengbo commented on issue #177: Can we remove gradle-wrapper.jar from TAR
files?
URL:
https://github.com/apache/incubator-openwhisk-release/issues/177#issuecomment-396382954
@mrutkows As we discussed, we can keep the gradle wrapper in the source code
release. Apache Beam can be our
houshengbo closed issue #177: Can we remove gradle-wrapper.jar from TAR files?
URL: https://github.com/apache/incubator-openwhisk-release/issues/177
This is an automated message from the Apache Git Service.
To respond to the
rabbah commented on issue #181: Add openwhisk-kube-deploy to list of release
repos.
URL:
https://github.com/apache/incubator-openwhisk-release/pull/181#issuecomment-396382802
Vincent what’s the rationale for notn ncluding this in the first release
with the increasing prevalence of kube?
houshengbo commented on issue #179: Do we need to add JIRA automation
(changes.md gen) and track Fix Versions
URL:
https://github.com/apache/incubator-openwhisk-release/issues/179#issuecomment-396382555
@mrutkows So far, we can stay with our process as documented in the
instruction of
houshengbo commented on issue #178: Verify each TAR has changes(.md or .txt)
included
URL:
https://github.com/apache/incubator-openwhisk-release/issues/178#issuecomment-396381447
@mrutkows Since the very first release contains the initial code, we do not
need it by far. This function
houshengbo commented on issue #181: Add openwhisk-kube-deploy to list of
release repos.
URL:
https://github.com/apache/incubator-openwhisk-release/pull/181#issuecomment-396380105
For the first source code release, we have decided to pick up the existing
12 openwhisk repositories. At this
csantanapr commented on issue #3685: Enable test code coverage collection for
containers
URL:
https://github.com/apache/incubator-openwhisk/pull/3685#issuecomment-396353952
PG3 2361 looks good
This is an automated message
dubee commented on issue #3619: Provide an activation store SPI
URL:
https://github.com/apache/incubator-openwhisk/pull/3619#issuecomment-396353658
@markusthoemmes, anything else on this one?
This is an automated message
csantanapr commented on issue #3453: Introduce a AttachmentStore SPI
URL:
https://github.com/apache/incubator-openwhisk/pull/3453#issuecomment-396353552
PG1 3010 looks good
This is an automated message from the Apache Git
csantanapr opened a new issue #58: can't override npm packages that are
embedded when extending runtime image.
URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/issues/58
Given that I have a `package.json` that I want to use to extend the nodejs
runtime
Let's say I want
rabbah commented on a change in pull request #3568: Add activation result
integration test
URL:
https://github.com/apache/incubator-openwhisk/pull/3568#discussion_r194485364
##
File path: tests/src/test/scala/system/basic/WskActivationTests.scala
##
@@ -54,4 +57,24 @@
dubee commented on issue #3568: Add activation result integration test
URL:
https://github.com/apache/incubator-openwhisk/pull/3568#issuecomment-396321661
@rabbah, removed the retry loop as it will not be needed with the future
ActivationStore changes.
dubee commented on issue #3568: Add activation result integration test
URL:
https://github.com/apache/incubator-openwhisk/pull/3568#issuecomment-396321661
@rabbah, removed the retry loop as it will not be need with the future
ActivationStore changes.
apercova commented on issue #132: Fix to fallback when LOCAL_IP cannot be
obtained
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/132#issuecomment-396318882
Hi @ddragosd thank you for reviewing the fix.
I wonder if you could give me an insight on what SO's are you
csantanapr closed issue #51: Bump nodejs 8 to latest LTS 8.11.2
URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/issues/51
This is an automated message from the Apache Git Service.
To respond to the message,
csantanapr commented on issue #51: Bump nodejs 8 to latest LTS 8.11.2
URL:
https://github.com/apache/incubator-openwhisk-runtime-nodejs/issues/51#issuecomment-396318121
this is done
This is an automated message from the
csantanapr opened a new issue #57: CWD is not the root of the zip action
URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/issues/57
Problem when your action wants to reference another file in your zip,
locally this is easy because the node process usually get's started
chetanmeh closed pull request #3748: Mention Slack and mailing list links in
issue template
URL: https://github.com/apache/incubator-openwhisk/pull/3748
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 #3740: Exclude logback-test.xml from test
sources jar
URL: https://github.com/apache/incubator-openwhisk/pull/3740
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:
codecov-io commented on issue #3568: Add activation result integration test
URL:
https://github.com/apache/incubator-openwhisk/pull/3568#issuecomment-396305057
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3568?src=pr=h1)
Report
> Merging
codecov-io commented on issue #3748: Mention Slack and mailing list links in
issue template
URL:
https://github.com/apache/incubator-openwhisk/pull/3748#issuecomment-396302337
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3748?src=pr=h1)
Report
> Merging
codecov-io commented on issue #3685: Enable test code coverage collection for
containers
URL:
https://github.com/apache/incubator-openwhisk/pull/3685#issuecomment-390943297
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3685?src=pr=h1)
Report
> Merging
codecov-io commented on issue #3685: Enable test code coverage collection for
containers
URL:
https://github.com/apache/incubator-openwhisk/pull/3685#issuecomment-390943297
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3685?src=pr=h1)
Report
> Merging
codecov-io commented on issue #3453: Introduce a AttachmentStore SPI
URL:
https://github.com/apache/incubator-openwhisk/pull/3453#issuecomment-395695075
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3453?src=pr=h1)
Report
> Merging
rabbah commented on issue #3173: incubator-openwhisk-2882 Delete printlns
URL:
https://github.com/apache/incubator-openwhisk/pull/3173#issuecomment-396292359
@vooolll did you try `./gradlew :tests:test --tests
"your.package.name.TestClass.evenMethodName"`
dubee commented on issue #270: Fix binary encoding
URL:
https://github.com/apache/incubator-openwhisk-package-kafka/pull/270#issuecomment-396288463
@abaruni, FYI.
This is an automated message from the Apache Git Service.
To
rabbah commented on a change in pull request #311: Display appropriate message
for trigger fire when 204 is returned
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/311#discussion_r194450149
##
File path: commands/trigger.go
##
@@ -92,13 +93,24 @@ var
1 - 100 of 148 matches
Mail list logo
