update travis badge for migration to travis-ci.com #5036

jiangpengcheng · 2020-12-18T03:14:36Z

No description provided.

jiangpengcheng · 2020-12-21T06:53:37Z

the unit test still failed in scan.sh step, but didn't find any error logs:

from above image, the scan.sh should be completed successfully

codecov-io · 2020-12-24T04:43:57Z

Codecov Report

Merging #5036 (ff87132) into master (4babe39) will decrease coverage by 11.27%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #5036       +/-   ##
===========================================
- Coverage   83.81%   72.54%   -11.28%     
===========================================
  Files         202      195        -7     
  Lines        9818     9553      -265     
  Branches      416      413        -3     
===========================================
- Hits         8229     6930     -1299     
- Misses       1589     2623     +1034

Impacted Files	Coverage Δ
...che/openwhisk/core/invoker/LogStoreCollector.scala	`0.00% <0.00%> (-100.00%)`	⬇️
...core/database/cosmosdb/RxObservableImplicits.scala	`0.00% <0.00%> (-100.00%)`	⬇️
...ore/database/cosmosdb/cache/CacheInvalidator.scala	`0.00% <0.00%> (-100.00%)`	⬇️
...e/database/cosmosdb/cache/ChangeFeedConsumer.scala	`0.00% <0.00%> (-100.00%)`	⬇️
...core/database/cosmosdb/CosmosDBArtifactStore.scala	`0.00% <0.00%> (-95.48%)`	⬇️
...sk/core/database/cosmosdb/CosmosDBViewMapper.scala	`0.00% <0.00%> (-93.90%)`	⬇️
...tabase/cosmosdb/cache/CacheInvalidatorConfig.scala	`0.00% <0.00%> (-92.31%)`	⬇️
...a/org/apache/openwhisk/http/BasicHttpService.scala	`3.33% <0.00%> (-86.67%)`	⬇️
...la/org/apache/openwhisk/http/BasicRasService.scala	`16.66% <0.00%> (-83.34%)`	⬇️
...enwhisk/connector/kafka/KamonMetricsReporter.scala	`0.00% <0.00%> (-83.34%)`	⬇️
... and 48 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4babe39...ff87132. Read the comment docs.

style95 · 2020-12-26T00:35:00Z

    stderr: Traceback (most recent call last):
      File "/home/travis/build/apache/openwhisk/ansible/../tools/db/cleanUpActivations.py", line 24, in <module>
        import couchdb.client
    ImportError: No module named couchdb.client

It complains about the couchdb.client module.
And it seems it's failing in conjunction with some heisenbugs.

jiangpengcheng · 2021-01-08T07:31:29Z

.travis.yml

@@ -54,6 +54,7 @@ cache:
 - $HOME/.gradle/wrapper/

 before_install:
+ - sudo systemctl stop epmd.socket epmd.service


they are using 4369 port

jiangpengcheng · 2021-01-08T07:32:30Z

tools/travis/scan.sh

@@ -41,7 +41,7 @@ cd $ROOTDIR
 TERM=dumb ./gradlew checkScalafmtAll

 # lint tests to all be actually runnable
-MISSING_TESTS=$(grep -rL "RunWith" --include="*Tests.scala" tests)
+MISSING_TESTS=$(grep -rL "RunWith" --include="*Tests.scala" tests || true)


seems ubuntu 20.04 will return an error for this command if grep matched nothing

jiangpengcheng · 2021-01-08T07:33:53Z

tools/travis/setup.sh

