Fix ECR Image build on master branch (#2076)

Signed-off-by: Tsotne Tabidze <tsotne@tecton.ai>
feast-dev · Nov 19, 2021 · 3917d5a · 3917d5a
1 parent 380cc62
commit 3917d5a
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 31 deletions.
diff --git a/.github/workflows/build_feature_server.yml b/.github/workflows/build_feature_server.yml
diff --git a/.github/workflows/integration_tests.yml b/.github/workflows/integration_tests.yml
@@ -6,7 +6,36 @@ on:
       - master
 
 jobs:
+  build-docker-image:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - name: Set up QEMU
+        uses: docker/setup-qemu-action@v1
+      - name: Set up Docker Buildx
+        uses: docker/setup-buildx-action@v1
+      - name: Set up AWS SDK
+        uses: aws-actions/configure-aws-credentials@v1
+        with:
+          aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }}
+          aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
+          aws-region: us-west-2
+      - name: Login to Amazon ECR
+        id: login-ecr
+        uses: aws-actions/amazon-ecr-login@v1
+      - name: Build and push
+        env:
+          ECR_REGISTRY: ${{ steps.login-ecr.outputs.registry }}
+          ECR_REPOSITORY: feast-python-server
+        # Note: the image tags should be in sync with sdk/python/feast/infra/aws.py:_get_docker_image_version
+        run: |
+          docker build \
+            --file sdk/python/feast/infra/feature_servers/aws_lambda/Dockerfile \
+            --tag $ECR_REGISTRY/$ECR_REPOSITORY:`git rev-parse HEAD` \
+            .
+          docker push $ECR_REGISTRY/$ECR_REPOSITORY:`git rev-parse HEAD`
   integration-test-python:
+    needs: build-docker-image
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: false