This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch add-more-disk-for-arm-instance in repository https://gitbox.apache.org/repos/asf/airflow.git
commit 494b5a4f95f9b6b111eea589186e378898d03a24 Author: Jarek Potiuk <[email protected]> AuthorDate: Sat Nov 18 19:33:12 2023 +0100 fixup! Add more disk space for ARM image --- .github/workflows/ci.yml | 38 +++++++++++----------- .../ci_start_arm_instance_and_connect_to_docker.sh | 2 +- 2 files changed, 20 insertions(+), 20 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 7a9ffb2251..35037c32f9 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -212,20 +212,20 @@ jobs: steps: - name: Cleanup repo run: docker run -v "${GITHUB_WORKSPACE}:/workspace" -u 0:0 bash -c "rm -rf /workspace/*" - if: needs.build-info.outputs.canary-run == 'true' && needs.build-info.outputs.default-branch == 'main' + # if: needs.build-info.outputs.canary-run == 'true' && needs.build-info.outputs.default-branch == 'main' - name: "Checkout ${{ github.ref }} ( ${{ github.sha }} )" uses: actions/checkout@v4 with: persist-credentials: false - if: needs.build-info.outputs.canary-run == 'true' && needs.build-info.outputs.default-branch == 'main' + # if: needs.build-info.outputs.canary-run == 'true' && needs.build-info.outputs.default-branch == 'main' - name: "Install Breeze" uses: ./.github/actions/breeze - if: needs.build-info.outputs.canary-run == 'true' && needs.build-info.outputs.default-branch == 'main' + # if: needs.build-info.outputs.canary-run == 'true' && needs.build-info.outputs.default-branch == 'main' - name: "Start ARM instance" run: ./scripts/ci/images/ci_start_arm_instance_and_connect_to_docker.sh - if: > - matrix.platform == 'linux/arm64' && needs.build-info.outputs.canary-run == 'true' - && needs.build-info.outputs.default-branch == 'main' + # if: > + # matrix.platform == 'linux/arm64' && needs.build-info.outputs.canary-run == 'true' + # && needs.build-info.outputs.default-branch == 'main' - name: "Push CI cache ${{ matrix.platform }}" run: > breeze ci-image build @@ -237,7 +237,7 @@ jobs: env: DEBUG_RESOURCES: ${{needs.build-info.outputs.debug-resources}} COMMIT_SHA: ${{ github.sha }} - if: needs.build-info.outputs.canary-run == 'true' && needs.build-info.outputs.default-branch == 'main' + # if: needs.build-info.outputs.canary-run == 'true' && needs.build-info.outputs.default-branch == 'main' - name: "Push CI latest image ${{ matrix.platform }}" run: > breeze ci-image build @@ -247,24 +247,24 @@ jobs: COMMIT_SHA: ${{ github.sha }} # We only push "amd" image as it is really only needed for any kind of automated builds in CI # and currently there is not an easy way to make multi-platform image from two separate builds - if: > - matrix.platform == 'linux/amd64' && needs.build-info.outputs.canary-run == 'true' - && needs.build-info.outputs.default-branch == 'main' + # if: > + # matrix.platform == 'linux/amd64' && needs.build-info.outputs.canary-run == 'true' + # && needs.build-info.outputs.default-branch == 'main' - name: "Stop ARM instance" run: ./scripts/ci/images/ci_stop_arm_instance.sh - if: > - always() && matrix.platform == 'linux/arm64' && needs.build-info.outputs.canary-run == 'true' - && needs.build-info.outputs.default-branch == 'main' + # if: > + # always() && matrix.platform == 'linux/arm64' && needs.build-info.outputs.canary-run == 'true' + # && needs.build-info.outputs.default-branch == 'main' - name: "Clean docker cache for ${{ matrix.platform }}" run: docker system prune --all --force - if: > - matrix.platform == 'linux/amd64' && needs.build-info.outputs.canary-run == 'true' - && needs.build-info.outputs.default-branch == 'main' + # if: > + # matrix.platform == 'linux/amd64' && needs.build-info.outputs.canary-run == 'true' + # && needs.build-info.outputs.default-branch == 'main' - name: "Fix ownership" run: breeze ci fix-ownership - if: > - always() && needs.build-info.outputs.canary-run == 'true' - && needs.build-info.outputs.default-branch == 'main' + # if: > + # always() && needs.build-info.outputs.canary-run == 'true' + # && needs.build-info.outputs.default-branch == 'main' # Check that after earlier cache push, breeze command will build quickly check-that-image-builds-quickly: timeout-minutes: 5 diff --git a/scripts/ci/images/ci_start_arm_instance_and_connect_to_docker.sh b/scripts/ci/images/ci_start_arm_instance_and_connect_to_docker.sh index 6e7047daf6..ef364e909d 100755 --- a/scripts/ci/images/ci_start_arm_instance_and_connect_to_docker.sh +++ b/scripts/ci/images/ci_start_arm_instance_and_connect_to_docker.sh @@ -39,7 +39,7 @@ function start_arm_instance() { --image-id "${ARM_AMI}" \ --count 1 \ --instance-type "${INSTANCE_TYPE}" \ - --blockd-device-mappings "file://${MAPPING_FILE}" \ + --block-device-mappings "file://${MAPPING_FILE}" \ --user-data "file://${USER_DATA_FILE}" \ --instance-market-options "${MARKET_OPTIONS}" \ --instance-initiated-shutdown-behavior terminate \
