Skip to content

Add NVHPC subset CI workflow - #7

Merged
cenamiller merged 2 commits into
masterfrom
feature-ci-subset-nvhpc
Mar 25, 2026
Merged

Add NVHPC subset CI workflow#7
cenamiller merged 2 commits into
masterfrom
feature-ci-subset-nvhpc

Conversation

@cenamiller

Copy link
Copy Markdown
Collaborator

Summary

  • Adds test-nvhpc.yml -- NVHPC-only subset workflow (no GPU)
  • Builds nvhpc with mpich3 and openmpi (SMIOL, 1 MPI rank)
  • Validates logs against reference output
  • Triggers on push to master/develop and workflow_dispatch

Test plan

  • Trigger manually via Actions tab after merge
  • Verify build/run/validate for both mpich3 and openmpi

Lightweight workflow that builds and runs NVHPC (nogpu) with mpich3
and openmpi (SMIOL, 1 proc). Validates against reference logs.
For GPU testing see test-cirrus-nvhpc.yml.

Made-with: Cursor
Copilot AI review requested due to automatic review settings March 25, 2026 04:24

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a lightweight GitHub Actions workflow to quickly sanity-check NVHPC builds/runs (CPU-only) on hosted runners, focusing on 1-rank SMIOL runs across two MPI implementations.

Changes:

  • Introduces a new test-nvhpc.yml workflow triggered on pushes to master/develop and workflow_dispatch.
  • Builds MPAS-A with NVHPC in container images for mpich3 and openmpi, uploads the executable, runs a 1-rank case, and uploads logs.
  • Validates produced logs against the 240km reference output and cleans up build artifacts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/test-nvhpc.yml
- Add pull_request trigger so workflow runs on PRs
- Remove continue-on-error from artifact download
- Make allow-missing configurable in validate-logs action, set false here

Made-with: Cursor
@cenamiller
cenamiller merged commit 35dc37e into master Mar 25, 2026
6 checks passed
@cenamiller
cenamiller deleted the feature-ci-subset-nvhpc branch March 26, 2026 21:28
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.

2 participants