[GitHub] [openwhisk-runtime-nodejs] dgrove-oss merged pull request #230: GitHub action to replace Travis CI build/test/publish logic

[GitBox]

dgrove-oss merged PR #230:
URL: https://github.com/apache/openwhisk-runtime-nodejs/pull/230


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-utilities] dgrove-oss merged pull request #87: Port CI from Travis to GitHub actions

[GitBox]

dgrove-oss merged PR #87:
URL: https://github.com/apache/openwhisk-utilities/pull/87


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] dgrove-oss merged pull request #5369: schedule actions to run at least 3 times a week

[GitBox]

dgrove-oss merged PR #5369:
URL: https://github.com/apache/openwhisk/pull/5369


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] codecov-commenter commented on pull request #5369: schedule actions to run at least 3 times a week

[GitBox]

codecov-commenter commented on PR #5369:
URL: https://github.com/apache/openwhisk/pull/5369#issuecomment-1397052146

   # 
[Codecov](https://codecov.io/gh/apache/openwhisk/pull/5369?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#5369](https://codecov.io/gh/apache/openwhisk/pull/5369?src=pr=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (1eeacd1) into 
[master](https://codecov.io/gh/apache/openwhisk/commit/21c9a6363fe14e581ec67da281e1762db02ae557?el=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (21c9a63) will **decrease** coverage by `71.26%`.
   > The diff coverage is `n/a`.
   
   > :exclamation: Current head 1eeacd1 differs from pull request most recent 
head fae737d. Consider uploading reports for the commit fae737d to get more 
accurate results
   
   ```diff
   @@Coverage Diff @@
   ##   master   #5369   +/-   ##
   ==
   - Coverage   75.78%   4.52%   -71.26% 
   ==
 Files 240 240   
 Lines   14407   14407   
 Branches  601 601   
   ==
   - Hits10918 652-10266 
   - Misses   3489   13755+10266 
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/openwhisk/pull/5369?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 | Coverage Δ | |
   |---|---|---|
   | 
[.../main/scala/org/apache/openwhisk/core/WarmUp.scala](https://codecov.io/gh/apache/openwhisk/pull/5369?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvV2FybVVwLnNjYWxh)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...ain/scala/org/apache/openwhisk/spi/SpiLoader.scala](https://codecov.io/gh/apache/openwhisk/pull/5369?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL3NwaS9TcGlMb2FkZXIuc2NhbGE=)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...n/scala/org/apache/openwhisk/utils/JsHelpers.scala](https://codecov.io/gh/apache/openwhisk/pull/5369?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL3V0aWxzL0pzSGVscGVycy5zY2FsYQ==)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...scala/org/apache/openwhisk/common/Prometheus.scala](https://codecov.io/gh/apache/openwhisk/pull/5369?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvbW1vbi9Qcm9tZXRoZXVzLnNjYWxh)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...scala/org/apache/openwhisk/common/time/Clock.scala](https://codecov.io/gh/apache/openwhisk/pull/5369?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvbW1vbi90aW1lL0Nsb2NrLnNjYWxh)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...scala/org/apache/openwhisk/core/FeatureFlags.scala](https://codecov.io/gh/apache/openwhisk/pull/5369?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvRmVhdHVyZUZsYWdzLnNjYWxh)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...scala/org/apache/openwhisk/http/CorsSettings.scala](https://codecov.io/gh/apache/openwhisk/pull/5369?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2h0dHAvQ29yc1NldHRpbmdzLnNjYWxh)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...ala/org/apache/openwhisk/common/ConfigMXBean.scala](https://codecov.io/gh/apache/openwhisk/pull/5369?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvbW1vbi9Db25maWdNWEJlYW4uc2NhbGE=)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 

[GitHub] [openwhisk] dgrove-oss opened a new pull request, #5369: schedule actions to run at least 3 times a week

[GitBox]

dgrove-oss opened a new pull request, #5369:
URL: https://github.com/apache/openwhisk/pull/5369

   Add schedule clause to GitHub actions to cause them to run 3 times a week.  
Schedule is offset by 1 hour for each workflow file to avoid heavy spikes.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-release] dgrove-oss opened a new pull request, #414: make scancode a mandatory check

[GitBox]

dgrove-oss opened a new pull request, #414:
URL: https://github.com/apache/openwhisk-release/pull/414

   Let's see if I understand the naming rules for GitHub actions in asf.yaml


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-release] dgrove-oss merged pull request #413: Add GitHub action badge

[GitBox]

dgrove-oss merged PR #413:
URL: https://github.com/apache/openwhisk-release/pull/413


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-release] style95 commented on a diff in pull request #413: Add GitHub action badge

[GitBox]

style95 commented on code in PR #413:
URL: https://github.com/apache/openwhisk-release/pull/413#discussion_r1080697250


##
README.md:
##
@@ -20,6 +20,7 @@
 # Apache OpenWhisk Project Release Management
 
 
[![License](https://img.shields.io/badge/license-Apache--2.0-blue.svg)](http://www.apache.org/licenses/LICENSE-2.0)
+[![Continuous 
Integration](https://github.com/apache/openwhisk-release/actions/workflows/ci.yaml/badge.svg)](https://github.com/apache/openwhisk-release/actions/workflows/ci.yaml)

Review Comment:
    



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-release] dependabot[bot] opened a new pull request, #412: Bump pyyaml from 5.3 to 5.4 in /tools

[GitBox]

dependabot[bot] opened a new pull request, #412:
URL: https://github.com/apache/openwhisk-release/pull/412

   Bumps [pyyaml](https://github.com/yaml/pyyaml) from 5.3 to 5.4.
   
   Changelog
   Sourced from https://github.com/yaml/pyyaml/blob/master/CHANGES;>pyyaml's 
changelog.
   
   5.4 (2021-01-19)
   
   https://github-redirect.dependabot.com/yaml/pyyaml/pull/407;>yaml/pyyaml#407
 -- Build modernization, remove distutils, fix metadata, build wheels, CI to 
GHA
   https://github-redirect.dependabot.com/yaml/pyyaml/pull/472;>yaml/pyyaml#472
 -- Fix for CVE-2020-14343, moves arbitrary python tags to UnsafeLoader
   https://github-redirect.dependabot.com/yaml/pyyaml/pull/441;>yaml/pyyaml#441
 -- Fix memory leak in implicit resolver setup
   https://github-redirect.dependabot.com/yaml/pyyaml/pull/392;>yaml/pyyaml#392
 -- Fix py2 copy support for timezone objects
   https://github-redirect.dependabot.com/yaml/pyyaml/pull/378;>yaml/pyyaml#378
 -- Fix compatibility with Jython
   
   5.3.1 (2020-03-18)
   
   https://github-redirect.dependabot.com/yaml/pyyaml/pull/386;>yaml/pyyaml#386
 -- Prevents arbitrary code execution during python/object/new constructor
   
   
   
   
   Commits
   
   https://github.com/yaml/pyyaml/commit/58d0cb7ee09954c67fabfbd714c5673b03e7a9e1;>58d0cb7
 5.4 release
   https://github.com/yaml/pyyaml/commit/a60f7a19c0b418fe95fcf2ec0957005ae39e1090;>a60f7a1
 Fix compatibility with Jython
   https://github.com/yaml/pyyaml/commit/ee98abd7d7bd2ca9c7b98aa19164fd0306a3f3d2;>ee98abd
 Run CI on PR base branch changes
   https://github.com/yaml/pyyaml/commit/ddf20330be1fae8813b8ce1789c48f244746d252;>ddf2033
 constructor.timezone: _copy  deepcopy
   https://github.com/yaml/pyyaml/commit/fc914d52c43f499224f7fb4c2d4c47623adc5b33;>fc914d5
 Avoid repeatedly appending to yaml_implicit_resolvers
   https://github.com/yaml/pyyaml/commit/a001f2782501ad2d24986959f0239a354675f9dc;>a001f27
 Fix for CVE-2020-14343
   https://github.com/yaml/pyyaml/commit/fe150624146ee631bb0f95e45731e8b01281fed6;>fe15062
 Add 3.9 to appveyor file for completeness sake
   https://github.com/yaml/pyyaml/commit/1e1c7fb7c09e9149967c208a6fd07276a6140d57;>1e1c7fb
 Add a newline character to end of pyproject.toml
   https://github.com/yaml/pyyaml/commit/0b6b7d61719fbe0a11f0980489f1bf8ce746c164;>0b6b7d6
 Start sentences and phrases for capital letters
   https://github.com/yaml/pyyaml/commit/c97691596eec279ef9191a9b3bba583a17139d5a;>c976915
 Shell code improvements
   Additional commits viewable in https://github.com/yaml/pyyaml/compare/5.3...5.4;>compare view
   
   
   
   
   
   [![Dependabot compatibility 
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=pyyaml=pip=5.3=5.4)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (dependabot-automerge-end)
   
   ---
   
   
   Dependabot commands and options
   
   
   You can trigger Dependabot actions by commenting on this PR:
   - `@dependabot rebase` will rebase this PR
   - `@dependabot recreate` will recreate this PR, overwriting any edits that 
have been made to it
   - `@dependabot merge` will merge this PR after your CI passes on it
   - `@dependabot squash and merge` will squash and merge this PR after your CI 
passes on it
   - `@dependabot cancel merge` will cancel a previously requested merge and 
block automerging
   - `@dependabot reopen` will reopen this PR if it is closed
   - `@dependabot close` will close this PR and stop Dependabot recreating it. 
You can achieve the same result by closing it manually
   - `@dependabot ignore this major version` will close this PR and stop 
Dependabot creating any more for this major version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this minor version` will close this PR and stop 
Dependabot creating any more for this minor version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this dependency` will close this PR and stop 
Dependabot creating any more for this dependency (unless you reopen the PR or 
upgrade to it yourself)
   - `@dependabot use these labels` will set the current labels as the default 
for future PRs for this repo and language
   - `@dependabot use these reviewers` will set the current reviewers as the 
default for future PRs for this repo and language
   - `@dependabot use these assignees` will set the current assignees as the 
default for future PRs for this repo and language
   - `@dependabot use this milestone` will set the current milestone as the 
default for future PRs for this repo and language
   
   You can disable automated security fix PRs for this repo from the [Security 
Alerts 

[GitHub] [openwhisk-release] dgrove-oss merged pull request #411: next stage in porting CI from Travis to GitHub actions

[GitBox]

dgrove-oss merged PR #411:
URL: https://github.com/apache/openwhisk-release/pull/411


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-release] dgrove-oss commented on a diff in pull request #411: next stage in porting CI from Travis to GitHub actions

[GitBox]

dgrove-oss commented on code in PR #411:
URL: https://github.com/apache/openwhisk-release/pull/411#discussion_r1073576428


##
README.md:
##
@@ -20,7 +20,6 @@
 # Apache OpenWhisk Project Release Management
 
 
[![License](https://img.shields.io/badge/license-Apache--2.0-blue.svg)](http://www.apache.org/licenses/LICENSE-2.0)
-[![Build 
Status](https://travis-ci.com/apache/openwhisk-release.svg?branch=master)](https://travis-ci.com/github/apache/openwhisk-release)

Review Comment:
   Once the workflow is running, I plan to add the badge in a follow up PR.  



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-release] style95 commented on a diff in pull request #411: next stage in porting CI from Travis to GitHub actions

[GitBox]

style95 commented on code in PR #411:
URL: https://github.com/apache/openwhisk-release/pull/411#discussion_r1073324536


##
README.md:
##
@@ -20,7 +20,6 @@
 # Apache OpenWhisk Project Release Management
 
 
[![License](https://img.shields.io/badge/license-Apache--2.0-blue.svg)](http://www.apache.org/licenses/LICENSE-2.0)
-[![Build 
Status](https://travis-ci.com/apache/openwhisk-release.svg?branch=master)](https://travis-ci.com/github/apache/openwhisk-release)

Review Comment:
   Could we add the status badge via Github workflow too or is this removed for 
some reason?
   
https://docs.github.com/en/actions/monitoring-and-troubleshooting-workflows/adding-a-workflow-status-badge



##
.github/workflows/ci.yaml:
##
@@ -0,0 +1,33 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+#
+
+name: Continuous Integration
+on:
+  push:
+branches: [ master ]
+  pull_request:
+branches: [ master ]
+types: [ opened, synchronize, reopened ]
+jobs:
+  scancode:
+runs-on: ubuntu-22.04
+steps:
+  - name: Checkout
+uses: actions/checkout@v3
+  - name: Scan Code
+uses: apache/openwhisk-utilities/scancode@master

Review Comment:
    



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-nodejs] dgrove-oss opened a new pull request, #230: GitHub action to replace Travis CI build/test/publish logic

[GitBox]

dgrove-oss opened a new pull request, #230:
URL: https://github.com/apache/openwhisk-runtime-nodejs/pull/230

   I believe this is now a full replacement for our existing Travis CI based 
workflow for this repository. 
   
   I've been testing on my fork and it looks good.  Assuming it also looks good 
on the real repo (including the cron job to drive publishing of new images a 
few times a week), we should be able to fairly easily adapt the workflow file 
to the rest of our runtime repos.
   
   I decided to port as much of the workflow as possible to a native GHA idioms 
and not just invoke our legacy scripts (which had accumulated a fair amount of 
historical baggage). 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-utilities] dgrove-oss opened a new pull request, #87: Port CI from Travis to GitHub actions

[GitBox]

dgrove-oss opened a new pull request, #87:
URL: https://github.com/apache/openwhisk-utilities/pull/87

   This adds a GHA that performs the same set of tests as we were doing on 
Travis CI.
   
   Once this is merged, I will do a followup to remove the .travis file, change 
the badge in the readme, and hopefully make the GHA required for merging PRs.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-release] dgrove-oss opened a new pull request, #411: remove Travis CI file/badge

[GitBox]

dgrove-oss opened a new pull request, #411:
URL: https://github.com/apache/openwhisk-release/pull/411

   We were not running any actual tests on this repo, so can be trivally 
dropped.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-release] dgrove-oss merged pull request #410: client-js 1.21.7 release config

[GitBox]

dgrove-oss merged PR #410:
URL: https://github.com/apache/openwhisk-release/pull/410


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-website] dgrove-oss merged pull request #504: Update to modern versions of tools; use rbenv

[GitBox]

dgrove-oss merged PR #504:
URL: https://github.com/apache/openwhisk-website/pull/504


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-go] Luke-Roy-IBM opened a new pull request, #180: update net package to fix CVE-2022-41717 + work around added in Dockerfile

[GitBox]

Luke-Roy-IBM opened a new pull request, #180:
URL: https://github.com/apache/openwhisk-runtime-go/pull/180

   Update net standard module to fix 
[CVE-2022-41717](https://www.cve.org/CVERecord?id=CVE-2022-41717)
   Work around can be removed when base image is update with fix


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-release] dgrove-oss opened a new pull request, #410: client-js 1.21.7 release config

[GitBox]

dgrove-oss opened a new pull request, #410:
URL: https://github.com/apache/openwhisk-release/pull/410

   This release was made in August, but because the website publish was broken 
I hadn't submitted the config file or archived the prior version.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-website] dgrove-oss opened a new pull request, #504: Update to modern versions of tools; use rbenv

[GitBox]

dgrove-oss opened a new pull request, #504:
URL: https://github.com/apache/openwhisk-website/pull/504

   This gets the website building again;  it has been broken for several months.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-vscode-extension] dependabot[bot] opened a new pull request, #25: Bump json5 from 1.0.1 to 1.0.2

[GitBox]

dependabot[bot] opened a new pull request, #25:
URL: https://github.com/apache/openwhisk-vscode-extension/pull/25

   Bumps [json5](https://github.com/json5/json5) from 1.0.1 to 1.0.2.
   
   Release notes
   Sourced from https://github.com/json5/json5/releases;>json5's releases.
   
   v1.0.2
   
   Fix: Properties with the name __proto__ are added to 
objects and arrays. (https://github-redirect.dependabot.com/json5/json5/issues/199;>#199) 
This also fixes a prototype pollution vulnerability reported by Jonathan 
Gregson! (https://github-redirect.dependabot.com/json5/json5/issues/295;>#295). 
This has been backported to v1. (https://github-redirect.dependabot.com/json5/json5/issues/298;>#298)
   
   
   
   
   Changelog
   Sourced from https://github.com/json5/json5/blob/main/CHANGELOG.md;>json5's 
changelog.
   
   Unreleased [https://github.com/json5/json5/tree/main;>code, 
https://github.com/json5/json5/compare/v2.2.3...HEAD;>diff]
   v2.2.3 [https://github.com/json5/json5/tree/v2.2.3;>code, 
https://github.com/json5/json5/compare/v2.2.2...v2.2.3;>diff]
   
   Fix: json5@2.2.3 is now the 'latest' release according to npm instead of
   v1.0.2. (https://github-redirect.dependabot.com/json5/json5/issues/299;>#299)
   
   v2.2.2 [https://github.com/json5/json5/tree/v2.2.2;>code, 
https://github.com/json5/json5/compare/v2.2.1...v2.2.2;>diff]
   
   Fix: Properties with the name __proto__ are added to 
objects and arrays.
   (https://github-redirect.dependabot.com/json5/json5/issues/199;>#199) 
This also fixes a prototype pollution vulnerability reported by
   Jonathan Gregson! (https://github-redirect.dependabot.com/json5/json5/issues/295;>#295).
   
   v2.2.1 [https://github.com/json5/json5/tree/v2.2.1;>code, 
https://github.com/json5/json5/compare/v2.2.0...v2.2.1;>diff]
   
   Fix: Removed dependence on minimist to patch CVE-2021-44906. (https://github-redirect.dependabot.com/json5/json5/issues/266;>#266)
   
   v2.2.0 [https://github.com/json5/json5/tree/v2.2.0;>code, 
https://github.com/json5/json5/compare/v2.1.3...v2.2.0;>diff]
   
   New: Accurate and documented TypeScript declarations are now included. 
There
   is no need to install @types/json5. (https://github-redirect.dependabot.com/json5/json5/issues/236;>#236, 
https://github-redirect.dependabot.com/json5/json5/issues/244;>#244)
   
   v2.1.3 [https://github.com/json5/json5/tree/v2.1.3;>code, 
https://github.com/json5/json5/compare/v2.1.2...v2.1.3;>diff]
   
   Fix: An out of memory bug when parsing numbers has been fixed. (https://github-redirect.dependabot.com/json5/json5/issues/228;>#228,
   https://github-redirect.dependabot.com/json5/json5/issues/229;>#229)
   
   v2.1.2 [https://github.com/json5/json5/tree/v2.1.2;>code, 
https://github.com/json5/json5/compare/v2.1.1...v2.1.2;>diff]
   
   
   ... (truncated)
   
   
   Commits
   
   https://github.com/json5/json5/commit/a62db1e51e1031d92ac260f5bb38bbed1fdbc754;>a62db1e
 1.0.2
   https://github.com/json5/json5/commit/e0c23fe458a77c0b2cdb271376be5d8d0908133c;>e0c23fe
 docs: update CHANGELOG for v1.0.2
   https://github.com/json5/json5/commit/62a65408408d40aeea14c7869ed327acead12972;>62a6540
 fix: add proto to objects and arrays
   See full diff in https://github.com/json5/json5/compare/v1.0.1...v1.0.2;>compare 
view
   
   
   
   
   
   [![Dependabot compatibility 
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=json5=npm_and_yarn=1.0.1=1.0.2)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (dependabot-automerge-end)
   
   ---
   
   
   Dependabot commands and options
   
   
   You can trigger Dependabot actions by commenting on this PR:
   - `@dependabot rebase` will rebase this PR
   - `@dependabot recreate` will recreate this PR, overwriting any edits that 
have been made to it
   - `@dependabot merge` will merge this PR after your CI passes on it
   - `@dependabot squash and merge` will squash and merge this PR after your CI 
passes on it
   - `@dependabot cancel merge` will cancel a previously requested merge and 
block automerging
   - `@dependabot reopen` will reopen this PR if it is closed
   - `@dependabot close` will close this PR and stop Dependabot recreating it. 
You can achieve the same result by closing it manually
   - `@dependabot ignore this major version` will close this PR and stop 
Dependabot creating any more for this major version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this minor version` will close this PR and stop 
Dependabot creating any more for this minor version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this dependency` will 

[GitHub] [openwhisk] bdoyle0182 commented on issue #5368: Action stucks in error forever after 'too many requests' response occurred

[GitBox]

bdoyle0182 commented on issue #5368:
URL: https://github.com/apache/openwhisk/issues/5368#issuecomment-1374271352

   Hi dino,
   
   I actually think I've managed to reproduce this. I don't think it's related 
to action concurrency, but rather just high scale number of executions of a 
specific action which results in the action throttling key getting stuck in the 
true state in etcd even after demand goes below the threshold such that 
throttling should be turned off. I will try to find time to debug next week.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] dinok97 commented on issue #5368: Action stucks in error forever after 'too many requests' response occurred

[GitBox]

dinok97 commented on issue #5368:
URL: https://github.com/apache/openwhisk/issues/5368#issuecomment-1373137162

   Thanks @bdoyle0182 !
   
   The error that I mention above seems like happened only when traffic is 
high. 
   Your explanation about 'scheduler decision on container creation', it makes 
sense for me.
   
   I'll open this issue until next 3 days, in case anyone want to give their 
thought related to this issue. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-wskdebug] dependabot[bot] opened a new pull request, #118: Bump json5 from 2.1.3 to 2.2.3

[GitBox]

dependabot[bot] opened a new pull request, #118:
URL: https://github.com/apache/openwhisk-wskdebug/pull/118

   Bumps [json5](https://github.com/json5/json5) from 2.1.3 to 2.2.3.
   
   Release notes
   Sourced from https://github.com/json5/json5/releases;>json5's releases.
   
   v2.2.3
   
   Fix: json5@2.2.3 is now the 'latest' release according to npm instead of 
v1.0.2. (https://github-redirect.dependabot.com/json5/json5/issues/299;>#299)
   
   v2.2.2
   
   Fix: Properties with the name __proto__ are added to 
objects and arrays.
   (https://github-redirect.dependabot.com/json5/json5/issues/199;>#199) 
This also fixes a prototype pollution vulnerability reported by
   Jonathan Gregson! (https://github-redirect.dependabot.com/json5/json5/issues/295;>#295).
   
   v2.2.1
   
   Fix: Removed dependence on minimist to patch CVE-2021-44906. (https://github-redirect.dependabot.com/json5/json5/issues/266;>#266)
   
   v2.2.0
   
   New: Accurate and documented TypeScript declarations are now included. 
There is no need to install @types/json5. (https://github-redirect.dependabot.com/json5/json5/issues/236;>#236, 
https://github-redirect.dependabot.com/json5/json5/issues/244;>#244)
   
   
   
   
   Changelog
   Sourced from https://github.com/json5/json5/blob/main/CHANGELOG.md;>json5's 
changelog.
   
   v2.2.3 [https://github.com/json5/json5/tree/v2.2.3;>code, 
https://github.com/json5/json5/compare/v2.2.2...v2.2.3;>diff]
   
   Fix: json5@2.2.3 is now the 'latest' release according to npm instead of
   v1.0.2. (https://github-redirect.dependabot.com/json5/json5/issues/299;>#299)
   
   v2.2.2 [https://github.com/json5/json5/tree/v2.2.2;>code, 
https://github.com/json5/json5/compare/v2.2.1...v2.2.2;>diff]
   
   Fix: Properties with the name __proto__ are added to 
objects and arrays.
   (https://github-redirect.dependabot.com/json5/json5/issues/199;>#199) 
This also fixes a prototype pollution vulnerability reported by
   Jonathan Gregson! (https://github-redirect.dependabot.com/json5/json5/issues/295;>#295).
   
   v2.2.1 [https://github.com/json5/json5/tree/v2.2.1;>code, 
https://github.com/json5/json5/compare/v2.2.0...v2.2.1;>diff]
   
   Fix: Removed dependence on minimist to patch CVE-2021-44906. (https://github-redirect.dependabot.com/json5/json5/issues/266;>#266)
   
   v2.2.0 [https://github.com/json5/json5/tree/v2.2.0;>code, 
https://github.com/json5/json5/compare/v2.1.3...v2.2.0;>diff]
   
   New: Accurate and documented TypeScript declarations are now included. 
There
   is no need to install @types/json5. (https://github-redirect.dependabot.com/json5/json5/issues/236;>#236, 
https://github-redirect.dependabot.com/json5/json5/issues/244;>#244)
   
   
   
   
   Commits
   
   https://github.com/json5/json5/commit/c3a75242772a5026a49c4017a16d9b3543b62776;>c3a7524
 2.2.3
   https://github.com/json5/json5/commit/94fd06d82eeed225fa172f6fb2ca27375cbd2e39;>94fd06d
 docs: update CHANGELOG for v2.2.3
   https://github.com/json5/json5/commit/3b8cebf0c474a8b20c78bd75c89cca0c4dce84ce;>3b8cebf
 docs(security): use GitHub security advisories
   https://github.com/json5/json5/commit/f0fd9e194dde282caff114a110f4fac635f3a62c;>f0fd9e1
 docs: publish a security policy
   https://github.com/json5/json5/commit/6a91a05fffeda16ff6b3b5008b6b340d42d31ec0;>6a91a05
 docs(template): bug - bug report
   https://github.com/json5/json5/commit/14f8cb186e8abdfaccf6527171da7b1224374650;>14f8cb1
 2.2.2
   https://github.com/json5/json5/commit/10cc7ca9169b59c5e0f5afc03dbd870cd06bcc46;>10cc7ca
 docs: update CHANGELOG for v2.2.2
   https://github.com/json5/json5/commit/7774c1097993bc3ce9f0ac4b722a32bf7d6871c8;>7774c10
 fix: add proto to objects and arrays
   https://github.com/json5/json5/commit/edde30abd8b22facf2c06c72586b9f6edf12700d;>edde30a
 Readme: slight tweak to intro
   https://github.com/json5/json5/commit/97286f8bd542c89dcee096bc05dd28ed2dfc1e16;>97286f8
 Improve example in readme
   Additional commits viewable in https://github.com/json5/json5/compare/v2.1.3...v2.2.3;>compare 
view
   
   
   
   
   
   [![Dependabot compatibility 
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=json5=npm_and_yarn=2.1.3=2.2.3)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (dependabot-automerge-end)
   
   ---
   
   
   Dependabot commands and options
   
   
   You can trigger Dependabot actions by commenting on this PR:
   - `@dependabot rebase` will rebase this PR
   - `@dependabot recreate` will recreate this PR, overwriting any edits that 
have been made to it
   - `@dependabot merge` will merge this PR after your CI passes on it
   - `@dependabot squash and merge` will 

[GitHub] [openwhisk-client-js] dependabot[bot] opened a new pull request, #251: Bump json5 from 2.1.3 to 2.2.3

[GitBox]

dependabot[bot] opened a new pull request, #251:
URL: https://github.com/apache/openwhisk-client-js/pull/251

   Bumps [json5](https://github.com/json5/json5) from 2.1.3 to 2.2.3.
   
   Release notes
   Sourced from https://github.com/json5/json5/releases;>json5's releases.
   
   v2.2.3
   
   Fix: json5@2.2.3 is now the 'latest' release according to npm instead of 
v1.0.2. (https://github-redirect.dependabot.com/json5/json5/issues/299;>#299)
   
   v2.2.2
   
   Fix: Properties with the name __proto__ are added to 
objects and arrays.
   (https://github-redirect.dependabot.com/json5/json5/issues/199;>#199) 
This also fixes a prototype pollution vulnerability reported by
   Jonathan Gregson! (https://github-redirect.dependabot.com/json5/json5/issues/295;>#295).
   
   v2.2.1
   
   Fix: Removed dependence on minimist to patch CVE-2021-44906. (https://github-redirect.dependabot.com/json5/json5/issues/266;>#266)
   
   v2.2.0
   
   New: Accurate and documented TypeScript declarations are now included. 
There is no need to install @types/json5. (https://github-redirect.dependabot.com/json5/json5/issues/236;>#236, 
https://github-redirect.dependabot.com/json5/json5/issues/244;>#244)
   
   
   
   
   Changelog
   Sourced from https://github.com/json5/json5/blob/main/CHANGELOG.md;>json5's 
changelog.
   
   v2.2.3 [https://github.com/json5/json5/tree/v2.2.3;>code, 
https://github.com/json5/json5/compare/v2.2.2...v2.2.3;>diff]
   
   Fix: json5@2.2.3 is now the 'latest' release according to npm instead of
   v1.0.2. (https://github-redirect.dependabot.com/json5/json5/issues/299;>#299)
   
   v2.2.2 [https://github.com/json5/json5/tree/v2.2.2;>code, 
https://github.com/json5/json5/compare/v2.2.1...v2.2.2;>diff]
   
   Fix: Properties with the name __proto__ are added to 
objects and arrays.
   (https://github-redirect.dependabot.com/json5/json5/issues/199;>#199) 
This also fixes a prototype pollution vulnerability reported by
   Jonathan Gregson! (https://github-redirect.dependabot.com/json5/json5/issues/295;>#295).
   
   v2.2.1 [https://github.com/json5/json5/tree/v2.2.1;>code, 
https://github.com/json5/json5/compare/v2.2.0...v2.2.1;>diff]
   
   Fix: Removed dependence on minimist to patch CVE-2021-44906. (https://github-redirect.dependabot.com/json5/json5/issues/266;>#266)
   
   v2.2.0 [https://github.com/json5/json5/tree/v2.2.0;>code, 
https://github.com/json5/json5/compare/v2.1.3...v2.2.0;>diff]
   
   New: Accurate and documented TypeScript declarations are now included. 
There
   is no need to install @types/json5. (https://github-redirect.dependabot.com/json5/json5/issues/236;>#236, 
https://github-redirect.dependabot.com/json5/json5/issues/244;>#244)
   
   
   
   
   Commits
   
   https://github.com/json5/json5/commit/c3a75242772a5026a49c4017a16d9b3543b62776;>c3a7524
 2.2.3
   https://github.com/json5/json5/commit/94fd06d82eeed225fa172f6fb2ca27375cbd2e39;>94fd06d
 docs: update CHANGELOG for v2.2.3
   https://github.com/json5/json5/commit/3b8cebf0c474a8b20c78bd75c89cca0c4dce84ce;>3b8cebf
 docs(security): use GitHub security advisories
   https://github.com/json5/json5/commit/f0fd9e194dde282caff114a110f4fac635f3a62c;>f0fd9e1
 docs: publish a security policy
   https://github.com/json5/json5/commit/6a91a05fffeda16ff6b3b5008b6b340d42d31ec0;>6a91a05
 docs(template): bug - bug report
   https://github.com/json5/json5/commit/14f8cb186e8abdfaccf6527171da7b1224374650;>14f8cb1
 2.2.2
   https://github.com/json5/json5/commit/10cc7ca9169b59c5e0f5afc03dbd870cd06bcc46;>10cc7ca
 docs: update CHANGELOG for v2.2.2
   https://github.com/json5/json5/commit/7774c1097993bc3ce9f0ac4b722a32bf7d6871c8;>7774c10
 fix: add proto to objects and arrays
   https://github.com/json5/json5/commit/edde30abd8b22facf2c06c72586b9f6edf12700d;>edde30a
 Readme: slight tweak to intro
   https://github.com/json5/json5/commit/97286f8bd542c89dcee096bc05dd28ed2dfc1e16;>97286f8
 Improve example in readme
   Additional commits viewable in https://github.com/json5/json5/compare/v2.1.3...v2.2.3;>compare 
view
   
   
   
   
   
   [![Dependabot compatibility 
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=json5=npm_and_yarn=2.1.3=2.2.3)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (dependabot-automerge-end)
   
   ---
   
   
   Dependabot commands and options
   
   
   You can trigger Dependabot actions by commenting on this PR:
   - `@dependabot rebase` will rebase this PR
   - `@dependabot recreate` will recreate this PR, overwriting any edits that 
have been made to it
   - `@dependabot merge` will merge this PR after your CI passes on it
   - `@dependabot squash and merge` will 

[GitHub] [openwhisk-runtime-swift] Andrea-Scuderi commented on pull request #153: Add Support for Swift 5.7

[GitBox]

Andrea-Scuderi commented on PR #153:
URL: 
https://github.com/apache/openwhisk-runtime-swift/pull/153#issuecomment-1372557751

   > Do you have an Apache CLA on file I was not able to locate one.
   
   Yes, I've sent a CLA the 11/11/2019.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-cli] dgrove-oss closed pull request #513: fix akka versions

[GitBox]

dgrove-oss closed pull request #513: fix akka versions
URL: https://github.com/apache/openwhisk-cli/pull/513


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-cli] dgrove-oss commented on pull request #513: fix akka versions

[GitBox]

dgrove-oss commented on PR #513:
URL: https://github.com/apache/openwhisk-cli/pull/513#issuecomment-1372457515

   obsolete 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-cli] dgrove-oss commented on pull request #515: fix travis URL

[GitBox]

dgrove-oss commented on PR #515:
URL: https://github.com/apache/openwhisk-cli/pull/515#issuecomment-1372456826

   No longer relevant


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-cli] dgrove-oss closed pull request #515: fix travis URL

[GitBox]

dgrove-oss closed pull request #515: fix travis URL
URL: https://github.com/apache/openwhisk-cli/pull/515


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-nodejs] Rurouni-Jiaxin commented on issue #229: Gradle : Incompatible versions

[GitBox]

Rurouni-Jiaxin commented on issue #229:
URL: 
https://github.com/apache/openwhisk-runtime-nodejs/issues/229#issuecomment-1372346857

   > > Build successful, though with warning "Deprecated Gradle features were 
used in this build, making it incompatible with Gradle 7.0."
   > 
   > this looks like a warning rather than a build failure. Did the docker 
image not get built?
   
   Indeed, my mistake


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-swift] rabbah commented on pull request #153: Add Support for Swift 5.7

[GitBox]

rabbah commented on PR #153:
URL: 
https://github.com/apache/openwhisk-runtime-swift/pull/153#issuecomment-1372292708

   Thanks @Andrea-Scuderi for this contribution. 
   
   Do you have an Apache CLA on file I was not able to locate one.
   
   We may continue to hold merging this PR until we've cut over to the new GHA 
flow and away from Travis.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-utilities] dgrove-oss merged pull request #86: GitHub action to run scanCode.py

[GitBox]

dgrove-oss merged PR #86:
URL: https://github.com/apache/openwhisk-utilities/pull/86


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-nodejs] rabbah commented on issue #229: Gradle Task :core:nodejs1XAction:distDocker FAILED

[GitBox]

rabbah commented on issue #229:
URL: 
https://github.com/apache/openwhisk-runtime-nodejs/issues/229#issuecomment-1372279323

   > Build successful, though with warning "Deprecated Gradle features were 
used in this build, making it incompatible with Gradle 7.0."
   
   this looks like a warning rather than a build failure. Did the docker image 
not get built?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] bdoyle0182 commented on issue #5368: Action stucks in error forever after 'too many requests' response occurred

[GitBox]

bdoyle0182 commented on issue #5368:
URL: https://github.com/apache/openwhisk/issues/5368#issuecomment-1370185863

   I’ve played with action concurrency > 1 on the new scheduler and it works
   as is however I don’t think there’s been any verification to production
   readiness. The scheduler doesn’t make any decisions right now on container
   creations regarding an action with concurrency so I wouldn’t be shocked if
   there’s some bug with setting and inserting the action throttled flag in
   etcd when action concurrency is in use.
   
   On Fri, Dec 30, 2022 at 5:46 AM Dino Keylas ***@***.***>
   wrote:
   
   > Thanks for your response! @style95 
   > Also sorry for my late reply. Just try to find out the problem, but it
   > takes time for my since I'm beginner in Scala.
   >
   > I have seen the logs and didn't found any error log related to this issue.
   > According my experiment, the throttling is enable for action. I'm able to
   > invoke other actions under same namespace with perfTest.
   >
   > The error that I described above only happen when action concurrency = 100
   > (I increased the action concurrency from 1 to 100).
   > If the action concurrency = 1, the error not occur.
   >
   > Is new Scheduler well supported for action concurrency?
   >
   > —
   > Reply to this email directly, view it on GitHub
   > ,
   > or unsubscribe
   > 

   > .
   > You are receiving this because you are subscribed to this thread.Message
   > ID: ***@***.***>
   >
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-vscode-extension] dependabot[bot] opened a new pull request, #24: Bump json5, ts-loader, webpack and webpack-cli

[GitBox]

dependabot[bot] opened a new pull request, #24:
URL: https://github.com/apache/openwhisk-vscode-extension/pull/24

   Removes [json5](https://github.com/json5/json5). It's no longer used after 
updating ancestor dependencies [json5](https://github.com/json5/json5), 
[ts-loader](https://github.com/TypeStrong/ts-loader), 
[webpack](https://github.com/webpack/webpack) and 
[webpack-cli](https://github.com/webpack/webpack-cli). These dependencies need 
to be updated together.
   
   Removes `json5`
   
   Updates `ts-loader` from 7.0.5 to 9.4.2
   
   Release notes
   Sourced from https://github.com/TypeStrong/ts-loader/releases;>ts-loader's 
releases.
   
   v9.4.2
   
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/pull/1550;>Bug
 fix: Use custom transformer when building solution references https://github-redirect.dependabot.com/TypeStrong/ts-loader/issues/1025;>#1025
 - thanks https://github.com/feosuna1;>@​feosuna1
   
   9.4.1
   
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/pull/1505;>Hotfix:
 Disable enhanced-resolve - thanks https://github.com/manuth;>@​manuth
   
   v9.4.0
   
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/pull/1503;>Add
 Support for Resolving .cjs, .mjs, .cts 
and .mts Files https://github-redirect.dependabot.com/TypeStrong/ts-loader/issues/1503;>#1503
 - thanks https://github.com/manuth;>@​manuth
   
   v9.3.1
   
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/pull/1483;>Bug
 fix: Generate declaration files for js files if allowJs is set to true https://github-redirect.dependabot.com/TypeStrong/ts-loader/issues/1260;>#1260
 - thanks https://github.com/hediet;>@​hediet and https://github.com/mvilliger;>@​mvilliger
   
   v9.3.0
   
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/pull/1451;>simplify
 configuration for fork-ts-checker-webpack-plugin - thanks https://github.com/piotr-oles;>@​piotr-oles
   
   v9.2.9
   
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/pull/1447;>make
 v9 latest following v8 release - thanks https://github.com/johnnyreilly;>@​johnnyreilly
   
   v9.2.8
   
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/pull/1439;>Bug
 fix: support webpack 5 in ts-loader https://github-redirect.dependabot.com/TypeStrong/ts-loader/issues/1438;>#1438
 - thanks https://github.com/einatbar;>@​einatbar
   
   v9.2.7
   
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/pull/1422;>cater
 for change in resolveTypeReferenceDirective API in TypeScript 4.7  https://github-redirect.dependabot.com/TypeStrong/ts-loader/issues/1421;>#1421
 - thanks https://github.com/johnnyreilly;>@​johnnyreilly and https://github.com/cspotcode;>@​cspotcode for 
inspiration in ts-node work here: https://github-redirect.dependabot.com/TypeStrong/ts-node/pull/1648;>TypeStrong/ts-node#1648
   
   v9.2.6
   
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/pull/1377;>Docs
 fix for thread-loader / history - thanks https://github.com/johnnyreilly;>@​johnnyreilly
   
   v9.2.5
   
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/pull/1352;>Add
 function to get the latest program - thanks https://github.com/Zn4rK;>@​Zn4rK
   
   v9.2.4
   
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/pull/1346;>Fix
 undefined configPath now falls back to default - thanks https://github.com/johnnyreilly;>@​johnnyreilly
   
   v9.2.3
   
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/issues/1334;>Fix
 error message for invalid getCustomTransformers modules - thanks https://github.com/blaky;>@​blaky
   
   v9.2.2
   
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/issues/1325;>Start
 consuming webpack loader types - thanks https://github.com/johnnyreilly;>@​johnnyreilly
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/issues/1324;>Add
 webpack minimum version in peerDependencies - thanks https://github.com/afdev82;>@​afdev82
   
   v9.2.1
   
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/issues/1320;>Make
 v9 latest in npm again - thanks https://github.com/johnnyreilly;>@​johnnyreilly
   
   v9.2.0
   
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/issues/1316;>Fixed
 impossibility to have several instances of ts-loader with different compiler 
options - thanks https://github.com/timocov;>@​timocov
   
   v9.1.2
   
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/pull/1293;>Fix
 removed files handling in watch mode - thanks https://github.com/gasnier;>@​gasnier
   
   v9.1.1
   
   
   ... (truncated)
   
   
   Changelog
   Sourced from https://github.com/TypeStrong/ts-loader/blob/main/CHANGELOG.md;>ts-loader's
 changelog.
   
   9.4.2
   
   https://github-redirect.dependabot.com/TypeStrong/ts-loader/pull/1550;>Bug
 fix: Use custom transformer when building solution references https://github-redirect.dependabot.com/TypeStrong/ts-loader/issues/1025;>#1025
 - thanks 

[GitHub] [openwhisk-vscode-extension] dependabot[bot] opened a new pull request, #23: Bump flat and mocha

[GitBox]

dependabot[bot] opened a new pull request, #23:
URL: https://github.com/apache/openwhisk-vscode-extension/pull/23

   Bumps [flat](https://github.com/hughsk/flat) to 5.0.2 and updates ancestor 
dependency [mocha](https://github.com/mochajs/mocha). These dependencies need 
to be updated together.
   
   Updates `flat` from 4.1.0 to 5.0.2
   
   Commits
   
   https://github.com/hughsk/flat/commit/e5ffd664df8a1fcc05adc22dd0ac6a2b32a0955d;>e5ffd66
 Release 5.0.2
   https://github.com/hughsk/flat/commit/fdb79d537748c827e8b886f897d8e1eb3c1acd17;>fdb79d5
 Update dependencies, refresh lockfile, format with standard.
   https://github.com/hughsk/flat/commit/e52185dded05768a1036327c5e79a399778d9191;>e52185d
 Test against node 14 in CI.
   https://github.com/hughsk/flat/commit/0189cb11dbc942447af78930bcb0ebc132b88384;>0189cb1
 Avoid arrow function syntax.
   https://github.com/hughsk/flat/commit/f25d3a11306bc460e43a14affa64d44a1d3cf8ed;>f25d3a1
 Release 5.0.1
   https://github.com/hughsk/flat/commit/54cc7ad380ebfbdf22e6654934dde5a34fdf3104;>54cc7ad
 use standard formatting
   https://github.com/hughsk/flat/commit/779816e81b6546da12280ee529d78dc57a7a5e1c;>779816e
 drop dependencies
   https://github.com/hughsk/flat/commit/2eea6d3a556feb1bdb02dc2f376c935da59a66e4;>2eea6d3
 Bump lodash from 4.17.15 to 4.17.19
   https://github.com/hughsk/flat/commit/a61a554952cfb550f276acc02ceea403afe01700;>a61a554
 Bump acorn from 7.1.0 to 7.4.0
   https://github.com/hughsk/flat/commit/20ef0ef55dfa028caddaedbcb33efbdb04d18e13;>20ef0ef
 Fix prototype pollution on unflatten
   Additional commits viewable in https://github.com/hughsk/flat/compare/4.1.0...5.0.2;>compare 
view
   
   
   
   Maintainer changes
   This version was pushed to npm by https://www.npmjs.com/~timoxley;>timoxley, a new releaser for flat 
since your current version.
   
   
   
   Updates `mocha` from 7.2.0 to 10.2.0
   
   Release notes
   Sourced from https://github.com/mochajs/mocha/releases;>mocha's releases.
   
   v10.2.0
   10.2.0 / 2022-12-11
   :tada: Enhancements
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4945;>#4945:
 API: add possibility to decorate ESM name before import (https://github.com/j0tunn;>@​j0tunn)
   
   :bug: Fixes
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4946;>#4946:
 Browser: color of failed test icon (https://github.com/kleisauke;>@​kleisauke)
   
   :book: Documentation
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4944;>#4944:
 Remove duplicated header (https://github.com/PauloGoncalvesBH;>@​PauloGoncalvesBH)
   
   v10.1.0
   10.1.0 / 2022-10-16
   :tada: Enhancements
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4896;>#4896:
 Browser: add support for prefers-color-scheme: dark (https://github.com/greggman;>@​greggman)
   
   :nut_and_bolt: Other
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4912;>#4912:
 Browser: increase contrast for replay buttons (https://github.com/JoshuaKGoldberg;>@​JoshuaKGoldberg)
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4905;>#4905:
 Use standard Promise.allSettled instead of polyfill (https://github.com/outsideris;>@​outsideris)
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4899;>#4899:
 Upgrade official GitHub actions to latest (https://github.com/ddzz;>@​ddzz)
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4770;>#4770:
 Fix regex in function clean(https://github.com/yetingli;>@​yetingli)
   
   v10.0.0
   10.0.0 / 2022-05-01
   :boom: Breaking Changes
   
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4845;>#4845:
 Drop Node.js v12.x support (https://github.com/juergba;>@​juergba)
   
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4848;>#4848:
 Drop Internet-Explorer-11 support (https://github.com/juergba;>@​juergba)
   
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4857;>#4857:
 Drop AMD/RequireJS support (https://github.com/juergba;>@​juergba)
   
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4866;>#4866:
 Drop Growl notification support (https://github.com/juergba;>@​juergba)
   
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4863;>#4863:
 Rename executable bin/mocha to bin/mocha.js (https://github.com/juergba;>@​juergba)
   
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4865;>#4865:
 --ignore option in Windows: upgrade Minimatch (https://github.com/juergba;>@​juergba)
   
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4861;>#4861:
 Remove deprecated Runner signature (https://github.com/juergba;>@​juergba)
   
   
   :nut_and_bolt: Other
   
   
   ... (truncated)
   
   
   Changelog
   Sourced from https://github.com/mochajs/mocha/blob/master/CHANGELOG.md;>mocha's 
changelog.
   
   10.2.0 / 2022-12-11
   :tada: Enhancements
   
   

[GitHub] [openwhisk-wskdebug] dependabot[bot] opened a new pull request, #117: Bump flat and mocha

[GitBox]

dependabot[bot] opened a new pull request, #117:
URL: https://github.com/apache/openwhisk-wskdebug/pull/117

   Bumps [flat](https://github.com/hughsk/flat) to 5.0.2 and updates ancestor 
dependency [mocha](https://github.com/mochajs/mocha). These dependencies need 
to be updated together.
   
   Updates `flat` from 4.1.0 to 5.0.2
   
   Commits
   
   https://github.com/hughsk/flat/commit/e5ffd664df8a1fcc05adc22dd0ac6a2b32a0955d;>e5ffd66
 Release 5.0.2
   https://github.com/hughsk/flat/commit/fdb79d537748c827e8b886f897d8e1eb3c1acd17;>fdb79d5
 Update dependencies, refresh lockfile, format with standard.
   https://github.com/hughsk/flat/commit/e52185dded05768a1036327c5e79a399778d9191;>e52185d
 Test against node 14 in CI.
   https://github.com/hughsk/flat/commit/0189cb11dbc942447af78930bcb0ebc132b88384;>0189cb1
 Avoid arrow function syntax.
   https://github.com/hughsk/flat/commit/f25d3a11306bc460e43a14affa64d44a1d3cf8ed;>f25d3a1
 Release 5.0.1
   https://github.com/hughsk/flat/commit/54cc7ad380ebfbdf22e6654934dde5a34fdf3104;>54cc7ad
 use standard formatting
   https://github.com/hughsk/flat/commit/779816e81b6546da12280ee529d78dc57a7a5e1c;>779816e
 drop dependencies
   https://github.com/hughsk/flat/commit/2eea6d3a556feb1bdb02dc2f376c935da59a66e4;>2eea6d3
 Bump lodash from 4.17.15 to 4.17.19
   https://github.com/hughsk/flat/commit/a61a554952cfb550f276acc02ceea403afe01700;>a61a554
 Bump acorn from 7.1.0 to 7.4.0
   https://github.com/hughsk/flat/commit/20ef0ef55dfa028caddaedbcb33efbdb04d18e13;>20ef0ef
 Fix prototype pollution on unflatten
   Additional commits viewable in https://github.com/hughsk/flat/compare/4.1.0...5.0.2;>compare 
view
   
   
   
   Maintainer changes
   This version was pushed to npm by https://www.npmjs.com/~timoxley;>timoxley, a new releaser for flat 
since your current version.
   
   
   
   Updates `mocha` from 7.1.1 to 10.2.0
   
   Release notes
   Sourced from https://github.com/mochajs/mocha/releases;>mocha's releases.
   
   v10.2.0
   10.2.0 / 2022-12-11
   :tada: Enhancements
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4945;>#4945:
 API: add possibility to decorate ESM name before import (https://github.com/j0tunn;>@​j0tunn)
   
   :bug: Fixes
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4946;>#4946:
 Browser: color of failed test icon (https://github.com/kleisauke;>@​kleisauke)
   
   :book: Documentation
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4944;>#4944:
 Remove duplicated header (https://github.com/PauloGoncalvesBH;>@​PauloGoncalvesBH)
   
   v10.1.0
   10.1.0 / 2022-10-16
   :tada: Enhancements
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4896;>#4896:
 Browser: add support for prefers-color-scheme: dark (https://github.com/greggman;>@​greggman)
   
   :nut_and_bolt: Other
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4912;>#4912:
 Browser: increase contrast for replay buttons (https://github.com/JoshuaKGoldberg;>@​JoshuaKGoldberg)
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4905;>#4905:
 Use standard Promise.allSettled instead of polyfill (https://github.com/outsideris;>@​outsideris)
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4899;>#4899:
 Upgrade official GitHub actions to latest (https://github.com/ddzz;>@​ddzz)
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4770;>#4770:
 Fix regex in function clean(https://github.com/yetingli;>@​yetingli)
   
   v10.0.0
   10.0.0 / 2022-05-01
   :boom: Breaking Changes
   
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4845;>#4845:
 Drop Node.js v12.x support (https://github.com/juergba;>@​juergba)
   
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4848;>#4848:
 Drop Internet-Explorer-11 support (https://github.com/juergba;>@​juergba)
   
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4857;>#4857:
 Drop AMD/RequireJS support (https://github.com/juergba;>@​juergba)
   
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4866;>#4866:
 Drop Growl notification support (https://github.com/juergba;>@​juergba)
   
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4863;>#4863:
 Rename executable bin/mocha to bin/mocha.js (https://github.com/juergba;>@​juergba)
   
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4865;>#4865:
 --ignore option in Windows: upgrade Minimatch (https://github.com/juergba;>@​juergba)
   
   
   https://github-redirect.dependabot.com/mochajs/mocha/issues/4861;>#4861:
 Remove deprecated Runner signature (https://github.com/juergba;>@​juergba)
   
   
   :nut_and_bolt: Other
   
   
   ... (truncated)
   
   
   Changelog
   Sourced from https://github.com/mochajs/mocha/blob/master/CHANGELOG.md;>mocha's 
changelog.
   
   10.2.0 / 2022-12-11
   :tada: Enhancements
   
   

[GitHub] [openwhisk-runtime-nodejs] Rurouni-Jiaxin commented on issue #229: Gradle Task :core:nodejs1XAction:distDocker FAILED

[GitBox]

Rurouni-Jiaxin commented on issue #229:
URL: 
https://github.com/apache/openwhisk-runtime-nodejs/issues/229#issuecomment-1367968324

   @upgl. Would you please be so kind and give me a hint? Because I see you 
previously bumped the gradle version


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-nodejs] Rurouni-Jiaxin opened a new issue, #229: Gradle Task :core:nodejs1XAction:distDocker FAILED

[GitBox]

Rurouni-Jiaxin opened a new issue, #229:
URL: https://github.com/apache/openwhisk-runtime-nodejs/issues/229

   
   
   ## Environment details:
   
   * local deployment, vagrant, native ubuntu, Mac OS, Bluemix, ...   
   * version of docker, vagrant, ubuntu, ...
   
   ## Steps to reproduce the issue:
   
   1.   
   2.   
   3.   
   
   
   ## Provide the expected results and outputs:
   
   ```
   output comes here
   ```
   
   
   ## Provide the actual results and outputs:
   
   ```
   output comes here
   ```
   
   ## Additional information you deem important:
   * issue happens only occasionally or under certain circumstances   
   * changes you did or observed in the environment
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] dinok97 commented on issue #5368: Action stucks in error forever after 'too many requests' response occurred

[GitBox]

dinok97 commented on issue #5368:
URL: https://github.com/apache/openwhisk/issues/5368#issuecomment-1367852029

   Thanks for your response! @style95  
   Also sorry for my late reply. Just try to find out the problem, but it takes 
time for my since I'm beginner in Scala.
   
   I have seen the logs and didn't found any error log related to this issue.
   According my experiment, the throttling is enable for action. I'm able to 
invoke other actions under same namespace with `perfTest`.
   
   The error that I described above only happen when action concurrency = 100 
(I increased the action concurrency from 1 to 100).
   If the action concurrency = 1, the error not occur.
   
   Is new Scheduler well supported for action concurrency? 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-utilities] dgrove-oss opened a new pull request, #86: GitHub action to run scanCode.py

[GitBox]

dgrove-oss opened a new pull request, #86:
URL: https://github.com/apache/openwhisk-utilities/pull/86

   Define a Dockerfile-based GitHub action to run scanCode.py.  This action can 
be used as a step in workflows to perform code scanning on the checked out 
source code.  For example:
   ```
   jobs:
 scancode:
   runs-on: ubuntu-22.04
   steps:
 - name: Checkout
   uses: actions/checkout@v3
 - name: Scan Code
   uses: dgrove-oss/openwhisk-utilities/scancode@action
   ```
   
   Once this PR is merged, we can refer simply to 
`apache/openwhisk-utilities/scancode` as the action name in workflow files.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] style95 commented on issue #5368: Action stucks in error forever after 'too many requests' response occurred

[GitBox]

style95 commented on issue #5368:
URL: https://github.com/apache/openwhisk/issues/5368#issuecomment-1366557617

   Hello @dinok97 
   
   Have you looked into logs?
   Generally, the throttling flag resides in ETCD in the new scheduler.
   You can look into the etcd data to see if the throttling is enabled and 
which throttling(between namespace and action) is enabled.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] dinok97 opened a new issue, #5368: Action stucks in error forever after 'too many requests' response occurred

[GitBox]

dinok97 opened a new issue, #5368:
URL: https://github.com/apache/openwhisk/issues/5368

   
   
   I did performance test for **Openwhisk with New Scheduler** (pull-based 
activation). When it comes to test with 5000 RPS, hundred of error happened. 
Then, I checked it by invoked the tested action from WSK CLI and get error 
response 
   
   `error: Unable to invoke action 'perfTest': Too many requests (code 
74bd570f6b048c437664907592f07bbe)`.
   
   I waited for 4 hours with hope that the action will be normal again. But it 
seems like the action cannot be normal anymore (always return 'too many 
requests' error).
   
   After long of waiting, I try to re-created the function (delete existing 
function and create the new one with the same name and kind) and still got `too 
many requests` error.
   
   Is that the true behavior? or that is a bug? 
   
   ## Environment details:
   
   * distributed deployment, all VMs  (AWS EC2 Instance) are Linux
   * 2 VMs Controller, 4 VMs Invoker, 2 VMs Scheduler, 1 VMs ETCD
   
   ## Steps to reproduce:
   1. Create simple nodejs action, for example `perfTest` action  
   2. Test  `perfTest` action with 5000 RPS for 1 minutes. We use `wrk 2` as a 
tool. Below is the commad 
   `wrk -t10 -c10 -d60s -R5000 --script post.lua --latency 
https://owserver/api/v1/namespaces/_/actions/perfTest?blocking=false`
   3. Wait until test finished with some of errors
   4. Try to invoke `perfTest` action
 
   
   ## Expected results:
   
   ```
   invocation success:
   
   {
   "params": {}
   }
   ```
   
   
   ## Actual results:
   
   ```
   invocation error:
   
   error: Unable to invoke action 'perfTest': Too many requests (code 
74bd570f6b048c437664907592f07bbe)
   Run 'wsk --help' for usage.
   ```
   
   ## Additional information:
   * The `perfTest`action back to normal after Scheduler was restarted. There 
us no other way besides that.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-devtools] christiansicari commented on issue #323: exec user process caused "exec format error" on raspberry 4B 8GB

[GitBox]

christiansicari commented on issue #323:
URL: 
https://github.com/apache/openwhisk-devtools/issues/323#issuecomment-1364746007

   > I followed this: 
[medium.com/openwhisk/apache-openwhisk-meets-raspberry-pi](https://medium.com/openwhisk/apache-openwhisk-meets-raspberry-pi-e346e555b56a)
 it works
   
   unfortunately does not work on Armv8 Raspberry.
   I changed the Minio Tag using this: RELEASE.2020-11-25T22-36-25Z-arm64
   But I got this
   ```
   host ip address: 155.98.20.227
   checking required ports ... 
... OK
 ... preparing api-gateway configuration
   pinging minio...
   .Traceback (most recent call last):
 File "/usr/local/bin/docker-compose", line 5, in 
   from compose.cli.main import main
 File "/usr/local/lib/python3.9/dist-packages/compose/cli/main.py", line 
29, in 
   from ..metrics.decorator import metrics
 File 
"/usr/local/lib/python3.9/dist-packages/compose/metrics/decorator.py", line 3, 
in 
   from compose.metrics.client import MetricsCommand
 File "/usr/local/lib/python3.9/dist-packages/compose/metrics/client.py", 
line 5, in 
   from docker import ContextAPI
   ImportError: cannot import name 'ContextAPI' from 'docker' (/usr/l
   ```
   Honestly I think there is no interest on ARM deployment without Kubernetes


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-swift] Andrea-Scuderi opened a new pull request, #153: Add Support for Swift 5.7

[GitBox]

Andrea-Scuderi opened a new pull request, #153:
URL: https://github.com/apache/openwhisk-runtime-swift/pull/153

   # Add support for Swift 5.7
   
   - Add new action signatures with async/await and throws
   - Backward compatibility with old action signatures
   
   ### swift57Action
   - Add support async to Whisk class
   
   ### swiftbuild.py
   - simplify the code added by the build process to:
   ```swift
   try await _WhiskRuntime.wiskRunLoop { jsonData in
  await _WhiskRuntime.runAsyncMain(mainFunction: , json: jsonData)
   }
   ```
   
   ### swiftbuild.py.launcher.swift
   - Add `_WhiskRuntime` struct 
   - Refactor Error messages
   - Add new signatures
   
   ```swift
   // Examples of Actions supported by Swift 5.7
   
   // Action with Any Input and Any Output
   func main(args: Any) -> Any {
   let newArgs = args as! [String:Any]
   if let name = newArgs["name"] as? String {
   return [ "greeting" : "Hello \(name)!" ]
   } else {
   return [ "greeting" : "Hello stranger!" ]
   }
   }

   // Async Action with Any Input and Any Output
   func mainAsync(args: Any) async -> Any {
   do {
   //async code sleep for 1 sec
   try await Task.sleep(nanoseconds: 1_000_000_000)
   
   let newArgs = args as! [String:Any]
   if let name = newArgs["name"] as? String {
   return [ "greeting" : "Hello \(name)!" ]
   } else {
   return [ "greeting" : "Hello stranger!" ]
   }
   } catch {
   return ["error" : error.localizedDescription]
   }
   }
   
   // Async throwing Action with Any Input and Any Output
   func mainAsyncThrows(args: Any) async throws -> Any {
   //async code sleep for 1 sec
   try await Task.sleep(nanoseconds: 1_000_000_000)
   
   let newArgs = args as! [String:Any]
   if let name = newArgs["name"] as? String {
   return [ "greeting" : "Hello \(name)!" ]
   } else {
   return [ "greeting" : "Hello stranger!" ]
   }
   }
   
   struct Input: Codable {
   let name: String?
   }
   
   struct Output: Codable {
   let count: Int
   }
   
   // Action with Codable Input and completion with Codable Output and Error
   func mainCompletionCodable(input: Input, completion: @escaping (Output?, 
Error?) -> Void) -> Void {
   if let name = input.name {
   let output = Output(count: name.count)
   completion(output, nil)
   } else {
   let output = Output(count: 0)
   completion(output, nil)
   }
   }
   
   // Action with Codable Input and completion with Codable Output and Error
   func mainCompletionCodableNoInput(completion: @escaping (Output?, Error?) -> 
Void) -> Void {
   let output = Output(count: 0)
   completion(output, nil)
   }
   
   // Async throwing Action with Codable Output
   func mainCodableAsyncThrowsNoInput() async throws -> Output? {
   try await Task.sleep(nanoseconds: 1_000_000_000)
   return Output(count: 0)
   }
   
   // Async throwing Action with a Codable Input and a Codable Output
   func mainCodableAsyncThrows(input: Input) async throws -> Output? {
   try await Task.sleep(nanoseconds: 1_000_000_000)
   if let name = input.name {
   return Output(count: name.count)
   } else {
   return Output(count: 0)
   }
   }
   ```
   
   
   Merry Christmas and Happy New Year! 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] sciabarracom merged pull request #5367: fixes to use different secrets, show the results in the log

