From c5cad2ee8d80b9d226ea6b4f520b6cd9cd1fa0f1 Mon Sep 17 00:00:00 2001 From: Michael He <53622546+yiyuan-he@users.noreply.github.com> Date: Wed, 12 Feb 2025 10:06:28 -0800 Subject: [PATCH] Fix: Deprecated artifacts follow up (#327) *Issue #, if available:* The previous fix did not work because `download-artifacts@v3` is also deprecated and must be updated. ``` Error: This request has been automatically failed because it uses a deprecated version of `actions/download-artifact: v3`. Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/ ``` Previous fix: https://github.com/aws-observability/aws-otel-python-instrumentation/pull/326 *Description of changes:* - Updating `download-artifacts@v3` to `v4`. - Also fixing syntax error for name pattern. By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice. --- .github/workflows/application-signals-e2e-test.yml | 2 +- .github/workflows/release-lambda.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/application-signals-e2e-test.yml b/.github/workflows/application-signals-e2e-test.yml index c7a5731a..288d93ad 100644 --- a/.github/workflows/application-signals-e2e-test.yml +++ b/.github/workflows/application-signals-e2e-test.yml @@ -34,7 +34,7 @@ jobs: role-to-assume: arn:aws:iam::${{ secrets.APPLICATION_SIGNALS_E2E_TEST_ACCOUNT_ID }}:role/${{ secrets.APPLICATION_SIGNALS_E2E_TEST_ROLE_NAME }} aws-region: us-east-1 - - uses: actions/download-artifact@v3 + - uses: actions/download-artifact@v4 with: name: ${{ inputs.staging-wheel-name }} diff --git a/.github/workflows/release-lambda.yml b/.github/workflows/release-lambda.yml index f621e2ed..77f3d90f 100644 --- a/.github/workflows/release-lambda.yml +++ b/.github/workflows/release-lambda.yml @@ -146,7 +146,7 @@ jobs: - name: download layerARNs uses: actions/download-artifact@v4 with: - name: ${{ env.LAYER_NAME }}-* + pattern: ${{ env.LAYER_NAME }}-* path: ${{ env.LAYER_NAME }} merge-multiple: true - name: show layerARNs