Add unit tests and crossdock in github actions (#775)

Author: Ashmita152

.github/actions/setup-branch/action.yml

@@ -0,0 +1,18 @@
+name: 'Setup BRANCH'
+description: 'Make BRANCH var accessible to job'
+runs:
+  using: "composite"
+  steps:
+    - name: Setup BRANCH
+      shell: bash
+      run: |
+        case ${GITHUB_EVENT_NAME} in
+          pull_request)
+            BRANCH=${GITHUB_HEAD_REF}
+            ;;
+          push | release)
+            BRANCH=${GITHUB_REF##*/}
+            ;;
+        esac
+        echo "we are on branch=${BRANCH}"
+        echo "BRANCH=${BRANCH}" >> ${GITHUB_ENV}

.github/workflows/ci-crossdock.yml

@@ -0,0 +1,48 @@
+name: Crossdock
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  crossdock:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v2
+      with:
+        submodules: true
+
+    - uses: actions/setup-java@v1
+      with:
+        java-version: 8
+
+    - uses: docker/login-action@v1
+      id: dockerhub-login
+      with:
+        username: jaegertracingbot
+        password: ${{ secrets.DOCKERHUB_TOKEN }}
+      env:
+        DOCKERHUB_TOKEN: ${{ secrets.DOCKERHUB_TOKEN }}
+      if: env.DOCKERHUB_TOKEN != null
+
+    - name: Export DOCKERHUB_LOGIN variable
+      run: |
+        echo "DOCKERHUB_LOGIN=true" >> $GITHUB_ENV
+      if: steps.dockerhub-login.outcome == 'success'
+
+    - name: Export BRANCH variable
+      uses: ./.github/actions/setup-branch
+
+    - name: Run Crossdock
+      id: run-crossdock
+      run: make crossdock-fresh
+
+    - name: Output Crossdock Logs
+      run: make crossdock-logs
+      if: ${{ failure() }}
+
+    - name: Publish Crossdock Image
+      run: bash scripts/publish-crossdock.sh
+      if: steps.run-crossdock.outcome == 'success'

.github/workflows/ci-unit-tests.yml

@@ -0,0 +1,31 @@
+name: Unit Tests
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  unit-tests:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v2
+      with:
+        submodules: true
+
+    - uses: actions/setup-java@v1
+      with:
+        java-version: 8
+
+    - name: Run unit tests
+      run: make test-ci
+
+    - name: Generate codecov report
+      run: ./gradlew codeCoverageReport
+
+    - name: Upload coverage to codecov
+      uses: codecov/codecov-action@v1
+      with:
+        verbose: true
+        fail_ci_if_error: true

Makefile

@@ -10,8 +10,8 @@ clean:
 test:
 	$(GRADLE) check
 
-.PHONY: test-travis
-test-travis:
+.PHONY: test-ci
+test-ci:
 	$(GRADLE) -is check
 
 .PHONY: release

jaeger-crossdock/Dockerfile

@@ -1,4 +1,4 @@
-FROM openjdk:8u121-jdk-alpine
+FROM openjdk:8-jdk-alpine
 EXPOSE 8080-8082
 
 ADD build/libs/jaeger-crossdock.jar /

jaeger-thrift/build.gradle

@@ -22,7 +22,7 @@ dependencies {
 }
 
 compileThrift {
-    thriftExecutable "${projectDir}/../travis/docker-thrift/thrift"
+    thriftExecutable "${projectDir}/../scripts/docker-thrift/thrift"
     sourceDir "${projectDir}/../idl/thrift"
     outputDir 'src/main/gen-java'
     generator 'java', 'private-members'

travis/deploy.sh scripts/deploy.shtravis/deploy.sh renamed to scripts/deploy.sh

@@ -15,13 +15,13 @@ if [ "x${ENCRYPTION_IV}" == "x" ]; then
 fi
 
 if [ "${TRAVIS_SECURE_ENV_VARS}" == true ]; then
-    openssl aes-256-cbc -K "${ENCRYPTION_KEY}" -iv "${ENCRYPTION_IV}" -in travis/signing-key.asc.enc -out travis/signing-key.asc -d
+    openssl aes-256-cbc -K "${ENCRYPTION_KEY}" -iv "${ENCRYPTION_IV}" -in scripts/signing-key.asc.enc -out scripts/signing-key.asc -d
     if (( $? != 0 )); then
         echo "Failed to decrypt the signing key. Skipping."
         exit 1
     fi
 
-    gpg --no-tty --batch --allow-secret-key-import --import travis/signing-key.asc
+    gpg --no-tty --batch --allow-secret-key-import --import scripts/signing-key.asc
     rm -rf "$HOME/.gradle/gradle.properties"
     echo signing.keyId="${SIGNING_KEY_ID}" > "$HOME/.gradle/gradle.properties"
     echo signing.password="${SIGNING_KEY_PASSPHRASE}" >> "$HOME/.gradle/gradle.properties"