Skip to content

add ci on UOregon Frank cluster - #2080

Open
balos1 wants to merge 8 commits into
developfrom
balos1/add-uoregon-gitlab-ci
Open

add ci on UOregon Frank cluster#2080
balos1 wants to merge 8 commits into
developfrom
balos1/add-uoregon-gitlab-ci

Conversation

@balos1

@balos1 balos1 commented Aug 21, 2026

Copy link
Copy Markdown
Member

Summary

This PR adds CI testing on the UOregon Frank cluster with CPU builds on NVIDIA Grace (ARM) and GPU builds on NVIDIA RTX5080, DGX Spark (with GB10), GH200, Intel Data Center Max 1100, and AMD MI300A. These are accessed through spack.gitlab.io mirroring.

pipeline status

set(RAJA_HOST_CONFIG_LOADED ON CACHE BOOL "")

set(CMAKE_CXX_COMPILER g++ CACHE FILEPATH "")
set(CMAKE_CXX_FLAGS "-Werror -Wno-error=template-id-cdtor -Wno-error=free-nonheap-object" CACHE STRING "")

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The free-nonheap-object warning will be fixed by llnl/camp#226.

@balos1
balos1 marked this pull request as draft August 21, 2026 23:24
@balos1 balos1 changed the title add ci on UOregon Frank cluster DRAFT: add ci on UOregon Frank cluster Aug 21, 2026
@balos1

balos1 commented Aug 21, 2026

Copy link
Copy Markdown
Member Author

I need to connect the GitLab CI status back to this repo still.

Comment thread .gitlab/uoregon-gitlab-ci.yml Outdated
@balos1

balos1 commented Aug 21, 2026

Copy link
Copy Markdown
Member Author

@rhornung67 I think previously we talked about setting this up to just run on develop pushes. We could set it up to run on PRs as an optional check too if you prefer.

@rhornung67 rhornung67 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@lc-hubcast approve

@rhornung67

Copy link
Copy Markdown
Member

@rhornung67 I think previously we talked about setting this up to just run on develop pushes. We could set it up to run on PRs as an optional check too if you prefer.

Running on all PRs as an optional check seems like a good option. Thank you @balos1

@balos1
balos1 marked this pull request as ready for review August 26, 2026 22:29
@balos1

balos1 commented Aug 26, 2026

Copy link
Copy Markdown
Member Author

@rhornung67 I think previously we talked about setting this up to just run on develop pushes. We could set it up to run on PRs as an optional check too if you prefer.

Running on all PRs as an optional check seems like a good option. Thank you @balos1

Done. The status now reports back from GitLab via polling in a GitHub action (you can see it in the checks below). This avoids the need for a GitHub PAT.

@balos1 balos1 changed the title DRAFT: add ci on UOregon Frank cluster add ci on UOregon Frank cluster Aug 26, 2026
@balos1
balos1 requested a review from smithsg84 August 26, 2026 22:31
@smithsg84

Copy link
Copy Markdown
Member

@lc-hubcast approve

@lc-hubcast

lc-hubcast Bot commented Aug 27, 2026

Copy link
Copy Markdown

To mirror this PR, please use the GitHub review comment feature. This ensures the approval is tied to a specific commit to avoid mirroring malicious data.

@smithsg84 smithsg84 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@lc-hubcast approve

@smithsg84

Copy link
Copy Markdown
Member

Done. The status now reports back from GitLab via polling in a GitHub action (you can see it in the checks below). This avoids the need for a GitHub PAT.

So the polling with actions increases the pressure on GitHub runner limits since it is running in a busy loop? Based on the logs ~2 hours? If I understand this correctly we could use a "wait-timer" to delay the polling job so it doesn't consume as much runner time. I'd have to add a GitHub environment in RAJA with the delay.

@smithsg84 smithsg84 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@lc-hubcast approve

@balos1
balos1 deployed to delayed-90-minutes August 27, 2026 22:23 — with GitHub Actions Active

@smithsg84 smithsg84 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@lc-hubcast approve

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