Commit Graph

118 Commits

Author SHA1 Message Date
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
Bobby R. Bruce
4acc20dac1 misc,tests: Download all gem5 bins via one artifact (#1178)
The Daily Tests are failing when downloading artifacts as part of the
`testlib-long-tests` matrix:
https://github.com/gem5/gem5/actions/runs/9250821764/job/25448583827.

It _could_ be that since upgrading to `actions/download@v4`, we're
hitting a limit as the `testlib-long-tests` are downloading every gem5
binary compiled in the `build-gem5` step, each with it's own
`actions/download` step, for every test.

This change adds a small job after `build-gem5` which creates a merged
artifact containing all the gem5 binaries then uses this to lessen the
number of times this action is called in such a short period of time.

Even if the bug still persists, this solution is neater than what was
there previously.
2024-05-28 12:55:30 -07:00
Bobby R. Bruce
52fbc8ebcf misc: Revert Dramsys Ubuntu to 22.04 to compile in gcc <13 (#1146)
Until https://github.com/gem5/gem5/issues/1121 is fixed, this change
will ensure our Weekly tests pass.
2024-05-21 10:57:16 -07:00
Bobby R. Bruce
ba1c22f143 misc,tests: Remove gcc-8 from compiler tests
GCC Version 10 is no longer supported by the gem5 project.

Change-Id: If657654299c1a018764d5f92e814ed5cd18c50f0
2024-05-20 06:27:45 -07:00
Bobby R. Bruce
dd2689905f misc,tests: Remove zip step from Workflows (#1048)
This is not needed with upload-artifact v4 directories are archived and
compressed by default.

This zip step was also causing Daily/Weekly test failures due to not
running `apt update` before the `apt install` for the zip utility. Ergo
this patch fixes these errors.
2024-04-21 09:15:20 -07:00
Bobby R. Bruce
29d56d3d65 misc,tests: Add Pyunit tests to CI GitHub Action Workflow
Due to an oversight, the PyUnit tests were not being run as part of the
gem5 CI tests. This was because they are located in "tests/pyunit"
instead of "tests/gem5", where the CI GitHub Action workflow searched
for tests to run and where all other tests reside.

This adds the Pyunit tests as a seperate job in the CI GitHub Action's
workflow.

Change-Id: I63d93571fde11c19bf3d281c034eddf4b455ae4e
2024-04-18 17:33:30 -07:00
Bobby R. Bruce
cbf0334762 misc: Fix jq install for testlib-quick-matrix (#1038) 2024-04-18 17:30:53 -07:00
Bobby R. Bruce
1aa0bf8ec6 tests,github: Update CI Tests' GitHub Actions versions (#1021) 2024-04-15 13:35:33 -07:00
Bobby R. Bruce
56a2346b8d tests,util-docker,github: Add Ubuntu 24.04 Docker image & updated tests/actions to use it (#1018)
This ensures gem5 compiles and runs in 24.04 environments. A necessary
PR, for ensuring gem5 support Ubuntu 24.04 (related issue: #909)
2024-04-15 13:34:22 -07:00
Bobby R. Bruce
630f3822b8 github: Update 'ubuntu-22.04' to 'ubuntu-latest' (#1022)
There was some inconsistency in the GitHub Workflow files on using
'ubuntu-latest' (which gets the latest Ubuntu version) or
'ubuntu-22.04'. To keep things consistent 'ubuntu-latest' is now used in
all cases. This also saves us updating workloads upon release of a new
Ubuntu version.
2024-04-15 09:55:56 -07:00
Bobby R. Bruce
b986c542ca tests,misc: Set pre-commit/action to v3.0.1
v3.0.0 of pre-commit/action caused a deprecation warning in actions.
v3.0.1 was released to deal with this.

Change-Id: Ib5654e465565ad4356754ac097983aec4166b98f
2024-04-13 20:30:34 -07:00
Bobby R. Bruce
3f45a2d08d misc,tests: Up actions/setup-python version to v5
This was causing a deprecation warning in GitHub Actions.

Change-Id: I9ab147acf12e3763ab731769468ce5b1dc5e4dea
2024-04-13 20:20:26 -07:00
Bobby R. Bruce
05bc85aa9b misc: Update Update GitHub Actions to use 24.04 over 22.04
This change ensures all our tests run on our most recent supported LTS
release of Ubuntu.

In the case of compiler tests we still test 22.04 all-dep but test 24.04
all-dep and min-dep (i.e., we drop 22.04 min-dep as it's somewhat
redundant).

Change-Id: I63666d1017594b496523a48e5112a8994f57885f
2024-04-13 14:13:35 -07:00
Bobby R. Bruce
b310ddf79a misc: Upgrade {download/upload}-artifact to v4
v3 was causing a 'Node.js 16 actions are deprecated' error.

Note: download-artifact@v4 must be used with upload-artifact@v4 and
vice-versa.

Change-Id: Icb8ab6d27aed4557be95ce31dd89d4655010968e
2024-03-30 01:22:28 -07:00
Bobby R. Bruce
21a00be6eb misc: Fix 'checkout@v2' to 'checkout@v4'
This caused a 'Node.js 16 actions are deprecated;' error.

With this commit all our checkout actions are set to '@v4'.

Change-Id: I0f931bf7967f49ee44b7bf1d6a56e19f017fb948
2024-03-30 01:14:57 -07:00
Bobby R. Bruce
1f0075bdbd misc: Remove incorrect 'working-directory' in Weekly tests
"build/VEGA_X86/gem5.opt" is not available in directory "hip". `${
github.workspace}` is default should be run from there. This patch fixes
this.

Change-Id: I99875270c77dde92d3ec2ae0a07760905eaf903e
2024-03-11 06:09:47 -07:00
Bobby R. Bruce
2bec61e69a misc: Revert download-artifact v4 to v3
This was accidently added in a previous commit. It breaks downloading.

Change-Id: I790b184e852ff732ea1106cb2cde79a83828bdcf
2024-03-11 05:59:37 -07:00
Harshil Patel
920497c19f tests: Add compiler test for gcc 13 (#858)
Change-Id: I41bdf3ab7ffff21c4148ef17fc5229b5597ec953
2024-02-26 18:03:14 -05:00
Harshil Patel
de0342128c tests: move to obtain-resources from wget (#845) 2024-02-06 09:34:03 -08:00
Bobby R. Bruce
c7426f9427 misc: Add 'workflow_dispatch' to daily tests (#850)
This allows us to manually trigger daily test runs rather than wait for
the scheduled time. This can be useful in cases where a fix for a broken
test is pushed and we wish to verify it works as intended ASAP.
2024-02-06 09:32:31 -08:00
Bobby R. Bruce
6f1d9b47e9 misc: Update actions/checkout from v3 to v4 (#836)
The `checkout` action now has a v4. v3 utilizes Node.js 16 which is now
deprecated by GitHub actions. Migrating to v4 is therefore encouraged.
2024-02-05 08:54:32 -08:00
Harshil Patel
858acacb20 tests: fix wget link for gpu tests (#840) 2024-02-02 10:34:41 -08:00
Harshil Patel
b5fae2f620 tests: Switch to vega_x86 from gcn3_x86 in daily tests (#817)
Change-Id: Ic2ed8cc4488ddd361b5773b91100d806b94f1b8a
2024-02-01 09:06:04 -08:00
Harshil Patel
c92ddf90e6 tests: update binaries for gpu tests
Change-Id: I057f76e472bc0f9fdeacd59238a05980389c92c8
2024-01-31 13:37:48 -08:00
Harshil Patel
76c3c02acb tests: remove GCN3_X86 from compiler tests (#819)
Change-Id: Ibb75e08abb9051b70e474d721fbafd71957db701
2024-01-30 15:55:15 -08:00
Harshil Patel
47369e786a tests: Switch to vega_x86 from gcn3_x86 in daily tests
Change-Id: Ic2ed8cc4488ddd361b5773b91100d806b94f1b8a
2024-01-29 10:58:07 -08:00
Harshil Patel
5a7d61d990 misc: move dependabot.yml to .github (#812)
Change-Id: I5c882afd1e15420b8fcdcc14895a77b275aedc4e
2024-01-29 10:07:32 -08:00
Ivana Mitrovic
8a6804231c misc: Added dependabot config file (#767)
- Added a yaml file to make dependabot target develop instead of stable.
2024-01-25 19:25:51 -08:00
Harshil Patel
7cf5c8c840 misc: Added dependabot config file
- Added a yaml file to make dependabot target develop instead of stable.

Change-Id: I5b28c06960c5a346b40e2af8f9284b11d9cc07cd
2024-01-25 08:57:32 -08:00
Bobby R. Bruce
5f767d7836 misc: Fixing comment indentation in weekly-tests.yaml
Change-Id: I047ef921703e635b37bacb54cd5b091c2a41b1d3
2024-01-18 15:55:25 -08:00
Bobby R. Bruce
e0c5f95110 misc: Merge Weekly GPU tests into Weekly Tests
This seperation was only for convenience while GPU tests were under
development and rapidly changing. This test merges the GPU tests into
the weekly tests where they belong.

Change-Id: I0e7118e863dba51334de89b3bbc3592374ef63ec
2023-12-03 13:46:55 -08:00
Harshil Patel
9d108826b0 tests: fix artifact reference in HACC tests (#638)
Change-Id: I181f17a598885b59c186ff7e810d5b8b3b304e05
2023-12-01 15:19:17 -08:00
Harshil Patel
b59a398312 tests: change HACC tests to VEGA_X86
Change-Id: I846229db1ab1480d79471c717b714698c3132df9
2023-11-29 16:02:30 -08:00
Harshil Patel
23cadf0886 tests: switch lulesh to use vega_x86
Change-Id: Ifbf0fdfd7d8c2bbaad0b6094090acecd1cb8055c
2023-11-29 07:41:51 -08:00
Bobby R. Bruce
d4b7c8a26d Merge branch 'develop' into develop-kconfig 2023-11-27 09:39:08 -08:00
Bobby R. Bruce
0f6eabe8c9 ext,github,tests: Update DRAMSys tests to v5.0 and handle new dependencies (#577)
#525 Updated DRAMSys to v5.0. This PR further improves v5.0
inforporation into gem5 by better managing its new dependencies and
updating the DRAMSys tests to use v5.0.

This PR:

1. Adds a check which throws warning if DRAMSys cannot be build due to a
missing `cmake` instead of failing with a build error. `cmake` is not a
hard gem5 requirement. It is only required to build DRAMSys in the cases
it is required. It is therefore prudent to not fail a build in cases
`cmake` is not present on the host system.
2. Updates the "all-dependency" Docker images to include the optional
dependencies `git-lfs` (needed to clone the DRAMSys repo when running
the command outlined in ext/dramsys/README -- introduced in #525) and
`cmake` (needed to build DRAMSys).
3. Updates the Weekly workflow's `dramsys-tests`' `Checkout DRAMSys` job
to clone DRAMSys in the same manner as outlined in ext/dramsys/README.
This ensures the `dram-systests` test the instructions we give users.
4. `.gitignore` is added to ext/dramsys to ignore the
ext/dramsys/DRAMSys directory when cloned for building and integration
into gem5.

(2.) Should fix our failing weekly tests:
https://github.com/gem5/gem5/actions/runs/6912511984/job/18808339821 and
(3.) will ensure the changes introduced in #525 are tested.
2023-11-27 09:37:11 -08:00
Harshil Patel
1de992bc75 tests: fix lulesh (#600)
- fixed the broken command that was causing lulesh to fail the run
Change-Id: I4e8a310f153d86deb8829f41b5ddd0c317df23cb
2023-11-27 07:42:59 -08:00