feat(ci): skip smoke-test lint if there are no changes (#9945)

datahub-project · Feb 28, 2024 · 55bc955 · 55bc955
1 parent b45c04f
commit 55bc955
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 28 deletions.
diff --git a/.github/actions/ci-optimization/action.yml b/.github/actions/ci-optimization/action.yml
@@ -35,6 +35,9 @@ outputs:
   elasticsearch-setup-change:
     description: "Elasticsearch setup docker change"
     value: ${{ steps.filter.outputs.elasticsearch-setup == 'true' }}
+  smoke-test-change:
+    description: "Smoke test change"
+    value: ${{ steps.filter.outputs.smoke-test == 'true' }}
 runs:
   using: "composite"
   steps:
@@ -76,4 +79,6 @@ runs:
           postgres-setup:
             - "docker/postgres-setup/**"
           elasticsearch-setup:
-            - "docker/elasticsearch-setup/**"
+            - "docker/elasticsearch-setup/**"
+          smoke-test:
+            - "smoke-test/**"
diff --git a/.github/workflows/docker-unified.yml b/.github/workflows/docker-unified.yml
@@ -29,35 +29,8 @@ env:
   DATAHUB_INGESTION_IMAGE: "acryldata/datahub-ingestion"
 
 jobs:
-  check_lint:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Check out the repo
-        uses: acryldata/sane-checkout-action@v3
-      - uses: actions/setup-python@v4
-        with:
-          python-version: "3.10"
-          cache: "pip"
-      - uses: actions/cache@v4
-        with:
-          path: |
-            ~/.cache/uv
-          key: ${{ runner.os }}-uv-${{ hashFiles('**/requirements.txt') }}
-      - name: Set up JDK 17
-        uses: actions/setup-java@v3
-        with:
-          distribution: "zulu"
-          java-version: 17
-      - uses: gradle/gradle-build-action@v2
-      - name: Ensure packages are correct
-        run: |
-          python ./.github/scripts/check_python_package.py
-      - name: Run lint on smoke test
-        run: |
-          ./gradlew :smoke-test:lint
   setup:
     runs-on: ubuntu-latest
-    needs: check_lint
     outputs:
       tag: ${{ steps.tag.outputs.tag }}
       slim_tag: ${{ steps.tag.outputs.slim_tag }}
@@ -106,6 +79,29 @@ jobs:
           echo "publish=${{ env.ENABLE_PUBLISH }}" >> $GITHUB_OUTPUT
       - uses: ./.github/actions/ci-optimization
         id: ci-optimize
+      - uses: actions/setup-python@v4
+        if: ${{ steps.ci-optimize.outputs.smoke-test-change == 'true' }}
+        with:
+          python-version: "3.10"
+          cache: "pip"
+      - uses: actions/cache@v4
+        if: ${{ steps.ci-optimize.outputs.smoke-test-change == 'true' }}
+        with:
+          path: |
+            ~/.cache/uv
+          key: ${{ runner.os }}-uv-${{ hashFiles('**/requirements.txt') }}
+      - name: Set up JDK 17
+        uses: actions/setup-java@v3
+        if: ${{ steps.ci-optimize.outputs.smoke-test-change == 'true' }}
+        with:
+          distribution: "zulu"
+          java-version: 17
+      - uses: gradle/gradle-build-action@v2
+      - name: Run lint on smoke test
+        if: ${{ steps.ci-optimize.outputs.smoke-test-change == 'true' }}
+        run: |
+          python ./.github/scripts/check_python_package.py
+          ./gradlew :smoke-test:lint
 
   gms_build:
     name: Build and Push DataHub GMS Docker Image