openfoam:v2406-ubuntu24.04#83
Open
AlexisEspinosaGayosso wants to merge 4 commits into
Open
Conversation
Main difference with previous style: - The Dockerfile has been refactored with the use of Stages. - The Dockerfile now includes a COPY of the dockerfile together with all the files called during building - As in previous versions, the OpenFOAM environment is set by a "startup" script that sources the OpenFOAM bashrc. This works for Docker and Singularity. - The Docker Entrypoint Script is now created by COPYing a template that exists next to the Dockerfile. After COPY, the template tags are replaced with the correct values for the version being built. - The Singularity environment script is also created by COPYing a template and replacing the tags with the correct values for the version being built. - The Docker ENTRYPOINT instruction has been changed to directly execute a specific Docker Entrypoint Script instead of the complicated previous setup. - The Singularity environment script is still placed in /.singularity.d/env - The directory "functionalTests" contains scripts for testing functionality of the Singularity image on Setonix - The directory "auxiliaryScripts" contains scripts that were useful for the developing of the Dockerfile. There are scripts for intermediate testing of ideas and troubleshooting of problems found on the way. These scripts are not necessarily well documented and are not expected to be maintained or reviewed. But will be kept for possible future use or reference.
- Building and testing scripts are provided - Some small changes were made on the openfoam-v2406.dockerfile
This also includes: - Minimal updates to the Dockerfile - Minimal updates to the testing scripts - Minimal updates to the checking commands in each of the test Steps
Collaborator
Author
|
@Me-Basha , can you be the main reviewer? This means to perform detailed review and testing of the container. Thanks. |
Collaborator
|
yes, I can. |
Me-Basha
approved these changes
May 6, 2026
Collaborator
Me-Basha
left a comment
There was a problem hiding this comment.
looks good, I reran all the steps sucessfully.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
New recipe. For building:
openfoam:v2406-ubuntu24.04This is a new recipe, but in comparison with other recipes this one has a few differences:
More detailed information is in the
README.mdfile.Building and testing has been performed with success. We are using some auxiliary scripts to assist the workflow. The use of these scripts is also explained with more detail in the
README.mdfile.The building and testing procedure from the podman nodes was:
buildingScriptsdirectory and build the docker image:output:
output:
output:
output:
output from (
tail slurm-1234.out):A.Extract the tutorial case:
output:
B.Decompose case:
output (from
tail slurm-1234.out):C. Execute the case:
output (from
tail -20 slurm-1234.out):