Skip to content

Refactoring the image building process (CORSIKA/sim_telarray/simtools)#1917

Merged
GernotMaier merged 90 commits intomainfrom
docker-corsika
Dec 16, 2025
Merged

Refactoring the image building process (CORSIKA/sim_telarray/simtools)#1917
GernotMaier merged 90 commits intomainfrom
docker-corsika

Conversation

@GernotMaier
Copy link
Copy Markdown
Contributor

@GernotMaier GernotMaier commented Nov 20, 2025

Refactoring the image building process (CORSIKA/sim_telarray/simtools):

  • separate images for CORSIKA and sim_telarray
  • directory change in images to /workdir/simulation_software/ with e.g.
    • /workdir/simulation_software/corsika
    • /workdir/simulation_software/sim_telarray
  • simulation software packages pulled from git repositories
    • CORSIKA7 from KIT gitlab and CORSIKA build configuration from CTAO gitlab
    • sim_telarray, hessio, and stdtools from CTAO gitlab
  • new build process for CORSIKA7 using coconut and configuration files

New Docker files / workflows:

  1. docker/Dockerfile-corsika7, .github/workflows/build-corsika7.yml: build CORSIKA7 for different opimization using KIT gitlab (no QGSTables included)
  2. docker/Dockerfile-simtel_array, .github/workflows/build-sim_telarray.yml: build sim_telarray, hessioxxx, stdools using CTAO gitlab. No CORSIKA included (temporary: hessioxxx from tar package)
  3. docker/Dockerfile-simtools-dev, .github/workflows/build-simtools-dev.yml: develop image for simtools include the images build in the steps 1 and 2; QGSJet-III Table added; no_opt build.
  4. docker/Dockerfile-simtools-prod, .github/workflows/build-simtools-prod.yml (missing):productions images for simtools (several images for each optimization step; includes images from steps 1 and 2)

Legacy Docker files :

  • docker/Dockerfile-corsika-simtel, .github/workflows/build-corsika-simtel.yml : build CORSIKA and sim_telarray using tar packages
  • docker/Dockerfile-simtools-legacy, .github/workflows/build-simtools-legacy.yml : simtools production image (includes qgs tables) based on Dockerfile-corsika-simtel images

Open points:

  • build-corsika7, build-simtools-dev, build-simtel_array: change push back to manual only
  • switch to released versions for the code cloned from gitlab --> issue Setting / release of sim_telarray, hessio, stdtools versions. #1947
  • bernloher / iact code is part of CORSIKA7 (released and tested) and part of sim_telarray. In a future step, this should be added to the corsika compilation? Decided to work for now with the CORSIKA7 released version.

EPOS executable is included, but corsika_autoinputs probably needs an update to get it working (issue #1925 )

@GernotMaier GernotMaier self-assigned this Nov 20, 2025
@GernotMaier GernotMaier marked this pull request as ready for review December 1, 2025 18:40
@GernotMaier GernotMaier requested a review from orelgueta December 1, 2025 18:40
Copy link
Copy Markdown
Contributor

@orelgueta orelgueta left a comment

Choose a reason for hiding this comment

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

This all looks great. I only made a few comments/questions/suggestions, but none of them is critical.

Comment thread .github/workflows/build-corsika7.yml
Comment thread .github/workflows/build-corsika7.yml
Comment thread .github/workflows/build-sim_telarray.yml Outdated
Comment thread .github/workflows/build-sim_telarray.yml Outdated
Comment thread .github/workflows/build-sim_telarray.yml Outdated
Comment thread docker/Dockerfile-corsika7
Comment thread docker/Dockerfile-simtel_array
Comment thread docker/Dockerfile-simtel_array Outdated
Comment thread docker/Dockerfile-simtel_array
Comment thread docker/Dockerfile-simtools-dev
@GernotMaier
Copy link
Copy Markdown
Contributor Author

Thanks @orelgueta - I've added now everything or moved tasks into issues.

Tests / linters fail due to #1941 and need #1939 to be merged - we can wait for that if needed.

Let me know if there is anything else I can do.

Copy link
Copy Markdown
Contributor

@orelgueta orelgueta left a comment

Choose a reason for hiding this comment

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

All good, you can merge before or after fixing the tests, I don't mind.

@ctao-sonarqube
Copy link
Copy Markdown

@GernotMaier GernotMaier merged commit 2b457f1 into main Dec 16, 2025
32 checks passed
@GernotMaier GernotMaier deleted the docker-corsika branch December 16, 2025 12:18
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.

3 participants