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

2022-12-20 Thread 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

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

2022-12-20 Thread 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&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache

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

2022-12-20 Thread 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 ## Relate

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

2022-12-20 Thread 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

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

2022-12-20 Thread 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 GitH

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

2022-12-20 Thread 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