From 6a4b2fc7f37c47c95f46738c8b80caf3e30560db Mon Sep 17 00:00:00 2001 From: Atif Ali Date: Thu, 26 Oct 2023 15:41:22 -0400 Subject: [PATCH] added test failure comment --- .github/workflows/integrated-test.yml | 2 +- .github/workflows/pr-ci.yml | 46 +++++++++++++++++++++++---- 2 files changed, 40 insertions(+), 8 deletions(-) diff --git a/.github/workflows/integrated-test.yml b/.github/workflows/integrated-test.yml index 3c34eae58f..5f60f808f9 100644 --- a/.github/workflows/integrated-test.yml +++ b/.github/workflows/integrated-test.yml @@ -54,7 +54,7 @@ jobs: - name: maven-settings uses: s4u/maven-settings-action@v2 with: - githubServer: false + githubServer: true - name: ghcr login uses: redhat-actions/podman-login@v1 with: diff --git a/.github/workflows/pr-ci.yml b/.github/workflows/pr-ci.yml index 035aa77c4c..52a0b6fbca 100644 --- a/.github/workflows/pr-ci.yml +++ b/.github/workflows/pr-ci.yml @@ -180,26 +180,24 @@ jobs: needs: [checkout-branch, push-to-ghcr] strategy: matrix: - arch: [amd64, arm64] + arch: [amd64] uses: ./.github/workflows/integrated-test.yml with: build-arch: ${{ matrix.arch }} pr-number: ${{ github.event.issue.number }} sha-value: ${{ needs.checkout-branch.outputs.PR_head_sha }} - skip-itests: ${{ matrix.arch == 'arm64' }} - + retest-integration: needs: [checkout-branch] if: contains(github.event.comment.body, '/retest') strategy: matrix: - arch: [amd64, arm64] + arch: [md64] uses: ./.github/workflows/integrated-test.yml with: build-arch: ${{ matrix.arch }} pr-number: ${{ github.event.issue.number }} sha-value: ${{ needs.checkout-branch.outputs.PR_head_sha }} - skip-itests: ${{ matrix.arch == 'arm64' }} integration-test-pass: runs-on: ubuntu-latest @@ -210,7 +208,7 @@ jobs: with: script: | const runURL = `https://github.com/${context.repo.owner}/${context.repo.repo}/actions/runs/${{ github.run_id }}`; - const commentBody = `both build and itest completed successfully ✅. \n[View Actions Run](${runURL}).`; + const commentBody = `\`/build_test\` completed successfully ✅. \n[View Actions Run](${runURL}).`; github.rest.issues.createComment({ issue_number: context.issue.number, owner: context.repo.owner, @@ -227,7 +225,41 @@ jobs: with: script: | const runURL = `https://github.com/${context.repo.owner}/${context.repo.repo}/actions/runs/${{ github.run_id }}`; - const commentBody = `retest Integration: All tests pass ✅. \n[View Actions Run](${runURL}).`; + const commentBody = `\`/retest\` Integration: All tests pass ✅. \n[View Actions Run](${runURL}).`; + github.rest.issues.createComment({ + issue_number: context.issue.number, + owner: context.repo.owner, + repo: context.repo.repo, + body: commentBody + }); + + comment-integration-test-fail: + if: (always() && contains(needs.*.result, 'failure')) + needs: [integration-test] + runs-on: ubuntu-latest + steps: + - uses: actions/github-script@v6 + with: + script: | + const runURL = `https://github.com/${context.repo.owner}/${context.repo.repo}/actions/runs/${{ github.run_id }}`; + const commentBody = `\`/build_test\` : At least one test failed ❌. \n[View Actions Run](${runURL}).`; + github.rest.issues.createComment({ + issue_number: context.issue.number, + owner: context.repo.owner, + repo: context.repo.repo, + body: commentBody + }); + + comment-retest-integration-fail: + if: (always() && contains(needs.*.result, 'failure')) + needs: [retest-integration] + runs-on: ubuntu-latest + steps: + - uses: actions/github-script@v6 + with: + script: | + const runURL = `https://github.com/${context.repo.owner}/${context.repo.repo}/actions/runs/${{ github.run_id }}`; + const commentBody = `\`/retest\` Integration: At least one test failed ❌. \n[View Actions Run](${runURL}).`; github.rest.issues.createComment({ issue_number: context.issue.number, owner: context.repo.owner,