Skip to content

ci: consolidate container image build matrices#391

Merged
plexoos merged 1 commit into
mainfrom
consolidate-build-matrix
Jul 2, 2026
Merged

ci: consolidate container image build matrices#391
plexoos merged 1 commit into
mainfrom
consolidate-build-matrix

Conversation

@plexoos

@plexoos plexoos commented Jun 29, 2026

Copy link
Copy Markdown
Member

This PR consolidates the container build matrices so release and develop images are generated from the same configuration list, instead of maintaining separate duplicated matrix entries. It also refreshes the default CUDA/Geant4 image versions and keeps the README image table aligned with the workflow tags.

Changes

  • Consolidates build-push.yaml around a shared target: [release, develop] and config matrix.
  • Adds matrix-level default_alias metadata so alias tagging no longer depends on hard-coded version checks.
  • Updates default container versions:
    • CUDA 13.0.2 -> 13.0.3
    • Geant4 11.4.1 -> 11.4.2
    • Adds Ubuntu 26.04 / CUDA 13.3.0 / OptiX 9.1.0 / CMake 4.3.4
  • Updates PR build coverage to test both release and develop targets across the refreshed matrix.
  • Keeps the Ubuntu 26.04 / CUDA 13.3.0 GPU test excluded until runner support is available.
  • Adds workflow concurrency controls for push and release builds.
  • Updates release tagging so latest and the short release tag are created as aliases for the default release configuration.
  • Updates the Dockerfile defaults and makes uv use managed Python 3.13.
  • Refreshes the README container image table to match the new image tags and aliases.

Copilot AI review requested due to automatic review settings June 29, 2026 18:54

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@plexoos plexoos self-assigned this Jun 29, 2026
@plexoos plexoos added the ci label Jun 29, 2026
@plexoos plexoos force-pushed the consolidate-build-matrix branch 2 times, most recently from 0927d8a to 371fa43 Compare June 30, 2026 00:43
Share a single build configuration matrix across release and develop targets so PR image jobs cover the same CUDA, OptiX, Geant4, and CMake combinations.

Push same-repository PR images for every target, then pull, test, and clean up both release and develop variants.
@plexoos plexoos force-pushed the consolidate-build-matrix branch from 371fa43 to 1fe3cad Compare July 2, 2026 12:47
@plexoos plexoos changed the title ci: align PR build matrices for release and develop targets ci: consolidate container image build matrices Jul 2, 2026
@plexoos plexoos merged commit 8938566 into main Jul 2, 2026
23 checks passed
@plexoos plexoos deleted the consolidate-build-matrix branch July 2, 2026 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants