Skip to content

[DON'T MERGE] check current master's travis ci whether pass#171

Closed
ningyougang wants to merge 4 commits intoapache:masterfrom
ningyougang:test-jenkins
Closed

[DON'T MERGE] check current master's travis ci whether pass#171
ningyougang wants to merge 4 commits intoapache:masterfrom
ningyougang:test-jenkins

Conversation

@ningyougang
Copy link
Contributor

@ningyougang ningyougang commented Aug 2, 2022

Found why travis ci failed in current master

@ningyougang ningyougang changed the title [DON'T MERGE]Test jenkins [DON'T MERGE] check current master's travis ci whether pass Aug 2, 2022
# Cleanup apt data, we do not need them later on.
apt-get clean && rm -rf /var/lib/apt/lists/* &&\
go get -u github.com/go-delve/delve/cmd/dlv@v1.8.1 &&\
go install github.com/go-delve/delve/cmd/dlv@v1.8.1 &&\
Copy link
Member

@upgle upgle Aug 2, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change is required to build a docker image 👍

go get: installing executables with 'go get' in module mode is deprecated.
        Use 'go install pkg@version' instead.
        For more information, see https://golang.org/doc/go-get-install-deprecation
        or run 'go help get' or 'go help install'.
# github.com/rivo/uniseg
pkg/mod/github.com/rivo/uniseg@v0.3.1/properties.go:130:6: missing function body
pkg/mod/github.com/rivo/uniseg@v0.3.1/properties.go:130:20: syntax error: unexpected [, expecting (
note: module requires Go 1.18
The command '/bin/sh -c echo "deb http://deb.debian.org/debian buster-backports main contrib non-free"      >>/etc/apt/sources.list &&    echo 'debconf debconf/frontend select Noninteractive' | debconf-set-selections &&    apt-get update &&    apt-get upgrade -y --no-install-recommends &&    apt-get install -y apt-utils &&    apt-get install -y      curl      jq      git      zip      vim &&    apt-get -y install      librdkafka1=0.11.6-1.1      librdkafka++1=0.11.6-1.1 &&    apt-get -y install      librdkafka-dev=0.11.6-1.1 &&    apt-get clean && rm -rf /var/lib/apt/lists/* &&    go get -u github.com/go-delve/delve/cmd/dlv@v1.8.1 &&    mkdir /action' returned a non-zero code: 2

@upgle
Copy link
Member

upgle commented Aug 3, 2022

Please add the dependencies explicitly.

diff --git a/tests/build.gradle b/tests/build.gradle
index de9e0e0..b718fb8 100644
--- a/tests/build.gradle
+++ b/tests/build.gradle
@@ -34,7 +34,10 @@ tasks.withType(Test) {
 }
 dependencies {
+    implementation "junit:junit:4.11"
     implementation "org.scala-lang:scala-library:${gradle.scala.version}"
+    implementation "org.scalatest:scalatest_${gradle.scala.depVersion}:3.0.8"
+    implementation "org.apache.openwhisk:openwhisk-common:${gradle.openwhisk.version}"
     implementation "org.apache.openwhisk:openwhisk-tests:${gradle.openwhisk.version}:tests"
     implementation "org.apache.openwhisk:openwhisk-tests:${gradle.openwhisk.version}:test-sources"
     implementation group: 'com.typesafe.akka', name: "akka-http2-support_${gradle.scala.depVersion}", version: "${gradle.akka_http.version}"

@ningyougang
Copy link
Contributor Author

Closed as solved in #170

@ningyougang ningyougang closed this Aug 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants