From 4b85af25867b9f682218087cfeda24a129704381 Mon Sep 17 00:00:00 2001 From: Daniel Vaz Gaspar Date: Wed, 31 Jan 2024 18:40:07 +0000 Subject: [PATCH] fix(ci): remove deprecated set-output on github workflows (#26921) --- .github/workflows/cancel_duplicates.yml | 2 +- .github/workflows/ephemeral-env-pr-close.yml | 2 +- .github/workflows/ephemeral-env.yml | 9 ++++----- .github/workflows/prefer-typescript.yml | 2 +- .github/workflows/release.yml | 2 +- .github/workflows/superset-helm-lint.yml | 2 +- 6 files changed, 9 insertions(+), 10 deletions(-) diff --git a/.github/workflows/cancel_duplicates.yml b/.github/workflows/cancel_duplicates.yml index a78ebc07fdbe7..3c2ac8e0b8a49 100644 --- a/.github/workflows/cancel_duplicates.yml +++ b/.github/workflows/cancel_duplicates.yml @@ -27,7 +27,7 @@ jobs: } count=$(( `get_count queued` + `get_count in_progress` )) echo "Found $count unfinished jobs." - echo "::set-output name=count::$count" + echo "count=$count" >> $GITHUB_OUTPUT - name: "Checkout ${{ github.ref }} ( ${{ github.sha }} )" if: steps.check_queued.outputs.count >= 20 diff --git a/.github/workflows/ephemeral-env-pr-close.yml b/.github/workflows/ephemeral-env-pr-close.yml index 7430950b453f6..30f0f684f2744 100644 --- a/.github/workflows/ephemeral-env-pr-close.yml +++ b/.github/workflows/ephemeral-env-pr-close.yml @@ -36,7 +36,7 @@ jobs: - name: Describe ECS service id: describe-services run: | - echo "::set-output name=active::$(aws ecs describe-services --cluster superset-ci --services pr-${{ github.event.number }}-service | jq '.services[] | select(.status == "ACTIVE") | any')" + echo "active=$(aws ecs describe-services --cluster superset-ci --services pr-${{ github.event.number }}-service | jq '.services[] | select(.status == "ACTIVE") | any')" >> $GITHUB_OUTPUT - name: Delete ECS service if: steps.describe-services.outputs.active == 'true' diff --git a/.github/workflows/ephemeral-env.yml b/.github/workflows/ephemeral-env.yml index 543c5e2862b35..25b5336aa639f 100644 --- a/.github/workflows/ephemeral-env.yml +++ b/.github/workflows/ephemeral-env.yml @@ -212,8 +212,7 @@ jobs: - name: Describe ECS service id: describe-services run: | - echo "::set-output name=active::$(aws ecs describe-services --cluster superset-ci --services pr-${{ github.event.issue.number }}-service | jq '.services[] | select(.status == "ACTIVE") | any')" - + echo "active=$(aws ecs describe-services --cluster superset-ci --services pr-${{ github.event.issue.number }}-service | jq '.services[] | select(.status == "ACTIVE") | any')" >> $GITHUB_OUTPUT - name: Create ECS service if: steps.describe-services.outputs.active != 'true' id: create-service @@ -244,17 +243,17 @@ jobs: - name: List tasks id: list-tasks run: | - echo "::set-output name=task::$(aws ecs list-tasks --cluster superset-ci --service-name pr-${{ github.event.issue.number }}-service | jq '.taskArns | first')" + echo "task=$(aws ecs list-tasks --cluster superset-ci --service-name pr-${{ github.event.issue.number }}-service | jq '.taskArns | first')" >> $GITHUB_OUTPUT - name: Get network interface id: get-eni run: | - echo "::set-output name=eni::$(aws ecs describe-tasks --cluster superset-ci --tasks ${{ steps.list-tasks.outputs.task }} | jq '.tasks | .[0] | .attachments | .[0] | .details | map(select(.name=="networkInterfaceId")) | .[0] | .value')" + echo "eni=$(aws ecs describe-tasks --cluster superset-ci --tasks ${{ steps.list-tasks.outputs.task }} | jq '.tasks | .[0] | .attachments | .[0] | .details | map(select(.name=="networkInterfaceId")) | .[0] | .value')" >> $GITHUB_OUTPUT - name: Get public IP id: get-ip run: | - echo "::set-output name=ip::$(aws ec2 describe-network-interfaces --network-interface-ids ${{ steps.get-eni.outputs.eni }} | jq -r '.NetworkInterfaces | first | .Association.PublicIp')" + echo "ip=$(aws ec2 describe-network-interfaces --network-interface-ids ${{ steps.get-eni.outputs.eni }} | jq -r '.NetworkInterfaces | first | .Association.PublicIp')" >> $GITHUB_OUTPUT - name: Comment (success) if: ${{ success() }} diff --git a/.github/workflows/prefer-typescript.yml b/.github/workflows/prefer-typescript.yml index 65fa8de11dc14..29dfcc0f7cfe6 100644 --- a/.github/workflows/prefer-typescript.yml +++ b/.github/workflows/prefer-typescript.yml @@ -42,7 +42,7 @@ jobs: ) | join("\n") ' ${HOME}/files_added.json } - echo ::set-output name=js_files_added::$(js_files_added) + echo "js_files_added=$(js_files_added)" >> $GITHUB_OUTPUT - if: steps.check.outputs.js_files_added name: Add Comment to PR diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index 7080620c95939..333868280acdb 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -64,7 +64,7 @@ jobs: - name: Get npm cache directory path if: env.HAS_TAGS id: npm-cache-dir-path - run: echo "::set-output name=dir::$(npm config get cache)" + run: echo "dir=$(npm config get cache)" >> $GITHUB_OUTPUT - name: Cache npm if: env.HAS_TAGS uses: actions/cache@v1 diff --git a/.github/workflows/superset-helm-lint.yml b/.github/workflows/superset-helm-lint.yml index 7ca8f11887459..23100ebcf09f3 100644 --- a/.github/workflows/superset-helm-lint.yml +++ b/.github/workflows/superset-helm-lint.yml @@ -34,7 +34,7 @@ jobs: run: | changed=$(ct list-changed --print-config) if [[ -n "$changed" ]]; then - echo "::set-output name=changed::true" + echo "changed=true" >> $GITHUB_OUTPUT fi env: CT_CHART_DIRS: helm