From 91a912093ebafb92ba78f7a7868835aa0c0ef6eb Mon Sep 17 00:00:00 2001
From: Robert Cohn <rscohn2@gmail.com>
Date: Fri, 7 Jun 2024 18:23:46 -0400
Subject: [PATCH] Revert "Selective CI basecd on changes to a domain" (#509)

---
 .github/workflows/pr.yml | 53 +++++++++++-----------------------------
 1 file changed, 14 insertions(+), 39 deletions(-)

diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml
index 6b263739f..7c22d384f 100644
--- a/.github/workflows/pr.yml
+++ b/.github/workflows/pr.yml
@@ -21,22 +21,23 @@ jobs:
       matrix:
         include:
         - config: portBLAS
-          domain: blas
-          build_options: -DREF_BLAS_ROOT=${PWD}/lapack/install -DENABLE_PORTBLAS_BACKEND=ON -DENABLE_MKLCPU_BACKEND=OFF -DPORTBLAS_TUNING_TARGET=INTEL_CPU
+          options: -DTARGET_DOMAINS=blas -DREF_BLAS_ROOT=${PWD}/lapack/install -DENABLE_PORTBLAS_BACKEND=ON -DENABLE_MKLCPU_BACKEND=OFF -DPORTBLAS_TUNING_TARGET=INTEL_CPU
+          tests: '.*'
         - config: portFFT
-          domain: dft
-          build_options: -DENABLE_PORTFFT_BACKEND=ON -DENABLE_MKLCPU_BACKEND=OFF -DCMAKE_CXX_FLAGS="-fsycl -fsycl-targets=spir64"
-          test_options: -R 'DFT/CT/.*ComputeTests_in_place_COMPLEX.COMPLEX_SINGLE_in_place_buffer.sizes_8_batches_1*'
+          options: -DENABLE_PORTFFT_BACKEND=ON -DENABLE_MKLCPU_BACKEND=OFF -DTARGET_DOMAINS=dft -DCMAKE_CXX_FLAGS="-fsycl -fsycl-targets=spir64"
+          tests: 'DFT/CT/.*ComputeTests_in_place_COMPLEX.COMPLEX_SINGLE_in_place_buffer.sizes_8_batches_1*'
         - config: MKL BLAS
-          domain: blas
-          build_options: -DREF_BLAS_ROOT=${PWD}/lapack/install
+          options: -DTARGET_DOMAINS=blas -DREF_BLAS_ROOT=${PWD}/lapack/install
+          tests: '.*'
         - config: MKL DFT
-          domain: dft
+          options: -DTARGET_DOMAINS=dft
+          tests: '.*'
         - config: MKL LAPACK
-          domain: lapack
-          build_options: -DREF_LAPACK_ROOT=${PWD}/lapack/install
+          options: -DTARGET_DOMAINS=lapack -DREF_LAPACK_ROOT=${PWD}/lapack/install
+          tests: '.*'
         - config: MKL RNG
-          domain: rng
+          options: -DTARGET_DOMAINS=rng
+          tests: '.*'
     name: unit tests ${{ matrix.config }} CPU
     steps:
     - uses: actions/checkout@44c2b7a8a4ea60a981eaca3cf939b5f4305c123b # v4.1.5
@@ -57,44 +58,18 @@ jobs:
         # 64 bit int
         cmake ${SHARED_OPT} -DBUILD_INDEX64=on -B lapack/build64
         cmake --build lapack/build64 ${PARALLEL} --target install
-    - name: Test if the changes affect this domain
-      id: test_domain
-      uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
-      with:
-        script: |
-          function matchesPattern(filePaths) {
-            console.log('files:', filePaths)
-            // These directories contain domain specific code
-            const dirs = '(tests/unit_tests|examples|src|include/oneapi/mkl)'
-            const domains = '(blas|lapack|rng|dft)'
-            const domain = '${{ matrix.domain }}'
-            // matches changes to the domain of interest or non domain-specific code
-            const re = new RegExp(`^(${dirs}/${domain}|(?!${dirs}/${domains}))`);
-            return filePaths.some(filePath => re.test(filePath));
-          }
-          const pr = context.payload.pull_request
-          const compare = await github.rest.repos.compareCommits({
-            owner: context.repo.owner,
-            repo: context.repo.repo,
-            base: pr.base.sha,
-            head: pr.head.sha
-          });
-          return matchesPattern(compare.data.files.map((f) => f.filename));
     - name: Install oneapi
-      if: steps.test_domain.outputs.result == 'true'
       uses: rscohn2/setup-oneapi@2ad0cf6b74bc2426bdcee825cf88f9db719dd727 # v0.1.0
       with:
         components: |
           icx@2024.1.0
           mkl@2024.1.0
     - name: Configure/Build for a domain
-      if: steps.test_domain.outputs.result == 'true'
       run: |
         source /opt/intel/oneapi/setvars.sh
-        cmake -DTARGET_DOMAINS=${{ matrix.domain }} -DENABLE_MKLGPU_BACKEND=off -DCMAKE_VERBOSE_MAKEFILE=on ${{ matrix.build_options }} -B build
+        cmake -DENABLE_MKLGPU_BACKEND=off -DCMAKE_VERBOSE_MAKEFILE=on ${{ matrix.options }} -B build
         cmake --build build ${PARALLEL}
     - name: Run tests
-      if: steps.test_domain.outputs.result == 'true'
       run: |
         source /opt/intel/oneapi/setvars.sh
-        ctest --test-dir build ${{ matrix.test_options }}
+        ctest --test-dir build -R ${{ matrix.tests }}