Merge pull request moby#3617 from crazy-max/ci-merge-test-nydus

ci: merge test-nydus job in test one
Antiarchitect · Feb 13, 2023 · 71a3a18 · 71a3a18
2 parents add607c + d6b20f1
commit 71a3a18
Showing 1 changed file with 22 additions and 52 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -87,13 +87,27 @@ jobs:
         typ:
           - integration
           - dockerfile
+        tags:
+          - ''
         exclude:
           - pkg: ./client ./cmd/buildctl ./worker/containerd ./solver ./frontend
             typ: dockerfile
         include:
           - pkg: ./...
             skip-integration-tests: 1
             typ: integration gateway
+          - pkg: ./client
+            worker: containerd
+            tags: nydus
+            typ: integration
+          - pkg: ./client
+            worker: oci
+            tags: nydus
+            typ: integration
+          - pkg: ./...
+            tags: nydus
+            skip-integration-tests: 1
+            typ: integration
     steps:
       -
         name: Checkout
@@ -112,75 +126,31 @@ jobs:
           driver-opts: image=${{ env.REPO_SLUG_ORIGIN }}
           buildkitd-flags: --debug
       -
-        name: Test pkg=${{ matrix.pkg }} ; typ=${{ matrix.typ }} ; skipit=${{ matrix.skip-integration-tests }} ; worker=${{ matrix.worker }}
+        name: Test
+        continue-on-error: ${{ matrix.tags == 'nydus' }}
         run: |
+          if [ -n "${{ matrix.tags }}" ]; then
+            TESTFLAGS="${TESTFLAGS} --tags=${{ matrix.tags }}"
+            export BUILDKITD_TAGS="${{ matrix.tags }}"
+          fi
           if [ -n "${{ matrix.worker }}" ]; then
             export TESTFLAGS="${TESTFLAGS} --run=//worker=${{ matrix.worker }}$"
           fi
           ./hack/test ${{ matrix.typ }}
-          mv ./coverage/coverage.txt ./coverage/coverage-${{ github.job }}-$(echo "${{ matrix.pkg }}-${{ matrix.skip-integration-tests }}-${{ matrix.typ }}-${{ matrix.worker }}" | tr -dc '[:alnum:]-\n\r' | tr '[:upper:]' '[:lower:]').txt
+          mv ./coverage/coverage.txt ./coverage/coverage-${{ github.job }}-$(echo "${{ matrix.pkg }}-${{ matrix.skip-integration-tests }}-${{ matrix.typ }}-${{ matrix.worker }}-${{ matrix.tags }}" | tr -dc '[:alnum:]-\n\r' | tr '[:upper:]' '[:lower:]').txt
         env:
           TEST_COVERAGE: 1
           TESTPKGS: ${{ matrix.pkg }}
           SKIP_INTEGRATION_TESTS: ${{ matrix.skip-integration-tests }}
           CACHE_FROM: type=gha,scope=${{ env.CACHE_GHA_SCOPE_IT }} type=gha,scope=${{ env.CACHE_GHA_SCOPE_BINARIES }}
       -
         name: Upload coverage file
+        continue-on-error: ${{ matrix.tags == 'nydus' }}
         uses: actions/upload-artifact@v3
         with:
           name: coverage
           path: ./coverage
 
-  test-nydus:
-    runs-on: ubuntu-20.04
-    needs: [base]
-    strategy:
-      fail-fast: false
-      matrix:
-        pkg:
-          - ./client
-        worker:
-          - containerd
-          - oci
-        typ:
-          - integration
-        exclude:
-          - pkg: ./client ./cmd/buildctl ./worker/containerd ./solver ./frontend
-            typ: dockerfile
-        include:
-          - pkg: ./...
-            skip-integration-tests: 1
-            typ: integration
-    steps:
-      -
-        name: Checkout
-        uses: actions/checkout@v3
-      -
-        name: Expose GitHub Runtime
-        uses: crazy-max/ghaction-github-runtime@v2
-      -
-        name: Set up QEMU
-        uses: docker/setup-qemu-action@v2
-      -
-        name: Set up Docker Buildx
-        uses: docker/setup-buildx-action@v2
-        with:
-          version: ${{ env.BUILDX_VERSION }}
-          driver-opts: image=${{ env.REPO_SLUG_ORIGIN }}
-          buildkitd-flags: --debug
-      -
-        name: Test pkg=${{ matrix.pkg }} ; typ=${{ matrix.typ }} ; skipit=${{ matrix.skip-integration-tests }} ; worker=${{ matrix.worker }}
-        run: |
-          if [ -n "${{ matrix.worker }}" ]; then
-            export TESTFLAGS="${TESTFLAGS} --tags=nydus --run=//worker=${{ matrix.worker }}$"
-          fi
-          ./hack/test ${{ matrix.typ }}
-        env:
-          BUILDKITD_TAGS: nydus
-          TESTPKGS: ${{ matrix.pkg }}
-          SKIP_INTEGRATION_TESTS: ${{ matrix.skip-integration-tests }}
-          CACHE_FROM: type=gha,scope=${{ env.CACHE_GHA_SCOPE_IT }} type=gha,scope=${{ env.CACHE_GHA_SCOPE_BINARIES }}
-
   test-os:
     runs-on: ${{ matrix.os }}
     strategy: