Skip to content

Commit a9a3a16

Browse files
author
Tsotne Tabidze
committed
Another try at fixing docker image tagging
Signed-off-by: Tsotne Tabidze <tsotne@tecton.ai>
1 parent c8ff73e commit a9a3a16

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

.github/workflows/pr_integration_tests.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,13 +38,12 @@ jobs:
3838
env:
3939
ECR_REGISTRY: ${{ steps.login-ecr.outputs.registry }}
4040
ECR_REPOSITORY: feast-python-server
41-
IMAGE_TAG: ${{ github.event.pull_request.head.ref }}
4241
run: |
4342
docker build \
4443
--file sdk/python/feast/infra/feature_servers/aws_lambda/Dockerfile \
45-
--tag $ECR_REGISTRY/$ECR_REPOSITORY:$IMAGE_TAG \
44+
--tag $ECR_REGISTRY/$ECR_REPOSITORY:`git rev-parse HEAD` \
4645
.
47-
docker push $ECR_REGISTRY/$ECR_REPOSITORY:$IMAGE_TAG
46+
docker push $ECR_REGISTRY/$ECR_REPOSITORY:`git rev-parse HEAD`
4847
integration-test-python:
4948
# all jobs MUST have this if check for 'ok-to-test' or 'approved' for security purposes.
5049
if: (github.event.action == 'labeled' && (github.event.label.name == 'approved' || github.event.label.name == 'ok-to-test'))

0 commit comments

Comments
 (0)