Skip to content

Commit

Permalink
[CI] Allow detect changes and detect Arc tests to run on more Linux m…
Browse files Browse the repository at this point in the history
…achines (#13992)

Even the Windows detect changes job requires a Linux build machine
today, and that can cause a bottleneck.

We may still get bottlenecked in the Linux tree because of the low
number of Linux build systems, but there are more Windows build systems,
so that tree might see a speedup.

We can hit a similar thing for the determine arc tests job, so update
that too.

Also remove a comment that seems out of date.

---------

Signed-off-by: Sarnie, Nick <nick.sarnie@intel.com>
  • Loading branch information
sarnex authored May 31, 2024
1 parent 1d24713 commit 692c1d6
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
4 changes: 1 addition & 3 deletions .github/workflows/sycl-detect-changes.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,7 @@ permissions: read-all
jobs:
need_check:
name: Decide which tests could be affected by the changes
# Github's ubuntu-* runners are slow to allocate. Use our CUDA runner since
# we don't use it for anything right now.
runs-on: [Linux, build]
runs-on: [Linux, aux-tasks]
timeout-minutes: 3
outputs:
filters: ${{ steps.result.outputs.result }}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/sycl-linux-precommit.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ jobs:
name: Decide which Arc tests to run
needs: [build, detect_changes]
if: ${{ always() && !cancelled() && needs.build.outputs.build_conclusion == 'success' }}
runs-on: [Linux, build]
runs-on: [Linux, aux-tasks]
timeout-minutes: 3
outputs:
arc_tests: ${{ steps.arc_tests.outputs.arc_tests }}
Expand Down

0 comments on commit 692c1d6

Please sign in to comment.