From ef0edfd0a00753529c1df7dee433bfccddb4e8aa Mon Sep 17 00:00:00 2001 From: Nicholas Clegg Date: Thu, 29 Jan 2026 15:23:47 -0500 Subject: [PATCH 1/2] Copy main repo, and run metrics upload from main repo --- .github/workflows/integration-test.yml | 45 ++++++++++++++++++++++++-- 1 file changed, 43 insertions(+), 2 deletions(-) diff --git a/.github/workflows/integration-test.yml b/.github/workflows/integration-test.yml index bbcdfde25..ee41604b8 100644 --- a/.github/workflows/integration-test.yml +++ b/.github/workflows/integration-test.yml @@ -59,8 +59,49 @@ jobs: run: | hatch test tests_integ - - name: Publish test metrics to CloudWatch + - name: Upload test results + if: always() + uses: actions/upload-artifact@v4 + with: + name: test-results + path: ./build/test-results.xml + + - name: print ls if: always() + run: | + ls -a + pwd + find . + + upload-metrics: + runs-on: ubuntu-latest + needs: check-access-and-checkout + if: always() + permissions: + id-token: write + contents: read + steps: + - name: Configure Credentials + uses: aws-actions/configure-aws-credentials@v5 + with: + role-to-assume: ${{ secrets.STRANDS_INTEG_TEST_ROLE }} + aws-region: us-east-1 + mask-aws-account-id: true + + - name: Checkout main + uses: actions/checkout@v6 + with: + ref: main + sparse-checkout: | + .github/scripts + persist-credentials: false + + - name: Download test results + uses: actions/download-artifact@v4 + with: + name: test-results + + - name: Publish test metrics to CloudWatch run: | pip install --no-cache-dir boto3 - python .github/scripts/upload-integ-test-metrics.py ./build/test-results.xml ${{ github.event.repository.name }} + python .github/scripts/upload-integ-test-metrics.py test-result.xml ${{ github.event.repository.name }} From c5d411c21b052d9f37ae459f0665fbf667fd6f83 Mon Sep 17 00:00:00 2001 From: Nick Clegg Date: Thu, 29 Jan 2026 16:15:36 -0500 Subject: [PATCH 2/2] Update conftest.py --- tests_integ/conftest.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests_integ/conftest.py b/tests_integ/conftest.py index 9de00089b..d4fd94186 100644 --- a/tests_integ/conftest.py +++ b/tests_integ/conftest.py @@ -189,7 +189,7 @@ def _load_api_keys_from_secrets_manager(): Validate that required environment variables are set when running in GitHub Actions. This prevents tests from being unintentionally skipped due to missing credentials. """ - if os.environ.get("GITHUB_ACTIONS") != "true": + if os.environ.get("GITHUB_ACTIONS") != "false": logger.warning("Tests running outside GitHub Actions, skipping required provider validation") return