@@ -34,7 +34,7 @@ function retry() {
 pip install --user couchdb

 # Ansible
-pip install --user ansible==2.5.2
+pip install --user ansible==2.8.18


ansible 2.5.2 is not compatible with python3, and some of our ansible script doesn't compatible ansible versions beyond 2.8

So do we need to migrate all scripts?
If so, it might be easier to fix the flake8 issue, for now, to normalize CI tests and incrementally introduce python3.

I found just one error while using ansible2.10, which is in the cli/deploy.yml#name: "Download release archive to build directory ..."

the headers option for get_url should use map instead of string

jiangpengcheng · 2021-01-11T09:48:43Z

strange failed test:

org.apache.openwhisk.core.admin.WskAdminTests > Wsk Admin CLI should adjust whitelist for namespace FAILED
    org.scalatest.exceptions.TestFailedException: "allowedKinds = [['nodejs:10', ]'blackbox']" was not equal to "allowedKinds = [[u'nodejs:10', u]'blackbox']"

jiangpengcheng · 2021-01-15T01:06:40Z

tests/src/test/scala/org/apache/openwhisk/core/admin/WskAdminTests.scala

@@ -252,7 +252,8 @@ class WskAdminTests extends TestHelpers with WskActorSystem with Matchers with B
 // check correctly set
 val lines = wskadmin.cli(Seq("limits", "get", subject)).stdout.linesIterator.toSeq
 lines should have size 1
- lines(0) shouldBe "allowedKinds = [u'nodejs:10', u'blackbox']"
+ lines(0) should (be("allowedKinds = [u'nodejs:10', u'blackbox']") or be(
+ "allowedKinds = ['nodejs:10', 'blackbox']"))


to be compatible with ubuntu20.04

jiangpengcheng · 2021-01-15T01:07:23Z

tools/travis/runStandaloneTests.sh

@@ -44,6 +44,8 @@ kind create cluster --wait 5m
 export KUBECONFIG="$(kind get kubeconfig-path)"
 kubectl config set-context --current --namespace=default

+docker pull openwhisk/action-nodejs-v10:nightly
+docker pull openwhisk/dockerskeleton:nightly


seems travis ci machine will take a long time to pull images which make some tests failed

rabbah · 2021-01-15T06:27:57Z

The PR shows there are two conflicts to resolve.

jiangpengcheng · 2021-01-15T06:45:45Z

since this PR: #5042 has already resolved the CI error, I will close this one

dgrove-oss added 2 commits December 11, 2020 14:12

update travis badge for migration to travis-ci.com

270b42f

re-encrypt travis2slack url

62c62af

jiangpengcheng changed the title ~~Change travis~~ update travis badge for migration to travis-ci.com Dec 18, 2020

rabbah approved these changes Dec 18, 2020

View reviewed changes

jiangpengcheng closed this Dec 22, 2020

jiangpengcheng reopened this Dec 22, 2020

Fix error

68c8b74

jiangpengcheng force-pushed the change_travis branch from 5df94bc to 68c8b74 Compare January 8, 2021 07:30

jiangpengcheng commented Jan 8, 2021

View reviewed changes

jiangpengcheng added 3 commits January 13, 2021 15:53

Fix WskAdminTests

ff87132

Pull nodejs10 first when do standalone test

b287c24

Pull required images before test

084f1c4

jiangpengcheng commented Jan 15, 2021

View reviewed changes

jiangpengcheng closed this Jan 15, 2021

jiangpengcheng reopened this Jan 15, 2021

style95 mentioned this pull request Jan 15, 2021

Migrate the Travis configuration to travis-ci.com #5042

Merged

21 tasks

jiangpengcheng closed this Jan 15, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

update travis badge for migration to travis-ci.com #5036

update travis badge for migration to travis-ci.com #5036

jiangpengcheng commented Dec 18, 2020

jiangpengcheng commented Dec 21, 2020

codecov-io commented Dec 24, 2020 •

edited

Loading

style95 commented Dec 26, 2020 •

edited

Loading

jiangpengcheng Jan 8, 2021

jiangpengcheng Jan 8, 2021

jiangpengcheng Jan 8, 2021

style95 Jan 8, 2021

jiangpengcheng Jan 8, 2021

jiangpengcheng commented Jan 11, 2021

jiangpengcheng Jan 15, 2021

jiangpengcheng Jan 15, 2021

rabbah commented Jan 15, 2021 •

edited

Loading

jiangpengcheng commented Jan 15, 2021

update travis badge for migration to travis-ci.com #5036

update travis badge for migration to travis-ci.com #5036

Conversation

jiangpengcheng commented Dec 18, 2020

jiangpengcheng commented Dec 21, 2020

codecov-io commented Dec 24, 2020 • edited Loading

Codecov Report

style95 commented Dec 26, 2020 • edited Loading

jiangpengcheng Jan 8, 2021

Choose a reason for hiding this comment

jiangpengcheng Jan 8, 2021

Choose a reason for hiding this comment

jiangpengcheng Jan 8, 2021

Choose a reason for hiding this comment

style95 Jan 8, 2021

Choose a reason for hiding this comment

jiangpengcheng Jan 8, 2021

Choose a reason for hiding this comment

jiangpengcheng commented Jan 11, 2021

jiangpengcheng Jan 15, 2021

Choose a reason for hiding this comment

jiangpengcheng Jan 15, 2021

Choose a reason for hiding this comment

rabbah commented Jan 15, 2021 • edited Loading

jiangpengcheng commented Jan 15, 2021

codecov-io commented Dec 24, 2020 •

edited

Loading

style95 commented Dec 26, 2020 •

edited

Loading

rabbah commented Jan 15, 2021 •

edited

Loading