Skip to content

Commit dfd8ec9

Browse files
authored
Merge pull request #1734 from oneapi-src/benie/gha-revert-e2e-workflow
[GHA] Revert e2e workflow & disable comment
2 parents 4f13cd1 + 6ea9078 commit dfd8ec9

File tree

3 files changed

+45
-56
lines changed

3 files changed

+45
-56
lines changed

.github/workflows/cmake.yml

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ concurrency:
88

99
permissions:
1010
contents: read
11+
pull-requests: write
1112

1213
jobs:
1314
ubuntu-build:
@@ -191,6 +192,30 @@ jobs:
191192
with:
192193
name: NATIVE_CPU
193194

195+
e2e-level-zero:
196+
name: E2E L0
197+
permissions:
198+
contents: read
199+
pull-requests: write
200+
needs: [ubuntu-build, level-zero]
201+
uses: ./.github/workflows/e2e_level_zero.yml
202+
203+
e2e-opencl:
204+
name: E2E OpenCL
205+
permissions:
206+
contents: read
207+
pull-requests: write
208+
needs: [ubuntu-build, opencl]
209+
uses: ./.github/workflows/e2e_opencl.yml
210+
211+
e2e-cuda:
212+
name: E2E CUDA
213+
permissions:
214+
contents: read
215+
pull-requests: write
216+
needs: [ubuntu-build, cuda]
217+
uses: ./.github/workflows/e2e_cuda.yml
218+
194219
windows-build:
195220
name: Build - Windows
196221
strategy:

.github/workflows/e2e.yml

Lines changed: 0 additions & 39 deletions
This file was deleted.

.github/workflows/e2e_core.yml

Lines changed: 20 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -178,20 +178,23 @@ jobs:
178178
id: tests
179179
run: ninja -C build-e2e check-sycl-e2e
180180

181-
- name: Add comment to PR
182-
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
183-
if: ${{ always() }}
184-
with:
185-
script: |
186-
const adapter = '${{ matrix.adapter.name }}';
187-
const url = '${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}';
188-
const test_status = '${{ steps.tests.outcome }}';
189-
const job_status = '${{ job.status }}';
190-
const body = `E2E ${adapter} build:\n${url}\nJob status: ${job_status}. Test status: ${test_status}`;
191-
192-
github.rest.issues.createComment({
193-
issue_number: context.issue.number,
194-
owner: context.repo.owner,
195-
repo: context.repo.repo,
196-
body: body
197-
})
181+
# FIXME: Requires pull-request: write permissions but this is only granted
182+
# on pull requests from forks if using pull_request_target workflow
183+
# trigger but not the pull_request trigger..
184+
# - name: Add comment to PR
185+
# uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
186+
# if: ${{ always() }}
187+
# with:
188+
# script: |
189+
# const adapter = '${{ matrix.adapter.name }}';
190+
# const url = '${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}';
191+
# const test_status = '${{ steps.tests.outcome }}';
192+
# const job_status = '${{ job.status }}';
193+
# const body = `E2E ${adapter} build:\n${url}\nJob status: ${job_status}. Test status: ${test_status}`;
194+
195+
# github.rest.issues.createComment({
196+
# issue_number: context.issue.number,
197+
# owner: context.repo.owner,
198+
# repo: context.repo.repo,
199+
# body: body
200+
# })

0 commit comments

Comments
 (0)