[GitBox]

sciabarracom merged PR #5367:
URL: https://github.com/apache/openwhisk/pull/5367


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] msciabarra commented on pull request #5367: fixes to use different secrets, show the results in the log

[GitBox]

msciabarra commented on PR #5367:
URL: https://github.com/apache/openwhisk/pull/5367#issuecomment-1362431880

   ![Screenshot 2022-12-22 at 05 17 
07](https://user-images.githubusercontent.com/1595312/209062384-3a6b434b-1b63-4119-93c3-f7d659d2e8cd.png)
   
   This is odd... when I remove the AWS variables this test starts to fail...
   If I enable them it will be the S3StorageTest to fail
   I need to figure out why.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] sciabarracom commented on pull request #5367: fixes to use different secrets, show the results in the log

[GitBox]

sciabarracom commented on PR #5367:
URL: https://github.com/apache/openwhisk/pull/5367#issuecomment-1362418878

   > BTW, if we change the variable, how can we test the S3 uploading tests?
   
   we cannot test it anyway and they are not tested with travis - secrets are 
not passed to pull requests (otherwise it would be trivial to grab them, just 
make a pull request that dumps and writes them somewhere)
   
   you can test them only locally and if you have all the expected environment 
variables, what I did wrong was to use the environment variables that activated 
the tests
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] style95 commented on pull request #5367: fixes to use different secrets, show the results in the log

[GitBox]

style95 commented on PR #5367:
URL: https://github.com/apache/openwhisk/pull/5367#issuecomment-1362286784

   BTW, if we change the variable, how can we test the S3 uploading tests?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-composer-python] dgrove-oss merged pull request #16: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #16:
URL: https://github.com/apache/openwhisk-composer-python/pull/16


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-cli] dgrove-oss merged pull request #519: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #519:
URL: https://github.com/apache/openwhisk-cli/pull/519


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-client-go] dgrove-oss merged pull request #157: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #157:
URL: https://github.com/apache/openwhisk-client-go/pull/157


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-wskdeploy] dgrove-oss merged pull request #1155: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #1155:
URL: https://github.com/apache/openwhisk-wskdeploy/pull/1155


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-apigateway] dgrove-oss merged pull request #398: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #398:
URL: https://github.com/apache/openwhisk-apigateway/pull/398


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-catalog] dgrove-oss merged pull request #326: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #326:
URL: https://github.com/apache/openwhisk-catalog/pull/326


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-catalog] dgrove-oss closed pull request #325: fix travis URL

[GitBox]

dgrove-oss closed pull request #325: fix travis URL
URL: https://github.com/apache/openwhisk-catalog/pull/325


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-devtools] dgrove-oss merged pull request #336: synch runtimes with core repo

[GitBox]

dgrove-oss merged PR #336:
URL: https://github.com/apache/openwhisk-devtools/pull/336


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-composer] dgrove-oss merged pull request #87: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #87:
URL: https://github.com/apache/openwhisk-composer/pull/87


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-devtools] dgrove-oss closed pull request #337: update travis url

[GitBox]

dgrove-oss closed pull request #337: update travis url
URL: https://github.com/apache/openwhisk-devtools/pull/337


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-devtools] dgrove-oss merged pull request #347: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #347:
URL: https://github.com/apache/openwhisk-devtools/pull/347


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-deno] dgrove-oss merged pull request #16: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #16:
URL: https://github.com/apache/openwhisk-runtime-deno/pull/16


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-deploy-kube] dgrove-oss merged pull request #758: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #758:
URL: https://github.com/apache/openwhisk-deploy-kube/pull/758


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-go] dgrove-oss merged pull request #179: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #179:
URL: https://github.com/apache/openwhisk-runtime-go/pull/179


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-package-kafka] dgrove-oss merged pull request #386: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #386:
URL: https://github.com/apache/openwhisk-package-kafka/pull/386


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-php] dgrove-oss merged pull request #126: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #126:
URL: https://github.com/apache/openwhisk-runtime-php/pull/126


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-python] dgrove-oss merged pull request #136: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #136:
URL: https://github.com/apache/openwhisk-runtime-python/pull/136


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-java] dgrove-oss merged pull request #144: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #144:
URL: https://github.com/apache/openwhisk-runtime-java/pull/144


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-nodejs] dgrove-oss merged pull request #228: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #228:
URL: https://github.com/apache/openwhisk-runtime-nodejs/pull/228


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-client-js] dgrove-oss merged pull request #250: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #250:
URL: https://github.com/apache/openwhisk-client-js/pull/250


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-utilities] dgrove-oss merged pull request #85: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #85:
URL: https://github.com/apache/openwhisk-utilities/pull/85


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-ruby] dgrove-oss merged pull request #75: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #75:
URL: https://github.com/apache/openwhisk-runtime-ruby/pull/75


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-release] dgrove-oss merged pull request #409: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #409:
URL: https://github.com/apache/openwhisk-release/pull/409


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-package-cloudant] dgrove-oss merged pull request #234: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #234:
URL: https://github.com/apache/openwhisk-package-cloudant/pull/234


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-docker] dgrove-oss merged pull request #97: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #97:
URL: https://github.com/apache/openwhisk-runtime-docker/pull/97


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-package-alarms] dgrove-oss merged pull request #237: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #237:
URL: https://github.com/apache/openwhisk-package-alarms/pull/237


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-website] dgrove-oss merged pull request #503: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #503:
URL: https://github.com/apache/openwhisk-website/pull/503


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-swift] dgrove-oss merged pull request #152: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #152:
URL: https://github.com/apache/openwhisk-runtime-swift/pull/152


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-dotnet] dgrove-oss merged pull request #73: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #73:
URL: https://github.com/apache/openwhisk-runtime-dotnet/pull/73


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-runtime-rust] dgrove-oss merged pull request #41: remove travis from required checks for merging PRs

