Skip to content

Add self-hosted C++ workflow#130

Open
scal444 wants to merge 12 commits intoNVIDIA-Digital-Bio:mainfrom
scal444:colossus_ci
Open

Add self-hosted C++ workflow#130
scal444 wants to merge 12 commits intoNVIDIA-Digital-Bio:mainfrom
scal444:colossus_ci

Conversation

@scal444
Copy link
Copy Markdown
Collaborator

@scal444 scal444 commented Apr 10, 2026

No description provided.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 10, 2026

Greptile Summary

This PR introduces a self-hosted C++ CI workflow that builds and tests the project inside a pinned nvcr.io/nvidia/cuda:12.6.3-devel-ubuntu22.04 container, along with a setup_dependencies.sh script that installs Miniforge and the required conda packages. The SHA-256 integrity check for the Miniforge installer has been added, resolving the prior security concern.

Confidence Score: 5/5

Safe to merge; all remaining findings are P2 style suggestions that do not block correct operation.

The previously flagged security concern (missing Miniforge checksum) is now resolved, the container image is pinned to a specific version, and the only new finding is an inconsistent -u flag in set options — a non-blocking style issue.

No files require special attention.

Important Files Changed

Filename Overview
.github/workflows/cpp-build-test.yml New self-hosted C++ CI workflow using a pinned CUDA 12.6.3 container; minor inconsistency in set flags (-u missing in Configure/Build/Test steps) and the CC/CXX compiler-selection issue discussed in prior threads remains unresolved.
admin/ci/setup_dependencies.sh Installs Miniforge and conda packages; SHA-256 integrity check for the installer has been added (resolving the prior security finding); compiler CC/CXX exports have been removed.

Reviews (12): Last reviewed commit: "Fix hash issue" | Re-trigger Greptile

@scal444 scal444 requested a review from evasnow1992 April 10, 2026 21:11
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.

1 participant