DO NOT overwrite master docker image

Signed-off-by: Oleksii Moskalenko <moskalenko.alexey@gmail.com>
feast-dev · Apr 5, 2021 · 52c7393 · 52c7393
1 parent 927ffbe
commit 52c7393
Showing 1 changed file with 2 additions and 7 deletions.
diff --git a/.github/workflows/pr_full_access.yml b/.github/workflows/pr_full_access.yml
@@ -29,7 +29,6 @@ jobs:
           # as such actions/checkout needs to be explicit configured to retrieve
           # code from the PR. 
           ref: ${{ github.event.pull_request.merge_commit_sha }}
-          submodules: recursive
       - uses: docker/setup-qemu-action@v1
       - name: Set up Docker Buildx
         uses: docker/setup-buildx-action@v1
@@ -46,11 +45,7 @@ jobs:
           --archive-uri ${MAVEN_CACHE} \
           --output-dir $HOME
       - name: Build image
-        run: make build-${{ matrix.component }}-docker REGISTRY=${REGISTRY} VERSION=${GITHUB_SHA}
+        run: make build-${{ matrix.component }}-docker REGISTRY=${REGISTRY} VERSION=${GITHUB_PR_SHA}
       - name: Push image
         run: |
-          docker push ${REGISTRY}/feast-${{ matrix.component }}:${GITHUB_SHA}
-          if [ -n "${GITHUB_PR_SHA}" ]; then
-            docker tag ${REGISTRY}/feast-${{ matrix.component }}:${GITHUB_SHA} gcr.io/kf-feast/feast-${{ matrix.component }}:${GITHUB_PR_SHA}
-            docker push ${REGISTRY}/feast-${{ matrix.component }}:${GITHUB_PR_SHA}
-          fi
+          docker push ${REGISTRY}/feast-${{ matrix.component }}:${GITHUB_PR_SHA}