Skip to content

Commit

Permalink
fix: Change image push to use --all-tags option (feast-dev#4926)
Browse files Browse the repository at this point in the history
change image push to use --all-tags option

Signed-off-by: Tommy Hughes <tohughes@redhat.com>
  • Loading branch information
tchughesiv authored Jan 15, 2025
1 parent 32aaf9a commit 02458fd
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions .github/workflows/master_only.yml
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,6 @@ jobs:
component: [ feature-server-dev, feature-transformation-server, feast-operator ]
env:
REGISTRY: quay.io/feastdev-ci
IMAGE: ${{ matrix.component }}
steps:
- uses: actions/checkout@v4
- name: Set up QEMU
Expand All @@ -117,10 +116,10 @@ jobs:
make build-${{ matrix.component }}-docker REGISTRY=${REGISTRY} VERSION=${GITHUB_SHA}
- name: Push image
run: |
make push-${{ matrix.component }}-docker REGISTRY=${REGISTRY} VERSION=${GITHUB_SHA}
if [[ ${{ matrix.component }} == "feature-server-dev" ]]; then
echo "IMAGE=feature-server" >> $GITHUB_ENV
if [[ "${{ matrix.component }}" == "feature-server-dev" ]]; then
docker tag ${REGISTRY}/feature-server:${GITHUB_SHA} ${REGISTRY}/feature-server:develop
docker push ${REGISTRY}/feature-server --all-tags
else
docker tag ${REGISTRY}/${{ matrix.component }}:${GITHUB_SHA} ${REGISTRY}/${{ matrix.component }}:develop
docker push ${REGISTRY}/${{ matrix.component }} --all-tags
fi
docker tag ${REGISTRY}/${IMAGE}:${GITHUB_SHA} ${REGISTRY}/${IMAGE}:develop
docker push ${REGISTRY}/${IMAGE}:develop

0 comments on commit 02458fd

Please sign in to comment.