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 build-hpc workflow #175

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Add build-hpc workflow #175

wants to merge 2 commits into from

Conversation

reuterbal
Copy link
Contributor

This adds a Github Actions workflow that dispatches to our HPC's GPU partition to enable testing for the GPU codepaths.

# Trigger the workflow on all pushes, except on tag creation
push:
branches:
- '**'

Choose a reason for hiding this comment

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

I think you might want to restrict general branch push runs to "main" and "develop". Pull requests (including draft PRs) and manual triggers (below) should suffice for a priori testing if that's needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, good point. I agree that PR and main/develop branches should suffice by default - on demand trigger is still available for other branches

@samhatfield
Copy link
Collaborator

Nice work. Is the idea that AC-based testing will replace the existing system? Or just supplement it?

@reuterbal
Copy link
Contributor Author

This is meant as a supplement that is able to utilize GPUs and the software versions available on our HPC. The testing of the full compiler matrix should remain on standard runners, which are also available to anyone when working on (public) forks.

@samhatfield samhatfield mentioned this pull request Nov 25, 2024
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