From 311d1c047e1d06a65e4328d62ff8a0ec8da8d8a2 Mon Sep 17 00:00:00 2001 From: Harshal Sheth Date: Tue, 28 Jan 2025 02:41:35 -0800 Subject: [PATCH] fix(ci): fix docker image save/load (#159) * fix(ci): fix docker image save/load I think the `>` character might have issues with yaml. * ensure it runs on all code with latest --- .github/workflows/datahub-actions-docker.yml | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/.github/workflows/datahub-actions-docker.yml b/.github/workflows/datahub-actions-docker.yml index 7309d7a..40f8076 100644 --- a/.github/workflows/datahub-actions-docker.yml +++ b/.github/workflows/datahub-actions-docker.yml @@ -11,6 +11,7 @@ on: - main paths: - "docker/**" + - "datahub-actions/**" - ".github/workflows/datahub-actions-docker.yml" - "!build/**" - "!**.md" @@ -148,7 +149,7 @@ jobs: "APP_ENV=prod-slim" - name: Save Docker image if: needs.setup.outputs.publish != 'true' - run: docker save ${{ steps.docker_meta_slim.outputs.tags }} > image.tar + run: docker image save -o image.tar ${{ steps.docker_meta_slim.outputs.tags }} - name: Upload artifact if: needs.setup.outputs.publish != 'true' uses: actions/upload-artifact@v3 @@ -205,7 +206,7 @@ jobs: name: docker-image - name: Load Docker image (if not publishing) if: needs.setup.outputs.publish != 'true' - run: docker load < image.tar + run: docker image load -i image.tar - name: Pull Docker image (if publishing) if: needs.setup.outputs.publish == 'true' run: docker pull acryldata/datahub-actions-slim:${{ needs.setup.outputs.unique_tag }} @@ -265,8 +266,6 @@ jobs: DATAHUB_TELEMETRY_ENABLED: false DATAHUB_ACTIONS_IMAGE: acryldata/datahub-actions-slim DATAHUB_ACTIONS_VERSION: ${{ needs.setup.outputs.unique_tag }} - ACTIONS_EXTRA_PACKAGES: "acryl-datahub-actions[executor]==0.0.13 acryl-datahub-actions==0.0.13 acryl-datahub==0.10.5" - ACTIONS_CONFIG: "https://raw.githubusercontent.com/acryldata/datahub-actions/main/docker/config/executor.yaml" run: | ./smoke-test/run-quickstart.sh - name: Disk Check