Skip to content

Bake PHOLD into the Docker image - #149

Open
bnmajor wants to merge 2 commits into
mainfrom
ross-as-dep
Open

Bake PHOLD into the Docker image#149
bnmajor wants to merge 2 commits into
mainfrom
ross-as-dep

Conversation

@bnmajor

@bnmajor bnmajor commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator
  • _ingest_output_files now marks the run FAILED when zero expected output files are found, instead of silently completing with no data.
  • added a "ross-builder" stage that clones ROSS at a pinned commit and compiles PHOLD against OpenMPI; the final image copies just the compiled binary to /opt/ross/phold.
  • removed the ${ROSS_PATH:-../ross} bind mounts
  • removed ROSS_PATH

@bnmajor
bnmajor marked this pull request as ready for review August 19, 2026 19:11
@bnmajor
bnmajor requested review from marySalvi and waxlamp August 19, 2026 19: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