Skip to content

Dockerfile for reproducible test environment#86

Merged
daavid00 merged 4 commits into
cssr-tools:mainfrom
frank1010111:docker
Jun 6, 2026
Merged

Dockerfile for reproducible test environment#86
daavid00 merged 4 commits into
cssr-tools:mainfrom
frank1010111:docker

Conversation

@frank1010111

Copy link
Copy Markdown
Contributor

Description

This adds a dockerfile to allow production and testing in a reproducible environment.

It can be tested with

docker buildx build -t pycopm:latest .
docker run -it --rm -v $(pwd):/shared_host pycopm:latest bash

I'm open to improvements; this is mostly so that I can test the code for openjournals/joss-reviews#10124.

@frank1010111 frank1010111 mentioned this pull request May 20, 2026
@daavid00

daavid00 commented Jun 6, 2026

Copy link
Copy Markdown
Member

Hi @frank1010111, I have created a temporal repository in my GitHub account https://github.com/daavid00/docker-pycopm, and I have setup an action docker-test.yml using a modified version of your suggested Dockerfile. The action has successfully run with and example also using ERT https://github.com/daavid00/docker-pycopm/actions/runs/27057013734.

If this works for you, could you please update the PR with the updated Dockerfile? Then we can merge this.

Closes #87

@frank1010111

Copy link
Copy Markdown
Contributor Author

It works!

Oh no, I typed that too soon. It works except not with the ERT tests. It appears that the code expects libhdf5 to have the so suffix for version 103, and it is given version 310.

@frank1010111

Copy link
Copy Markdown
Contributor Author

I had to install libxkbcommon (naturally), but I think I have a working dockerfile!

@daavid00

daavid00 commented Jun 6, 2026

Copy link
Copy Markdown
Member

Many thanks @frank1010111 for the Dockerfile, merging this.

@daavid00 daavid00 closed this Jun 6, 2026
@daavid00 daavid00 reopened this Jun 6, 2026
@daavid00 daavid00 merged commit ddabdfe into cssr-tools:main Jun 6, 2026
1 check passed
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.

2 participants