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

Adding ApPredict GitHub Actions workflow on self-hosted Ubuntu 20.04 runners #11

Merged
merged 10 commits into from
Jun 8, 2022

Conversation

kwabenantim
Copy link
Member

Description

  • This PR adds a GitHub Actions workflow to build and run the ApPredict project test pack with a self-hosted runner on Ubuntu 20.04.
  • The workflow, (1) is scheduled to run every midnight, (2) is triggered when a new commit is made to the ApPredict repository, and (3) can be triggered manually.

Motivation and Context

Testing

Additional Notes

  • A self-hosted runner should be added, preferably at the organization level so multiple repositories can share it: Adding a self-hosted runner to an organization (this will generate a token that is required for configuring the runner).
  • This workflow assumes that self-hosted runners have the required dependencies for building and testing Chaste pre-installed.

Copy link
Contributor

@MauriceHendrix MauriceHendrix left a comment

Choose a reason for hiding this comment

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

Overall the alooks good. I'm just wondering:

  • Can you add the docker file for the runner as well?
  • Maybe is it possible to set the number of processes in a different way instead of hard coding? Maybe an environment variable, or something?

Copy link
Member

@mirams mirams left a comment

Choose a reason for hiding this comment

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

Looks good to me, safe to give it a try I think?

@kwabenantim kwabenantim merged commit 21aaeff into Chaste:master Jun 8, 2022
@kwabenantim kwabenantim deleted the chaste_ticket_3099 branch June 8, 2022 16:27
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.

3 participants