From 7c7c27f0069c20e9ff3f70a38b9792c2002fb9be Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Philippe=20No=C3=ABl?= Date: Tue, 4 Feb 2025 23:53:49 -0800 Subject: [PATCH] Swap to Vars for DOCKERHUB_USERNAME Cleanup --- .github/workflows/tests-cluster-chainsaw.yaml | 27 +++++-------------- 1 file changed, 6 insertions(+), 21 deletions(-) diff --git a/.github/workflows/tests-cluster-chainsaw.yaml b/.github/workflows/tests-cluster-chainsaw.yaml index cbaf57ad8..d44834672 100644 --- a/.github/workflows/tests-cluster-chainsaw.yaml +++ b/.github/workflows/tests-cluster-chainsaw.yaml @@ -7,40 +7,25 @@ on: jobs: test-list: - runs-on: ubuntu-24.04 + runs-on: ubuntu-latest outputs: tests: ${{ steps.listTests.outputs.tests }} steps: - name: Checkout - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2 + uses: actions/checkout@4 with: fetch-depth: 1 - id: listTests run: | echo "tests=$(ls charts/cluster/test -1 | jq -cRn '{ include: [inputs | { test: "\(.)" }]}')" >> $GITHUB_OUTPUT - test: - needs: test-list - runs-on: ubuntu-24.04 - strategy: - fail-fast: false - matrix: ${{ fromJson(needs.test-list.outputs.tests) }} - name: ${{matrix.test}} - steps: - - name: Checkout - uses: actions/checkout@v4 - with: - fetch-depth: 1 - - id: listTests - run: | - echo "tests=$(ls charts/paradedb/test -1 | jq -cRn '{ include: [inputs | { test: "\(.)" }]}')" >> $GITHUB_OUTPUT - test: needs: test-list runs-on: ubuntu-latest strategy: + fail-fast: false matrix: ${{ fromJson(needs.test-list.outputs.tests) }} - + name: ${{ matrix.test }} steps: - name: Checkout uses: actions/checkout@v4 @@ -54,7 +39,7 @@ jobs: - name: Login to Docker Hub uses: docker/login-action@v3 with: - username: ${{ secrets.DOCKERHUB_USERNAME }} + username: ${{ vars.DOCKERHUB_USERNAME }} password: ${{ secrets.DOCKERHUB_ACCESS_TOKEN }} # Added by ParadeDB: Always pull the latest version of paradedb/paradedb @@ -110,5 +95,5 @@ jobs: - name: Run Kyverno/Chainsaw run: chainsaw test charts/paradedb/test/${{ matrix.test }} env: - PARADEDB_ENTERPRISE_DOCKER_USERNAME: ${{ secrets.DOCKERHUB_USERNAME }} + PARADEDB_ENTERPRISE_DOCKER_USERNAME: ${{ vars.DOCKERHUB_USERNAME }} PARADEDB_ENTERPRISE_DOCKER_PAT: ${{ secrets.DOCKERHUB_ACCESS_TOKEN }}