[GitBox]

dgrove-oss merged PR #41:
URL: https://github.com/apache/openwhisk-runtime-rust/pull/41


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-wskdebug] rabbah merged pull request #116: remove travis from required checks for merging PRs

[GitBox]

rabbah merged PR #116:
URL: https://github.com/apache/openwhisk-wskdebug/pull/116


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-client-js] codecov-commenter commented on pull request #250: remove travis from required checks for merging PRs

[GitBox]

codecov-commenter commented on PR #250:
URL: 
https://github.com/apache/openwhisk-client-js/pull/250#issuecomment-1361998621

   # 
[Codecov](https://codecov.io/gh/apache/openwhisk-client-js/pull/250?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#250](https://codecov.io/gh/apache/openwhisk-client-js/pull/250?src=pr=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (049b9c1) into 
[master](https://codecov.io/gh/apache/openwhisk-client-js/commit/95f66ca1148ed489060627187e1ddbfb6052cfb5?el=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (95f66ca) will **not change** coverage.
   > The diff coverage is `n/a`.
   
   ```diff
   @@   Coverage Diff   @@
   ##   master #250   +/-   ##
   ===
 Coverage   98.87%   98.87%   
   ===
 Files  14   14   
 Lines 443  443   
   ===
 Hits  438  438   
 Misses  55   
   ```
   
   
   
   :mega: We’re building smart automated test selection to slash your CI/CD 
build times. [Learn 
more](https://about.codecov.io/iterative-testing/?utm_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] sciabarracom commented on pull request #5367: fixes to use different secrets, show the results in the log

[GitBox]

sciabarracom commented on PR #5367:
URL: https://github.com/apache/openwhisk/pull/5367#issuecomment-1360909057

   My github actions pass the AWS_ACCESS_ID secrets to the build but since they 
are pull requests they are empty. This enables some tests and make some of them 
fail. So I changed the name of the variables used to upload logs. Furthermore I 
made optional the upload of logs and slack notification and printed the results 
of the tes in the job run for inspection. Now all test pass.  


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] codecov-commenter commented on pull request #5367: fixes to use different secrets, show the results in the log

[GitBox]

codecov-commenter commented on PR #5367:
URL: https://github.com/apache/openwhisk/pull/5367#issuecomment-1360553251

   # 
[Codecov](https://codecov.io/gh/apache/openwhisk/pull/5367?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#5367](https://codecov.io/gh/apache/openwhisk/pull/5367?src=pr=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (8f9a51c) into 
[master](https://codecov.io/gh/apache/openwhisk/commit/f717619c8c25b54294cfa195732732e85c5d9431?el=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (f717619) will **decrease** coverage by `52.85%`.
   > The diff coverage is `n/a`.
   
   ```diff
   @@Coverage Diff @@
   ##   master   #5367   +/-   ##
   ==
   - Coverage   57.37%   4.52%   -52.86% 
   ==
 Files 233 240+7 
 Lines   14143   14407  +264 
 Branches  597 601+4 
   ==
   - Hits 8115 652 -7463 
   - Misses   6028   13755 +7727 
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/openwhisk/pull/5367?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 | Coverage Δ | |
   |---|---|---|
   | 
[.../main/scala/org/apache/openwhisk/core/WarmUp.scala](https://codecov.io/gh/apache/openwhisk/pull/5367/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvV2FybVVwLnNjYWxh)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...ain/scala/org/apache/openwhisk/spi/SpiLoader.scala](https://codecov.io/gh/apache/openwhisk/pull/5367/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL3NwaS9TcGlMb2FkZXIuc2NhbGE=)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...scala/org/apache/openwhisk/common/time/Clock.scala](https://codecov.io/gh/apache/openwhisk/pull/5367/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvbW1vbi90aW1lL0Nsb2NrLnNjYWxh)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...scala/org/apache/openwhisk/core/FeatureFlags.scala](https://codecov.io/gh/apache/openwhisk/pull/5367/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvRmVhdHVyZUZsYWdzLnNjYWxh)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...scala/org/apache/openwhisk/http/CorsSettings.scala](https://codecov.io/gh/apache/openwhisk/pull/5367/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2h0dHAvQ29yc1NldHRpbmdzLnNjYWxh)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...la/org/apache/openwhisk/http/BasicRasService.scala](https://codecov.io/gh/apache/openwhisk/pull/5367/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2h0dHAvQmFzaWNSYXNTZXJ2aWNlLnNjYWxh)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...pache/openwhisk/http/LenientSprayJsonSupport.scala](https://codecov.io/gh/apache/openwhisk/pull/5367/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2h0dHAvTGVuaWVudFNwcmF5SnNvblN1cHBvcnQuc2NhbGE=)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...ache/openwhisk/core/database/DocumentFactory.scala](https://codecov.io/gh/apache/openwhisk/pull/5367/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvZGF0YWJhc2UvRG9jdW1lbnRGYWN0b3J5LnNjYWxh)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[.../apache/openwhisk/core/controller/Namespaces.scala](https://codecov.io/gh/apache/openwhisk/pull/5367/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29yZS9jb250cm9sbGVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvY29udHJvbGxlci9OYW1lc3BhY2VzLnNjYWxh)
 | `0.00% 

[GitHub] [openwhisk] msciabarra opened a new pull request, #5367: fixes to use different secrets, show the results in the log

[GitBox]

msciabarra opened a new pull request, #5367:
URL: https://github.com/apache/openwhisk/pull/5367

   
   Changes to run without errors
   
   ## Description
   
   updated the build to do not block if missing variables
   
   
   Build fails because of overlap of env variables
   
   
   ## Related issue and scope
   
   - [ ] I opened an issue to propose and discuss this change (#)
   
   ## My changes affect the following components
   
   
   - [ ] API
   - [ ] Controller
   - [ ] Message Bus (e.g., Kafka)
   - [ ] Loadbalancer
   - [ ] Scheduler
   - [ ] Invoker
   - [ ] Intrinsic actions (e.g., sequences, conductors)
   - [ ] Data stores (e.g., CouchDB)
   - [ ] Tests
   - [ ] Deployment
   - [ ] CLI
   - [ ] General tooling
   - [ ] Documentation
   
   ## Types of changes
   
   - [ ] Bug fix (generally a non-breaking change which closes an issue).
   - [ ] Enhancement or new feature (adds new functionality).
   - [ ] Breaking change (a bug fix or enhancement which changes existing 
behavior).
   
   ## Checklist:
   
   
   - [x] I signed an [Apache 
CLA](https://github.com/apache/openwhisk/blob/master/CONTRIBUTING.md).
   - [ ] I reviewed the [style 
guides](https://github.com/apache/openwhisk/blob/master/CONTRIBUTING.md#coding-standards)
 and followed the recommendations (Travis CI will check :).
   - [ ] I added tests to cover my changes.
   - [ ] My changes require further changes to the documentation.
   - [ ] I updated the documentation where necessary.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] msciabarra closed pull request #5366: making optional logs upload and slack notification, and showing test results in the job run

[GitBox]

msciabarra closed pull request #5366: making optional logs upload and slack 
notification, and showing test results in the job run
URL: https://github.com/apache/openwhisk/pull/5366


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] msciabarra commented on pull request #5366: making optional logs upload and slack notification, and showing test results in the job run

[GitBox]

msciabarra commented on PR #5366:
URL: https://github.com/apache/openwhisk/pull/5366#issuecomment-1360396541

   forget about this - I misunderstood a few things - restarting the patch


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] msciabarra commented on pull request #5366: making optional logs upload and slack notification, and showing test results in the job run

[GitBox]

msciabarra commented on PR #5366:
URL: https://github.com/apache/openwhisk/pull/5366#issuecomment-1360374551

   Hi I made a few changes, mostly I made the s3 test skipped it the bucket is 
not available, fixed a compilation error and now the build works. Can you merge?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk-cli] kaxi1993 commented on issue #483: Could not open settings remapped class cache

[GitBox]

kaxi1993 commented on issue #483:
URL: https://github.com/apache/openwhisk-cli/issues/483#issuecomment-1358398578

   > > Running `./gradlw` itself would give the same error. I was able to 
resolve it by opening Android Studio and syncing grade (building a react native 
project)
   > 
   > This solution only worked for me. Mine was a react-native project
   
   I resolved it in the same way, opening Android Studio and syncing.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] msciabarra commented on pull request #5366: making optional logs upload and slack notification, and showing test results in the job run

[GitBox]

msciabarra commented on PR #5366:
URL: https://github.com/apache/openwhisk/pull/5366#issuecomment-1357544635

   Yes true. AWS S3 tests are NOW failing, and I guess this is happening simply 
because someone removed the AWS_BUCKET secret that was there before. I used the 
bucket to upload logs there and I am sure he was not happy.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] style95 commented on pull request #5366: making optional logs upload and slack notification, and showing test results in the job run

[GitBox]

style95 commented on PR #5366:
URL: https://github.com/apache/openwhisk/pull/5366#issuecomment-1357522194

   Seems S3-related tests are failing.
   https://user-images.githubusercontent.com/3447251/208418333-debf3a98-ded2-4afb-93d4-eeabeb255a2e.png;>
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] codecov-commenter commented on pull request #5366: making optional logs upload and slack notification

