Skip to content

Commit 3917d5a

Browse files
author
Tsotne Tabidze
authored
Fix ECR Image build on master branch (feast-dev#2076)
Signed-off-by: Tsotne Tabidze <[email protected]>
1 parent 380cc62 commit 3917d5a

2 files changed

Lines changed: 29 additions & 31 deletions

File tree

‎.github/workflows/build_feature_server.yml‎

Lines changed: 0 additions & 31 deletions
This file was deleted.

‎.github/workflows/integration_tests.yml‎

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,36 @@ on:
66
- master
77

88
jobs:
9+
build-docker-image:
10+
runs-on: ubuntu-latest
11+
steps:
12+
- uses: actions/checkout@v2
13+
- name: Set up QEMU
14+
uses: docker/setup-qemu-action@v1
15+
- name: Set up Docker Buildx
16+
uses: docker/setup-buildx-action@v1
17+
- name: Set up AWS SDK
18+
uses: aws-actions/configure-aws-credentials@v1
19+
with:
20+
aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }}
21+
aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
22+
aws-region: us-west-2
23+
- name: Login to Amazon ECR
24+
id: login-ecr
25+
uses: aws-actions/amazon-ecr-login@v1
26+
- name: Build and push
27+
env:
28+
ECR_REGISTRY: ${{ steps.login-ecr.outputs.registry }}
29+
ECR_REPOSITORY: feast-python-server
30+
# Note: the image tags should be in sync with sdk/python/feast/infra/aws.py:_get_docker_image_version
31+
run: |
32+
docker build \
33+
--file sdk/python/feast/infra/feature_servers/aws_lambda/Dockerfile \
34+
--tag $ECR_REGISTRY/$ECR_REPOSITORY:`git rev-parse HEAD` \
35+
.
36+
docker push $ECR_REGISTRY/$ECR_REPOSITORY:`git rev-parse HEAD`
937
integration-test-python:
38+
needs: build-docker-image
1039
runs-on: ${{ matrix.os }}
1140
strategy:
1241
fail-fast: false

0 commit comments

Comments
 (0)