dimas-b commented on code in PR #3989:
URL: https://github.com/apache/polaris/pull/3989#discussion_r2925553049


##########
.github/workflows/ci.yml:
##########
@@ -183,9 +183,44 @@ jobs:
           name: upload-${{ github.job }}-artifacts
           path: *test-archive-path
 
+  helm-tests-image-build:
+    name: Helm Tests Image Build
+    runs-on: ubuntu-latest
+    timeout-minutes: 30
+    steps:
+      - name: Checkout
+        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
+        with:
+          fetch-depth: 0
+      - name: Set up JDK 21
+        uses: actions/setup-java@be666c2fcd27ec809703dec50e508c2fdc7f6654 # v5
+        with: *setup-java-vars
+      - name: Setup test environment
+        uses: ./.github/actions/setup-test-env
+      - name: Prepare Gradle build cache
+        uses: ./.github/actions/ci-incr-build-cache-prepare
+      - name: Image build
+        env: *gradle_env_vars
+        run: |
+          ./gradlew \
+            :polaris-server:assemble \
+            :polaris-server:quarkusAppPartsBuild --rerun \

Review Comment:
   nit: do we need `quarkusAppPartsBuild --rerun` in CI? The source never 
changes during the build 🤔 



##########
.github/workflows/ci.yml:
##########
@@ -183,9 +183,44 @@ jobs:
           name: upload-${{ github.job }}-artifacts
           path: *test-archive-path
 
+  helm-tests-image-build:
+    name: Helm Tests Image Build

Review Comment:
   nit: technically this job is not about Helm at all, it just makes a docker 
image for Polaris



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to