Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ock artefact building #574

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

coldav
Copy link
Collaborator

@coldav coldav commented Oct 23, 2024

Overview

Add ock artefact building, icd building and tornado run for planned testing

Reason for change

Need to port some of our local testing to planned to improve public level of testing.

Description of change

Added planned testing which is intended to run long running scheduled
tests. Added a base which they can all use which allows all jobs to
follow a similar method of using a matrix, a named target and a flag
which dictates what tests are enabled.

icd, ock artefacts and tornado are done as part of this.

@coldav coldav requested a review from a team as a code owner October 23, 2024 15:55
@coldav coldav force-pushed the colin/build_ock_artefact branch 29 times, most recently from 96959be to 8cd5f1e Compare October 28, 2024 10:26
@coldav coldav force-pushed the colin/build_ock_artefact branch 23 times, most recently from 685ad44 to 87c4889 Compare November 21, 2024 16:19
@coldav coldav changed the title DRAFT: Add ock artefact building Add ock artefact building Nov 21, 2024
@@ -25,6 +25,7 @@ permissions: {}

jobs:
analyze-host-x86_64-release:
if: false
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TBD remove

@@ -72,6 +73,7 @@ jobs:
category: "/language:c-cpp"

analyze-riscv-m1:
if: false
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TBD remove

sparse-checkout: |
platform
.github
# The following can also be done with the matrix with exclude: to limit the targets to what we want
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove comment

call_planned:
# This makes the diagram too big if we post much here so P_ for pull request.
name: P_
if: ${{ github.event_name == 'pull_request' }}
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove if:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant