-
Notifications
You must be signed in to change notification settings - Fork 748
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
Fix Arm Virtual Hardware CI workflow #2479
base: main
Are you sure you want to change the base?
Conversation
on: | ||
schedule: | ||
- cron: '0 4 * * *' | ||
on: [push, pull_request] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As written, this will run on every push to any branch and on every PR event. I'm going to revise this to a more narrow subset. I'm thinking ci:run_full and postmerge makes sense for this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mans: this could probably be run pre-merge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MatthiasHertel80 - I can't modify your main branch, so I can't update this myself. I'm going to suggest this:
on:
# Allow manually triggering of the workflow.
workflow_dispatch: {}
workflow_call:
inputs:
trigger-sha:
required: true
type: string
secrets:
tflm-bot-token:
required: true
I want to test it out in CI first, then start requiring it for PRs.
"This PR is being marked as stale due to inactivity. Remove label or comment to prevent closure in 5 days." |
Description
The existing tests for Cortex-M and Arm Compiler 6 use Arm Virtual Hardware instances on AWS. Same function is now supplied in a docker image available on the Github Container Registry, allowing to execute tests directly on GH runners.
The image:
docker pull ghcr.io/arm-software/avh-mlops/arm-mlops-docker-licensed-community:latest