ci: Fix build and ensure that building the java feature server is part of PR CI (feast-dev#3052)

adchia · web-flow · commit bc8b3a3ebfdd · 2022-08-09T19:06:27.000-04:00
ci: Fix build and ensure that building the java feature server is part of the java_pr workflow

Signed-off-by: Danny Chiao &lt;danny@tecton.ai&gt;
diff --git a/.github/workflows/java_pr.yml b/.github/workflows/java_pr.yml
@@ -53,6 +53,26 @@ jobs:
           name: java-coverage-report
           path: ${{ github.workspace }}/docs/coverage/java/target/site/jacoco-aggregate/
 
+  build-docker-image-java:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        component: [ feature-server-java ]
+    env:
+      REGISTRY: gcr.io/kf-feast
+    steps:
+      - uses: actions/checkout@v2
+        with:
+          submodules: 'true'
+      - run: gcloud auth configure-docker --quiet
+      - name: Get m2 cache
+        run: |
+          infra/scripts/download-maven-cache.sh \
+          --archive-uri ${MAVEN_CACHE} \
+          --output-dir .
+      - name: Build image
+        run: make build-${{ matrix.component }}-docker REGISTRY=${REGISTRY} VERSION=${GITHUB_SHA}
+
   integration-test:
     # all jobs MUST have this if check for 'ok-to-test' or 'approved' for security purposes.
     if:
diff --git a/java/infra/docker/feature-server/Dockerfile b/java/infra/docker/feature-server/Dockerfile
@@ -10,7 +10,7 @@ COPY java/pom.xml .
 COPY java/datatypes/pom.xml datatypes/pom.xml
 COPY java/serving/pom.xml serving/pom.xml
 COPY java/serving-client/pom.xml serving-client/pom.xml
-COPY java/docs/coverage/pom.xml docs/coverage/pom.xml
+COPY java/coverage/pom.xml coverage/pom.xml
 
 # Setting Maven repository .m2 directory relative to /build folder gives the
 # user to optionally use cached repository when building the image by copying
