gem5-dev@gem5.org

The gem5 Developer List

View all threads

[S] Change in gem5/gem5[stable]: misc: Adjust compiler tests and CI tests

MJ
Melissa Jost (Gerrit)
Thu, Jun 15, 2023 11:24 PM

Melissa Jost has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/71725?usp=email )

Change subject: misc: Adjust compiler tests and CI tests
......................................................................

misc: Adjust compiler tests and CI tests

This adds a job level timeout for the compiler tests, allows
them to run weekly instead of daily, and adds a workflow
dispatch option.  It also adds a ready for review option
to the CI tests to run when someone converts a draft
pull request.

Change-Id: Id32b5f7da90745d18de2e550bd48d32ba45fb4d9

M .github/workflows/ci-tests.yaml
M .github/workflows/compiler-tests.yaml
2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/ci-tests.yaml
b/.github/workflows/ci-tests.yaml
index b1c5a1a..61ba4b5 100644
--- a/.github/workflows/ci-tests.yaml
+++ b/.github/workflows/ci-tests.yaml
@@ -4,7 +4,7 @@

on:
pull_request:

  • types: [opened, edited, synchronize]
  • types: [opened, edited, synchronize, ready_for_review]

jobs:
diff --git a/.github/workflows/compiler-tests.yaml
b/.github/workflows/compiler-tests.yaml
index 013a710..ad8c8a5 100644
--- a/.github/workflows/compiler-tests.yaml
+++ b/.github/workflows/compiler-tests.yaml
@@ -5,9 +5,11 @@

on:

  • Runs every day from 7AM UTC

  • Runs every Friday from 7AM UTC

    schedule:
    • cron:  '0 7 * * *'
    • cron:  '00 7 * * 5'
  • Allows us to manually start workflow for testing

  • workflow_dispatch:

jobs:
# replication of compiler-tests.sh
@@ -17,6 +19,7 @@
image: [gcc-version-12, gcc-version-11, gcc-version-10,
gcc-version-9, gcc-version-8, gcc-version-7, clang-version-14,
clang-version-13, clang-version-12, clang-version-11, clang-version-10,
clang-version-9, clang-version-8, clang-version-7, clang-version-6.0,
ubuntu-18.04_all-dependencies, ubuntu-20.04_all-dependencies,
ubuntu-22.04_all-dependencies, ubuntu-22.04_min-dependencies]
opts: [.opt, .fast]
runs-on: [self-hosted, linux, x64, run]

  • timeout-minutes: 2880    # 48 hours
    container: gcr.io/gem5-test/${{ matrix.image }}:latest
    steps:
    • uses: actions/checkout@v3
      @@ -36,6 +39,7 @@
      image: [gcc-version-12, clang-version-14]
      opts: [.opt, .fast]
      runs-on: [self-hosted, linux, x64, run]
  • timeout-minutes: 2880    # 48 hours
    container: gcr.io/gem5-test/${{ matrix.image }}:latest
    steps:
    • uses: actions/checkout@v3

--
To view, visit
https://gem5-review.googlesource.com/c/public/gem5/+/71725?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings?usp=email

Gerrit-MessageType: newchange
Gerrit-Project: public/gem5
Gerrit-Branch: stable
Gerrit-Change-Id: Id32b5f7da90745d18de2e550bd48d32ba45fb4d9
Gerrit-Change-Number: 71725
Gerrit-PatchSet: 1
Gerrit-Owner: Melissa Jost melissakjost@gmail.com

Melissa Jost has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/71725?usp=email ) Change subject: misc: Adjust compiler tests and CI tests ...................................................................... misc: Adjust compiler tests and CI tests This adds a job level timeout for the compiler tests, allows them to run weekly instead of daily, and adds a workflow dispatch option. It also adds a ready for review option to the CI tests to run when someone converts a draft pull request. Change-Id: Id32b5f7da90745d18de2e550bd48d32ba45fb4d9 --- M .github/workflows/ci-tests.yaml M .github/workflows/compiler-tests.yaml 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/.github/workflows/ci-tests.yaml b/.github/workflows/ci-tests.yaml index b1c5a1a..61ba4b5 100644 --- a/.github/workflows/ci-tests.yaml +++ b/.github/workflows/ci-tests.yaml @@ -4,7 +4,7 @@ on: pull_request: - types: [opened, edited, synchronize] + types: [opened, edited, synchronize, ready_for_review] jobs: diff --git a/.github/workflows/compiler-tests.yaml b/.github/workflows/compiler-tests.yaml index 013a710..ad8c8a5 100644 --- a/.github/workflows/compiler-tests.yaml +++ b/.github/workflows/compiler-tests.yaml @@ -5,9 +5,11 @@ on: - # Runs every day from 7AM UTC + # Runs every Friday from 7AM UTC schedule: - - cron: '0 7 * * *' + - cron: '00 7 * * 5' + # Allows us to manually start workflow for testing + workflow_dispatch: jobs: # replication of compiler-tests.sh @@ -17,6 +19,7 @@ image: [gcc-version-12, gcc-version-11, gcc-version-10, gcc-version-9, gcc-version-8, gcc-version-7, clang-version-14, clang-version-13, clang-version-12, clang-version-11, clang-version-10, clang-version-9, clang-version-8, clang-version-7, clang-version-6.0, ubuntu-18.04_all-dependencies, ubuntu-20.04_all-dependencies, ubuntu-22.04_all-dependencies, ubuntu-22.04_min-dependencies] opts: [.opt, .fast] runs-on: [self-hosted, linux, x64, run] + timeout-minutes: 2880 # 48 hours container: gcr.io/gem5-test/${{ matrix.image }}:latest steps: - uses: actions/checkout@v3 @@ -36,6 +39,7 @@ image: [gcc-version-12, clang-version-14] opts: [.opt, .fast] runs-on: [self-hosted, linux, x64, run] + timeout-minutes: 2880 # 48 hours container: gcr.io/gem5-test/${{ matrix.image }}:latest steps: - uses: actions/checkout@v3 -- To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/71725?usp=email To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: public/gem5 Gerrit-Branch: stable Gerrit-Change-Id: Id32b5f7da90745d18de2e550bd48d32ba45fb4d9 Gerrit-Change-Number: 71725 Gerrit-PatchSet: 1 Gerrit-Owner: Melissa Jost <melissakjost@gmail.com>