[GitBox]

codecov-commenter commented on PR #5366:
URL: https://github.com/apache/openwhisk/pull/5366#issuecomment-1355990340

   # 
[Codecov](https://codecov.io/gh/apache/openwhisk/pull/5366?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#5366](https://codecov.io/gh/apache/openwhisk/pull/5366?src=pr=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (6ead0ee) into 
[master](https://codecov.io/gh/apache/openwhisk/commit/f717619c8c25b54294cfa195732732e85c5d9431?el=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (f717619) will **decrease** coverage by `52.85%`.
   > The diff coverage is `n/a`.
   
   ```diff
   @@Coverage Diff @@
   ##   master   #5366   +/-   ##
   ==
   - Coverage   57.37%   4.52%   -52.86% 
   ==
 Files 233 240+7 
 Lines   14143   14407  +264 
 Branches  597 601+4 
   ==
   - Hits 8115 652 -7463 
   - Misses   6028   13755 +7727 
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/openwhisk/pull/5366?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 | Coverage Δ | |
   |---|---|---|
   | 
[.../main/scala/org/apache/openwhisk/core/WarmUp.scala](https://codecov.io/gh/apache/openwhisk/pull/5366/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvV2FybVVwLnNjYWxh)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...ain/scala/org/apache/openwhisk/spi/SpiLoader.scala](https://codecov.io/gh/apache/openwhisk/pull/5366/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL3NwaS9TcGlMb2FkZXIuc2NhbGE=)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...scala/org/apache/openwhisk/common/time/Clock.scala](https://codecov.io/gh/apache/openwhisk/pull/5366/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvbW1vbi90aW1lL0Nsb2NrLnNjYWxh)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...scala/org/apache/openwhisk/core/FeatureFlags.scala](https://codecov.io/gh/apache/openwhisk/pull/5366/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvRmVhdHVyZUZsYWdzLnNjYWxh)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...scala/org/apache/openwhisk/http/CorsSettings.scala](https://codecov.io/gh/apache/openwhisk/pull/5366/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2h0dHAvQ29yc1NldHRpbmdzLnNjYWxh)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...la/org/apache/openwhisk/http/BasicRasService.scala](https://codecov.io/gh/apache/openwhisk/pull/5366/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2h0dHAvQmFzaWNSYXNTZXJ2aWNlLnNjYWxh)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...pache/openwhisk/http/LenientSprayJsonSupport.scala](https://codecov.io/gh/apache/openwhisk/pull/5366/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2h0dHAvTGVuaWVudFNwcmF5SnNvblN1cHBvcnQuc2NhbGE=)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...ache/openwhisk/core/database/DocumentFactory.scala](https://codecov.io/gh/apache/openwhisk/pull/5366/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29tbW9uL3NjYWxhL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvZGF0YWJhc2UvRG9jdW1lbnRGYWN0b3J5LnNjYWxh)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[.../apache/openwhisk/core/controller/Namespaces.scala](https://codecov.io/gh/apache/openwhisk/pull/5366/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-Y29yZS9jb250cm9sbGVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvb3BlbndoaXNrL2NvcmUvY29udHJvbGxlci9OYW1lc3BhY2VzLnNjYWxh)
 | `0.00% 

[GitHub] [openwhisk] msciabarra opened a new pull request, #5366: making optional logs upload and slack notification

[GitBox]

msciabarra opened a new pull request, #5366:
URL: https://github.com/apache/openwhisk/pull/5366

   
   making optional logs upload and slack notification
   
   ## Description
   
   
   errors in build for missing env vars 
   
   ## Related issue and scope
   
   - [ ] I opened an issue to propose and discuss this change (#)
   
   ## My changes affect the following components
   
   
   - [ ] API
   - [ ] Controller
   - [ ] Message Bus (e.g., Kafka)
   - [ ] Loadbalancer
   - [ ] Scheduler
   - [ ] Invoker
   - [ ] Intrinsic actions (e.g., sequences, conductors)
   - [ ] Data stores (e.g., CouchDB)
   - [ ] Tests
   - [ ] Deployment
   - [ ] CLI
   - [ ] General tooling
   - [ ] Documentation
   
   ## Types of changes
   
   - [ ] Bug fix (generally a non-breaking change which closes an issue).
   - [ ] Enhancement or new feature (adds new functionality).
   - [ ] Breaking change (a bug fix or enhancement which changes existing 
behavior).
   
   ## Checklist:
   
   
   - [ ] I signed an [Apache 
CLA](https://github.com/apache/openwhisk/blob/master/CONTRIBUTING.md).
   - [ ] I reviewed the [style 
guides](https://github.com/apache/openwhisk/blob/master/CONTRIBUTING.md#coding-standards)
 and followed the recommendations (Travis CI will check :).
   - [ ] I added tests to cover my changes.
   - [ ] My changes require further changes to the documentation.
   - [ ] I updated the documentation where necessary.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] msciabarra closed pull request #5365: Making optional log upload and slack notifications if variables are missing

[GitBox]

msciabarra closed pull request #5365: Making optional log upload and slack 
notifications if variables are missing
URL: https://github.com/apache/openwhisk/pull/5365


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] msciabarra opened a new pull request, #5365: Making optional log upload and slack notifications if variables are missing

[GitBox]

msciabarra opened a new pull request, #5365:
URL: https://github.com/apache/openwhisk/pull/5365

   
   
   ## Description
   
   
   
   ## Related issue and scope
   
   - [ ] I opened an issue to propose and discuss this change (#)
   
   ## My changes affect the following components
   
   
   - [ ] API
   - [ ] Controller
   - [ ] Message Bus (e.g., Kafka)
   - [ ] Loadbalancer
   - [ ] Scheduler
   - [ ] Invoker
   - [ ] Intrinsic actions (e.g., sequences, conductors)
   - [ ] Data stores (e.g., CouchDB)
   - [ ] Tests
   - [ ] Deployment
   - [ ] CLI
   - [ ] General tooling
   - [ ] Documentation
   
   ## Types of changes
   
   - [ ] Bug fix (generally a non-breaking change which closes an issue).
   - [ ] Enhancement or new feature (adds new functionality).
   - [ ] Breaking change (a bug fix or enhancement which changes existing 
behavior).
   
   ## Checklist:
   
   
   - [ ] I signed an [Apache 
CLA](https://github.com/apache/openwhisk/blob/master/CONTRIBUTING.md).
   - [ ] I reviewed the [style 
guides](https://github.com/apache/openwhisk/blob/master/CONTRIBUTING.md#coding-standards)
 and followed the recommendations (Travis CI will check :).
   - [ ] I added tests to cover my changes.
   - [ ] My changes require further changes to the documentation.
   - [ ] I updated the documentation where necessary.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] sciabarracom merged pull request #5361: GitHub action

[GitBox]

sciabarracom merged PR #5361:
URL: https://github.com/apache/openwhisk/pull/5361


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] msciabarra commented on pull request #5361: GitHub action

[GitBox]

msciabarra commented on PR #5361:
URL: https://github.com/apache/openwhisk/pull/5361#issuecomment-1354549513

   Definitely ready to merge. This is what I get on my fork:
   ![Screenshot 2022-12-16 at 10 52 
51](https://user-images.githubusercontent.com/1595312/208082941-d902970d-787b-4b84-bedc-2bbeaf7199e9.png)
   All tests are passing.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] style95 commented on a diff in pull request #5361: GitHub action

[GitBox]

style95 commented on code in PR #5361:
URL: https://github.com/apache/openwhisk/pull/5361#discussion_r1050494234


##
tools/github/setup.sh:
##
@@ -0,0 +1,79 @@
+#!/bin/bash
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+#if [[ $TEST_SUITE =~ Dummy ]]

Review Comment:
   ah ok I got confused.
   My last question is what is the purpose of this commented block?
   There is already uncommnted one.
   
   ```
   if [[ $TEST_SUITE =~ Dummy ]]
   then echo skiping setup ; exit 0
   fi



##
tools/github/setup.sh:
##
@@ -0,0 +1,79 @@
+#!/bin/bash
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+#if [[ $TEST_SUITE =~ Dummy ]]

Review Comment:
   ah ok, I got confused.
   My last question is what is the purpose of this commented block?
   There is already uncommented one.
   
   ```
   if [[ $TEST_SUITE =~ Dummy ]]
   then echo skiping setup ; exit 0
   fi```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] style95 commented on a diff in pull request #5361: GitHub action

[GitBox]

style95 commented on code in PR #5361:
URL: https://github.com/apache/openwhisk/pull/5361#discussion_r1050494234


##
tools/github/setup.sh:
##
@@ -0,0 +1,79 @@
+#!/bin/bash
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+#if [[ $TEST_SUITE =~ Dummy ]]

Review Comment:
   ah ok I got confused.
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [openwhisk] msciabarra commented on a diff in pull request #5361: GitHub action

[GitBox]

msciabarra commented on code in PR #5361:
URL: https://github.com/apache/openwhisk/pull/5361#discussion_r1050472679


##
tools/github/setup.sh:
##
@@ -0,0 +1,79 @@
+#!/bin/bash
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+#if [[ $TEST_SUITE =~ Dummy ]]

Review Comment:
   Yes you control the test from there. Select "Dummy" to run the dummy test. 
It is useful when also you enable Ngrok debugging so you get an ssh command to 
get into the CI build VM to experiment. 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org