Commit Graph

154 Commits

Author SHA1 Message Date
Bobby R. Bruce
d49d0272ff misc,tests: Create Daily GPU Test timeout 2024-10-04 07:36:46 -07:00
Bobby R. Bruce
866b51a1cc misc,tests: Increase Weekly GPU test timeout
The Weekly GPU tests are failing due to a timeout but I found the testing
timeout was set to 5 hours and  we have been frequently close to reaching this
but have recently changes the test enought o consistently go over.

 The main two things that appear to have caused this are:

1. Moving the X86_VEGA compilation into the the same step as the running of
   the tests.
2. Reducing the number of threads per GitHub Actions runner, thus slowing
   job execution.

In addition we've added more tests to this weekly GPU suite though I don't
believe have got to running these tests yet. The timeout appears to
always been triggered before this.

This PR increases the timout to 3 days and moves the compilation into a
seperate step.
2024-10-04 06:12:13 -07:00
Bobby R. Bruce
c9408828a1 misc,tests: Revert "Test docker runners vs self-hosted"
This reverts commit 0792d94b6f.
2024-10-01 16:03:57 -07:00
Bobby R. Bruce
0792d94b6f misc,tests: Test docker runners vs self-hosted 2024-10-01 08:53:31 -07:00
Bobby R. Bruce
34f6bc4501 misc,tests: Fix caching in daily tests 2024-10-01 08:18:55 -07:00
Bobby R. Bruce
054790ad47 ext: Fix GCC v13+ comp of systemc due to problematic overloaded-virtual warn (#1576)
Fixes #1121 in line with the following suggesting:

https://github.com/gem5/gem5/issues/1121#issuecomment-2352743409
2024-09-26 14:32:20 -07:00
Bobby R. Bruce
a240ff8d32 misc,tests: Fix caching in daily tests 2024-09-26 11:12:52 -07:00
Bobby R. Bruce
e3fd7dcaec misc,tests: Remove cache store from dramsys test 2024-09-26 11:10:32 -07:00
Bobby R. Bruce
d74d550af4 misc,tests: Improve daily cache handling. 2024-09-23 14:20:29 -07:00
Bobby R. Bruce
6af68bcf81 tests,misc: Update weekly/daily caches 2024-09-23 13:16:01 -07:00
Bobby R. Bruce
5214c8b0cb misc, tests: Add missing build/ALL cache in daily-tests.yaml 2024-09-23 12:05:42 -07:00
Bobby R. Bruce
162ea1fa74 tests,misc: Add caching to daily and weekly test workflows 2024-09-23 12:01:57 -07:00
Bobby R. Bruce
41b02c5020 misc: Revert "Revert Dramsys Ubuntu to 22.04 to ..."
This reverts commit 52fbc8ebcf.

This commit used Ubuntu 22.04 instead of the typucal 24.04 as 24.04
has GCC v13 installed by default. GCC v13 (and new compilrs introduce a
'oerloaderdf-virtual' check that is triggered in systemc. Systemc
developers suggest this fix to proceed.
2024-09-23 05:20:30 -07:00
Bobby R. Bruce
9b83fc8736 misc: Add caching to weekly tests 2024-09-23 05:02:38 -07:00
Bobby R. Bruce
8aa58714c4 misc: Update docker-build.yaml to target default 2024-09-23 02:40:37 -07:00
Bobby R. Bruce
ba02266260 misc: Fix 'target' field in docker-build.yaml 2024-09-21 08:05:09 -07:00
Bobby R. Bruce
7a5b8d9a9c misc: Fix 'username' field in docker-build.yaml 2024-09-21 08:01:22 -07:00
Bobby R. Bruce
b47dc0d5e6 misc: Fix 'needs' field in docker-build.yaml 2024-09-21 07:41:58 -07:00
Bobby R. Bruce
c01aaf83f7 misc: Add matrix to docker-build.yaml 2024-09-21 07:39:06 -07:00
Bobby R. Bruce
c88f0d0097 misc: docker-build.yaml test 2024-09-21 07:27:42 -07:00
Bobby R. Bruce
50aac87c71 misc: docker-build.yaml fix 2024-09-21 07:17:32 -07:00
Bobby R. Bruce
cae4852606 misc: Fix docker-build.yaml (#1588)
This is an attempt to get the docker build workflow working
2024-09-21 07:08:12 -07:00
Bobby R. Bruce
a1105cf234 misc,github,tests: Remove gerrit change ID requirement (#1486) 2024-09-13 20:22:04 -07:00
Bobby R. Bruce
0a188850fe tests,gpu-compute: Fix artifact upload for GPU tests
actions/upload-artifact@v4 does not understand periods in artifact
names.

Change-Id: Ia272f9dcf9cb2213fb78b1814007921232395914
2024-08-22 05:49:07 -07:00
Bobby R. Bruce
28a6ca201b misc,tests: Remove Gerrit ID check from CI Workflow
Change-Id: I86933f3b315f3233e135de2e32498c1641f7443e
2024-08-22 04:24:56 -07:00
Bobby R. Bruce
e7442036a5 tests,gpu-compute: Fix Daily/Weekly GPU tests failures (#1485)
Without specifying the "gem5/gpu" directory, this test attempted to run
the entire test suite. This caused the daily and weekly tests to fail.
This change fixes this.
2024-08-20 14:18:51 -07:00
Bobby R. Bruce
0857442e44 util-docker: Cleanup, refactor, better document Dockerfiles (#1292)
* Removes the "docker-compose.yaml" in favor of "docker-bake.hcl". This
uses the `docker buildx` tool which has the advantage of enabling
multi-platformm builds where desired. By default all images are built
targeting `linux/arm64`, `linux/amd64` and `linux/riscv64` as targets
with the exception of the GPU images where only `linux/amd64` makes
sense.
* Remove unused/older Docker build targets (these can easily be re-added
but they were not regularly built or have any current usage).
* Update "README.md" to better describe these Dockerfiles and how they
are built.
* Simplify GCC and Clang compiler images. Each uses the Ubuntu 24.04 All
Deps image as a base then specialized the compiler on top.
* To simply things, all compiler versions are built from 24.04. This
means **narrowing the supported versions from GCC v10 to v14 and Clang
v14 to v18**.
* Fix some bugs in the "docker-bake.hcl" thus ensuring all targets may
be built from it.
* Cleanup the systemc and sst images: reducing their size and building
them off the common 24.04 ubuntu base image.
2024-08-20 09:45:47 -07:00
Bobby R. Bruce
f600db4a98 gpu-compute,tests: Move GPU tests to testlib (#1270)
A new host tag `gcn_gpu` has been added. This allows for selection of
those GPU tests which depend upon the gcn-gpu docker image to run.

In addition to this, the square GPU tests has been moved to the CI
tests. This ensures some GPU code is compiled and run on every PR.
2024-08-19 10:58:06 -07:00
Bobby R. Bruce
dcb04a72fc util-docker,tests: Remove Ubuntu 20.04 Docker
Change-Id: I1d4bbebaa4b6f064b5f40a95d066bbf092cf103f
2024-08-13 16:15:49 -07:00
Bobby R. Bruce
bef452ce72 misc,tests: Update supported GCC and Clang compilers
- GCC: v10 to v14
- Clang: v14 to v18

Change-Id: I6cd1686ffff0f08686a231b6b4936da343d53831
2024-08-13 16:09:06 -07:00
Bobby R. Bruce
3640559a12 misc,tests: Fix compiler tests (add missing ,) (#1459) 2024-08-13 06:54:12 -07:00
Bobby R. Bruce
eabb625870 util-docker,tests: Up clang support: >=v10
The compiler tests are failing to to a compile bug in Clang 7:
https://github.com/gem5/gem5/actions/runs/10170081794

Given Ubuntu 20.04 APT installs v10 by default (i.e., with `apt install
clang`). This is the oldest LTS Ubuntu version. It therefore seems
sensible to drop support for older (<v10) versions of clang.

Change-Id: I4c48223b80306422beac1464c09f03397c156ba1
2024-08-07 00:35:34 -07:00
Harshil Patel
679000f91d tests: remove dependant job (#1386) 2024-07-26 16:36:35 -07:00
Bobby R. Bruce
b11718536e misc,tests: Rm gem5 binary pre-build from dailys (#1383) 2024-07-26 09:35:08 -07:00
Harshil Patel
99aa8307b6 misc,tests: Revert "Attempt fix daily downloads (#1369)"
This reverts commit 97f6f3c4da.

Change-Id: I406c0c3d5429266da5ca037999247e21f1859ce5
2024-07-25 11:03:04 -07:00
Harshil Patel
bb2fd84111 misc,tests: Revert "Second attempt at fixing Daily test (#1373)"
This reverts commit e7d1c90aeb.

Change-Id: I5aed0c59d55c20b1774abcaa6396f6dcad11699b
2024-07-25 11:02:57 -07:00
Harshil Patel
22ebbd17dc misc,test: Revert "Third attempt at fixing Daily test"
This reverts commit 7722f84d1e.
2024-07-25 10:51:29 -07:00
Bobby R. Bruce
7722f84d1e misc,tests: Third attempt at fixing Daily test (#1375) 2024-07-24 05:41:22 -07:00
Bobby R. Bruce
e7d1c90aeb misc,tests: Second attempt at fixing Daily test (#1373)
This fix works by only downloading the gem5 binaries needed for each
test, instead of overwhelming the downloader by fetching them all at
once.
2024-07-23 10:21:53 -07:00
Bobby R. Bruce
97f6f3c4da misc,tests: Attempt fix daily downloads (#1369)
This tests attempts to infer which tests to download per job in the
matrix thereby significantly reducing the download times for each job.

Change-Id: I61b4f4b6410aa86de7437caf213499d805861e0c
2024-07-22 11:44:39 -07:00
Bobby R. Bruce
8d88a37ec9 misc: v4.0.0 -> v4 for actions/download-artifact
This was fixed to v4.0.0 under the assumption the flaky nature or the
daily-tests.yaml workflow was due to a later, minor v4 version causing
issue. This did not work. Ergo this patch reverts back to using the
latest v4 version.

Change-Id: I72b8811022268f34309de193445987dbe0085951
2024-07-01 15:08:19 -07:00
Bobby R. Bruce
801b86e860 misc: Remove artifact merge from daily-tests.yaml
This Workflow is flakey and it appears to msotly around the usage of the
the merging of all the gem5 builds into a single artifact. In attempt to
stabalize the workflow this merge step has been removed. ALL jobs now
download all gem5 binaries.

Change-Id: Ib1e9d82514c3d5e5af9de974a477e213f8af2aaa
2024-07-01 15:05:08 -07:00
Bobby R. Bruce
3142464ff7 misc: Add 'scheduler.yaml' workflow (#1307)
This is made to run on the 'stable' branch to schedule workflow runs on
the `develop` branch. This solves the problem of GitHub Workflows being
scheduled to only run on 'stable' branch' thus ignoring changes made to
them on 'develop'

With this schedule we no longer need to force a checkout of 'develop' in
the workflows. As such these have been removed.

The scheduled workflows are now triggered via "workflow_dispatch" via
the "scheduler.yaml" workflow
2024-07-01 12:36:56 -07:00
Bobby R. Bruce
e5414a80a3 misc, tests: Fix missing 's' in GPU tests (#1305)
This caused the weekly tests to fail. It's 'tests' not 'test'.
2024-07-01 07:37:32 -07:00
Bobby R. Bruce
09781fd78f misc: Update dummy jobs for workflows
These give us clear indications if a workflow has passed or failed.

Change-Id: If61b9ac5dc4d2da54b4ad68e427b149bbcb4a30b
2024-06-22 12:58:35 -07:00
Bobby R. Bruce
261490f23c misc,tests: Revert merge version to 'v4' from 'v4.0.0'
'v4.0.0' wasn't working. The following error was occurred:

```
Can't find 'action.yml', 'action.yaml' or 'Dockerfile' for action 'actions/upload-artifact/merge@v4.0.0'.
```

Change-Id: I658b0fe292df029501fbc1286acb06f4014ae4e1
2024-06-12 00:14:27 -07:00
Bobby R. Bruce
8fc4d3f793 misc,tests: Update daily test artifact actions to v4.0.0
Change-Id: I711fa36639e925ce958e0484a31ee6a4dde87dbe
2024-06-11 15:43:40 -07:00
Bobby R. Bruce
3b7307182f misc: Fix daily-tests
1. Typo in container.
2. Add compression level to minimize size of generated artifact.

Change-Id: I854e814162fb434ad50a64e3070b943905e4134b
2024-05-30 10:15:35 -07:00
Bobby R. Bruce
7c1207d5c4 misc: Another attempt to fix the merge-upload in for daily (#1188)
Change-Id: I6a6064ec3b5be4ac1f7d6cd3c2f6c0ca62d2cfcd
2024-05-30 07:45:35 -07:00
Bobby R. Bruce
65b86cfac9 misc: Fix daily tests merge-artifacts (#1184) 2024-05-30 04:27:40 -07:00